Remove use of global variable at runtime in this lib
authormdkinney <mdkinney@6f19259b-4bc3-4df7-8a09-765794883524>
Sat, 2 Dec 2006 19:09:56 +0000 (19:09 +0000)
committermdkinney <mdkinney@6f19259b-4bc3-4df7-8a09-765794883524>
Sat, 2 Dec 2006 19:09:56 +0000 (19:09 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@2046 6f19259b-4bc3-4df7-8a09-765794883524

EdkModulePkg/Library/EdkDxeSalLib/EdkDxeSalLib.msa
EdkModulePkg/Library/EdkDxeSalLib/Ipf/EsalServiceLib.c

index a7a6214a30e592735816295303a557c47b0670b5..10c3f8cb4fbd39c5d59a8e40c73881a61d5c1988 100644 (file)
@@ -54,6 +54,9 @@
   <Externs>\r
     <Specification>EFI_SPECIFICATION_VERSION 0x00020000</Specification>\r
     <Specification>EDK_RELEASE_VERSION 0x00020000</Specification>\r
+    <Extern>\r
+      <Constructor>DxeSalLibConstructor</Constructor>\r
+    </Extern>\r
     <Extern>\r
       <SetVirtualAddressMapCallBack>DxeSalVirtualNotifyEvent</SetVirtualAddressMapCallBack>\r
     </Extern>\r
index 706dda2873e92a278ce3f5bb668f927e7068c7f9..9eb909dcd30dbd56d3ea209e92fc54ca8f6e7a66 100644 (file)
@@ -19,25 +19,21 @@ Abstract:
 \r
 #include <Ipf/IpfDefines.h>\r
 \r
-BOOLEAN mLibraryInitialized = FALSE;\r
-STATIC EXTENDED_SAL_BOOT_SERVICE_PROTOCOL *mEsalBootService;\r
-STATIC EFI_PLABEL                         mPlabel;\r
+EXTENDED_SAL_BOOT_SERVICE_PROTOCOL  *mEsalBootService = NULL;\r
+EFI_PLABEL                          mPlabel;\r
 \r
 EFI_STATUS\r
 EFIAPI\r
-DxeSalLibConstruct (\r
-//  IN EFI_HANDLE        ImageHandle,\r
-//  IN EFI_SYSTEM_TABLE  *SystemTable\r
+DxeSalLibInitialize (\r
   VOID\r
   )\r
 {\r
   EFI_PLABEL  *Plabel;\r
   EFI_STATUS  Status;\r
 \r
-  if (mLibraryInitialized == TRUE) {\r
+  if (mEsalBootService != NULL) {\r
     return EFI_SUCCESS;\r
   }\r
-  mLibraryInitialized = TRUE;\r
 \r
   //\r
   // The protocol contains a function pointer, which is an indirect procedure call.\r
@@ -47,8 +43,11 @@ DxeSalLibConstruct (
   // virtual). So lets grap the physical PLABEL for the EsalEntryPoint and store it\r
   // away. We cache it in a module global, so we can register the vitrual version.\r
   //\r
-  Status              = gBS->LocateProtocol (&gEfiExtendedSalBootServiceProtocolGuid, NULL, &mEsalBootService);\r
-  ASSERT_EFI_ERROR (Status);\r
+  Status = gBS->LocateProtocol (&gEfiExtendedSalBootServiceProtocolGuid, NULL, &mEsalBootService);\r
+  if (EFI_ERROR (Status)) {\r
+    mEsalBootService = NULL;\r
+    return EFI_SUCCESS;\r
+  }\r
 \r
   Plabel              = (EFI_PLABEL *) (UINTN) mEsalBootService->ExtendedSalProc;\r
 \r
@@ -56,7 +55,17 @@ DxeSalLibConstruct (
   mPlabel.GP          = Plabel->GP;\r
   SetEsalPhysicalEntryPoint (mPlabel.EntryPoint, mPlabel.GP);\r
 \r
-  return Status;\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+EFI_STATUS\r
+EFIAPI\r
+DxeSalLibConstructor (\r
+  IN EFI_HANDLE        ImageHandle,\r
+  IN EFI_SYSTEM_TABLE  *SystemTable\r
+  )\r
+{\r
+  return DxeSalLibInitialize ();\r
 }\r
 \r
 VOID\r
@@ -115,7 +124,7 @@ Returns:
 \r
 --*/\r
 {\r
-  DxeSalLibConstruct ();\r
+  DxeSalLibInitialize ();\r
   return mEsalBootService->AddExtendedSalProc (\r
                             mEsalBootService,\r
                             ClassGuid,\r
@@ -224,12 +233,32 @@ Returns:
   SAL_RETURN_REGS       ReturnReg;\r
   SAL_EXTENDED_SAL_PROC EsalProc;\r
 \r
-  DxeSalLibConstruct ();\r
   ReturnReg = GetEsalEntryPoint ();\r
   if (ReturnReg.Status != EFI_SAL_SUCCESS) {\r
     return ReturnReg;\r
   }\r
 \r
+  //\r
+  // Look at the physical mode ESAL entry point to determine of the ESAL entry point has been initialized\r
+  //\r
+  if (*(UINT64 *)ReturnReg.r9 == 0 && *(UINT64 *)(ReturnReg.r9 + 8) == 0) {\r
+    //\r
+    // Both the function ponter and the GP value are zero, so attempt to initialize the ESAL Entry Point\r
+    //\r
+    DxeSalLibInitialize ();\r
+    ReturnReg = GetEsalEntryPoint ();\r
+    if (ReturnReg.Status != EFI_SAL_SUCCESS) {\r
+      return ReturnReg;\r
+    }\r
+    if (*(UINT64 *)ReturnReg.r9 == 0 && *(UINT64 *)(ReturnReg.r9 + 8) == 0) {\r
+      //\r
+      // The ESAL Entry Point could not be initialized\r
+      //\r
+      ReturnReg.Status = EFI_SAL_ERROR;\r
+      return ReturnReg;\r
+    }\r
+  }\r
+\r
   if (ReturnReg.r11 & PSR_IT_MASK) {\r
     //\r
     // Virtual mode plabel to entry point\r