[ovs-dev] [of1.2 0/5] enable oxm flow format

[Ben Pfaff]

On Fri, Nov 16, 2012 at 07:52:46PM +0900, Simon Horman wrote:
> On Thu, Nov 15, 2012 at 10:44:19PM -0800, Ben Pfaff wrote:
> > See http://openvswitch.org/pipermail/dev/2012-November/022990.html
> > for background.
> > 
> > Ben Pfaff (5):
> >   ovs-ofctl: Style fix.
> >   ofp-util: Rename OFPUTIL_P_* constants and update comments for
> >     clarity.
> >   ofp-util: Allow use of OpenFlow 12 flow format
> >   ofp-util: Make ofputil_encode_set_protocol() able to return failure.
> >   nicira-ext: Remove NXFF_OPENFLOW12.
> > 
> >  include/openflow/nicira-ext.h |    3 +-
> >  lib/learning-switch.c         |   20 +++--
> >  lib/ofp-msgs.h                |    2 +-
> >  lib/ofp-util.c                |  191 +++++++++++++++++++++--------------------
> >  lib/ofp-util.h                |   52 +++++++----
> >  ofproto/connmgr.c             |    4 +-
> >  tests/learn.at                |    2 +-
> >  tests/ovs-ofctl.at            |   12 +--
> >  utilities/ovs-ofctl.c         |    5 +-
> >  9 files changed, 159 insertions(+), 132 deletions(-)
> 
> This all seems reasonable to me.
> 
> Reviewed-by: Simon Horman <horms at verge.net.au>

Thanks for the reviews, I pushed these to master.

I'll continue to work along your series as I find time.

Thanks,

Ben.