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

Simon Horman horms at verge.net.au
Sun Oct 14 00:43:40 UTC 2012


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>



More information about the dev mailing list