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

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

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.


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