[AMD Official Use Only - General] Hi Morten,
> -----Original Message----- > From: Morten Brørup <m...@smartsharesystems.com> > Sent: Wednesday, December 20, 2023 12:58 PM > To: Tummala, Sivaprasad <sivaprasad.tumm...@amd.com>; > david.march...@redhat.com; ktray...@redhat.com; tho...@monjalon.net; > konstantin.anan...@huawei.com; konstantin.v.anan...@yandex.ru; > bruce.richard...@intel.com; maxime.coque...@redhat.com; > acon...@redhat.com; Yigit, Ferruh <ferruh.yi...@amd.com> > Cc: dev@dpdk.org > 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:sivaprasad.tumm...@amd.com] > > 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/ > > 20230925151027.558546-1-sivaprasad.tumm...@amd.com/ > > > > As agreed earlier in mailing list, this patch adjusts the limit for > > specific AMD EPYC target/cross builds. > > > > Signed-off-by: Sivaprasad Tummala <sivaprasad.tumm...@amd.com> > > --- > > [...] > > > + 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 <m...@smartsharesystems.com>
