From: Cornelia Huck <cohuck@redhat.com>
To: Stefano Garzarella <sgarzare@redhat.com>
Cc: Halil Pasic <pasic@linux.ibm.com>,
Qinghua Cheng <qcheng@redhat.com>,
qemu devel list <qemu-devel@nongnu.org>,
Stefan Hajnoczi <stefanha@redhat.com>,
"Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: virtio-vsock requires 'disable-legacy=on' in QEMU 5.1
Date: Tue, 18 Aug 2020 16:31:01 +0200 [thread overview]
Message-ID: <20200818163101.72abd288.cohuck@redhat.com> (raw)
In-Reply-To: <20200818140120.c46fsf6cia2ol53v@steredhat>
On Tue, 18 Aug 2020 16:01:20 +0200
Stefano Garzarella <sgarzare@redhat.com> wrote:
> On Tue, Aug 18, 2020 at 02:44:50PM +0200, Cornelia Huck wrote:
> > It seems to me that the status before this was "works by accident, but
> > only if we're not negotiating to legacy, or the guest/host are both
> > little endian". IOW, no visible breakage for most people (or we'd
> > probably have heard of it already). Now we have a setup that's correct,
> > but forces users to adapt their QEMU command lines. Option 1 would
> > eliminate the need to do that, but would cause possibly
> > not-really-fixable migration issues (you can probably deal with that
> > manually, detaching and re-attaching the device as a last resort.)
> >
> > So, force modern, probably also remove the -transitional device type,
> > and put a prominent explanation into the change log?
> >
>
> I completely agree with your analysis and solution.
>
> So, for now we need to patch vhost-vsock-pci and vhost-user-vsock-pci,
> and queue the patches in stable.
I think we should also change -ccw; even though users won't get an
error when starting QEMU, they might still run into the legacy problems
in theory.
Not sure how fast we'll have a stable release, though.
>
> Do you prefer to send them? Otherwise I can do that.
If you already have something on your disk, please go ahead :)
>
> Thanks again for the help and the test with s390x guest!
np; especially as it was my patch which started this in the first place
:/
next prev parent reply other threads:[~2020-08-18 15:39 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-13 9:16 virtio-vsock requires 'disable-legacy=on' in QEMU 5.1 Stefano Garzarella
2020-08-13 9:28 ` Cornelia Huck
2020-08-13 10:24 ` Stefano Garzarella
2020-08-13 10:37 ` Cornelia Huck
2020-08-13 12:04 ` Stefano Garzarella
2020-08-17 10:27 ` Cornelia Huck
2020-08-17 13:11 ` Stefano Garzarella
2020-08-17 14:37 ` Cornelia Huck
2020-08-18 12:44 ` Cornelia Huck
2020-08-18 14:01 ` Stefano Garzarella
2020-08-18 14:31 ` Cornelia Huck [this message]
2020-08-18 15:28 ` Stefano Garzarella
2020-08-18 15:49 ` Cornelia Huck
2020-08-19 9:48 ` Auger Eric
2020-08-13 15:36 ` no-reply
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=20200818163101.72abd288.cohuck@redhat.com \
--to=cohuck@redhat.com \
--cc=mst@redhat.com \
--cc=pasic@linux.ibm.com \
--cc=qcheng@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=sgarzare@redhat.com \
--cc=stefanha@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.