[ovs-dev] [PATCH v4 00/10] instruction apply-actions/goto-table support

Isaku Yamahata yamahata at valinux.co.jp
Wed Aug 1 15:24:05 UTC 2012


This patch adds instruction OF11 apply-actions/goto-table supports.

Now the patch is in good shape and unit tests pass.
The first and second patch (1,2/10) allows action encoder to return error.
The changes to caller of the action decoder would be arguable. Especially
the behaviour of monitor.
The last patch (10/10) is just for showing how write-actions will be, and
the action-set support is on my TODO yet.

Changes v3 -> v4:
- patch reorganization
- unit test
- allow encoder to return error

Changes v2 -> v3:
- introduce OFPACT_{CLEAR_ACTIONS, WRITE_ACTIONS, GOTO_TABLE}

Changes v1 -> v2:
- introduce ofpact_instruction to represent instruction delimiter

thanks,

Isaku Yamahata (10):
  ofp-actions: allow ofpacts_put_openflow10() return error
  ofp-actions: allow ofpacts_to_openflow11_{actions, instructions}
    return error
  ofp-actions: export OVSINST_OFPIT11_xxx
  ofp-actions: sort OVSINST_OFPIT11_xxx value in execution order
  ofp-actions/instruction: helper functions for intructions
  instruction: support goto-table action
  ofproto-dpif: refactor rule_dpif_lookup() for table miss whose id > 0
  ofproto-dpif: teach goto-table instruction packet-in
  instruction/clear-actions: string parser/formater, of packet
    decoder/encoder
  instruction/write-actions: add text parser/formater, of packet
    decoder/encoder

 lib/learning-switch.c    |   35 ++++-
 lib/ofp-actions.c        |  330 +++++++++++++++++++++++++++++++++++++---------
 lib/ofp-actions.h        |   87 +++++++++++--
 lib/ofp-parse.c          |  120 ++++++++++++++++--
 lib/ofp-util.c           |   59 +++++++--
 lib/ofp-util.h           |   15 +-
 ofproto/connmgr.c        |   16 ++-
 ofproto/ofproto-dpif.c   |   75 ++++++++---
 ofproto/ofproto.c        |   13 ++-
 tests/ofp-actions.at     |   44 +++++-
 utilities/ovs-ofctl.8.in |   25 ++++
 utilities/ovs-ofctl.c    |   50 ++++++-
 12 files changed, 720 insertions(+), 149 deletions(-)




More information about the dev mailing list