> -----Original Message----- > From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: 04 September 2015 15:42 > To: Wei Liu; Paul Durrant > Cc: xen-de...@lists.xenproject.org; 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.durr...@citrix.com> > > > Cc: Ian Campbell <ian.campb...@citrix.com> > > > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > > > Cc: Jan Beulich <jbeul...@suse.com> > > > Cc: Keir Fraser <k...@xen.org> > > > Cc: Tim Deegan <t...@xen.org> > > > > Acked-by: Wei Liu <wei.l...@citrix.com> > > 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@lists.xen.org http://lists.xen.org/xen-devel
