Thanks for posting this as a separate patch, it makes the review easier. The idea and the patch look good to me.
One comment: with this code we don't need the ctxs array, we can just have a single ctx in the for loop. Other than that: Acked-by: Daniele Di Proietto <[email protected]> 2017-02-21 9:36 GMT-08:00 Flavio Leitner <[email protected]>: > On Thu, Feb 16, 2017 at 12:47:34AM -0800, Darrell Ball wrote: >> Packet batch sorting is removed for three reasons: >> >> 1) The following patches for NAT change the locking >> marshalling so batching loses benefit. >> >> 2) For real mixtures of flows either in hypervisors >> or gateways, the batch sorting won't provide benefit >> and will just be a tax. >> >> 3) Code clarity. >> >> Signed-off-by: Darrell Ball <[email protected]> >> --- > > I can't tell about the real performance impact but I'd agree > with the second point. > > Acked-by: Flavio Leitner <[email protected]> > > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
