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 <[email protected]> Reported-by: Girish Moodalbail <[email protected]> Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-June/050186.html Signed-off-by: Dumitru Ceara <[email protected]> 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 | 829 +++++++++++++++++++++++++++++------------------ tests/ovn-northd.at | 149 ++++++++ tests/ovn.at | 8 4 files changed, 684 insertions(+), 348 deletions(-) --- v4: - Addressed Han's comments: - Patch 1: mentioned pipeline stages where R0 and R1 are used. - Patch 5: simplified drop flows for DNAT ARP resolver (prio 91). - Patch 5: run the loop for DNAT ARP resolver flow generation twice, once for SNAT, once for DNAT. - Added Han's acks on patches 2-4. - Patch 4: Coding style: fix function definition. v3: - Addressed Han's comment: - fixed register diagram in patch 1 and 2. - added Han's ack on patch 1. 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. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
