Thanks. I rebased and applied this series to master.
On Mon, Sep 13, 2021 at 02:13:28PM -0400, Mark Michelson wrote: > 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