| Message ID | 20170327122216.29760-1-ard.biesheuvel@linaro.org |
|---|---|
| State | Accepted |
| Commit | 6e7ec25aaaf0dfc2b4c84ffd4c7ee7cd442aecb6 |
| Headers | show |
Reviewed-by: Liming Gao <liming.gao@intel.com> >-----Original Message----- >From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ard >Biesheuvel >Sent: Monday, March 27, 2017 8:22 PM >To: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com> >Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> >Subject: [edk2] [PATCH] MdeModulePkg/PeiCore: avoid >EFI_IMAGE_MACHINE_TYPE_SUPPORTED to check arch > >The EFI_IMAGE_MACHINE_TYPE_SUPPORTED() macro is abused in the >PeiCore >code to decide whether the system we are compiling for can deal with >executable code being copied elsewhere and executed from there. > >As stated in the comment, this is fundamentally a property of the compiler >target, and so this should be made dependent on MDE_CPU_xxx >preprocessor >defines, and not on whether or not the runtime target can deal with >PE/COFF images of a certain machine type. > >On X86/IA32, this mostly boils down to the same thing, but not on other >architectures, so let's clean this up. > >Contributed-under: TianoCore Contribution Agreement 1.0 >Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >--- > MdeModulePkg/Core/Pei/Image/Image.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > >diff --git a/MdeModulePkg/Core/Pei/Image/Image.c >b/MdeModulePkg/Core/Pei/Image/Image.c >index c8bb2300a0a6..198541128512 100644 >--- a/MdeModulePkg/Core/Pei/Image/Image.c >+++ b/MdeModulePkg/Core/Pei/Image/Image.c >@@ -112,6 +112,7 @@ GetImageReadFunction ( > IN PE_COFF_LOADER_IMAGE_CONTEXT *ImageContext > ) > { >+#if defined (MDE_CPU_IA32) || defined (MDE_CPU_X64) > PEI_CORE_INSTANCE *Private; > EFI_PHYSICAL_ADDRESS MemoryBuffer; > >@@ -119,8 +120,7 @@ GetImageReadFunction ( > MemoryBuffer = 0; > > if (Private->PeiMemoryInstalled && (((Private- >>HobList.HandoffInformationTable->BootMode != BOOT_ON_S3_RESUME) >&& PcdGetBool (PcdShadowPeimOnBoot)) || >- ((Private->HobList.HandoffInformationTable->BootMode == >BOOT_ON_S3_RESUME) && PcdGetBool (PcdShadowPeimOnS3Boot))) && >- (EFI_IMAGE_MACHINE_TYPE_SUPPORTED(EFI_IMAGE_MACHINE_X64) >|| EFI_IMAGE_MACHINE_TYPE_SUPPORTED(EFI_IMAGE_MACHINE_IA32))) { >+ ((Private->HobList.HandoffInformationTable->BootMode == >BOOT_ON_S3_RESUME) && PcdGetBool (PcdShadowPeimOnS3Boot)))) { > // > // Shadow algorithm makes lots of non ANSI C assumptions and only works >for IA32 and X64 > // compilers that have been tested >@@ -136,7 +136,9 @@ GetImageReadFunction ( > } else { > ImageContext->ImageRead = PeiImageRead; > } >- >+#else >+ ImageContext->ImageRead = PeiImageRead; >+#endif > return EFI_SUCCESS; > } > /** >-- >2.9.3 > >_______________________________________________ >edk2-devel mailing list >edk2-devel@lists.01.org >https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
diff --git a/MdeModulePkg/Core/Pei/Image/Image.c b/MdeModulePkg/Core/Pei/Image/Image.c index c8bb2300a0a6..198541128512 100644 --- a/MdeModulePkg/Core/Pei/Image/Image.c +++ b/MdeModulePkg/Core/Pei/Image/Image.c @@ -112,6 +112,7 @@ GetImageReadFunction ( IN PE_COFF_LOADER_IMAGE_CONTEXT *ImageContext ) { +#if defined (MDE_CPU_IA32) || defined (MDE_CPU_X64) PEI_CORE_INSTANCE *Private; EFI_PHYSICAL_ADDRESS MemoryBuffer; @@ -119,8 +120,7 @@ GetImageReadFunction ( MemoryBuffer = 0; if (Private->PeiMemoryInstalled && (((Private->HobList.HandoffInformationTable->BootMode != BOOT_ON_S3_RESUME) && PcdGetBool (PcdShadowPeimOnBoot)) || - ((Private->HobList.HandoffInformationTable->BootMode == BOOT_ON_S3_RESUME) && PcdGetBool (PcdShadowPeimOnS3Boot))) && - (EFI_IMAGE_MACHINE_TYPE_SUPPORTED(EFI_IMAGE_MACHINE_X64) || EFI_IMAGE_MACHINE_TYPE_SUPPORTED(EFI_IMAGE_MACHINE_IA32))) { + ((Private->HobList.HandoffInformationTable->BootMode == BOOT_ON_S3_RESUME) && PcdGetBool (PcdShadowPeimOnS3Boot)))) { // // Shadow algorithm makes lots of non ANSI C assumptions and only works for IA32 and X64 // compilers that have been tested @@ -136,7 +136,9 @@ GetImageReadFunction ( } else { ImageContext->ImageRead = PeiImageRead; } - +#else + ImageContext->ImageRead = PeiImageRead; +#endif return EFI_SUCCESS; } /**
The EFI_IMAGE_MACHINE_TYPE_SUPPORTED() macro is abused in the PeiCore code to decide whether the system we are compiling for can deal with executable code being copied elsewhere and executed from there. As stated in the comment, this is fundamentally a property of the compiler target, and so this should be made dependent on MDE_CPU_xxx preprocessor defines, and not on whether or not the runtime target can deal with PE/COFF images of a certain machine type. On X86/IA32, this mostly boils down to the same thing, but not on other architectures, so let's clean this up. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> --- MdeModulePkg/Core/Pei/Image/Image.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) -- 2.9.3 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel