diff options
| author | Maurice Ma <maurice.ma@intel.com> | 2016-10-12 18:00:44 -0700 |
|---|---|---|
| committer | Hao Wu <hao.a.wu@intel.com> | 2016-10-31 09:13:18 +0800 |
| commit | 31056a5f187d36d84cbddaa9b3f88b6e0f01a8f6 (patch) | |
| tree | d2f2aab2ebe2e52e13def3ac112e22b39521a92d | |
| parent | b50e4779dfe2f26cd1fa25230fab72416dfaf049 (diff) | |
| download | edk2-31056a5f187d36d84cbddaa9b3f88b6e0f01a8f6.tar.gz | |
IntelFsp2Pkg/FspSecCore: Make FSP functions position independent
The current AsmGetFspInfoHeader function in FspHeader.nasm is
position dependent code since it uses absolute address. Change
to use relative address instead to make it position independent.
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Giri P Mudusuru <giri.p.mudusuru@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Maurice Ma <maurice.ma@intel.com>
Reviewed-by: Giri P Mudusuru <giri.p.mudusuru@intel.com>
(cherry picked from commit 08354c34486947da17a36a605f9a4b000132123f)
| -rw-r--r-- | IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm index a848dcbc02..ad631943e3 100644 --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm @@ -31,7 +31,7 @@ ASM_PFX(NextInstruction): pop eax
sub eax, ASM_PFX(NextInstruction)
add eax, ASM_PFX(AsmGetFspInfoHeader)
- sub eax, dword [ASM_PFX(FspInfoHeaderRelativeOff)]
+ sub eax, dword [eax - ASM_PFX(AsmGetFspInfoHeader) + ASM_PFX(FspInfoHeaderRelativeOff)]
ret
global ASM_PFX(AsmGetFspInfoHeaderNoStack)
|