mbox series

[net-next,v2,0/2] MACB NAPI improvements

Message ID 20220509194635.3094080-1-robert.hancock@calian.com (mailing list archive)
Headers show
Series MACB NAPI improvements | expand

Message

Robert Hancock May 9, 2022, 7:46 p.m. UTC
Simplify the logic in the Cadence MACB/GEM driver for determining
when to reschedule NAPI processing, and update it to use NAPI for the
TX path as well as the RX path.

Changes since v1: Changed to use separate TX and RX NAPI instances and
poll functions to avoid unnecessary checks of the other ring (TX/RX)
states during polling and to use budget handling for both RX and TX.
Fixed locking to protect against concurrent access to TX ring on
TX transmit and TX poll paths.

Robert Hancock (2):
  net: macb: simplify/cleanup NAPI reschedule checking
  net: macb: use NAPI for TX completion path

 drivers/net/ethernet/cadence/macb.h      |   6 +-
 drivers/net/ethernet/cadence/macb_main.c | 291 +++++++++++++++--------
 2 files changed, 191 insertions(+), 106 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org May 11, 2022, 11:20 p.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Mon,  9 May 2022 13:46:33 -0600 you wrote:
> Simplify the logic in the Cadence MACB/GEM driver for determining
> when to reschedule NAPI processing, and update it to use NAPI for the
> TX path as well as the RX path.
> 
> Changes since v1: Changed to use separate TX and RX NAPI instances and
> poll functions to avoid unnecessary checks of the other ring (TX/RX)
> states during polling and to use budget handling for both RX and TX.
> Fixed locking to protect against concurrent access to TX ring on
> TX transmit and TX poll paths.
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/2] net: macb: simplify/cleanup NAPI reschedule checking
    https://git.kernel.org/netdev/net-next/c/1900e30d0ef7
  - [net-next,v2,2/2] net: macb: use NAPI for TX completion path
    https://git.kernel.org/netdev/net-next/c/138badbc21a0

You are awesome, thank you!