Hi > -----Original Message----- > From: David Christensen <[email protected]> > Sent: Tuesday, September 28, 2021 4:53 PM > To: De Lara Guarch, Pablo <[email protected]>; Wang, Yipeng1 > <[email protected]>; Marohn, Byron <[email protected]> > Cc: [email protected]; Mcnamara, John <[email protected]> > 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 <[email protected]> > > Acked-by: John McNamara <[email protected]> > > --- > > > > v4: Set socket id in EFD tests > > > > v3: Fixed commit message > > > > v2: Fixed EFD tests > > Results with v4 on a non-consecutive NUMA system:
... > Test OK Great! Thanks a lot for checking. Would you mind adding tested-by to the patch? Pablo
