[U-Boot] [PATCH v1] Makefile: Don't generate position independent code

[Andy Shevchenko]

On Mon, 2018-08-06 at 18:56 +0200, Heinrich Schuchardt wrote:
> On 08/06/2018 06:00 PM, Andy Shevchenko wrote:

> > Fix all these by disabling PIE on Makefile level.

> With the patch building with gcc-8.1 works on i386.

Does it mean you are actually run it and it works?

>  But the interesting
> question is whether the EFI subsystem will be able to relocate the
> runtime code when the EFI service SetVirtualAddressMap() is called.

EFI code should have different CFLAGS I suppose.

> Did you boot Linux with the patch via bootefi and call any of the EFI
> runtime services from Linux?

Nope, I have no platform with UEFI to test.

> As you are changing this for all architectures this needs to be tested
> on all (ARM, RISC-V, and x86) architectures supporting the EFI
> subsystem.

Agree. Unfortunately I have almost none of them to play with.
I leave this to others who able to confirm the patch works.

My understanding we need this anyway and if there are some problems, we
need to fix them on individual basis.

-- 
Andy Shevchenko <andriy.shevchenko at linux.intel.com>
Intel Finland Oy