On Thu, 12 Jun 2025 at 10:21, Xuewei Niu <niuxuewe...@gmail.com> wrote: > > > On Thu, 12 Jun 2025 at 08:50, Xuewei Niu <niuxuewe...@gmail.com> wrote: > > > > > > > On Thu, Jun 12, 2025 at 01:32:01PM +0800, Xuewei Niu wrote: > > > > > 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 > > > > > > > > > > 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.
As Michael pointed out, it was merged several weeks ago in net tree, see https://lore.kernel.org/netdev/174827942876.985160.7017354014266756923.git-patchwork-not...@kernel.org/ And it also landed in Linus tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=45ca7e9f0730ae36fc610e675b990e9cc9ca0714 So, I think you can go head with your patch, right? Please remember to target net-next, since it will be a new feature IIRC. Thanks, Stefano