On Fri, Jan 24, 2014 at 2:58 PM, Jarno Rajahalme <jrajaha...@nicira.com> wrote: > Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > --- > datapath/datapath.c | 59 > +++++++++++++++++++++++++---------------------- > datapath/flow.h | 2 ++ > datapath/flow_netlink.c | 10 +++++--- > 3 files changed, 41 insertions(+), 30 deletions(-)
Do you have separate performance numbers for this patch? The other patches cited reducing lock contention as the main benefit but there is no contention here (actually it would be good to break out performance numbers in general across this series). Reducing the use of ksize() might also be considered an optimization in its own right. It seems like it might be a good idea to use a wrapper function when freeing the actions so that all paths are guaranteed to handle different allocation types correctly. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
