[ovs-dev] [PATCH 0/9] Drop most OVS patch ports from OVN

Ben Pfaff blp at ovn.org
Mon Dec 5 07:17:37 UTC 2016


This series takes advantage of William's new "clone" action for OVS to get
rid of most of the OVS patch ports that ovn-controller has used for a while
now.  It points out a bug in that code (see the commit "XXX Fix
segfault...").

Along the way, this series implements something that we've wanted for a
while: to make ovn-controller only implement the datapaths and ports and
flows that are actually relevant on any given hypervisor.  It turns out to
actually be really simple and a nice code cleanup.  See commit
"ovn-controller: Handle only relevant ports and flows." for details.

Ben Pfaff (8):
  ovn-controller: Make indexes more broadly available.
  lport: Be a little more careful building lport index.
  lport: Tolerate null pointers in destroy functions.
  lport: Add index for logical datapaths.
  ovn-controller: Handle only relevant ports and flows.
  pnysical: Factor code out of consider_port_binding().
  XXX Fix segfault in clone action.
  ovn-controller: Drop most uses of OVS patch ports.

William Tu (1):
  ofp-actions: Add clone action.

 include/openvswitch/ofp-actions.h |  23 +++
 lib/ofp-actions.c                 | 101 ++++++++++-
 ofproto/ofproto-dpif-xlate.c      |  21 +++
 ovn/controller/binding.c          |  75 ++++++--
 ovn/controller/binding.h          |   7 +-
 ovn/controller/lflow.c            |  46 +----
 ovn/controller/lflow.h            |   1 -
 ovn/controller/lport.c            |  82 +++++++++
 ovn/controller/lport.h            |  33 +++-
 ovn/controller/ovn-controller.c   |  71 +++-----
 ovn/controller/ovn-controller.h   |  33 ++--
 ovn/controller/patch.c            | 123 +++-----------
 ovn/controller/patch.h            |   6 +-
 ovn/controller/physical.c         | 349 +++++++++++++++++++++++---------------
 ovn/controller/physical.h         |   9 +-
 ovn/controller/pinctrl.c          |  36 ++--
 ovn/controller/pinctrl.h          |   4 +-
 tests/ofproto-dpif.at             |  19 +++
 tests/ovn-controller.at           |  50 +-----
 tests/system-traffic.at           |  29 ++++
 20 files changed, 698 insertions(+), 420 deletions(-)

-- 
2.10.2



More information about the dev mailing list