[ovs-dev] [ovs-dev,v2,2/3] Add skeleton for OF1.6 support.

Ben Pfaff blp at ovn.org
Mon Apr 18 15:48:19 UTC 2016


On Sun, Apr 17, 2016 at 05:21:53PM -0500, Ryan Moats wrote:
> 
> > --- Original Message ---
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> > ---
> >  FAQ.md                             | 33 +++++++++------
> >  build-aux/extract-ofp-errors       |  3 +-
> >  build-aux/extract-ofp-msgs         |  3 +-
> >  include/openflow/openflow-common.h |  5 ++-
> >  lib/ofp-actions.c                  |  1 +
> >  lib/ofp-errors.c                   |  4 +-
> >  lib/ofp-msgs.c                     |  5 +++
> >  lib/ofp-print.c                    |  4 ++
> >  lib/ofp-util.c                     | 82 ++++++++++++++++++++++++++++++
> +------
> >  lib/ofp-util.h                     |  9 ++--
> 
> I believe lib/ofp-util.h needs to be changed to include/openflow/ofp-util.h

Thanks.  ("git rebase" is so freakishly smart that it figured this out
on its own, though.)

> >  tests/ofp-errors.at                |  2 +
> >  tests/ofproto-macros.at            |  3 +-
> >  tests/ofproto.at                   | 84 ++++++++++++++++++
> +-------------------
> >  tests/ovs-ofctl.at                 | 18 ++++----
> >  vswitchd/vswitch.ovsschema         |  7 ++--
> >  vswitchd/vswitch.xml               |  5 ++-
> >  16 files changed, 177 insertions(+), 91 deletions(-)
> 
> Given the above modification,
> 
> Acked-by: Ryan Moats <rmoats at us.ibm.com>

Thanks for the reviews.  I'll apply these in a minute.



More information about the dev mailing list