ArmVirtPkg/PrePi: remove global variable allocation from lowlevel init

Now that we dropped all ArmPlatformGlobalVariableLib dependencies,
there is no longer a need to allocate and clear out the global
variable region in the PrePi init code. So remove it.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18991 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 2 insertions, 31 deletions

diff --git a/ArmVirtPkg/PrePi/PrePi.c b/ArmVirtPkg/PrePi/PrePi.c
index fe7612cec7..c69cff249e 100755
--- a/ArmVirtPkg/PrePi/PrePi.c
+++ b/ArmVirtPkg/PrePi/PrePi.c
@@ -25,14 +25,10 @@
 #include <Ppi/GuidedSectionExtraction.h>

 #include <Ppi/ArmMpCoreInfo.h>

 #include <Guid/LzmaDecompress.h>

-#include <Guid/ArmGlobalVariableHob.h>

 

 #include "PrePi.h"

 #include "LzmaDecompress.h"

 

-// Not used when PrePi in run in XIP mode

-UINTN mGlobalVariableBase = 0;

-

 EFI_STATUS

 EFIAPI

 ExtractGuidedSectionLibConstructor (

@@ -45,23 +41,6 @@ LzmaDecompressLibConstructor (
   VOID

   );

 

-VOID

-EFIAPI

-BuildGlobalVariableHob (

-  IN EFI_PHYSICAL_ADDRESS         GlobalVariableBase,

-  IN UINT32                       GlobalVariableSize

-  )

-{

-  ARM_HOB_GLOBAL_VARIABLE  *Hob;

-

-  Hob = CreateHob (EFI_HOB_TYPE_GUID_EXTENSION, sizeof (ARM_HOB_GLOBAL_VARIABLE));

-  ASSERT(Hob != NULL);

-

-  CopyGuid (&(Hob->Header.Name), &gArmGlobalVariableGuid);

-  Hob->GlobalVariableBase = GlobalVariableBase;

-  Hob->GlobalVariableSize = GlobalVariableSize;

-}

-

 EFI_STATUS

 GetPlatformPpi (

   IN  EFI_GUID  *PpiGuid,

@@ -90,7 +69,6 @@ VOID
 PrePiMain (

   IN  UINTN                     UefiMemoryBase,

   IN  UINTN                     StacksBase,

-  IN  UINTN                     GlobalVariableBase,

   IN  UINT64                    StartTimeStamp

   )

 {

@@ -133,9 +111,6 @@ PrePiMain (
   StacksSize = PcdGet32 (PcdCPUCorePrimaryStackSize);

   BuildStackHob (StacksBase, StacksSize);

 

-  // Declare the Global Variable HOB

-  BuildGlobalVariableHob (GlobalVariableBase, FixedPcdGet32 (PcdPeiGlobalVariableSize));

-

   //TODO: Call CpuPei as a library

   BuildCpuHob (PcdGet8 (PcdPrePiCpuMemorySize), PcdGet8 (PcdPrePiCpuIoSize));

 

@@ -174,8 +149,7 @@ VOID
 CEntryPoint (

   IN  UINTN                     MpId,

   IN  UINTN                     UefiMemoryBase,

-  IN  UINTN                     StacksBase,

-  IN  UINTN                     GlobalVariableBase

+  IN  UINTN                     StacksBase

   )

 {

   UINT64   StartTimeStamp;

@@ -199,10 +173,7 @@ CEntryPoint (
   // Enable Instruction Caches on all cores.

   ArmEnableInstructionCache ();

 

-  // Define the Global Variable region

-  mGlobalVariableBase = GlobalVariableBase;

-

-  PrePiMain (UefiMemoryBase, StacksBase, GlobalVariableBase, StartTimeStamp);

+  PrePiMain (UefiMemoryBase, StacksBase, StartTimeStamp);

 

   // DXE Core should always load and never return

   ASSERT (FALSE);