netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Siwei Liu <loseweigh@gmail.com>,
	"Samudrala, Sridhar" <sridhar.samudrala@intel.com>,
	Alexander Duyck <alexander.h.duyck@intel.com>,
	virtio-dev@lists.oasis-open.org, aaron.f.brown@intel.com,
	Jiri Pirko <jiri@resnulli.us>, Jakub Kicinski <kubakici@wp.pl>,
	Netdev <netdev@vger.kernel.org>,
	qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org
Subject: Re: Re: [Qemu-devel] [PATCH] qemu: Introduce VIRTIO_NET_F_STANDBY feature bit to virtio_net
Date: Fri, 15 Jun 2018 11:32:42 +0200	[thread overview]
Message-ID: <20180615113242.799974f6.cohuck@redhat.com> (raw)
In-Reply-To: <20180615052743-mutt-send-email-mst@kernel.org>

On Fri, 15 Jun 2018 05:34:24 +0300
"Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Thu, Jun 14, 2018 at 12:02:31PM +0200, Cornelia Huck wrote:

> > > > I am not all that familiar with how Qemu manages network devices. If we can
> > > > do all the
> > > > required management of the primary/standby devices within Qemu, that is
> > > > definitely a better
> > > > approach without upper layer involvement.    
> > > 
> > > Right. I would imagine in the extreme case the upper layer doesn't
> > > have to be involved at all if QEMU manages all hot plug/unplug logic.
> > > The management tool can supply passthrough device and virtio with the
> > > same group UUID, QEMU auto-manages the presence of the primary, and
> > > hot plug the device as needed before or after the migration.  
> > 
> > I do not really see how you can manage that kind of stuff in QEMU only.  
> 
> So right now failover is limited to pci passthrough devices only.
> The idea is to realize the vfio device but not expose it
> to guest. Have a separate command to expose it to guest.
> Hotunplug would also hide it from guest but not unrealize it.

So, this would not be real hot(un)plug, but 'hide it from the guest',
right? The concept of "we have it realized in QEMU, but the guest can't
discover and use it" should be translatable to non-pci as well (at
least for ccw).

> 
> This will help ensure that e.g. on migration failure we can
> re-expose the device without risk of running out of resources.

Makes sense.

Should that 'hidden' state be visible/settable from outside as well
(e.g. via a property)? I guess yes, so that management software has a
chance to see whether a device is visible. Settable may be useful if we
find another use case for hiding realized devices.

  reply	other threads:[~2018-06-15  9:32 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-07 23:09 [PATCH] qemu: Introduce VIRTIO_NET_F_STANDBY feature bit to virtio_net Sridhar Samudrala
2018-06-11 17:26 ` Michael S. Tsirkin
2018-06-12  1:54   ` [Qemu-devel] " Jason Wang
2018-06-12  2:17     ` Michael S. Tsirkin
2018-06-12  5:02       ` Samudrala, Sridhar
2018-06-12 11:34         ` Michael S. Tsirkin
2018-06-13  0:08           ` [virtio-dev] " Samudrala, Sridhar
2018-06-14  1:02             ` Siwei Liu
2018-06-14 10:02               ` Cornelia Huck
2018-06-15  1:57                 ` Siwei Liu
2018-06-15 11:48                   ` Cornelia Huck
2018-06-15 17:06                     ` [virtio-dev] " Siwei Liu
2018-06-19 10:54                       ` Cornelia Huck
2018-06-19 20:09                         ` Siwei Liu
2018-06-20 14:34                           ` Cornelia Huck
2018-06-20 19:59                             ` Siwei Liu
2018-06-19 20:32                         ` [virtio-dev] " Michael S. Tsirkin
2018-06-20  9:53                           ` Cornelia Huck
2018-06-20 14:11                             ` Michael S. Tsirkin
2018-06-20 16:06                               ` Cornelia Huck
2018-06-20 19:48                                 ` Michael S. Tsirkin
2018-06-21 14:59                                   ` [virtio-dev] " Cornelia Huck
2018-06-21 18:20                                     ` Michael S. Tsirkin
2018-06-22 15:09                                       ` [virtio-dev] " Cornelia Huck
2018-06-22 19:05                                         ` Michael S. Tsirkin
2018-06-22 20:21                                           ` Siwei Liu
2018-06-22 21:32                                             ` Michael S. Tsirkin
2018-06-22 21:57                                               ` [virtio-dev] " Siwei Liu
2018-06-22 22:33                                                 ` Michael S. Tsirkin
2018-06-23  0:05                                                   ` Siwei Liu
2018-06-26 15:08                                                     ` Cornelia Huck
2018-06-26 17:50                                                       ` Michael S. Tsirkin
2018-06-27  9:11                                                         ` Cornelia Huck
2018-06-25  9:55                                           ` Cornelia Huck
2018-06-26  1:46                                             ` Michael S. Tsirkin
2018-06-26 11:55                                               ` [virtio-dev] " Cornelia Huck
2018-06-26 13:54                                                 ` Michael S. Tsirkin
2018-06-22 21:43                                         ` Michael S. Tsirkin
2018-06-27 10:10                                           ` Cornelia Huck
2018-06-22  1:21                                     ` Siwei Liu
2018-06-22  2:25                                       ` Venu Busireddy
2018-06-22  2:32                                       ` [virtio-dev] " Michael S. Tsirkin
2018-06-22 20:00                                         ` Siwei Liu
2018-06-22 20:03                                           ` Siwei Liu
2018-06-22 21:29                                             ` Michael S. Tsirkin
2018-06-22 21:51                                               ` Siwei Liu
2018-06-22 22:25                                                 ` Michael S. Tsirkin
2018-06-22 23:40                                                   ` Siwei Liu
2018-06-23  0:17                                                     ` Siwei Liu
2018-06-24  1:45                                                       ` Michael S. Tsirkin
2018-06-25 17:54                                                       ` Samudrala, Sridhar
2018-06-26  1:50                                                         ` [virtio-dev] " Michael S. Tsirkin
2018-06-26 15:17                                                           ` Cornelia Huck
2018-06-26 15:38                                                             ` Michael S. Tsirkin
2018-06-26 16:03                                                               ` Cornelia Huck
2018-06-26 17:42                                                                 ` Michael S. Tsirkin
2018-06-26 23:38                                                           ` Siwei Liu
2018-06-27  0:29                                                             ` Michael S. Tsirkin
2018-06-27  6:21                                                               ` [virtio-dev] " Siwei Liu
2018-06-27  6:49                                                                 ` Samudrala, Sridhar
2018-06-27  7:03                                                                   ` [virtio-dev] " Siwei Liu
2018-06-15  2:34                 ` Michael S. Tsirkin
2018-06-15  9:32                   ` Cornelia Huck [this message]
2018-06-15 12:31                     ` Michael S. Tsirkin
2018-06-18 13:27                       ` Cornelia Huck
2018-06-14 12:50               ` Michael S. Tsirkin

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=20180615113242.799974f6.cohuck@redhat.com \
    --to=cohuck@redhat.com \
    --cc=aaron.f.brown@intel.com \
    --cc=alexander.h.duyck@intel.com \
    --cc=jiri@resnulli.us \
    --cc=kubakici@wp.pl \
    --cc=loseweigh@gmail.com \
    --cc=mst@redhat.com \
    --cc=netdev@vger.kernel.org \
    --cc=qemu-devel@nongnu.org \
    --cc=sridhar.samudrala@intel.com \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=virtualization@lists.linux-foundation.org \
    /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).