From: Numan Siddique <num...@ovn.org>

This patch series optimizes ovn-controller to add only relevant
datapaths to its "local datapaths" map if the logical topology
has a single flat provider logical switch connected to multiple
routers with a distributed gateway port.

(Patch 2 has a detailed commit message.)

v3 -> v4
-------
  * Addressed review comments in patch 2 from Mark.
  * Added Han's and Mark's Acked-by in patch 1.

v2 -> v3
------
  * There are many changes in v3 compared to v2.
  * Patch 1 and 2 of v2 are dropped completely.
  * Some of the changes of patch 3 of v2 were
    split into separate patch series which is merged.
  * In this version, we don't rely on the flag 'only_dgp_peer_ports'
    which was used in v2.  Instead I used the approach suggested by Han,
    which makes more sense and simpler. In v2, while handling the patch port
    binding changes incrementally, it was removing a datapath from the
    local_datapaths map if it was no longer relevant.  But now to make
    the code simpler,  it is not handled.  A recompute (whenever is
    triggered next) will not add those datapaths anymore.


v1 -> v2
-----
  * Rebased to resolve the conflicts.

Numan Siddique (2):
  controller: Expand the helper lport_get_peer().
  controller: Optimize adding 'dps' to the local datapaths.

 controller/binding.c    |  35 ++--
 controller/local_data.c | 110 ++++++------
 controller/local_data.h |   6 +-
 controller/lport.c      |  10 ++
 controller/lport.h      |   3 +
 tests/ovn.at            | 375 ++++++++++++++++++++++++++++++++++++++++
 tests/system-ovn.at     |  28 +++
 7 files changed, 493 insertions(+), 74 deletions(-)

-- 
2.49.0

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to