> -----Original Message----- > From: David Christensen <d...@linux.vnet.ibm.com> > Sent: Tuesday, September 28, 2021 8:53 AM > To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Wang, Yipeng1 > <yipeng1.w...@intel.com>; Marohn, Byron <byron.mar...@intel.com> > Cc: dev@dpdk.org; Mcnamara, John <john.mcnam...@intel.com> > Subject: Re: [PATCH v4] efd: change data type of parameter > > > > 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: 56b6ef874f8 ("efd: new Elastic Flow Distributor library") > > > > Signed-off-by: Pablo de Lara <pablo.de.lara.gua...@intel.com> > > Acked-by: John McNamara <john.mcnam...@intel.com> > > --- > > > > v4: Set socket id in EFD tests > > > > v3: Fixed commit message > > > > v2: Fixed EFD tests > [Wang, Yipeng] Acked-by: Yipeng Wang <yipeng1.w...@intel.com>
Thanks Pablo!