Re: [ovs-dev] [PATCH 2/8] ofp-util: Flow Dump Protocol for OpenFlow 12

2012-11-19 Thread Ben Pfaff
On Tue, Nov 20, 2012 at 10:50:39AM +0900, Simon Horman wrote: > On Mon, Nov 19, 2012 at 05:11:21PM -0800, Ben Pfaff wrote: > > On Tue, Nov 20, 2012 at 09:39:56AM +0900, Simon Horman wrote: > > > On Mon, Nov 19, 2012 at 10:13:22AM -0800, Ben Pfaff wrote: > > > > On Mon, Nov 19, 2012 at 02:59:29PM +0

Re: [ovs-dev] [PATCH 2/8] ofp-util: Flow Dump Protocol for OpenFlow 12

2012-11-19 Thread Simon Horman
On Mon, Nov 19, 2012 at 05:11:21PM -0800, Ben Pfaff wrote: > On Tue, Nov 20, 2012 at 09:39:56AM +0900, Simon Horman wrote: > > On Mon, Nov 19, 2012 at 10:13:22AM -0800, Ben Pfaff wrote: > > > On Mon, Nov 19, 2012 at 02:59:29PM +0900, Simon Horman wrote: > > > > Allow only OpenFlow 12 as a flow dump

Re: [ovs-dev] [PATCH 2/8] ofp-util: Flow Dump Protocol for OpenFlow 12

2012-11-19 Thread Ben Pfaff
On Tue, Nov 20, 2012 at 09:39:56AM +0900, Simon Horman wrote: > On Mon, Nov 19, 2012 at 10:13:22AM -0800, Ben Pfaff wrote: > > On Mon, Nov 19, 2012 at 02:59:29PM +0900, Simon Horman wrote: > > > Allow only OpenFlow 12 as a flow dump protocol for OpenFlow12. > > > Allow OpenFlow10 and NXM as flow du

Re: [ovs-dev] [PATCH 2/8] ofp-util: Flow Dump Protocol for OpenFlow 12

2012-11-19 Thread Simon Horman
On Mon, Nov 19, 2012 at 10:13:22AM -0800, Ben Pfaff wrote: > On Mon, Nov 19, 2012 at 02:59:29PM +0900, Simon Horman wrote: > > Allow only OpenFlow 12 as a flow dump protocol for OpenFlow12. > > Allow OpenFlow10 and NXM as flow dump protocols for other OpenFlow versions. > > This only changes the be

Re: [ovs-dev] [PATCH 2/8] ofp-util: Flow Dump Protocol for OpenFlow 12

2012-11-19 Thread Ben Pfaff
On Mon, Nov 19, 2012 at 02:59:29PM +0900, Simon Horman wrote: > Allow only OpenFlow 12 as a flow dump protocol for OpenFlow12. > Allow OpenFlow10 and NXM as flow dump protocols for other OpenFlow versions. > This only changes the behaviour prior to this patch in the case of OpenFlow12 > which seeme