Revision: 18995
          http://sourceforge.net/p/edk2/code/18995
Author:   abiesheuvel
Date:     2015-11-27 17:07:43 +0000 (Fri, 27 Nov 2015)
Log Message:
-----------
ArmPlatformPkg/Sec: 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 Sec init code. So remove it.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <[email protected]>
Reviewed-by: Leif Lindholm <[email protected]>

Modified Paths:
--------------
    trunk/edk2/ArmPlatformPkg/Sec/AArch64/SecEntryPoint.S
    trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.S
    trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.asm
    trunk/edk2/ArmPlatformPkg/Sec/Sec.inf

Modified: trunk/edk2/ArmPlatformPkg/Sec/AArch64/SecEntryPoint.S
===================================================================
--- trunk/edk2/ArmPlatformPkg/Sec/AArch64/SecEntryPoint.S       2015-11-27 
17:07:31 UTC (rev 18994)
+++ trunk/edk2/ArmPlatformPkg/Sec/AArch64/SecEntryPoint.S       2015-11-27 
17:07:43 UTC (rev 18995)
@@ -99,11 +99,7 @@
   LoadConstantToReg (FixedPcdGet32(PcdCPUCoreSecPrimaryStackSize), x2)
   add   x1, x1, x2
 
-  LoadConstantToReg (FixedPcdGet32(PcdSecGlobalVariableSize), x2)
-
-  // The reserved space for global variable must be 8-bytes aligned for pushing
-  // 64-bit variable on the stack
-  SetPrimaryStack (x1, x2, x3, x4)
+  mov   sp, x1
   b     _PrepareArguments
 
 _SetupSecondaryCoreStack:

Modified: trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.S
===================================================================
--- trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.S   2015-11-27 17:07:31 UTC 
(rev 18994)
+++ trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.S   2015-11-27 17:07:43 UTC 
(rev 18995)
@@ -81,11 +81,7 @@
   LoadConstantToReg (FixedPcdGet32(PcdCPUCoreSecPrimaryStackSize), r2)
   add   r1, r1, r2
 
-  LoadConstantToReg (FixedPcdGet32(PcdSecGlobalVariableSize), r2)
-
-  // The reserved space for global variable must be 8-bytes aligned for pushing
-  // 64-bit variable on the stack
-  SetPrimaryStack (r1, r2, r3)
+  mov   sp, r1
   b     _PrepareArguments
 
 _SetupSecondaryCoreStack:

Modified: trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.asm
===================================================================
--- trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.asm 2015-11-27 17:07:31 UTC 
(rev 18994)
+++ trunk/edk2/ArmPlatformPkg/Sec/Arm/SecEntryPoint.asm 2015-11-27 17:07:43 UTC 
(rev 18995)
@@ -83,11 +83,7 @@
   LoadConstantToReg (FixedPcdGet32(PcdCPUCoreSecPrimaryStackSize), r2)
   add   r1, r1, r2
 
-  LoadConstantToReg (FixedPcdGet32(PcdSecGlobalVariableSize), r2)
-
-  // The reserved space for global variable must be 8-bytes aligned for pushing
-  // 64-bit variable on the stack
-  SetPrimaryStack (r1, r2, r3)
+  mov   sp, r1
   b     _PrepareArguments
 
 _SetupSecondaryCoreStack

Modified: trunk/edk2/ArmPlatformPkg/Sec/Sec.inf
===================================================================
--- trunk/edk2/ArmPlatformPkg/Sec/Sec.inf       2015-11-27 17:07:31 UTC (rev 
18994)
+++ trunk/edk2/ArmPlatformPkg/Sec/Sec.inf       2015-11-27 17:07:43 UTC (rev 
18995)
@@ -81,7 +81,5 @@
   gArmTokenSpaceGuid.PcdGicDistributorBase
   gArmTokenSpaceGuid.PcdGicInterruptInterfaceBase
 
-  gArmPlatformTokenSpaceGuid.PcdSecGlobalVariableSize
-
 [FixedPcd.ARM]
   gArmTokenSpaceGuid.PcdArmNsacr


------------------------------------------------------------------------------
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to