Re: [PATCH V7 19/19] virtio_ring: add in order support

From: Jason Wang

Date: Mon Oct 13 2025 - 23:51:05 EST


On Fri, Sep 26, 2025 at 10:48 PM Michael S. Tsirkin <mst@xxxxxxxxxx> wrote:
>
> On Thu, Sep 25, 2025 at 06:37:08PM +0800, Jason Wang wrote:
> > This patch implements in order support for both split virtqueue and
> > packed virtqueue. Performance could be gained for the device where the
> > memory access could be expensive (e.g vhost-net or a real PCI device):
> >
> > Benchmark with KVM guest:
> >
> > Vhost-net on the host: (pktgen + XDP_DROP):
> >
> > in_order=off | in_order=on | +%
> > TX: 5.20Mpps | 6.20Mpps | +19%
> > RX: 3.47Mpps | 3.61Mpps | + 4%
> >
> > Vhost-user(testpmd) on the host: (pktgen/XDP_DROP):
> >
> > For split virtqueue:
> >
> > in_order=off | in_order=on | +%
> > TX: 5.60Mpps | 5.60Mpps | +0.0%
> > RX: 9.16Mpps | 9.61Mpps | +4.9%
> >
> > For packed virtqueue:
> >
> > in_order=off | in_order=on | +%
> > TX: 5.60Mpps | 5.70Mpps | +1.7%
> > RX: 10.6Mpps | 10.8Mpps | +1.8%
> >
> > Benchmark also shows no performance impact for in_order=off for queue
> > size with 256 and 1024.
> >
> > Signed-off-by: Jason Wang <jasowang@xxxxxxxxxx>
> > Signed-off-by: Michael S. Tsirkin <mst@xxxxxxxxxx>
>
>
> I'm not sure why my signoff is here but it does not look my
> comments have been addressed.

I will remove it.

>
> For example, see 20250921133711-mutt-send-email-mst@xxxxxxxxxx
>
>
>
> Could you pls go over my comments again and check they
> have all been addressed?
>

It has been replied here:

https://lkml.org/lkml/2025/9/22/28

Please tell me if it's ok for you or not.

Thanks