[ovs-dev] [PATCH 0/4] Openflow 1.2 role request/reply support, 1.3 testing

Jarno Rajahalme jarno.rajahalme at nsn.com
Fri Dec 7 13:48:20 UTC 2012


This series adds placeholders for the unimplemented OF 1.1+ queue set/get
config messages, adds OF 1.2 role request/reply support, enables OF 1.3 testing
for ofproto, and updates and adds test cases to use OF 1.2 role and OF 1.3 set
async messaging.

Since OF 1.2 role enum is almost the same as nx_role, but adds a pseudo-role
for a non-changing role request, I decided to map the OF enum to nx_role.  This
minimises changes, and also removes any hassle of using enum
ofp12_controller_role as the role variable inside connmgr, since the
"nochange" value cannot be used as an actual role.

The printing of the default role is changed from "other" to "equal" to be
compatible with Openflow 1.2 wording.  This change did not break any existing
test cases.

The Openflow 1.3 version of the asynchronous message control test drops the
separate invalid_ttl_to_controller check, as that flag has been deprecated in
Openflow 1.3.  The new set async config message can be used instead.

Jarno Rajahalme (4):
  Openflow 1.1 queue set/get config placeholders.
  Add Openflow 1.2 role request/reply processing, update OF 1.2 tests.
  Add OpenFlow13 to the allowed protocols for ofproto testing.
  Add Openflow 1.3 test for asynchronous message control.

 lib/learning-switch.c   |    2 +
 lib/ofp-msgs.h          |   34 ++++++++---
 lib/ofp-print.c         |   45 ++++++++++++---
 lib/rconn.c             |    2 +
 ofproto/connmgr.c       |   25 ++++++++
 ofproto/connmgr.h       |    1 +
 ofproto/ofproto.c       |   64 ++++++++++++++++++---
 tests/ofp-print.at      |   30 ++++++++++
 tests/ofproto-macros.at |    2 +-
 tests/ofproto.at        |  147 +++++++++++++++++++++++++++++++++++++++++++++--
 10 files changed, 321 insertions(+), 31 deletions(-)

-- 
1.7.10.4




More information about the dev mailing list