Re: [dpdk-dev] [PATCH v4] efd: change data type of parameter

2021-10-01 Thread Thomas Monjalon
> > > rte_efd_create() function was using uint8_t for a socket bitmask, for > > > one of its parameters. > > > This limits the maximum of NUMA sockets to be 8. > > > Changing to to uint64_t increases it to 64, which should be more > > > future-proof. > > > > > > Coverity issue: 366390 > > > Fixes:

Re: [dpdk-dev] [PATCH v4] efd: change data type of parameter

2021-09-29 Thread Wang, Yipeng1
> -Original Message- > From: David Christensen > Sent: Tuesday, September 28, 2021 8:53 AM > To: De Lara Guarch, Pablo ; Wang, Yipeng1 > ; Marohn, Byron > Cc: dev@dpdk.org; Mcnamara, John > Subject: Re: [PATCH v4] efd: change data type of parameter > > > > On 9/28/21 6:58 AM, Pablo de

Re: [dpdk-dev] [PATCH v4] efd: change data type of parameter

2021-09-29 Thread David Christensen
On 9/29/21 12:51 AM, De Lara Guarch, Pablo wrote: Hi -Original Message- From: David Christensen Sent: Tuesday, September 28, 2021 4:53 PM To: De Lara Guarch, Pablo ; Wang, Yipeng1 ; Marohn, Byron Cc: dev@dpdk.org; Mcnamara, John Subject: Re: [PATCH v4] efd: change data type of par

Re: [dpdk-dev] [PATCH v4] efd: change data type of parameter

2021-09-29 Thread De Lara Guarch, Pablo
Hi > -Original Message- > From: David Christensen > Sent: Tuesday, September 28, 2021 4:53 PM > To: De Lara Guarch, Pablo ; Wang, Yipeng1 > ; Marohn, Byron > Cc: dev@dpdk.org; Mcnamara, John > Subject: Re: [PATCH v4] efd: change data type of parameter > > > > On 9/28/21 6:58 AM, Pabl

Re: [dpdk-dev] [PATCH v4] efd: change data type of parameter

2021-09-28 Thread David Christensen
On 9/28/21 6:58 AM, Pablo de Lara wrote: rte_efd_create() function was using uint8_t for a socket bitmask, for one of its parameters. This limits the maximum of NUMA sockets to be 8. Changing to to uint64_t increases it to 64, which should be more future-proof. Coverity issue: 366390 Fixes: 5

[dpdk-dev] [PATCH v4] efd: change data type of parameter

2021-09-28 Thread Pablo de Lara
rte_efd_create() function was using uint8_t for a socket bitmask, for one of its parameters. This limits the maximum of NUMA sockets to be 8. Changing to to uint64_t increases it to 64, which should be more future-proof. Coverity issue: 366390 Fixes: 56b6ef874f8 ("efd: new Elastic Flow Distributor