Hi Ben, for the series: Acked-by: Mark Michelson <mmich...@redhat.com>
I had a question on patch 1, but it's minor and otherwise shouldn't block an ACK.
On 9/7/21 6:45 PM, Ben Pfaff wrote:
With and without these patches, I see the following performance when I run the load-balancer heavy benchmark. The measurements include cold start with all the load balancers, then deleting the sctp load balancer and waiting for it to finish, then the same with the tcp load balancer, then the same with the udp load balancer. The measurements only include ddlog time; there is additional overhead talking to the database, but that's a constant factor. Without patches: 783 CPU seconds, 23.5 GB RAM, 13:06 elapsed time. With patches: 224 CPU seconds, 14.3 GB RAM, 3:45 elapsed time. I am continuing to work on performance. These patches are also available in: https://github.com/blp/ovs-reviews/tree/ovn-memory-8 v1->v2: - Add two new patches. - Rebase and rerun benchmarks (revisions reported above). Ben Pfaff (9): ovn-northd-ddlog: Make joins for ARP/ND flows slightly more efficient. ovn-northd-ddlog: Derive load balancer IP addresses in new LoadBalancer. ovn-northd-ddlog: Reverse order of joins for connection tracking flows. ovn-northd-ddlog: Avoid re-parsing LB IP addresses and ports. ovn-northd-ddlog: Simplify LBVIPWithStatus to include up_backends string. ovn-northd-ddlog: Avoid storing unused 'lbs' field in Router. ovn-northd-ddlog: Intern strings before joining when possible. ovn-northd-ddlog: Avoid map(ival) for ARP flows. ovn-northd-ddlog: Avoid unnecessary joins for SwitchPortARPForwards. Leonid Ryzhyk (1): ovn-northd-ddlog: Intern all strings in OVSDB tables. configure.ac | 2 +- manpages.mk | 1 - northd/copp.dl | 32 +- northd/helpers.dl | 14 +- northd/ipam.dl | 17 +- northd/lrouter.dl | 213 +++-- northd/lswitch.dl | 177 ++-- northd/multicast.dl | 44 +- northd/ovn-nb.dlopts | 1 + northd/ovn-sb.dlopts | 1 + northd/ovn.dl | 7 + northd/ovn_northd.dl | 2021 ++++++++++++++++++++---------------------- northd/ovsdb2ddlog2c | 6 +- tests/ovn-ic.at | 8 +- tests/ovn-northd.at | 20 +- tests/ovn.at | 6 +- 16 files changed, 1280 insertions(+), 1290 deletions(-)
_______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev