Re: [edk2-devel] [PATCH edk2-platforms v1 1/1] Platform/ARM: Reduce System Memory Size for FVP with RME extensions
Merged as ab9805e0020b..310d31231d69 Regards, Sami Mujawar -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103295): https://edk2.groups.io/g/devel/message/103295 Mute This Topic: https://groups.io/mt/98319017/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 1/1] Platform/ARM: Reduce System Memory Size for FVP with RME extensions
Hi Leif, Thank you for reviewing the patch and for the feedback. Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 18/04/2023 02:30 pm, Leif Lindholm wrote: Hi Sami, On Mon, Apr 17, 2023 at 14:22:49 +0100, Sami Mujawar wrote: For older FVPs (without support for RME extension) the top 16MB of DRAM1 is reserved as Trusted DRAM. However, the latest FVP Base RevC AEM Model [1] has support for RME extension. When RME extension is present the top 64MB of DRAM1 (i.e. at the top of the 32bit address space) is carved out for four-world support in TF-A [2]. Therefore, reduce the System Memory size by 64MB. This is clearly needed in order to not break things, so Reviewed-by: Leif Lindholm *but* this is really just extending the life of a horrible hack. Ideally, the ARM ltd. platforms should lead the way in providing reference implementations - since inevitably people will look to those for creating new platform ports. Things like this should be runtime discoverable. [SAMI] TF-A has this information in the platform DT, which we should be able to consume and setup the system memory size dynamically. However, it will be good to get this information from TF-A using the firmware handoff protocol (https://developer.arm.com/documentation/den0135/a). I will check with the TF-A team to understand when they plan to implement this feature. I will merge this patch for now. Once we have the firmware handoff protocol implemented in TF-A, we can add corresponding support in edk2. [/SAMI] / Leif Reference: [1] FVP Base RevC AEM Model (available on x86_64 / Arm64 Linux) (https://developer.arm.com/Tools%20and%20Software/ Fixed%20Virtual%20Platforms) [2] commit c872072 (https://review.trustedfirmware.org/plugins/gitiles/ TF-A/trusted-firmware-a/+/c8720729726faffc39ec64f3a02440a48c8c305a)) Signed-off-by: Sami Mujawar --- Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc index dc081794cf98a27c667ef85bd27dacd80e9e8bd2..f70a4d52ba06f570e017ab5286f06d87193753e5 100644 --- a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc +++ b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc @@ -107,9 +107,12 @@ [PcdsFixedAtBuild.common] gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000 gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0 - # System Memory (2GB - 16MB of Trusted DRAM at the top of the 32bit address space) + # System Memory + # When RME is supported by the FVP the top 64MB of DRAM1 (i.e. at the top + # of the 32bit address space) is reserved for four-world support in TF-A. + # Therefore, set the default System Memory size to (2GB - 64MB). gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000 - gArmTokenSpaceGuid.PcdSystemMemorySize|0x7F00 + gArmTokenSpaceGuid.PcdSystemMemorySize|0x7C00 # Size of the region used by UEFI in permanent memory (Reserved 64MB) gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x0400 -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103291): https://edk2.groups.io/g/devel/message/103291 Mute This Topic: https://groups.io/mt/98319017/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 1/1] Platform/ARM: Reduce System Memory Size for FVP with RME extensions
Hi Sami, Tested-by: Pierre Gondois On 4/17/23 15:22, Sami Mujawar wrote: For older FVPs (without support for RME extension) the top 16MB of DRAM1 is reserved as Trusted DRAM. However, the latest FVP Base RevC AEM Model [1] has support for RME extension. When RME extension is present the top 64MB of DRAM1 (i.e. at the top of the 32bit address space) is carved out for four-world support in TF-A [2]. Therefore, reduce the System Memory size by 64MB. Reference: [1] FVP Base RevC AEM Model (available on x86_64 / Arm64 Linux) (https://developer.arm.com/Tools%20and%20Software/ Fixed%20Virtual%20Platforms) [2] commit c872072 (https://review.trustedfirmware.org/plugins/gitiles/ TF-A/trusted-firmware-a/+/c8720729726faffc39ec64f3a02440a48c8c305a)) Signed-off-by: Sami Mujawar --- Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc index dc081794cf98a27c667ef85bd27dacd80e9e8bd2..f70a4d52ba06f570e017ab5286f06d87193753e5 100644 --- a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc +++ b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc @@ -107,9 +107,12 @@ [PcdsFixedAtBuild.common] gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000 gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0 - # System Memory (2GB - 16MB of Trusted DRAM at the top of the 32bit address space) + # System Memory + # When RME is supported by the FVP the top 64MB of DRAM1 (i.e. at the top + # of the 32bit address space) is reserved for four-world support in TF-A. + # Therefore, set the default System Memory size to (2GB - 64MB). gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000 - gArmTokenSpaceGuid.PcdSystemMemorySize|0x7F00 + gArmTokenSpaceGuid.PcdSystemMemorySize|0x7C00 # Size of the region used by UEFI in permanent memory (Reserved 64MB) gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x0400 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103165): https://edk2.groups.io/g/devel/message/103165 Mute This Topic: https://groups.io/mt/98319017/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH edk2-platforms v1 1/1] Platform/ARM: Reduce System Memory Size for FVP with RME extensions
Hi Sami, On Mon, Apr 17, 2023 at 14:22:49 +0100, Sami Mujawar wrote: > For older FVPs (without support for RME extension) the top 16MB of > DRAM1 is reserved as Trusted DRAM. However, the latest FVP Base RevC > AEM Model [1] has support for RME extension. When RME extension is > present the top 64MB of DRAM1 (i.e. at the top of the 32bit address > space) is carved out for four-world support in TF-A [2]. > > Therefore, reduce the System Memory size by 64MB. This is clearly needed in order to not break things, so Reviewed-by: Leif Lindholm *but* this is really just extending the life of a horrible hack. Ideally, the ARM ltd. platforms should lead the way in providing reference implementations - since inevitably people will look to those for creating new platform ports. Things like this should be runtime discoverable. / Leif > Reference: > [1] FVP Base RevC AEM Model (available on x86_64 / Arm64 Linux) > (https://developer.arm.com/Tools%20and%20Software/ > Fixed%20Virtual%20Platforms) > > [2] commit c872072 (https://review.trustedfirmware.org/plugins/gitiles/ > TF-A/trusted-firmware-a/+/c8720729726faffc39ec64f3a02440a48c8c305a)) > > Signed-off-by: Sami Mujawar > --- > Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc > b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc > index > dc081794cf98a27c667ef85bd27dacd80e9e8bd2..f70a4d52ba06f570e017ab5286f06d87193753e5 > 100644 > --- a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc > +++ b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc > @@ -107,9 +107,12 @@ [PcdsFixedAtBuild.common] >gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000 >gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0 > > - # System Memory (2GB - 16MB of Trusted DRAM at the top of the 32bit > address space) > + # System Memory > + # When RME is supported by the FVP the top 64MB of DRAM1 (i.e. at the top > + # of the 32bit address space) is reserved for four-world support in TF-A. > + # Therefore, set the default System Memory size to (2GB - 64MB). >gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000 > - gArmTokenSpaceGuid.PcdSystemMemorySize|0x7F00 > + gArmTokenSpaceGuid.PcdSystemMemorySize|0x7C00 > ># Size of the region used by UEFI in permanent memory (Reserved 64MB) >gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x0400 > -- > 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103163): https://edk2.groups.io/g/devel/message/103163 Mute This Topic: https://groups.io/mt/98319017/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] [PATCH edk2-platforms v1 1/1] Platform/ARM: Reduce System Memory Size for FVP with RME extensions
For older FVPs (without support for RME extension) the top 16MB of DRAM1 is reserved as Trusted DRAM. However, the latest FVP Base RevC AEM Model [1] has support for RME extension. When RME extension is present the top 64MB of DRAM1 (i.e. at the top of the 32bit address space) is carved out for four-world support in TF-A [2]. Therefore, reduce the System Memory size by 64MB. Reference: [1] FVP Base RevC AEM Model (available on x86_64 / Arm64 Linux) (https://developer.arm.com/Tools%20and%20Software/ Fixed%20Virtual%20Platforms) [2] commit c872072 (https://review.trustedfirmware.org/plugins/gitiles/ TF-A/trusted-firmware-a/+/c8720729726faffc39ec64f3a02440a48c8c305a)) Signed-off-by: Sami Mujawar --- Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc index dc081794cf98a27c667ef85bd27dacd80e9e8bd2..f70a4d52ba06f570e017ab5286f06d87193753e5 100644 --- a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc +++ b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc @@ -107,9 +107,12 @@ [PcdsFixedAtBuild.common] gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000 gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0 - # System Memory (2GB - 16MB of Trusted DRAM at the top of the 32bit address space) + # System Memory + # When RME is supported by the FVP the top 64MB of DRAM1 (i.e. at the top + # of the 32bit address space) is reserved for four-world support in TF-A. + # Therefore, set the default System Memory size to (2GB - 64MB). gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000 - gArmTokenSpaceGuid.PcdSystemMemorySize|0x7F00 + gArmTokenSpaceGuid.PcdSystemMemorySize|0x7C00 # Size of the region used by UEFI in permanent memory (Reserved 64MB) gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x0400 -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103096): https://edk2.groups.io/g/devel/message/103096 Mute This Topic: https://groups.io/mt/98319017/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
