* Re: [edk2-devel] [PATCH] IntelFsp2Pkg: Support FSP private temporary memory.
[not found] <16574B3F98009B5A.29174@groups.io>
@ 2021-01-07 2:52 ` Chiu, Chasel
0 siblings, 0 replies; only message in thread
From: Chiu, Chasel @ 2021-01-07 2:52 UTC (permalink / raw)
To: devel@edk2.groups.io, Chiu, Chasel
Cc: Ma, Maurice, Desimone, Nathaniel L, Zeng, Star
Patch pushed: 379e5098a4836b21b893b4801baa34b570c24599
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chiu, Chasel
> Sent: Tuesday, January 5, 2021 5:15 PM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Ma, Maurice
> <maurice.ma@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [edk2-devel] [PATCH] IntelFsp2Pkg: Support FSP private temporary
> memory.
>
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3153
>
> When FSP sharing same bootloader stack, the bootloader provided temporary
> memory for FSP will be totally given to PeiCore as Heap, but in some cases FSP
> may have to reserve some more temporary memory for private usage.
>
> This commit adds this flexibility for FSP to reserve some temporary memory
> before giving them to PeiCore.
>
> Cc: Maurice Ma <maurice.ma@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
> ---
> IntelFsp2Pkg/FspSecCore/SecMain.c | 11 +++++++----
> IntelFsp2Pkg/FspSecCore/FspSecCoreM.inf | 3 ++-
> IntelFsp2Pkg/IntelFsp2Pkg.dec | 7 ++++++-
> 3 files changed, 15 insertions(+), 6 deletions(-)
>
> diff --git a/IntelFsp2Pkg/FspSecCore/SecMain.c
> b/IntelFsp2Pkg/FspSecCore/SecMain.c
> index c8de52e1d5..540571187a 100644
> --- a/IntelFsp2Pkg/FspSecCore/SecMain.c
> +++ b/IntelFsp2Pkg/FspSecCore/SecMain.c
> @@ -1,6 +1,6 @@
> /** @file - Copyright (c) 2014 - 2020, Intel Corporation. All rights
> reserved.<BR>+ Copyright (c) 2014 - 2021, Intel Corporation. All rights
> reserved.<BR> SPDX-License-Identifier: BSD-2-Clause-Patent **/@@ -150,15
> +150,18 @@ SecStartup (
> SecCoreData.BootFirmwareVolumeBase = BootFirmwareVolume;
> SecCoreData.BootFirmwareVolumeSize =
> (UINT32)((EFI_FIRMWARE_VOLUME_HEADER *)BootFirmwareVolume)-
> >FvLength; - SecCoreData.TemporaryRamBase = (VOID*)(UINTN)
> TempRamBase;+ //+ // Support FSP reserved temporary memory from the
> whole temporary memory provided by bootloader.+ // FSP reserved temporary
> memory will not be given to PeiCore.+ //+ SecCoreData.TemporaryRamBase
> = (UINT8 *)(UINTN) TempRamBase + PcdGet32
> (PcdFspPrivateTemporaryRamSize);+ SecCoreData.TemporaryRamSize =
> SizeOfRam - PcdGet32 (PcdFspPrivateTemporaryRamSize); if (PcdGet8
> (PcdFspHeapSizePercentage) == 0) {- SecCoreData.TemporaryRamSize =
> SizeOfRam; // stack size that is going to be copied to the permanent memory
> SecCoreData.PeiTemporaryRamBase = SecCoreData.TemporaryRamBase;
> SecCoreData.PeiTemporaryRamSize = SecCoreData.TemporaryRamSize;
> SecCoreData.StackBase = (VOID *)GetFspEntryStack(); // Share the same
> boot loader stack SecCoreData.StackSize = 0; } else {-
> SecCoreData.TemporaryRamSize = SizeOfRam;
> SecCoreData.PeiTemporaryRamBase = SecCoreData.TemporaryRamBase;
> SecCoreData.PeiTemporaryRamSize = SecCoreData.TemporaryRamSize *
> PcdGet8 (PcdFspHeapSizePercentage) / 100; SecCoreData.StackBase =
> (VOID*)(UINTN)((UINTN)SecCoreData.TemporaryRamBase +
> SecCoreData.PeiTemporaryRamSize);diff --git
> a/IntelFsp2Pkg/FspSecCore/FspSecCoreM.inf
> b/IntelFsp2Pkg/FspSecCore/FspSecCoreM.inf
> index 61b7ddca4c..7b05cae641 100644
> --- a/IntelFsp2Pkg/FspSecCore/FspSecCoreM.inf
> +++ b/IntelFsp2Pkg/FspSecCore/FspSecCoreM.inf
> @@ -1,7 +1,7 @@
> ## @file # Sec Core for FSP #-# Copyright (c) 2016 - 2019, Intel Corporation. All
> rights reserved.<BR>+# Copyright (c) 2016 - 2021, Intel Corporation. All rights
> reserved.<BR> # # SPDX-License-Identifier: BSD-2-Clause-Patent #@@ -59,6
> +59,7 @@
> gIntelFsp2PkgTokenSpaceGuid.PcdFspTemporaryRamSize ## CONSUMES
> gIntelFsp2PkgTokenSpaceGuid.PcdFspHeapSizePercentage ## CONSUMES
> gIntelFsp2PkgTokenSpaceGuid.PcdFspMaxInterruptSupported ##
> CONSUMES+ gIntelFsp2PkgTokenSpaceGuid.PcdFspPrivateTemporaryRamSize
> ## CONSUMES [Ppis] gEfiTemporaryRamSupportPpiGuid ##
> PRODUCESdiff --git a/IntelFsp2Pkg/IntelFsp2Pkg.dec
> b/IntelFsp2Pkg/IntelFsp2Pkg.dec
> index 60d40ab6b9..ec7b9a7702 100644
> --- a/IntelFsp2Pkg/IntelFsp2Pkg.dec
> +++ b/IntelFsp2Pkg/IntelFsp2Pkg.dec
> @@ -1,7 +1,7 @@
> ## @file # Provides driver and definitions to build fsp in EDKII bios. #-#
> Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.<BR>+#
> Copyright (c) 2014 - 2021, Intel Corporation. All rights reserved.<BR> # SPDX-
> License-Identifier: BSD-2-Clause-Patent # ##@@ -98,6 +98,11 @@
> # Maximal Interrupt supported in IDT table. #
> gIntelFsp2PkgTokenSpaceGuid.PcdFspMaxInterruptSupported | 34|
> UINT8|0x10000005+ #+ # Allows FSP-M to reserve a section of Temporary
> RAM for implementation specific use.+ # Reduces the amount of memory
> available for the PeiCore heap.+ #+
> gIntelFsp2PkgTokenSpaceGuid.PcdFspPrivateTemporaryRamSize
> |0x00000000|UINT32|0x10000006
> [PcdsFixedAtBuild,PcdsDynamic,PcdsDynamicEx]
> gIntelFsp2PkgTokenSpaceGuid.PcdFspReservedMemoryLength
> |0x00100000|UINT32|0x46530000--
> 2.28.0.windows.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#69685): https://edk2.groups.io/g/devel/message/69685
> Mute This Topic: https://groups.io/mt/79446867/1777047
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [chasel.chiu@intel.com] -=-
> =-=-=-=-=
>
^ permalink raw reply [flat|nested] only message in thread