Re: [PATCH net] vsock/virtio: fix `rx_bytes` accounting for stream sockets

On Thu, 12 Jun 2025 at 08:50, Xuewei Niu [off-list ref] wrote:

quoted

quoted

On Thu, Jun 12, 2025 at 01:32:01PM +0800, Xuewei Niu wrote:

quoted

No comments since last month.

The patch [1], which adds SIOCINQ ioctl support for vsock, depends on this
patch. Could I get more eyes on this one?

[1]: https://lore.kernel.org/lkml/bbn4lvdwh42m2zvi3rdyws66y5ulew32rchtz3kxirqlllkr63@7toa4tcepax3/#t (local)

Thanks,
Xuewei

it's been in net for two weeks now, no?

Umm sorry, I didn't check the date carefully, because there are several
ongoing patches. Next time I'll check it carefully. Sorry again.

It looks like no one is paying attention to this patch. I am requesting
someone interested in vsock to review this. I'd appreciate that!

Which patch do you mean?

Thanks,
Stefano

I am saying your patch, "vsock/virtio: fix `rx_bytes` accounting for stream
sockets".

Once this gets merged, I will send a new version of my patch to support
SIOCINQ ioctl. Thus, I can reuse `rx_bytes` to count unread bytes, as we
discussed.

Thanks,
Xuewei