Re: [PATCH] ArmVirtPkg: apply PE/COFF memory protection to DxeCore as well

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 7 March 2017 at 14:56, Laszlo Ersek <lersek@redhat.com> wrote:
> On 03/07/17 12:24, Ard Biesheuvel wrote:
>> Include DXE_CORE in the BuildOptions that are set to force 4 KB section
>> alignment for PE/COFF images in order to allow them to be mapped with
>> strict memory permissions.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.0
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> ---
>>  ArmVirtPkg/ArmVirt.dsc.inc | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc
>> index e2d3dcce7945..f61fd0680612 100644
>> --- a/ArmVirtPkg/ArmVirt.dsc.inc
>> +++ b/ArmVirtPkg/ArmVirt.dsc.inc
>> @@ -17,7 +17,7 @@ [Defines]
>>    DEFINE DEBUG_PRINT_ERROR_LEVEL = 0x8000004F
>>    DEFINE TTY_TERMINAL            = FALSE
>>
>> -[BuildOptions.common.EDKII.DXE_DRIVER,BuildOptions.common.EDKII.UEFI_DRIVER,BuildOptions.common.EDKII.UEFI_APPLICATION]
>> +[BuildOptions.common.EDKII.DXE_CORE,BuildOptions.common.EDKII.DXE_DRIVER,BuildOptions.common.EDKII.UEFI_DRIVER,BuildOptions.common.EDKII.UEFI_APPLICATION]
>>    GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000
>>
>>  [BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER]
>>
>
> Reviewed-by: Laszlo Ersek <lersek@redhat.com>

Thanks, pushed.