[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240730-ioctl-v4-0-16d89286a8f0@outlook.com>
Date: Tue, 30 Jul 2024 21:43:05 +0200
From: Luigi Leonardi via B4 Relay <devnull+luigi.leonardi.outlook.com@...nel.org>
To: Stefano Garzarella <sgarzare@...hat.com>,
"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Stefan Hajnoczi <stefanha@...hat.com>,
"Michael S. Tsirkin" <mst@...hat.com>, Jason Wang <jasowang@...hat.com>,
Eugenio PĂ©rez <eperezma@...hat.com>,
Xuan Zhuo <xuanzhuo@...ux.alibaba.com>
Cc: virtualization@...ts.linux.dev, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, kvm@...r.kernel.org,
Luigi Leonardi <luigi.leonardi@...look.com>,
Daan De Meyer <daan.j.demeyer@...il.com>
Subject: [PATCH net-next v4 0/3] ioctl support for AF_VSOCK and
virtio-based transports
This patch series introduce the support for ioctl(s) in AF_VSOCK.
The only ioctl currently available is SIOCOUTQ, which returns
the number of unsent or unacked packets. It is available for
SOCK_STREAM, SOCK_SEQPACKET and SOCK_DGRAM.
As this information is transport-dependent, a new optional callback
is introduced: unsent_bytes.
The first patch add ioctl support in AF_VSOCK, while the second
patch introduce support for SOCK_STREAM and SOCK_SEQPACKET
in all virtio-based transports: virtio_transport (G2H),
vhost-vsock (H2G) and vsock-loopback.
The latest patch introduce two tests for this new feature.
More details can be found in each patch changelog.
v3->v4
- Fixed warnings produced by kernel test robots
- Functions are now *_unsent_bytes
- Minor style changes pointed out by Stefano
- Using opts->peer_port in tests
- Rebased to latest net-next
- Link to v3: https://lore.kernel.org/r/20240626-ioctl_next-v3-0-63be5bf19a40@outlook.com
v2->v3
Applied all reviewers' suggetions:
- Minor style and code changes
- atomic_int replaced with an existing spin_lock.
Introduced lock_sock on ioctl call.
Rebased to latest net-next
v1->v2
Applied all Stefano's suggestions:
- vsock_do_ioctl has been rewritten
- ioctl(SIOCOUTQ) test is skipped when it is not supported
- Minor variable/function name changes
- rebased to latest net-next
Signed-off-by: Luigi Leonardi <luigi.leonardi@...look.com>
---
Luigi Leonardi (3):
vsock: add support for SIOCOUTQ ioctl
vsock/virtio: add SIOCOUTQ support for all virtio based transports
test/vsock: add ioctl unsent bytes test
drivers/vhost/vsock.c | 4 +-
include/linux/virtio_vsock.h | 6 +++
include/net/af_vsock.h | 3 ++
net/vmw_vsock/af_vsock.c | 58 ++++++++++++++++++++--
net/vmw_vsock/virtio_transport.c | 4 +-
net/vmw_vsock/virtio_transport_common.c | 35 ++++++++++++++
net/vmw_vsock/vsock_loopback.c | 6 +++
tools/testing/vsock/util.c | 6 +--
tools/testing/vsock/util.h | 3 ++
tools/testing/vsock/vsock_test.c | 85 +++++++++++++++++++++++++++++++++
10 files changed, 202 insertions(+), 8 deletions(-)
---
base-commit: 1722389b0d863056d78287a120a1d6cadb8d4f7b
change-id: 20240730-ioctl-0da7dee1b725
Best regards,
--
Luigi Leonardi <luigi.leonardi@...look.com>
Powered by blists - more mailing lists