On Tue, Sep 24, 2013 at 04:41:43PM -0700, Justin Pettit wrote:
> On Sep 24, 2013, at 2:11 PM, Ben Pfaff <b...@nicira.com> wrote:
> 
> > On Mon, Sep 23, 2013 at 02:36:39PM -0700, Justin Pettit wrote:
> >> Signed-off-by: Justin Pettit <jpet...@nicira.com>
> > 
> > Should the tcpdump patch check for the old and the new OpenFlow port?
> > 
> > There are many remaining hits for 6633 in the tree.  I'd suggest
> > updating at least lib/vconn-active.man, lib/vconn-passive.man,
> > utilities/ovs-controller.8.in, and vswitchd/vswitch.xml to mention
> > that the default port number will change.  (Please don't update
> > lib/aes128.c despite the three hits ;-)
> > 
> > Acked-by: Ben Pfaff <b...@nicira.com>
> 
> Okay, how about the incremental at the end of this message?

Looks good, thanks.

I see one use of "when this form is used", which seems unnecessary in
context.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to