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 <ho...@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 <ho...@verge.net.au>

Thanks.  I pushed this to master.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to