Stefano Garzarella
2019-Oct-15 15:00 UTC
[PATCH] vsock/virtio: remove unused 'work' field from 'struct virtio_vsock_pkt'
The 'work' field was introduced with commit 06a8fc78367d0 ("VSOCK: Introduce virtio_vsock_common.ko") but it is never used in the code, so we can remove it to save memory allocated in the per-packet 'struct virtio_vsock_pkt' Suggested-by: Michael S. Tsirkin <mst at redhat.com> Signed-off-by: Stefano Garzarella <sgarzare at redhat.com> --- include/linux/virtio_vsock.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/linux/virtio_vsock.h b/include/linux/virtio_vsock.h index 4c7781f4b29b..07875ccc7bb5 100644 --- a/include/linux/virtio_vsock.h +++ b/include/linux/virtio_vsock.h @@ -48,7 +48,6 @@ struct virtio_vsock_sock { struct virtio_vsock_pkt { struct virtio_vsock_hdr hdr; - struct work_struct work; struct list_head list; /* socket refcnt not held, only use for cancellation */ struct vsock_sock *vsk; -- 2.21.0
David Miller
2019-Oct-16 03:55 UTC
[PATCH] vsock/virtio: remove unused 'work' field from 'struct virtio_vsock_pkt'
From: Stefano Garzarella <sgarzare at redhat.com> Date: Tue, 15 Oct 2019 17:00:51 +0200> The 'work' field was introduced with commit 06a8fc78367d0 > ("VSOCK: Introduce virtio_vsock_common.ko") > but it is never used in the code, so we can remove it to save > memory allocated in the per-packet 'struct virtio_vsock_pkt' > > Suggested-by: Michael S. Tsirkin <mst at redhat.com> > Signed-off-by: Stefano Garzarella <sgarzare at redhat.com>Michael, will you take this? I assume so... Thanks.
Stefan Hajnoczi
2019-Oct-16 14:50 UTC
[PATCH] vsock/virtio: remove unused 'work' field from 'struct virtio_vsock_pkt'
On Tue, Oct 15, 2019 at 05:00:51PM +0200, Stefano Garzarella wrote:> The 'work' field was introduced with commit 06a8fc78367d0 > ("VSOCK: Introduce virtio_vsock_common.ko") > but it is never used in the code, so we can remove it to save > memory allocated in the per-packet 'struct virtio_vsock_pkt' > > Suggested-by: Michael S. Tsirkin <mst at redhat.com> > Signed-off-by: Stefano Garzarella <sgarzare at redhat.com> > --- > include/linux/virtio_vsock.h | 1 - > 1 file changed, 1 deletion(-)Reviewed-by: Stefan Hajnoczi <stefanha at redhat.com> -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 488 bytes Desc: not available URL: <http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20191016/67142b91/attachment.sig>
Possibly Parallel Threads
- [PATCH v4 3/5] vsock/virtio: fix locking in virtio_transport_inc_tx_pkt()
- [PATCH v2 7/8] vsock/virtio: increase RX buffer size to 64 KiB
- [PATCH v2 7/8] vsock/virtio: increase RX buffer size to 64 KiB
- [PATCH v2 7/8] vsock/virtio: increase RX buffer size to 64 KiB
- [PATCH v2 7/8] vsock/virtio: increase RX buffer size to 64 KiB