[ovs-dev] [PATCH 18/18] multicast snooping: Update documentation

Flavio Leitner fbl at redhat.com
Sat May 24 01:09:28 UTC 2014


On Tue, May 20, 2014 at 04:33:45PM -0700, Ben Pfaff wrote:
> On Tue, May 20, 2014 at 04:32:48PM -0700, Ben Pfaff wrote:
> > On Fri, Apr 11, 2014 at 06:34:23PM -0300, Flavio Leitner wrote:
> > > Acked-by: Thomas Graf <tgraf at redhat.com>
> > > Acked-by: Daniel Borkmann <dborkman at redhat.com>
> > > Signed-off-by: Flavio Leitner <fbl at redhat.com>
> > 
> > Thanks for adding examples to the ovs-vsctl manpage!
> > 
> > I see two spelling errors in vswitch.xml: s/packers/packets/,
> > s/inconditionally/unconditionally/.
> > 
> > Please squash this with the commit that enables the features that it
> > documents.
> > 
> > I don't recall seeing a NEWS item among the patches in the series.  If
> > there isn't one already, please add one.
> 
> I think I've reviewed the whole series now, if not please let me know
> which patches I missed.

Yes, you did and I will fix the patchset accordingly.

> Thanks a lot for sending this our way, by the way.  I think it is
> nicely done.
 
Thanks a lot for your throughout review.
I appreciate it
fbl



More information about the dev mailing list