diff options
| author |   Igor Russkikh <Igor.Russkikh@aquantia.com> | 2019-03-23 15:23:34 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-03-23 22:16:53 -0400 |
| commit | 9773ef18b83d8acc5862ea7e727d6e4d52846dd7 (patch) | |
| tree | d63bc6457458f5bc17b93e469ba70248defe7a31 /drivers/net/ethernet/aquantia/atlantic/aq_ring.c | |
| parent | net: aquantia: optimize rx performance by page reuse strategy (diff) | |
| download | linux-dev-9773ef18b83d8acc5862ea7e727d6e4d52846dd7.tar.xz linux-dev-9773ef18b83d8acc5862ea7e727d6e4d52846dd7.zip | |
net: aquantia: Introduce rx refill threshold value
Before that, we've refilled ring even on single descriptor move.
Under high packet load that caused page allocation logic to be triggered
too often. That made overall ring processing slower.
Moreover, with page buffer reuse implemented, we should give a chance
higher networking levels to process received packets faster, release
the pages they consumed and therefore give a higher chance for these
pages to be reused.
RX ring is now refilled only when AQ_CFG_RX_REFILL_THRES or more
descriptors were processed (32 by default). Under regular traffic this
gives quite enough time for packet to be consumed and page to be reused.
Signed-off-by: Igor Russkikh <igor.russkikh@aquantia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/aquantia/atlantic/aq_ring.c')
| -rw-r--r-- | drivers/net/ethernet/aquantia/atlantic/aq_ring.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_ring.c b/drivers/net/ethernet/aquantia/atlantic/aq_ring.c index c48696123193..c64e2fb5a4f1 100644 --- a/drivers/net/ethernet/aquantia/atlantic/aq_ring.c +++ b/drivers/net/ethernet/aquantia/atlantic/aq_ring.c @@ -419,6 +419,10 @@ int aq_ring_rx_fill(struct aq_ring_s *self) int err = 0; int i = 0; + if (aq_ring_avail_dx(self) < min_t(unsigned int, AQ_CFG_RX_REFILL_THRES, + self->size / 2)) + return err; + for (i = aq_ring_avail_dx(self); i--; self->sw_tail = aq_ring_next_dx(self, self->sw_tail)) { buff = &self->buff_ring[self->sw_tail]; |