RE: [virtio-comment] Re: [PATCH v3 6/8] admin: Add theory of operation for write recording commands

[Parav Pandit <parav@nvidia.com>]

> From: virtio-comment@lists.oasis-open.org <virtio-comment@lists.oasis-
> open.org> On Behalf Of Michael S. Tsirkin
> Sent: Friday, November 17, 2023 6:11 PM
> 
> On Fri, Nov 17, 2023 at 12:22:59PM +0000, Parav Pandit wrote:
> >
> >
> > > From: Michael S. Tsirkin <mst@redhat.com>
> > > Sent: Friday, November 17, 2023 5:03 PM
> > > To: Parav Pandit <parav@nvidia.com>
> >
> > > > Somehow the claim of shadow vq is great without sharing any
> > > > performance
> > > numbers is what I don't agree with.
> > >
> > > It's upstream in QEMU. Test it youself.
> > >
> > We did few minutes back.
> > It results in a call trace.
> > Vhost_vdpa_setup_vq_irq crashes on list corruption on net-next.
> 
> Wrong list for this bug report.
> 
> > We are stopping any shadow vq tests on unstable stuff.
> 
> If you don't want to benchmark against alternatives how are you going to prove
> your stuff is worth everyone's time?

Comparing performance of the functional things count.
You suggest shadow vq, frankly you should post the grand numbers of shadow vq.

It is really not my role to report bug of unstable stuff and compare the perf against.

We propose device context and provided the numbers you asked. Mostly wont be able to go farther than this.