mbox series

[net-next,0/2] vsock: avoid queuing on workqueue if possible

Message ID AS2P194MB2170EB1827729FB1666311FA9AC22@AS2P194MB2170.EURP194.PROD.OUTLOOK.COM (mailing list archive)
Headers show
Series vsock: avoid queuing on workqueue if possible | expand

Message

Luigi Leonardi June 14, 2024, 1:55 p.m. UTC
This patch series introduces an optimization for vsock/virtio to reduce latency:
When the guest sends a packet to the host, and the workqueue is empty,
if there is enough space, the packet is put directly in the virtqueue.

The first one contains some code refactoring.
More details and some performance tests in the second patch.

Marco Pinna (2):
  vsock/virtio: refactor virtio_transport_send_pkt_work
  vsock/virtio: avoid enqueue packets when work queue is empty

 net/vmw_vsock/virtio_transport.c | 166 +++++++++++++++++++------------
 1 file changed, 104 insertions(+), 62 deletions(-)

Comments

Stefano Garzarella June 14, 2024, 2:40 p.m. UTC | #1
On Fri, Jun 14, 2024 at 03:55:41PM GMT, Luigi Leonardi wrote:
>This patch series introduces an optimization for vsock/virtio to reduce latency:
>When the guest sends a packet to the host, and the workqueue is empty,
>if there is enough space, the packet is put directly in the virtqueue.

Thanks for working on this!!

I left few small comments.
I'm at DevConf this weekend, so I'll do a better review and some testing
next week.

Stefano

>
>The first one contains some code refactoring.
>More details and some performance tests in the second patch.
>
>Marco Pinna (2):
>  vsock/virtio: refactor virtio_transport_send_pkt_work
>  vsock/virtio: avoid enqueue packets when work queue is empty
>
> net/vmw_vsock/virtio_transport.c | 166 +++++++++++++++++++------------
> 1 file changed, 104 insertions(+), 62 deletions(-)
>
>-- 
>2.45.2
>