Re: [ovs-dev] [PATCH 3/4] Add some missing Open Flow 1.1 definitions

2012-03-28 Thread Simon Horman
On Wed, Mar 28, 2012 at 05:20:30PM -0700, Ben Pfaff wrote: > On Thu, Mar 29, 2012 at 09:18:57AM +0900, Simon Horman wrote: > > On Thu, Mar 29, 2012 at 09:06:36AM +0900, Simon Horman wrote: > > > On Wed, Mar 28, 2012 at 12:21:18PM -0700, Ben Pfaff wrote: > > > > On Wed, Mar 28, 2012 at 09:44:16AM +0

Re: [ovs-dev] [PATCH 3/4] Add some missing Open Flow 1.1 definitions

2012-03-28 Thread Ben Pfaff
On Thu, Mar 29, 2012 at 09:18:57AM +0900, Simon Horman wrote: > On Thu, Mar 29, 2012 at 09:06:36AM +0900, Simon Horman wrote: > > On Wed, Mar 28, 2012 at 12:21:18PM -0700, Ben Pfaff wrote: > > > On Wed, Mar 28, 2012 at 09:44:16AM +0900, Simon Horman wrote: > > > > Signed-off-by: Simon Horman > > >

Re: [ovs-dev] [PATCH 3/4] Add some missing Open Flow 1.1 definitions

2012-03-28 Thread Simon Horman
On Thu, Mar 29, 2012 at 09:06:36AM +0900, Simon Horman wrote: > On Wed, Mar 28, 2012 at 12:21:18PM -0700, Ben Pfaff wrote: > > On Wed, Mar 28, 2012 at 09:44:16AM +0900, Simon Horman wrote: > > > Signed-off-by: Simon Horman > > > > The comment here is wrong, since it talks about OXM but OXM was >

Re: [ovs-dev] [PATCH 3/4] Add some missing Open Flow 1.1 definitions

2012-03-28 Thread Simon Horman
On Wed, Mar 28, 2012 at 12:21:18PM -0700, Ben Pfaff wrote: > On Wed, Mar 28, 2012 at 09:44:16AM +0900, Simon Horman wrote: > > Signed-off-by: Simon Horman > > The comment here is wrong, since it talks about OXM but OXM was > introduced in OpenFlow 1.2: > > +/* The match type indicates the match s

Re: [ovs-dev] [PATCH 3/4] Add some missing Open Flow 1.1 definitions

2012-03-28 Thread Ben Pfaff
On Wed, Mar 28, 2012 at 09:44:16AM +0900, Simon Horman wrote: > Signed-off-by: Simon Horman The comment here is wrong, since it talks about OXM but OXM was introduced in OpenFlow 1.2: > +/* The match type indicates the match structure (set of fields that compose > the > + * match) in use. The ma

[ovs-dev] [PATCH 3/4] Add some missing Open Flow 1.1 definitions

2012-03-27 Thread Simon Horman
Signed-off-by: Simon Horman --- include/openflow/openflow-1.1.h | 564 +++ 1 files changed, 564 insertions(+), 0 deletions(-) diff --git a/include/openflow/openflow-1.1.h b/include/openflow/openflow-1.1.h index d02ece3..cdcfbd0 100644 --- a/include/openflow/o