On 9/9/21 10:57 AM, Simon Glass wrote:
Hi Heinrich,
On Wed, 8 Sept 2021 at 12:25, Heinrich Schuchardt xypron.glpk@gmx.de wrote:
On 9/8/21 3:34 PM, Simon Glass wrote:
Add a message here so that both paths of memory allocation are reported.
Signed-off-by: Simon Glass sjg@chromium.org
lib/efi/efi_app.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/lib/efi/efi_app.c b/lib/efi/efi_app.c index e0666d1fae2..c8f4784eb7d 100644 --- a/lib/efi/efi_app.c +++ b/lib/efi/efi_app.c @@ -97,6 +97,7 @@ static efi_status_t setup_memory(struct efi_priv *priv) return ret; priv->use_pool_for_malloc = true; } else {
printf("(using allocated RAM address %lx) ", (ulong)addr);Shouldn't this be log_debug()?
It matches the one a few lines higher. I suppose it could be a debug statement, but allocation is an area that seems to different on different UEFIs, so I have it more prominent for the moment, until we figure it out.
If you want it more prominent, isn't it log_info() that you were promoting?
Best regards
Heinrich