[ovs-dev] [PATCH v3 0/8] Add actions for egress loopback

Ben Pfaff blp at ovn.org
Sat Jan 21 19:13:17 UTC 2017


This series is fully reviewed except for the new bug fix patch 1.

I believe that, with these patches, egress loopback as proposed by Mickey's
patches can be implemented with:
    clone { inport = outport; outport = ""; flags = 0;
            reg0 = 0; reg1 = 0; ... regN = 0;
            next(pipeline=ingress, table=0); }

v1->v2:
  - Patches 1, 2, and 3 applied and dropped.
  - Acks applied to other patches.
  - Typo fixed in patch 5 (now patch 2).
  - Stupid bugs fixed in patch 7 (now patch 4).

v2->v3:
  - New bug fix patch (patch 1).
  - Bug fixes in last 3 patches.  (Thanks Mickey!)

Ben Pfaff (8):
  ovn-trace: Fix selection of table that "next" jumps to.
  actions: Make "free" functions per-struct, not per-action.
  actions: Add new OVN action "clone".
  actions: Separate action structures for "next" and "ct_next".
  actions: Omit table number when possible for formatting "next" action.
  actions: Introduce enum ovnact_pipeline.
  actions: Make "next" action able to jump from egress to ingress
    pipeline.
  actions: Add new "ct_clear" action.

 include/ovn/actions.h     |  91 ++++++++++------
 ovn/controller/lflow.c    |   7 +-
 ovn/lib/actions.c         | 261 +++++++++++++++++++++++++++-------------------
 ovn/ovn-sb.xml            |  26 ++++-
 ovn/utilities/ovn-trace.c |  97 ++++++++++++-----
 tests/ovn.at              |  37 ++++++-
 tests/test-ovn.c          |   6 +-
 7 files changed, 354 insertions(+), 171 deletions(-)

-- 
2.10.2



More information about the dev mailing list