Re: [PATCH net] vsock/loopback: use only sk_buff_head.lock to protect the packet queue

From: patchwork-bot+netdevbpf
Date: Mon Mar 27 2023 - 03:20:26 EST


Hello:

This patch was applied to netdev/net.git (main)
by David S. Miller <davem@xxxxxxxxxxxxx>:

On Fri, 24 Mar 2023 12:54:50 +0100 you wrote:
> pkt_list_lock was used before commit 71dc9ec9ac7d ("virtio/vsock:
> replace virtio_vsock_pkt with sk_buff") to protect the packet queue.
> After that commit we switched to sk_buff and we are using
> sk_buff_head.lock in almost every place to protect the packet queue
> except in vsock_loopback_work() when we call skb_queue_splice_init().
>
> As reported by syzbot, this caused unlocked concurrent access to the
> packet queue between vsock_loopback_work() and
> vsock_loopback_cancel_pkt() since it is not holding pkt_list_lock.
>
> [...]

Here is the summary with links:
- [net] vsock/loopback: use only sk_buff_head.lock to protect the packet queue
https://git.kernel.org/netdev/net/c/b465518dc27d

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html