[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] public/io/netif.h: document a mechanism to advertise carrier state


  • To: Jürgen Groß <jgross@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: "Durrant, Paul" <pdurrant@xxxxxxxxxx>
  • Date: Fri, 13 Dec 2019 15:33:45 +0000
  • Accept-language: en-GB, en-US
  • Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
  • Delivery-date: Fri, 13 Dec 2019 15:34:02 +0000
  • Ironport-sdr: VFyMQywmkTBDNnX3FPrEJV7St7OaiMhYa8xZi78JTxRC6aGSys0E3yX+MLtmFUUIYu1cDm0H0F 4WvCDStfrvzQ==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHVsbWxxnT/OROzMkWhaLrTzZMz8qe4HJmAgAAVLnA=
  • Thread-topic: [PATCH] public/io/netif.h: document a mechanism to advertise carrier state

> -----Original Message-----
> From: Jürgen Groß <jgross@xxxxxxxx>
> Sent: 13 December 2019 14:17
> To: Durrant, Paul <pdurrant@xxxxxxxxxx>; xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> Subject: Re: [PATCH] public/io/netif.h: document a mechanism to advertise
> carrier state
> 
> On 13.12.19 14:03, Paul Durrant wrote:
> > This patch adds a specification for a 'carrier' node in xenstore to
> allow
> > a backend to notify a frontend of it's virtual carrier/link state. E.g.
> > a backend that is unable to forward packets from the guest because it is
> > not attached to a bridge may wish to advertise 'no carrier'.
> >
> > NOTE: This is purely a documentation patch. No functional change.
> >
> > Signed-off-by: Paul Durrant <pdurrant@xxxxxxxxxx>
> > ---
> > Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> > Cc: Juergen Gross <jgross@xxxxxxxx>
> > ---
> >   xen/include/public/io/netif.h | 14 ++++++++++++++
> >   1 file changed, 14 insertions(+)
> >
> > diff --git a/xen/include/public/io/netif.h
> b/xen/include/public/io/netif.h
> > index 2454448baa..e587055f68 100644
> > --- a/xen/include/public/io/netif.h
> > +++ b/xen/include/public/io/netif.h
> > @@ -190,6 +190,20 @@
> >    * order as requests.
> >    */
> >
> > +/*
> > + * Link state
> > + * ==========
> > + *
> > + * The backend can advertise it is current link (carrier) state to the
> 
> s/it is/its/ ?
> 

Oh yes.

> > + * frontend using the /local/domain/X/backend/<domid>/<vif>/carrier
> node.
> 
> Hmm, I just realized that the other mentioned backend path in this file
> is wrong, it should be: /local/domain/X/backend/vif/<domid>/<vif>/...
> 
> Mind correcting that in your patch, too?
> 

Sure.

  Paul
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.