[U-Boot] [PATCH 2/6] x86: Change __kernel_size_t conditionals to use compiler provided defines

Bin Meng bmeng.cn at gmail.com
Sun Jun 24 00:54:01 UTC 2018


On Sat, Jun 23, 2018 at 10:22 PM, Simon Glass <sjg at chromium.org> wrote:
> On 22 June 2018 at 02:38, Bin Meng <bmeng.cn at gmail.com> wrote:
>> Since commit bb0bb91cf0aa ("efi_stub: Use efi_uintn_t"), EFI x86
>> 64-bit payload does not work anymore. The call to GetMemoryMap()
>> in efi_stub.c fails with return code EFI_INVALID_PARAMETER. Since
>> the payload itself is still 32-bit U-Boot, efi_uintn_t gets wrongly
>> interpreted as int, but it should actually be long in a 64-bit EFI
>> environment.
>>
>> This changes the x86 __kernel_size_t conditionals to use compiler
>> provided defines instead. That way we always adhere to the build
>> environment we're in and the definitions adjust automatically.
>>
>> Fixes: bb0bb91cf0aa ("efi_stub: Use efi_uintn_t")
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>  arch/x86/include/asm/posix_types.h | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> Reviewed-by: Simon Glass <sjg at chromium.org>

applied to u-boot-x86, thanks!


More information about the U-Boot mailing list