Re: [ovs-dev] [PATCH 3/4] ofproto: Define official OpenFlow port number.

2013-10-01 Thread Justin Pettit
On Sep 24, 2013, at 9:09 PM, Ben Pfaff wrote: > On Tue, Sep 24, 2013 at 04:41:43PM -0700, Justin Pettit wrote: >> On Sep 24, 2013, at 2:11 PM, Ben Pfaff wrote: >> >>> On Mon, Sep 23, 2013 at 02:36:39PM -0700, Justin Pettit wrote: Signed-off-by: Justin Pettit >>> >>> Should the tcpdump p

Re: [ovs-dev] [PATCH 3/4] ofproto: Define official OpenFlow port number.

2013-09-24 Thread Ben Pfaff
On Tue, Sep 24, 2013 at 04:41:43PM -0700, Justin Pettit wrote: > On Sep 24, 2013, at 2:11 PM, Ben Pfaff wrote: > > > On Mon, Sep 23, 2013 at 02:36:39PM -0700, Justin Pettit wrote: > >> Signed-off-by: Justin Pettit > > > > Should the tcpdump patch check for the old and the new OpenFlow port? > >

Re: [ovs-dev] [PATCH 3/4] ofproto: Define official OpenFlow port number.

2013-09-24 Thread Justin Pettit
On Sep 24, 2013, at 2:11 PM, Ben Pfaff wrote: > On Mon, Sep 23, 2013 at 02:36:39PM -0700, Justin Pettit wrote: >> Signed-off-by: Justin Pettit > > 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 > updati

Re: [ovs-dev] [PATCH 3/4] ofproto: Define official OpenFlow port number.

2013-09-24 Thread Ben Pfaff
On Mon, Sep 23, 2013 at 02:36:39PM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit 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, util

[ovs-dev] [PATCH 3/4] ofproto: Define official OpenFlow port number.

2013-09-23 Thread Justin Pettit
Signed-off-by: Justin Pettit --- include/openflow/openflow-common.h |3 ++- lib/stream-ssl.c |6 +++--- lib/vconn-stream.c |4 ++-- lib/vconn.c|8 ofproto/connmgr.c |2 +- third-party/ofp-tcpdu