Re: [dpdk-dev] [PATCH] common: introduce an integer log2 function

2017-07-03 Thread Thomas Monjalon
06/04/2017 16:15, Olivier Matz: > At some places, the log2() function is used despite this function > works on float. This introduces a dependency to the math lib but > most of the time it is not required because we want an integer log2. > > Add a new helper to do this job and fix 2 drivers. Only

Re: [dpdk-dev] [PATCH] common: introduce an integer log2 function

2017-07-03 Thread Alejandro Lucero
Hi, Sorry for the late answer. Yes, that's fine. Thank you On Sat, Jul 1, 2017 at 3:37 PM, Thomas Monjalon wrote: > Alejandro, > > Are you OK to replace log2 by rte_log2_u32? > I'm waiting your ack to apply this patch. > > > 06/04/2017 16:23, Olivier MATZ: > > I forgot to CC Alejandro for n

Re: [dpdk-dev] [PATCH] common: introduce an integer log2 function

2017-07-01 Thread Thomas Monjalon
Alejandro, Are you OK to replace log2 by rte_log2_u32? I'm waiting your ack to apply this patch. 06/04/2017 16:23, Olivier MATZ: > I forgot to CC Alejandro for nfp. [...] > > drivers/net/nfp/nfp_net.c | 6 ++ > > lib/librte_eal/common/include/rte_common.h | 17

Re: [dpdk-dev] [PATCH] common: introduce an integer log2 function

2017-04-06 Thread Jerin Jacob
-Original Message- > Date: Thu, 6 Apr 2017 16:23:55 +0200 > From: Olivier MATZ > To: dev@dpdk.org, Jerin Jacob > Cc: alejandro.luc...@netronome.com > Subject: Re: [dpdk-dev] [PATCH] common: introduce an integer log2 function > X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.3

Re: [dpdk-dev] [PATCH] common: introduce an integer log2 function

2017-04-06 Thread Olivier MATZ
I forgot to CC Alejandro for nfp. Also CC Jerin: I think the same could be done in thunderx, but I did not want to do the patch since it changes the base/ driver. On Thu, 6 Apr 2017 16:15:36 +0200, Olivier Matz wrote: > At some places, the log2() function is used despite this function > works

[dpdk-dev] [PATCH] common: introduce an integer log2 function

2017-04-06 Thread Olivier Matz
At some places, the log2() function is used despite this function works on float. This introduces a dependency to the math lib but most of the time it is not required because we want an integer log2. Add a new helper to do this job and fix 2 drivers. Signed-off-by: Olivier Matz --- drivers/net/