[ovs-dev] [PATCH v6 0/8] Run-Time Open Flow Version Configuration

Simon Horman horms at verge.net.au
Mon Nov 19 05:59:27 UTC 2012


Hi,

This series adds run-time configuration of allowed OpenFlow versions to
ovs-vswtichd, ovs-ofctl and ovs-controller; and adds tests for Open Flow
1.2 messages that utilise the run-time configuration of allowed OpenFlow
versions.

Changes since v5:

* Rebased, including dropping patches from v6 that have been merged

* Reorder, moving "ofp-util: Flow Dump Protocol for OpenFlow 12"
  from near the end to near the begining of the series. This
  seems to be a more appropriate place for it as it is a change
  to core code.

* Revise "vswitchd: Configuration of allowed OpenFlow versions" so
  that an unknown version string is not treated as a fatal error.

Changes are noted in the changelog of each patch.

Git and diffstat information is provided below to aid review.

Ben, please feel free to modify these patches or send me feedback,
whichever is more convenient for you.

----------------------------------------------------------------
The following changes since commit 568e23fc2772235204dcabb61e1425b312070da2:

  CodingStyle: Fix indentation. (2012-11-16 16:08:15 -0800)

are available in the git repository at:

  git://github.com/horms/openvswitch.git devel/of1.2-runtime-switch6

for you to fetch changes up to dfc62e193eabddf55712072c0193fd4b9a1a8272:

  test: add ofproto OpenFlow1.2 tests (2012-11-19 14:35:45 +0900)

----------------------------------------------------------------
Simon Horman (8):
      connmgr: Use version of underlying rconn
      ofp-util: Flow Dump Protocol for OpenFlow 12
      vswitchd: Configuration of allowed OpenFlow versions
      lib: Add helpers for OpenFlow version command line options
      ovs-ofctl: Add option to set allowed OpenFlow versions
      ovs-controller: Allow setting of allowed OpenFlow versions
      ovs-ofctl: By default, do not set flow format for OpenFlow 1.1+
      test: add ofproto OpenFlow1.2 tests

 lib/automake.mk               |    3 +
 lib/ofp-util.c                |   50 +++-
 lib/ofp-util.h                |    7 +-
 lib/ofp-version-opt.c         |   41 +++
 lib/ofp-version-opt.h         |   28 ++
 lib/ofp-version.man           |   26 ++
 manpages.mk                   |    2 +
 ofproto/connmgr.c             |   18 +-
 tests/ofproto-macros.at       |    2 +-
 tests/ofproto.at              |  595 ++++++++++++++++++++++++++++++++++++++---
 tests/ovs-vsctl.at            |    2 +
 utilities/ovs-controller.8.in |    1 +
 utilities/ovs-controller.c    |   20 +-
 utilities/ovs-ofctl.8.in      |    1 +
 utilities/ovs-ofctl.c         |   96 ++++---
 vswitchd/bridge.c             |   15 +-
 vswitchd/vswitch.ovsschema    |    8 +-
 vswitchd/vswitch.xml          |    7 +
 18 files changed, 821 insertions(+), 101 deletions(-)
 create mode 100644 lib/ofp-version-opt.c
 create mode 100644 lib/ofp-version-opt.h
 create mode 100644 lib/ofp-version.man



More information about the dev mailing list