From: "Michael S. Tsirkin" <mst@redhat.com>
To: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Cc: "John G Johnson" <john.g.johnson@oracle.com>,
"Daniel P. Berrangé" <berrange@redhat.com>,
quintela@redhat.com, "Jason Wang" <jasowang@redhat.com>,
"Felipe Franciosi" <felipe@nutanix.com>,
"Kirti Wankhede" <kwankhede@nvidia.com>,
qemu-devel@nongnu.org,
"Alex Williamson" <alex.williamson@redhat.com>,
"Stefan Hajnoczi" <stefanha@redhat.com>,
"Thanos Makatos" <thanos.makatos@nutanix.com>,
"Paolo Bonzini" <pbonzini@redhat.com>
Subject: Re: VFIO Migration
Date: Thu, 5 Nov 2020 07:47:55 -0500 [thread overview]
Message-ID: <20201105074428-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <20201105121324.GH3186@work-vm>
On Thu, Nov 05, 2020 at 12:13:24PM +0000, Dr. David Alan Gilbert wrote:
> * Stefan Hajnoczi (stefanha@redhat.com) wrote:
> > On Wed, Nov 04, 2020 at 05:32:02PM +0000, Dr. David Alan Gilbert wrote:
> > > * Stefan Hajnoczi (stefanha@redhat.com) wrote:
> > > > Michael replied in another sub-thread wondering if versions are really
> > > > necessary since tools do the migration checks. Let's try dropping
> > > > versions to simplify things. We can bring them back if needed later.
> > >
> > > What does a user facing tool do? If I say I want one of these NICs
> > > and I'm on the latest QEMU machine type, who sets all these parameters?
> >
> > The machine type is orthogonal since QEMU doesn't know about every
> > possible VFIO device. The device is like a PCI adapter that is added to
> > a physical machine aftermarket, it's not part of the base machine's
> > specs.
>
> OK, but ignoring migration, I think the same problem holds; if I'm a
> tool creating one of these VMs, and I plug this device in, what do I do
> with all it's configuration parameters? I'd assume most of the time
> that they don't know about or dont care about most of the parameters,
> they just want the sane defaults unless told otherwise.
I think that if you ignore migration then you can ignore parameters.
> > The migration tool queries the parameters from the source device.
> > VFIO/mdev will provide sysfs attrs. For vfio-user I'm not sure whether
> > to print the parameters during device instantiation, require a
> > VFIO-compatible FUSE directory, or to use a query-migration-params RPC
> > command.
>
> But on VM creation we have to answer the question of what config do we
> want; so for example lets say I'm creating a new VM in my cluster,
> but I want to be sure that later I can migrate it. I can read the
> config off one of the other machines; can I just use that even if my
> new machine has a later device implementation?
>
> Dave
I don't think so - we need a tool that can query a set of machines and then
produce a safe configuration.
The same problem exists with vhost as well it will just
explode exponentially with lots more devices and backends ...
Talked about it a bit in my kvm forum preso ...
> > Let's discuss this more when the next revision of the document is sent
> > out, because it modifies the approach so that migration parameters are
> > logically separate from device configuration parameters. That changes
> > things a bit.
> >
> > Stefan
>
>
> --
> Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
next prev parent reply other threads:[~2020-11-05 12:50 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-02 11:11 VFIO Migration Stefan Hajnoczi
2020-11-02 12:28 ` Cornelia Huck
2020-11-02 14:56 ` Stefan Hajnoczi
2020-11-04 8:07 ` Gerd Hoffmann
2020-11-04 16:40 ` Stefan Hajnoczi
2020-11-05 6:47 ` Gerd Hoffmann
2020-11-05 11:42 ` Stefan Hajnoczi
2020-11-02 19:38 ` Alex Williamson
2020-11-03 11:03 ` Stefan Hajnoczi
2020-11-03 17:13 ` Alex Williamson
2020-11-03 18:09 ` Stefan Hajnoczi
2020-11-05 23:37 ` Yan Zhao
2020-11-03 8:46 ` Jason Wang
2020-11-03 12:15 ` Stefan Hajnoczi
2020-11-04 3:32 ` Jason Wang
2020-11-04 7:16 ` Stefan Hajnoczi
2020-11-03 11:39 ` Daniel P. Berrangé
2020-11-03 15:05 ` Stefan Hajnoczi
2020-11-03 15:23 ` Daniel P. Berrangé
2020-11-03 18:16 ` Stefan Hajnoczi
2020-11-03 12:17 ` Dr. David Alan Gilbert
2020-11-03 15:27 ` Stefan Hajnoczi
2020-11-03 18:49 ` Dr. David Alan Gilbert
2020-11-04 7:36 ` Stefan Hajnoczi
2020-11-04 10:14 ` Dr. David Alan Gilbert
2020-11-04 16:47 ` Stefan Hajnoczi
2020-11-04 17:32 ` Dr. David Alan Gilbert
2020-11-05 11:40 ` Stefan Hajnoczi
2020-11-05 12:13 ` Dr. David Alan Gilbert
2020-11-05 12:47 ` Michael S. Tsirkin [this message]
2020-11-05 14:17 ` Dr. David Alan Gilbert
2020-11-05 12:53 ` Michael S. Tsirkin
2020-11-04 11:05 ` Christophe de Dinechin
2020-11-03 15:23 ` Christophe de Dinechin
2020-11-03 15:33 ` Daniel P. Berrangé
2020-11-03 17:31 ` Alex Williamson
2020-11-04 10:13 ` Stefan Hajnoczi
2020-11-04 11:10 ` Stefan Hajnoczi
2020-11-04 7:50 ` Michael S. Tsirkin
2020-11-04 16:37 ` Stefan Hajnoczi
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=20201105074428-mutt-send-email-mst@kernel.org \
--to=mst@redhat.com \
--cc=alex.williamson@redhat.com \
--cc=berrange@redhat.com \
--cc=dgilbert@redhat.com \
--cc=felipe@nutanix.com \
--cc=jasowang@redhat.com \
--cc=john.g.johnson@oracle.com \
--cc=kwankhede@nvidia.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=stefanha@redhat.com \
--cc=thanos.makatos@nutanix.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.