[U-Boot] [PATCH 2/3] efi_loader: use freestanding library for efi apps

Alexander Graf agraf at suse.de
Mon Jan 21 13:52:38 UTC 2019


On 01/20/2019 08:20 AM, Heinrich Schuchardt wrote:
> GCC requires that freestanding programs provide memcpy(), memmove(),
> memset(), and memcmp().
>
> Add the library functions when building a *.efi files.
>
> The EFI selftests might use other compilation flags. So use a symbolic
> link to provide lib/efi_selftest/efi_freestanding.c and compile it
> separately.
>
> Reported-by: Alexander Graf <agraf at suse.de>
> Fixes: 5be444d14b38 ("efi_loader: consistent build flags for EFI applications")
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
>   lib/efi_selftest/efi_freestanding.c | 1 +
>   scripts/Makefile.lib                | 2 +-
>   2 files changed, 2 insertions(+), 1 deletion(-)
>   create mode 120000 lib/efi_selftest/efi_freestanding.c
>
> diff --git a/lib/efi_selftest/efi_freestanding.c b/lib/efi_selftest/efi_freestanding.c
> new file mode 120000
> index 0000000000..4b7edd52bd
> --- /dev/null
> +++ b/lib/efi_selftest/efi_freestanding.c
> @@ -0,0 +1 @@
> +../efi_loader/efi_freestanding.c
> \ No newline at end of file

Please try to refrain from using symlinks. I will apply the patch 
regardless, but if you can think of a better way to model this, I'm all 
for it.


Thanks,

Alex



More information about the U-Boot mailing list