Eric Lee / smarc-ti-linux-kernel

Download as
Browse Code ยป

Commit 6980baed350d2e32647595fd06f5281d708fbee0

Authored by Ivan Vecera
Committed by Greg Kroah-Hartman
1 parent 4fe35cff3d

bna: fix performance regression 

[ Upstream commit c36c9d50cc6af5c5bfcc195f21b73f55520c15f9 ]

The recent commit "e29aa33 bna: Enable Multi Buffer RX" is causing
a performance regression. It does not properly update 'cmpl' pointer
at the end of the loop in NAPI handler bnad_cq_process(). The result is
only one packet / per NAPI-schedule is processed.

Signed-off-by: Ivan Vecera <ivecera@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff

drivers/net/ethernet/brocade/bna/bnad.c
Diff comments   View file @ 6980bae
... ... @@ -600,9 +600,9 @@
600 600 prefetch(bnad->netdev);
601 601  
602 602 cq = ccb->sw_q;
603   - cmpl = &cq[ccb->producer_index];
604 603  
605 604 while (packets < budget) {
  605 + cmpl = &cq[ccb->producer_index];
606 606 if (!cmpl->valid)
607 607 break;
608 608 /* The 'valid' field is set by the adapter, only after writing