diff options
| author | Yao, Jiewen <jiewen.yao@intel.com> | 2015-12-15 04:27:00 +0000 |
|---|---|---|
| committer | jyao1 <jyao1@Edk2> | 2015-12-15 04:27:00 +0000 |
| commit | 3b567f08f99df24a31fa06750171b7050e08f966 (patch) | |
| tree | 1c5e72c98ffbf75a9659de7c4a6f6f9f342248ec /IntelFspWrapperPkg | |
| parent | fb567b2f7400dc615ea8195efb33935e32a13f40 (diff) | |
| download | device_linaro_bootloader_edk2-3b567f08f99df24a31fa06750171b7050e08f966.tar.gz device_linaro_bootloader_edk2-3b567f08f99df24a31fa06750171b7050e08f966.tar.bz2 device_linaro_bootloader_edk2-3b567f08f99df24a31fa06750171b7050e08f966.zip | |
Fix >4G issue on IDT not restored correctly.
Idtr might be changed inside of FSP. 32bit FSP only knows the <4G address.
If IDTR.Base is >4G, FSP can not handle. So we need save/restore IDTR here.
Interrupt is already disabled here, so it is safety to update IDTR.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: "Yao, Jiewen" <jiewen.yao@intel.com>
Reviewed-by: "Ma, Maurice" <maurice.ma@intel.com>
Reviewed-by: "Mudusuru, Giri P" <giri.p.mudusuru@intel.com>
Reviewed-by: "Yarlagadda, Satya P" <satya.p.yarlagadda@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19246 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'IntelFspWrapperPkg')
| -rw-r--r-- | IntelFspWrapperPkg/Library/BaseFspApiLib/X64/DispatchExecute.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/IntelFspWrapperPkg/Library/BaseFspApiLib/X64/DispatchExecute.c b/IntelFspWrapperPkg/Library/BaseFspApiLib/X64/DispatchExecute.c index e2f442253..360327ed0 100644 --- a/IntelFspWrapperPkg/Library/BaseFspApiLib/X64/DispatchExecute.c +++ b/IntelFspWrapperPkg/Library/BaseFspApiLib/X64/DispatchExecute.c @@ -89,6 +89,18 @@ Execute32BitCode ( IN UINT64 Param1
)
{
- return AsmExecute32BitCode (Function, Param1, 0, &mGdt);
+ EFI_STATUS Status;
+ IA32_DESCRIPTOR Idtr;
+
+ //
+ // Idtr might be changed inside of FSP. 32bit FSP only knows the <4G address.
+ // If IDTR.Base is >4G, FSP can not handle. So we need save/restore IDTR here for X64 only.
+ // Interrupt is already disabled here, so it is safety to update IDTR.
+ //
+ AsmReadIdtr (&Idtr);
+ Status = AsmExecute32BitCode (Function, Param1, 0, &mGdt);
+ AsmWriteIdtr (&Idtr);
+
+ return Status;
}
|