Pushed to master with Co-authored-by relationship, Jarno
On Oct 6, 2014, at 3:54 PM, Ben Pfaff <b...@nicira.com> wrote: > On Fri, Oct 03, 2014 at 01:22:07PM -0700, Jarno Rajahalme wrote: >> Replace bitwise loops with a single operation, inline all bitmap >> functions. Inlining allows the compiler to remove unnecessary code >> due to some parameters being compile-time constants. >> >> Before: >> >> $ tests/ovstest test-bitmap benchmark 1000000 >> bitmap equal: 341 ms >> bitmap scan: 8089 ms >> >> After: >> >> $ tests/ovstest test-bitmap benchmark 1000000 >> bitmap equal: 152 ms >> bitmap scan: 146 ms >> >> Signed-off-by: Kmindg <kmi...@gmail.com> >> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > I can't argue with the results. > > I think you made significant changes to Kmindg's original patch. > You should probably either add a note saying how you changed it, e.g. > > Signed-off-by: Kmindg <kmi...@gmail.com> > [jrajaha...@nicira.com updated this to frob the quux] > Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com> > > or just change it to a co-author relationship. > > Acked-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
