On Thu, May 20, 2021 at 10:23:36PM -0700, Ben Pfaff wrote: > On Thu, May 20, 2021 at 04:50:58PM -0400, Mark Michelson wrote: > > Hi Ben, > > > > I gave these patches another look, and I can say that for patches 1-10, and > > patch 27: > > > > Acked-by: Mark Michelson <[email protected]> > > > > For patch 11, I had suggested previously a configure-time check to detect if > > the correct DDLog version was installed, and you replied with an example > > that you would roll into the next version of the series. But I don't see it > > here. Is it in a different patch series? > > > > For patches 12-26, I don't feel qualified to definitively state that it's > > the "best" way to optimize things, but the changes all make sense to me and > > don't seem to be incorrect, so for patches 12-26: > > > > Acked-by: Mark Michelson <[email protected]> > > > > but I wouldn't be averse to others having a look to be certain. > > Thanks. I've now applied the whole series, except the last patch, to > master. > > Somehow I omitted the configure-time check patch. I'll post it again.
It's a 3-patch series now: https://mail.openvswitch.org/pipermail/ovs-dev/2021-May/383261.html https://mail.openvswitch.org/pipermail/ovs-dev/2021-May/383259.html https://mail.openvswitch.org/pipermail/ovs-dev/2021-May/383260.html _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
