1) Make a slightly better work around for the EdkDxeSalLib by removing constructor from MSA and calling constructor from library functions.

2)	Fix pointer bug in the EdkUefiRuntimeLibrary in the event creation


git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@2034 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 11 insertions, 3 deletions

diff --git a/EdkModulePkg/Library/EdkDxeSalLib/Ipf/EsalServiceLib.c b/EdkModulePkg/Library/EdkDxeSalLib/Ipf/EsalServiceLib.c
index 2e0daeb89e..706dda2873 100644
--- a/EdkModulePkg/Library/EdkDxeSalLib/Ipf/EsalServiceLib.c
+++ b/EdkModulePkg/Library/EdkDxeSalLib/Ipf/EsalServiceLib.c
@@ -19,20 +19,26 @@ Abstract:
 

 #include <Ipf/IpfDefines.h>

 

-

+BOOLEAN mLibraryInitialized = FALSE;

 STATIC EXTENDED_SAL_BOOT_SERVICE_PROTOCOL *mEsalBootService;

 STATIC EFI_PLABEL                         mPlabel;

 

 EFI_STATUS

 EFIAPI

 DxeSalLibConstruct (

-  IN EFI_HANDLE        ImageHandle,

-  IN EFI_SYSTEM_TABLE  *SystemTable

+//  IN EFI_HANDLE        ImageHandle,

+//  IN EFI_SYSTEM_TABLE  *SystemTable

+  VOID

   )

 {

   EFI_PLABEL  *Plabel;

   EFI_STATUS  Status;

 

+  if (mLibraryInitialized == TRUE) {

+    return EFI_SUCCESS;

+  }

+  mLibraryInitialized = TRUE;

+

   //

   // The protocol contains a function pointer, which is an indirect procedure call.

   // An indirect procedure call goes through a plabel, and pointer to a function is

@@ -109,6 +115,7 @@ Returns:
 

 --*/

 {

+  DxeSalLibConstruct ();

   return mEsalBootService->AddExtendedSalProc (

                             mEsalBootService,

                             ClassGuid,

@@ -217,6 +224,7 @@ Returns:
   SAL_RETURN_REGS       ReturnReg;

   SAL_EXTENDED_SAL_PROC EsalProc;

 

+  DxeSalLibConstruct ();

   ReturnReg = GetEsalEntryPoint ();

   if (ReturnReg.Status != EFI_SAL_SUCCESS) {

     return ReturnReg;