Dnia wtorek, 9 lipca 2024 15:00:12 CEST Leif Lindholm via groups.io pisze: > On Tue, Jul 09, 2024 at 12:47:10 +0200, Marcin Juszkiewicz wrote: > > Function AddPpttTable() adding PPTT got too long. This change moves part > > of it into helper function AddCoresToPpttTable() which takes care of > > generating entries for Core and below (Cache, Thread). > > > > Signed-off-by: Marcin Juszkiewicz <marcin.juszkiew...@linaro.org> > > --- > > > > .../Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c | 237 > > +++++++++++--------- 1 file changed, 133 insertions(+), 104 deletions(-) > > > > diff --git > > a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c > > b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c index > > a7a9664abdcb..a4b2ee2fdcb0 100644 > > --- a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c > > +++ b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c > > @@ -29,6 +29,9 @@ > > > > static UINTN GicItsBase; > > > > +static UINTN CpuId; > > +static UINTN CacheId; > > + > > static variables are supposed to have g (global) or m (module) prefix. > This is local, so m. > (Yes, that means I missed that when reviewing the GitIts bits.) > > Also, why are these in a #pragma pack(1) block?
Added right after GicItsBase. Moved out of block. > > #pragma pack () > STATIC > > > +UINT32 > > +AddCoresToPpttTable ( > > + UINT8 *New, > > + UINT32 ClusterIndex, > > + CpuTopology CpuTopo > > + ) done > > - ClusterIndex = CoreIndex; > > + CoresPartSize = AddCoresToPpttTable (New, ClusterIndex, CpuTopo); > > + ClusterIndex += CoresPartSize; > > This sounds like ClisterIndex is no longer an Index after this patch. > Should it be renamed? It is still an Index. CoresPartSize is a size taken by Core/Cache/Thread part of this Cluster. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119843): https://edk2.groups.io/g/devel/message/119843 Mute This Topic: https://groups.io/mt/107120147/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-