RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Varghese, Vipin
Snipped > > > > > > I recall from the Cache Stashing community call... There is some > > > ACPI > > function to > > > get the (opaque) "location IDs" of various parts in the system, to > > > be > > used for setting > > > the Cache Stashing hints. > > > Is there only one "ACPI location ID" (I don't

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Morten Brørup
> > > > > > Does the API need to be prepared for L4 cache? > > > > > > https://www.anandtech.com/show/16924/did-ibm-just-preview- > the- > > > future > > > > > > - > > > > > of-caches > > > > > Thank you for the pointer, yes initial patch was considering L4 > > > cache > > > > > too. But I was not

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Varghese, Vipin
Snipped > > > > > > > Does the API need to be prepared for L4 cache? > > > > > https://www.anandtech.com/show/16924/did-ibm-just-preview-the- > > future > > > > > - > > > > of-caches > > > > Thank you for the pointer, yes initial patch was considering L4 > > cache > > > > too. But I was not able

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Morten Brørup
> > > > Does the API need to be prepared for L4 cache? > > > > https://www.anandtech.com/show/16924/did-ibm-just-preview-the- > future > > > > - > > > of-caches > > > Thank you for the pointer, yes initial patch was considering L4 > cache > > > too. But I was not able to get hold of system or get s

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Varghese, Vipin
Snipped > > > > Does the API need to be prepared for L4 cache? > > > https://www.anandtech.com/show/16924/did-ibm-just-preview-the-future > > > - > > of-caches > > Thank you for the pointer, yes initial patch was considering L4 cache > > too. But I was not able to get hold of system or get someo

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Morten Brørup
> > Does the API need to be prepared for L4 cache? > > https://www.anandtech.com/show/16924/did-ibm-just-preview-the-future- > of-caches > Thank you for the pointer, yes initial patch was considering L4 cache > too. But I was not able to get hold of system or get someone to test > this with L4. > H

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Varghese, Vipin
Snipped > > Does the API need to be prepared for L4 cache? > https://www.anandtech.com/show/16924/did-ibm-just-preview-the-future-of-caches Thank you for the pointer, yes initial patch was considering L4 cache too. But I was not able to get hold of system or get someone to test this with L4. Hen

Re: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-04 Thread Mattias Rönnblom
On 2024-10-30 16:44, Stephen Hemminger wrote: On Wed, 30 Oct 2024 11:11:31 +0530 Vipin Varghese wrote: + if (topo_cnfg.io) { + free(topo_cnfg.io); + topo_cnfg.io = NULL; + } No need to check for NULL before calling free. If you do need to check for N

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-03 Thread Morten Brørup
> From: Vipin Varghese [mailto:vipin.vargh...@amd.com] > Sent: Wednesday, 30 October 2024 06.42 > > Introduce topology aware lcore mapping into lcore API. > With higher core density, more and more cores are categorized > into various chiplets based on IO (memory and PCIe) and > Last Level Cache (m

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-03 Thread Varghese, Vipin
Snipped > > > +struct topology_config { > > +#ifdef RTE_EAL_HWLOC_TOPOLOGY_PROBE > > + hwloc_topology_t topology; > > +#endif > > + > > + /* domain count */ > > + uint16_t l1_count; > > + uint16_t l2_count; > > + uint8_t l3_count; > > + uint8_t io_count; > > + > > + /*

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-03 Thread Varghese, Vipin
Snipped > > > + /* domain count */ > > + uint16_t l1_count; > > + uint16_t l2_count; > > + uint8_t l3_count; > > + uint8_t io_count; > > Make them all uint16_t, the space is there already. Thank you, fixing this in v4

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-03 Thread Varghese, Vipin
Snipped > > On Wed, 30 Oct 2024 11:11:31 +0530 > Vipin Varghese wrote: > > > + if (topo_cnfg.io) { > > + free(topo_cnfg.io); > > + topo_cnfg.io = NULL; > > + } > > No need to check for NULL before calling free. Thank you Stepehen on this, the reason why we speci

RE: [RFC v3 1/3] eal/lcore: add topology based functions

2024-11-03 Thread Varghese, Vipin
Snipped > > On Wed, 30 Oct 2024 11:11:31 +0530 > Vipin Varghese wrote: > > > topo_cnfg.l3[j] = (struct core_domain_mapping *) > > + malloc(sizeof(struct > > core_domain_mapping)); > > Minor nit, no need to cast return value from malloc

[RFC v3 1/3] eal/lcore: add topology based functions

2024-10-29 Thread Vipin Varghese
Introduce topology aware lcore mapping into lcore API. With higher core density, more and more cores are categorized into various chiplets based on IO (memory and PCIe) and Last Level Cache (mainly L3). Using hwloc library, the dpdk available lcores can be grouped into various groups nameley L1, L