[AMD Official Use Only - General] Hi Morten,
> -----Original Message----- > From: Morten Brørup <[email protected]> > Sent: Wednesday, December 20, 2023 12:58 PM > To: Tummala, Sivaprasad <[email protected]>; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; Yigit, Ferruh <[email protected]> > Cc: [email protected] > Subject: RE: [PATCH] config/x86: config support for AMD EPYC processors > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > > From: Sivaprasad Tummala [mailto:[email protected]] > > Sent: Wednesday, 20 December 2023 08.11 > > > > On x86 platforms, max lcores are limited to 128 by default. > > > > On AMD EPYC processors, this limit was adjusted for native builds in > > the previous patch. > > https://patches.dpdk.org/project/dpdk/patch/ > > [email protected]/ > > > > As agreed earlier in mailing list, this patch adjusts the limit for > > specific AMD EPYC target/cross builds. > > > > Signed-off-by: Sivaprasad Tummala <[email protected]> > > --- > > [...] > > > + foreach m:epyc_zen_cores.keys() > > + if m.contains(cpu_instruction_set) > > + dpdk_conf.set('RTE_MAX_LCORE', epyc_zen_cores[m]) > > + break > > The indentation of "break" uses a mix of tab and spaces, and should be fixed. Thanks for spotting this. Will fix this in v2. > > Acked-by: Morten Brørup <[email protected]>
