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

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

2012-11-18 Thread Simon Horman
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 seemed broken as an attempt would be made to set NXM as the flow dump forma