mbox series

[net,stable,v1,0/3] add checking sq is full inside xdp xmit

Message ID 20230308024935.91686-1-xuanzhuo@linux.alibaba.com (mailing list archive)
Headers show
Series add checking sq is full inside xdp xmit | expand

Message

Xuan Zhuo March 8, 2023, 2:49 a.m. UTC
If the queue of xdp xmit is not an independent queue, then when the xdp
xmit used all the desc, the xmit from the __dev_queue_xmit() may encounter
the following error.

net ens4: Unexpected TXQ (0) queue failure: -28

This patch adds a check whether sq is full in XDP Xmit.

Thanks.

v1:
    1. rename to check_sq_full_and_disable
    2. reorder some funcs to avoid declaration

Xuan Zhuo (3):
  virtio_net: reorder some funcs
  virtio_net: separate the logic of checking whether sq is full
  virtio_net: add checking sq is full inside xdp xmit

 drivers/net/virtio_net.c | 155 +++++++++++++++++++++------------------
 1 file changed, 85 insertions(+), 70 deletions(-)

--
2.32.0.3.g01195cf9f

Comments

patchwork-bot+netdevbpf@kernel.org March 11, 2023, 12:50 a.m. UTC | #1
Hello:

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

On Wed,  8 Mar 2023 10:49:32 +0800 you wrote:
> If the queue of xdp xmit is not an independent queue, then when the xdp
> xmit used all the desc, the xmit from the __dev_queue_xmit() may encounter
> the following error.
> 
> net ens4: Unexpected TXQ (0) queue failure: -28
> 
> This patch adds a check whether sq is full in XDP Xmit.
> 
> [...]

Here is the summary with links:
  - [net,stable,v1,1/3] virtio_net: reorder some funcs
    https://git.kernel.org/netdev/net/c/25074a44ac4e
  - [net,stable,v1,2/3] virtio_net: separate the logic of checking whether sq is full
    https://git.kernel.org/netdev/net/c/b8ef4809bc7f
  - [net,stable,v1,3/3] virtio_net: add checking sq is full inside xdp xmit
    https://git.kernel.org/netdev/net/c/cd1c604aa1d8

You are awesome, thank you!