[edk2-devel] [PATCH] StandaloneMmPkg/StandaloneMmMemLib: Update AARCH64 address size

[Milosek, Damian]

-----Original Message-----
From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Ard Biesheuvel
Sent: Monday, January 23, 2023 6:46 PM
To: Milosek, Damian <damian.milosek at intel.com>
Cc: devel at edk2.groups.io; sami.mujawar at arm.com; Yao, Jiewen <jiewen.yao at intel.com>
Subject: Re: [edk2-devel] [PATCH] StandaloneMmPkg/StandaloneMmMemLib: Update AARCH64 address size

On Mon, 23 Jan 2023 at 17:14, Milosek, Damian <damian.milosek at intel.com> wrote:
>
>
>
> -----Original Message-----
> From: Ard Biesheuvel <ardb at kernel.org>
> Sent: Monday, January 23, 2023 5:05 PM
> To: Milosek, Damian <damian.milosek at intel.com>
> Cc: devel at edk2.groups.io; sami.mujawar at arm.com; Yao, Jiewen 
> <jiewen.yao at intel.com>
> Subject: Re: [PATCH] StandaloneMmPkg/StandaloneMmMemLib: Update 
> AARCH64 address size
>
> On Mon, 23 Jan 2023 at 16:51, Milosek, Damian <damian.milosek at intel.com> wrote:
> >
> >
> >
> >
> >
> > From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of 
> > Milosek, Damian
> > Sent: Monday, January 16, 2023 2:51 PM
> > To: devel at edk2.groups.io
> > Cc: Milosek, Damian <damian.milosek at intel.com>
> > Subject: [edk2-devel] [PATCH] StandaloneMmPkg/StandaloneMmMemLib:
> > Update AARCH64 address size
> >
> >
> >
> > Enable StandaloneMmMemLib to use longer physical addresses than 36 bits.
> >
>
> What is the purpose of this change?
>
> - Making this Lib working for newer systems that use longer addresses.
>

What is the symptom? How is it failing currently?

- Failure occurs in the first check in MmIsBufferOutsideMmValid () function when using longer address.

> > According to ARM "Learn the architecture - AArch64 memory management"
> >
> > since Armv8.2-A physical address size was extended to 52 bits.
> >
> > https://developer.arm.com/documentation/101811/0102/Address-spaces
> >
>
> This is an optional extension. Which hardware are you using that implements this extension?
>
> - System with Armv9-A Neoverse CPU cores.
>

Which system? What does the memory map look like?

- It is not productized yet so cannot give any more info. However it seems that specific system is not relevant here.
  Current definition assumes max 36 bits for AARCH64 which is not the case anymore.




---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Spolka oswiadcza, ze posiada status duzego przedsiebiorcy w rozumieniu ustawy z dnia 8 marca 2013 r. o przeciwdzialaniu nadmiernym opoznieniom w transakcjach handlowych.

Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#98975): https://edk2.groups.io/g/devel/message/98975
Mute This Topic: https://groups.io/mt/96320111/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-