OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

virtio-dev message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: Re: [virtio-dev] [PATCH] content: Introduce VIRTIO_NET_F_STANDBY feature


On Mon, 23 Jul 2018 12:22:38 +0300
"Michael S. Tsirkin" <mst@redhat.com> wrote:

> On Mon, Jul 23, 2018 at 11:03:11AM +0200, Cornelia Huck wrote:
> > On Sun, 22 Jul 2018 18:37:42 +0300
> > "Michael S. Tsirkin" <mst@redhat.com> wrote:
> >   
> > > On Fri, Jul 20, 2018 at 09:44:50AM +0200, Cornelia Huck wrote:  
> > > > On Thu, 19 Jul 2018 14:29:40 -0700
> > > > Sridhar Samudrala <sridhar.samudrala@intel.com> wrote:

> > > > > +
> > > > > +If the driver negotiates VIRTIO_NET_F_STANDBY, it should act as a standby    
> > > > for    
> > > > > +another device with the same MAC address when available. The hypervisor can
> > > > > +hot-plug a primary device with same MAC address if the feature is successfully
> > > > > +negotiated with the driver.    
> > > > 
> > > > I don't think you should add implementation details like hotplugging
> > > > into the spec.
> > > > 
> > > > What about:
> > > > 
> > > > "If the driver negotiates VIRTIO_NET_F_STANDBY, the device MAY act as a
> > > > standby device for another device with the same MAC address, the
> > > > 'failover device'.    
> > > 
> > > I find the name "failover device" confusing. Linux came up with
> > > names primary and standby.  
> > 
> > "...the device MAY act as a standby device for a primary device with
> > the same MAC address."
> > 
> > ?  
> 
> Right. But I agree we need a section that defines failover operation.

The basics for VIRTIO_NET_F_STANDBY are probably the most pressing
right now, we can still fill in the detailed description when we have
figured out whether the suggested approach actually works (it seems
there haven't been any QEMU patches implementing the visibility
handling yet?)


[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]