Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-12-03 Thread Ben Pfaff
On Tue, Dec 03, 2013 at 01:15:20PM +0900, Simon Horman wrote: > On Mon, Dec 02, 2013 at 02:34:12PM -0800, Ben Pfaff wrote: > > From: Simon Horman > > > > Add elements to enum ofp_table_config for table mod bits > > defined in OpenFlow1.4. > > > > Also tidy up the indentation of existing elements

Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-12-02 Thread Simon Horman
On Mon, Dec 02, 2013 at 02:34:57PM -0800, Ben Pfaff wrote: > On Tue, Nov 26, 2013 at 04:34:23PM +0900, Simon Horman wrote: > > Add elements to enum ofp_table_config for table mod bits > > defined in OpenFlow1.4. > > > > Also tidy up the indentation of existing elements. > > > > Signed-off-by: Sim

Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-12-02 Thread Simon Horman
On Mon, Dec 02, 2013 at 02:34:12PM -0800, Ben Pfaff wrote: > From: Simon Horman > > Add elements to enum ofp_table_config for table mod bits > defined in OpenFlow1.4. > > Also tidy up the indentation of existing elements. > > Signed-off-by: Simon Horman > Co-authored-by: Ben Pfaff > Signed-of

Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-12-02 Thread Ben Pfaff
On Tue, Nov 26, 2013 at 04:34:23PM +0900, Simon Horman wrote: > Add elements to enum ofp_table_config for table mod bits > defined in OpenFlow1.4. > > Also tidy up the indentation of existing elements. > > Signed-off-by: Simon Horman I sent out my own version of this. Want to look at it?

[ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-12-02 Thread Ben Pfaff
From: Simon Horman Add elements to enum ofp_table_config for table mod bits defined in OpenFlow1.4. Also tidy up the indentation of existing elements. Signed-off-by: Simon Horman Co-authored-by: Ben Pfaff Signed-off-by: Ben Pfaff --- include/openflow/openflow-1.1.h| 12 i

Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-11-27 Thread Jarno Rajahalme
> On Nov 26, 2013, at 9:23 PM, Simon Horman wrote: > > Ben previously indicated that he would like to stick with > the OFPTC_TABLE_MISS_MASK name. With that in mind perhaps it > would be best to put a comment above OFPTC_TABLE_MISS_MASK noting > that its has a different name but the same value i

Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-11-26 Thread Simon Horman
On Tue, Nov 26, 2013 at 01:47:06PM -0800, Jarno Rajahalme wrote: > > On Nov 25, 2013, at 11:34 PM, Simon Horman wrote: > > > Add elements to enum ofp_table_config for table mod bits > > defined in OpenFlow1.4. > > > > Also tidy up the indentation of existing elements. > > > > Signed-off-by: Si

Re: [ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-11-26 Thread Jarno Rajahalme
On Nov 25, 2013, at 11:34 PM, Simon Horman wrote: > Add elements to enum ofp_table_config for table mod bits > defined in OpenFlow1.4. > > Also tidy up the indentation of existing elements. > > Signed-off-by: Simon Horman > --- > include/openflow/openflow-common.h | 15 +++ > 1 fil

[ovs-dev] [PATCH] ofproto: Add OF1.4 elements to enum ofp_table_config

2013-11-25 Thread Simon Horman
Add elements to enum ofp_table_config for table mod bits defined in OpenFlow1.4. Also tidy up the indentation of existing elements. Signed-off-by: Simon Horman --- include/openflow/openflow-common.h | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/include/ope