qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Halil Pasic <pasic@linux.ibm.com>
To: Jason Wang <jasowang@redhat.com>
Cc: Stefano Garzarella <sgarzare@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	qemu-devel@nongnu.org, Marc Hartmayer <mhartmay@linux.ibm.com>,
	Christian Borntraeger <borntraeger@linux.ibm.com>,
	Boris Fiuczynski <fiuczy@linux.ibm.com>,
	Halil Pasic <pasic@linux.ibm.com>
Subject: Re: [PATCH 1/1] vhost-vsock: add VIRTIO_F_RING_PACKED to feaure_bits
Date: Mon, 3 Jun 2024 13:44:23 +0200	[thread overview]
Message-ID: <20240603134423.5d71dbde.pasic@linux.ibm.com> (raw)
In-Reply-To: <CACGkMEtoxSne90b75GEWWKaYOXzvvo5=rUS96ufjYsWYfN0ykA@mail.gmail.com>

On Thu, 30 May 2024 10:34:55 +0800
Jason Wang <jasowang@redhat.com> wrote:

> >
> > IMHO changing the semantic of the VHOST_GET_FEATURES ioctl is not viable,
> > but also not necessary. What I am proposing is changing the (in QEMU)
> > logic of processing the features returned by VHOST_GET_FEATURES, while
> > preserving the outcomes (essentially realize the same function in a
> > mathematical sense, but with code that is less fragile), modulo bugs like
> > the one addressed with this patch of course.  
> 
> Ok, I think I misunderstood you here. Maybe an RFC to see?

I will try to hack up an RFC that involves vsock and net so we can
discuss the approach. Unfortunately I currently have a lots of other
(non-development) stuff on my plate, please bear with me.

I suggest to go forward with this patch for fixing vsock, and look if we
can do something better for the rest.

By the way in the thread with Stefano, I've raised a question about live
migration (i.e. what would happen of someone were to implement
packed layout for vhost and make add the feature to vhost/net). I would
very much like to have your opinion on that!

Regards,
Halil


  reply	other threads:[~2024-06-03 11:45 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-29 11:33 [PATCH 1/1] vhost-vsock: add VIRTIO_F_RING_PACKED to feaure_bits Halil Pasic
2024-05-07 19:26 ` Halil Pasic
2024-05-15 22:41   ` Halil Pasic
2024-05-16  8:39 ` Stefano Garzarella
2024-05-27 11:27   ` Halil Pasic
2024-05-28  3:25     ` Jason Wang
2024-05-29 12:17       ` Halil Pasic
2024-05-30  2:34         ` Jason Wang
2024-06-03 11:44           ` Halil Pasic [this message]
2024-05-28 15:32     ` Stefano Garzarella
2024-05-29 12:49       ` Halil Pasic
2024-05-30  7:56         ` Stefano Garzarella

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240603134423.5d71dbde.pasic@linux.ibm.com \
    --to=pasic@linux.ibm.com \
    --cc=borntraeger@linux.ibm.com \
    --cc=fiuczy@linux.ibm.com \
    --cc=jasowang@redhat.com \
    --cc=mhartmay@linux.ibm.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=sgarzare@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).