[ovs-dev] [PATCH] ofproto: Use OFPERR_OFPFMFC_TABLE_FULL for OpenFlow 1.1+
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