Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-05 Thread Ben Pfaff
On Mon, Feb 05, 2018 at 05:40:28PM +0100, Jakub Sitnicki wrote: > On Mon, Feb 05, 2018 at 04:25 PM GMT, Ben Pfaff wrote: > > On Mon, Feb 05, 2018 at 05:03:13PM +0100, Jakub Sitnicki wrote: > >> > >> It caught my attention that the contract with hexit_value() callers has > >> changed. Previously we

Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-05 Thread Jakub Sitnicki
On Mon, Feb 05, 2018 at 04:25 PM GMT, Ben Pfaff wrote: > On Mon, Feb 05, 2018 at 05:03:13PM +0100, Jakub Sitnicki wrote: >> >> It caught my attention that the contract with hexit_value() callers has >> changed. Previously we had a catch-all clause for out-of-range values. >> Now there will be an o

Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-05 Thread Ben Pfaff
On Mon, Feb 05, 2018 at 05:03:13PM +0100, Jakub Sitnicki wrote: > Hi Ben, > > On Fri, Feb 02, 2018 at 11:16 PM GMT, Ben Pfaff wrote: > > Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd > > due to a similar patch. > > > > Reported-by: Daniel Alvarez Sanchez > > Reported

Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-05 Thread Jakub Sitnicki
Hi Ben, On Fri, Feb 02, 2018 at 11:16 PM GMT, Ben Pfaff wrote: > Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd > due to a similar patch. > > Reported-by: Daniel Alvarez Sanchez > Reported-at: > https://mail.openvswitch.org/pipermail/ovs-discuss/2018-February/046120.

Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-03 Thread Daniel Alvarez Sanchez
Thanks for the patch, Ben. Looks good to me. Acked-by: Daniel Alvarez On Sat, Feb 3, 2018 at 12:16 AM, Ben Pfaff wrote: > Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd > due to a similar patch. > > Reported-by: Daniel Alvarez Sanchez > Reported-at: https://mail.o

Re: [ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-02 Thread Yifeng Sun
Looks good to me, thanks. Reviewed-by: Yifeng Sun On Fri, Feb 2, 2018 at 3:16 PM, Ben Pfaff wrote: > Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd > due to a similar patch. > > Reported-by: Daniel Alvarez Sanchez > Reported-at: https://mail.openvswitch.org/piperm

[ovs-dev] [PATCH] util: Use lookup table to optimize hexit_value().

2018-02-02 Thread Ben Pfaff
Daniel Alvarez Sanchez reported a significant overall speedup in ovn-northd due to a similar patch. Reported-by: Daniel Alvarez Sanchez Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-February/046120.html Signed-off-by: Ben Pfaff --- lib/util.c | 39 +--