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

["Michael S. Tsirkin" <mst@redhat.com>]

On Wed, Nov 22, 2023 at 03:46:41AM +0000, Parav Pandit wrote:
> 
> 
> > From: Michael S. Tsirkin <mst@redhat.com>
> > Sent: Wednesday, November 22, 2023 2:31 AM
> > 
> > On Tue, Nov 21, 2023 at 04:29:36PM +0000, Parav Pandit wrote:
> > > Basic test with iperf is not working. Crashing it.
> > > All of this is complete unrelated discussion to this series to slow down the
> > work.
> > > I donât see any value.
> > > Michael asked to do the test, we did, it does not work. Functionally broken
> > code has no comparison.
> > 
> > It's unfortunate it's unstable for you, if you could show perf comparison that
> > would be a strong argument for your case. Reporting Linux/qemu failures to
> > virtio TC is not going to help you though, wrong forum.
> 
> As I explained, the basic requirements are not met hence, the comparison is not applicable.
> There is no point in discussing specific OS implementation anyway. You asked to remove vfio citations hence we remove other citations as well.
> 
> Thanks.

So it crashes, and even if it does not it does not perform well, and even
if it performs well it does not meet the requirements.
Really, you are shooting yourself in the foot with stuff like this.
Does not look good at all.

-- 
MST