Re: [ovs-dev] [PATCH] openflow-1.1+: move OFPTC11_TABLE_MISS_MASK out of enum

2013-09-11 Thread Ben Pfaff
Thanks, applied as follows (at end). On Wed, Sep 11, 2013 at 11:47:49AM -0700, Andy Zhou wrote: > This looks good to me. > > > On Wed, Sep 11, 2013 at 10:28 AM, Ben Pfaff wrote: > > > On Tue, Sep 10, 2013 at 01:33:08PM -0700, Andy Zhou wrote: > > > OFPTC11_TABLE_MISS_MASK is not a valid config

Re: [ovs-dev] [PATCH] openflow-1.1+: move OFPTC11_TABLE_MISS_MASK out of enum

2013-09-11 Thread Andy Zhou
This looks good to me. On Wed, Sep 11, 2013 at 10:28 AM, Ben Pfaff wrote: > On Tue, Sep 10, 2013 at 01:33:08PM -0700, Andy Zhou wrote: > > OFPTC11_TABLE_MISS_MASK is not a valid configuration, but to indicate > > there are only 2 bits being used for table miss configuration. Move > > it out of

Re: [ovs-dev] [PATCH] openflow-1.1+: move OFPTC11_TABLE_MISS_MASK out of enum

2013-09-11 Thread Ben Pfaff
On Tue, Sep 10, 2013 at 01:33:08PM -0700, Andy Zhou wrote: > OFPTC11_TABLE_MISS_MASK is not a valid configuration, but to indicate > there are only 2 bits being used for table miss configuration. Move > it out of the enum definition. > > Reported-by: Simon Horman > Signed-off-by: Andy Zhou I ag

[ovs-dev] [PATCH] openflow-1.1+: move OFPTC11_TABLE_MISS_MASK out of enum

2013-09-10 Thread Andy Zhou
OFPTC11_TABLE_MISS_MASK is not a valid configuration, but to indicate there are only 2 bits being used for table miss configuration. Move it out of the enum definition. Reported-by: Simon Horman Signed-off-by: Andy Zhou --- include/openflow/openflow-1.1.h |3 ++- 1 file changed, 2 insertion