[ovs-dev] [PATCH] ofproto: Use OFPERR_OFPFMFC_TABLE_FULL for OpenFlow 1.1+

Ben Pfaff blp at nicira.com
Sun Oct 14 01:40:50 UTC 2012


On Sun, Oct 14, 2012 at 09:43:40AM +0900, Simon Horman wrote:
> On Fri, Oct 12, 2012 at 10:16:11AM -0700, Ben Pfaff wrote:
> > On Fri, Oct 12, 2012 at 10:00:38AM -0700, Ben Pfaff wrote:
> > > On Fri, Oct 12, 2012 at 09:26:58AM +0900, Simon Horman wrote:
> > > > Use OFPERR_OFPFMFC_ALL_TABLES_FULL in place of
> > > > OFPERR_OFPFMFC_TABLE_FULL for OpenFlow 1.1+.
> > > > 
> > > > OFPERR_OFPFMFC_ALL_TABLES_FULL only exists in OpenFlow 1.0.
> > > > OFPERR_OFPFMFC_TABLE_FULL was added in OpenFlow 1.1.
> > > > 
> > > > Signed-off-by: Simon Horman <horms at verge.net.au>
> > > 
> > > These two codes represent essentially the same condition so I think
> > > the following would be a better change.  What you think?
> > 
> > I forgot to run the testsuite after making my change.  Here's an
> > updated commit.
> 
> Thanks, I think this approach is better than the one I took.
> 
> Acked-by: Simon Horman <horms at verge.net.au>

Thanks.  I pushed this to master.



More information about the dev mailing list