CSM should firstly set timer to enable state at first, then RestoreTpl is called.

Signed-off-by: li-elvin
Reviewed-by: rsun3
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@12555 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Thunk.c b/IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Thunk.c
index 16c873d8f5c40bcb42ec9b107d65fbe1388be7c3..3d9a8b96490ca54794954ed31f4f51c4dc0d10a9 100644 (file)
--- a/IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Thunk.c
+++ b/IntelFrameworkModulePkg/Csm/LegacyBiosDxe/Thunk.c
@@ -315,14 +315,14 @@ InternalLegacyBiosFarCall (
   mThunkContext.RealModeState = NULL;\r
 \r
   //\r
   mThunkContext.RealModeState = NULL;\r
 \r
   //\r

-  // End critical section\r
+  // Enable and restore rate of DXE Timer\r

   //\r
   //\r

-  gBS->RestoreTPL (OriginalTpl);\r
+  Private->Timer->SetTimerPeriod (Private->Timer, TimerPeriod);\r

 \r
   //\r
 \r
   //\r

-  // Enable and restore rate of DXE Timer\r
+  // End critical section\r

   //\r
   //\r

-  Private->Timer->SetTimerPeriod (Private->Timer, TimerPeriod);\r
+  gBS->RestoreTPL (OriginalTpl);\r

   \r
   //\r
   // Restore interrupt of debug timer\r
   \r
   //\r
   // Restore interrupt of debug timer\r