[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 06/11] qapi: add failover negotiated event
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH v6 06/11] qapi: add failover negotiated event |
Date: |
Tue, 29 Oct 2019 18:41:58 -0400 |
On Fri, Oct 25, 2019 at 07:39:21PM +0200, Jens Freimann wrote:
> On Fri, Oct 25, 2019 at 04:03:54PM +0200, Markus Armbruster wrote:
> > Bear with me, I know next to nothing about failover.
> >
> > Jens Freimann <address@hidden> writes:
> >
> > > This event is sent to let libvirt know that VIRTIO_NET_F_STANDBY feature
> > > was enabled. The primary device this virtio-net device is associated
> > > with, will now be hotplugged via qdev_device_add().
> >
> > Passive voice deftly avoids telling the reader who will do the
> > hot-plugging. Intentional?
>
> Not really, it's in the comment to the event. The hotplug will be
> done by the virtio-net device code that activates the feature, in
> virtio_net_set_features().
>
> >
> > > Signed-off-by: Jens Freimann <address@hidden>
> > > Acked-by: Cornelia Huck <address@hidden>
> > > ---
> > > qapi/net.json | 19 +++++++++++++++++++
> > > 1 file changed, 19 insertions(+)
> > >
> > > diff --git a/qapi/net.json b/qapi/net.json
> > > index 728990f4fb..ea6eeee4f7 100644
> > > --- a/qapi/net.json
> > > +++ b/qapi/net.json
> > > @@ -737,3 +737,22 @@
> > > ##
> > > { 'command': 'announce-self', 'boxed': true,
> > > 'data' : 'AnnounceParameters'}
> > > +
> > > +##
> > > +# @FAILOVER_NEGOTIATED:
> > > +#
> > > +# Emitted when VIRTIO_NET_F_STANDBY was enabled during feature
> > > negotiation.
> > > +# Failover primary devices which were hidden (not hotplugged when
> > > requested)
> > > +# before will now be hotplugged by the virtio-net standby device.
> > > +#
> > > +# device-id: QEMU device id of the unplugged device
> >
> > @device-id is new since v5.
> >
> > A quick skim of
> > https://www.kernel.org/doc/html/latest/networking/net_failover.html
> > tells me there are three devices involved: master, primary slave,
> > standby slave. Which one is @device-id? Or am I confused?
>
> Yes, the device-id is new and it's the device-id of the standby (i.e.
> virtio-net) device.
>
> regards,
> Jens
And now I am confused.
How is standby "the unplugged device"?
Why not just say "the standby device"?
--
MST
- [PATCH v6 01/11] qdev/qbus: add hidden device support, (continued)
- [PATCH v6 01/11] qdev/qbus: add hidden device support, Jens Freimann, 2019/10/25
- [PATCH v6 02/11] pci: add option for net failover, Jens Freimann, 2019/10/25
- [PATCH v6 03/11] pci: mark devices partially unplugged, Jens Freimann, 2019/10/25
- [PATCH v6 04/11] pci: mark device having guest unplug request pending, Jens Freimann, 2019/10/25
- [PATCH v6 05/11] qapi: add unplug primary event, Jens Freimann, 2019/10/25
- [PATCH v6 06/11] qapi: add failover negotiated event, Jens Freimann, 2019/10/25
[PATCH v6 07/11] migration: allow unplug during migration for failover devices, Jens Freimann, 2019/10/25
[PATCH v6 08/11] migration: add new migration state wait-unplug, Jens Freimann, 2019/10/25
[PATCH v6 09/11] libqos: tolerate wait-unplug migration state, Jens Freimann, 2019/10/25
[PATCH v6 10/11] net/virtio: add failover support, Jens Freimann, 2019/10/25
[PATCH v6 11/11] vfio: unplug failover primary device before migration, Jens Freimann, 2019/10/25
Re: [PATCH v6 0/11] add failover feature for assigned network devices, Roman Kagan, 2019/10/25