mbox series

[ovs-dev,v2,ovn,0/5] Reduce number of flows in IN_IP_INPUT table for DNAT.

Message ID 20200630214107.26800.21305.stgit@dceara.remote.csb
Headers show
Series Reduce number of flows in IN_IP_INPUT table for DNAT. | expand

Message

Dumitru Ceara June 30, 2020, 9:41 p.m. UTC
Patch 1 documents and refactors the usage of OVS registers in logical
flows.

Patches 2-4 refactor the ARP/NS responder code for logical routers in
order to make it easier for patch 5 to configure the flows with different
priorities depending on logical port type.

Suggested-by: Han Zhou <hzhou@ovn.org>
Reported-by: Girish Moodalbail <gmoodalbail@gmail.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-June/050186.html
Signed-off-by: Dumitru Ceara <dceara@redhat.com>

Dumitru Ceara (5):
      ovn-northd: Document OVS register usage in logical flows.
      ovn-northd: Store ETH address of router inport in xreg0.
      ovn-northd: Refactor ARP/NS responder in router pipeline.
      ovn-northd: Refactor NAT address parsing.
      ovn-northd: Minimize number of ARP/NS responder flows for DNAT.


 northd/ovn-northd.8.xml |   46 ++-
 northd/ovn-northd.c     |  733 +++++++++++++++++++++++++++++------------------
 tests/ovn-northd.at     |  149 ++++++++++
 tests/ovn.at            |    8 -
 4 files changed, 634 insertions(+), 302 deletions(-)


---
v2:
- Addressed Numan's comments:
  - Inserted a new patch in the beginning of the series to document
    OVS register usage in logical flows. Also refactored the code
    to avoid using bare register names.
  - Added unit tests to ovn-northd.at in every patch that changed
    logical flows.
  - Added/updated documentation in ovn-northd.8.xml in every patch that
    changed logical flows.