[U-Boot] [PATCH] efi_loader: Add debug output for efi_add_memory_map()

Tom Rini trini at konsulko.com
Mon Apr 18 18:26:37 CEST 2016


On Wed, Apr 13, 2016 at 05:33:50AM +0200, Andreas Färber wrote:

> Tracing the arguments has been helpful for pinpointing overflows.
> 
> Cc: Alexander Graf <agraf at suse.de>
> Signed-off-by: Andreas Färber <afaerber at suse.de>
> ---
>  lib/efi_loader/efi_memory.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c
> index df99585..138736f 100644
> --- a/lib/efi_loader/efi_memory.c
> +++ b/lib/efi_loader/efi_memory.c
> @@ -123,6 +123,11 @@ uint64_t efi_add_memory_map(uint64_t start, uint64_t pages, int memory_type,
>  	struct efi_mem_list *newlist;
>  	bool do_carving;
>  
> +#ifdef DEBUG_EFI
> +	printf("%s: 0x%" PRIx64 " 0x%" PRIx64 " %d %s\n", __func__,
> +		start, pages, memory_type, overlap_only_ram ? "yes" : "no");
> +#endif
> +
>  	if (!pages)
>  		return start;

This should be using debug() instead.  And in fact can you please do a
clean up series to make EFI use DEBUG (for toggling EFI_ENTRY extra
debug stuff) and debug() instead of adding a new symbol?  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160418/dd20c76c/attachment.sig>


More information about the U-Boot mailing list