eth: fbnic: update fbnic_poll return value
authorMohsin Bashir <mohsin.bashr@gmail.com>
Sat, 4 Jan 2025 01:53:16 +0000 (17:53 -0800)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 7 Jan 2025 12:22:02 +0000 (13:22 +0100)
In cases where the work done is less than the budget, `fbnic_poll` is
returning 0. This affects the tracing of `napi_poll`. Following is a
snippet of before and after result from `napi_poll` tracepoint. Instead,
returning the work done improves the manual tracing.

Before:
@[10]: 1
...
@[64]: 208175
@[0]: 2128008

After:
@[56]: 86
@[48]: 222
...
@[5]: 1885756
@[6]: 1933841

Signed-off-by: Mohsin Bashir <mohsin.bashr@gmail.com>
Reviewed-by: Jakub Kicinski <kuba@kernel.org>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Link: https://patch.msgid.link/20250104015316.3192946-1-mohsin.bashr@gmail.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/meta/fbnic/fbnic_txrx.c

index bb54ce5..d4d7027 100644 (file)
@@ -1033,7 +1033,7 @@ static int fbnic_poll(struct napi_struct *napi, int budget)
        if (likely(napi_complete_done(napi, work_done)))
                fbnic_nv_irq_rearm(nv);
 
-       return 0;
+       return work_done;
 }
 
 irqreturn_t fbnic_msix_clean_rings(int __always_unused irq, void *data)