Re: [PATCH RESEND net v4 0/4] several updates to virtio/vsock

From: patchwork-bot+netdevbpf
Date: Thu Mar 16 2023 - 13:30:59 EST


Hello:

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

On Tue, 14 Mar 2023 14:03:23 +0300 you wrote:
> Hello,
>
> this patchset evolved from previous v2 version (see link below). It does
> several updates to virtio/vsock:
> 1) Changes 'virtio_transport_inc/dec_rx_pkt()' interface. Now instead of
> using skbuff state ('head' and 'data' pointers) to update 'fwd_cnt'
> and 'rx_bytes', integer value is passed as an input argument. This
> makes code more simple, because in this case we don't need to update
> skbuff state before calling 'virtio_transport_inc/dec_rx_pkt()'. In
> more common words - we don't need to change skbuff state to update
> 'rx_bytes' and 'fwd_cnt' correctly.
> 2) For SOCK_STREAM, when copying data to user fails, current skbuff is
> not dropped. Next read attempt will use same skbuff and last offset.
> Instead of 'skb_dequeue()', 'skb_peek()' + '__skb_unlink()' are used.
> This behaviour was implemented before skbuff support.
> 3) For SOCK_SEQPACKET it removes unneeded 'skb_pull()' call, because for
> this type of socket each skbuff is used only once: after removing it
> from socket's queue, it will be freed anyway.
>
> [...]

Here is the summary with links:
- [RESEND,net,v4,1/4] virtio/vsock: don't use skbuff state to account credit
https://git.kernel.org/netdev/net/c/077706165717
- [RESEND,net,v4,2/4] virtio/vsock: remove redundant 'skb_pull()' call
https://git.kernel.org/netdev/net/c/6825e6b4f8e5
- [RESEND,net,v4,3/4] virtio/vsock: don't drop skbuff on copy failure
https://git.kernel.org/netdev/net/c/8daaf39f7f6e
- [RESEND,net,v4,4/4] test/vsock: copy to user failure test
https://git.kernel.org/netdev/net/c/7e699d2a4e81

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