Re: [Xen-devel] [PATCH v2] public/io/netif.h: move and amend multicast control documentation

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: Ian Campbell [mailto:ian.campbell@xxxxxxxxxx]
> Sent: 04 September 2015 15:42
> To: Wei Liu; Paul Durrant
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx; Keir (Xen.org); Tim (Xen.org); Ian
> Jackson; Jan Beulich
> Subject: Re: [Xen-devel] [PATCH v2] public/io/netif.h: move and amend
> multicast control documentation
> 
> On Fri, 2015-09-04 at 14:22 +0100, Wei Liu wrote:
> > On Wed, Sep 02, 2015 at 12:17:05PM +0100, Paul Durrant wrote:
> > > netif.h contains a specification of the
> > > XEN_NETIF_EXTRA_TYPE_MCAST_{ADD,DEL}
> > > extra info messages require to manipulate a multicast filter list
> > > maintained
> > > by a backend and specifies the xenstore negotiation protocol in a
> > > comment
> > > just above the structure defintion, which is easy to miss.
> > >
> > > This patch moves the documentation of the xenstore negotiation to be
> > > co-located with the documentation for other features and also amends
> > > the
> > > wording to be clearer.
> > >
> > > Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
> > > Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > > Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> > > Cc: Jan Beulich <jbeulich@xxxxxxxx>
> > > Cc: Keir Fraser <keir@xxxxxxx>
> > > Cc: Tim Deegan <tim@xxxxxxx>
> >
> > Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> 
> Applied, with:
> 
> > > + * connected state.
> >
> > I would prefer adding a blank line here if possible.
> 
> "\n *" added.

Cool. Thanks,

  Paul
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel