[ovs-dev] [ofp-print 00/18] OpenFlow and extensions printing and parsing.

Ben Pfaff blp at nicira.com
Thu Dec 9 00:26:52 UTC 2010


OVS has really fallen behind in terms of what actions and messages it
supports versus what it can print and parse.  This series fixes that
up.  Most importantly:

	* It makes NXST_FLOW replies printed properly, which means
	  that "ovs-ofctl dump-flows" does what everyone expects without
	  "-F openflow10".

	* It adds support for printing and parsing register actions.

Ben Pfaff (18):
  ofp-print: Verify size of OFPAT_ENQUEUE actions.
  ofp-util: Use ofpbuf_use_const() in a few more places.
  Use ofpbuf_pull() instead of ofpbuf_try_pull() where it is valid.
  ofp-print: Improve OFPST_FLOW stats reply printing.
  ofp-print: Print NXST_FLOW replies.
  ofp-print: Print durations more readably.
  ovs-ofctl: Fix del-flows command parsing bugs.
  ofp-print: Print OFPUTIL_NXT_ROLE_REQUEST and OFPUTIL_NXT_ROLE_REPLY.
  ofp-print: Print OFPUTIL_NXT_STATUS_REQUEST and
    OFPUTIL_NXT_STATUS_REPLY.
  ofp-print: Print NXST_FLOW_REQUEST and NXST_AGGREGATE_REQUEST.
  ofp-print: Print OFPUTIL_NXST_AGGREGATE_REPLY.
  ofp-util: Fix byte order of ofputil_cls_rule_from_match() parameter.
  ofp-print: Print OFPUTIL_NXT_FLOW_REMOVED.
  ofp-print: Print vendor errors.
  nicira-ext: Remove unused macro NICIRA_OUI_STR.
  Make compiler complain about unhandled OpenFlow and Nicira action
    types.
  ofp-util: Group everything related to actions together in header
    file.
  ofp-print, ofp-parse: Add support for NXAST_REG_MOVE and
    NXAST_REG_LOAD.

 include/openflow/nicira-ext.h |    2 -
 lib/nx-match.c                |  180 ++++++++++++++++-
 lib/nx-match.h                |    7 +
 lib/ofp-parse.c               |   12 +-
 lib/ofp-print.c               |  431 +++++++++++++++++++++++++++--------------
 lib/ofp-util.c                |  120 +++++++++---
 lib/ofp-util.h                |   31 +++-
 ofproto/ofproto.c             |   24 +--
 tests/ofproto.at              |   23 ++-
 tests/ovs-ofctl.at            |   12 +-
 utilities/ovs-ofctl.8.in      |   15 ++
 utilities/ovs-ofctl.c         |    3 +-
 12 files changed, 655 insertions(+), 205 deletions(-)





More information about the dev mailing list