[PATCH] efi_loader: Expose relocated address for gdb debugging purposes

[Alexander von Gluck IV]

March 24, 2022 12:06 PM, "Heinrich Schuchardt" <xypron.glpk at gmx.de> wrote:
> On 3/24/22 17:22, Alexander von Gluck IV wrote:
> Do we really always need this output for every invokation of LoadImage()?
> 
> Writing test messages during the runtime of a menu driven program like
> the UEFI SCT can be very disturbing.
> 
> Isn't it sufficient to show this output for binaries directly launched
> by U-Boot?

Ideally this would be exposed strictly for the binaries directly launched,
however the bare relocated address isn't exposed outside of the brief
usage by LoadImage.

I figured that adding a required pointer arg to expose the relocated address
might be overkill.

using log_debug over log_info was brought up as a good change, but that
still suffers the same potential impact.

Maybe adding reloc_addr storage to efi_loaded_image_obj would be cleaner?
(then digging it up during the efi launch for a quick log_info?)

 -- Alex