On Wed, Aug 07, 2024 at 13:34:38 +0200, Marcin Juszkiewicz wrote: > We want to make sure that CPU topology information given to QEMU would > be provided to the operating system. So we use SMC call to ask TF-A for > amount of sockets, clusters, cores and threads set in QEMU config. > > The TF-A part is already merged: > https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/27189 > > Signed-off-by: Xiong Yining <[email protected]> > Signed-off-by: Marcin Juszkiewicz <[email protected]>
For the series: Reviewed-by: Leif Lindholm <[email protected]> Thanks! / Leif > To: [email protected] > Cc: Leif Lindholm <[email protected]> > Cc: Ard Biesheuvel <[email protected]> > Cc: Graeme Gregory <[email protected]> > Cc: Chen Baozi <[email protected]> > Cc: Xiong Yining <[email protected]> > Cc: Jonathan Cameron <[email protected]> > > Changes in v6: > - use ArmMonitorLib for GetCpuTopology() as we need 5 return values > - Link to v5: > https://openfw.io/edk2-devel/[email protected] > > Changes in v5: > - added support for cache sizes on cores with FEAT_CCIDX (Neoverse V1+) > - Link to v4: > https://openfw.io/edk2-devel/[email protected] > > Changes in v4: > - renamed all *Index variables to *Offset ones for clarity > - renamed static CpuId/CacheId variable to mCpuId/mCacheId > - moved above variables outside of pragma pack > - moved all variables definitions to start of functions > - added reading cpu cache size from CCIDR registers > - changed wording in SbsaHardwareInfoLib header > - changed wording in 3rd patch commit message > - Link to v3: > https://openfw.io/edk2-devel/[email protected] > > Changes in v3: > - split ACPI 6.5 changes into separate patch > - moved adding cores/threads to separate function > - fixed cache offsets > - Link to v2: > https://openfw.io/edk2-devel/[email protected]/T/#t > > Changes in v2 (Marcin Juszkiewicz): > - use ACPI 6.5 structures (instead of 6.3) > - add patch to move cache data to cores (instead of clusters) > - this is for future MPAM support > - reformatted sources using uncrustify > - changed debug output to allow singular values (s/are/:/) > > --- > Marcin Juszkiewicz (5): > SbsaQemu: get the information of CPU topology via SMC calls > SbsaQemu: update PPTT to ACPI 6.5 > SbsaQemu: provide cache info per core in PPTT > SbsaQemu: introduce helper in PPTT generation > SbsaQemu: export proper cache values in PPTT > > Xiong Yining (1): > SbsaQemu: align the PPTT tables with QEMU > > .../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.h | 11 + > .../Include/IndustryStandard/SbsaQemuAcpi.h | 110 +++----- > .../SbsaQemu/Include/IndustryStandard/SbsaQemuSmc.h | 1 + > .../Qemu/SbsaQemu/Include/Library/HardwareInfoLib.h | 26 ++ > .../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c | 274 > +++++++++++++++++--- > .../SbsaQemuHardwareInfoLib.c | 35 +++ > 6 files changed, 341 insertions(+), 116 deletions(-) > --- > base-commit: a3c898956a4d48dc5980336fa6ce6eeb23c4f72b > change-id: 20240702-acpi65-1bfdb20bde1a > > Best regards, > -- > Marcin Juszkiewicz <[email protected]> > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#120283): https://edk2.groups.io/g/devel/message/120283 Mute This Topic: https://groups.io/mt/107767179/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
