On 24.01.2022 16:51, Roger Pau Monne wrote:
> I find it useful for debugging certain issues to have the memory map
> dom0 is using, so print it when using `dom0=verbose` on the command
> line.
> 
> Signed-off-by: Roger Pau Monné <roger....@citrix.com>

Reviewed-by: Jan Beulich <jbeul...@suse.com>
with one further request:

> --- a/xen/arch/x86/e820.c
> +++ b/xen/arch/x86/e820.c
> @@ -88,7 +88,7 @@ static void __init add_memory_region(unsigned long long 
> start,
>      e820.nr_map++;
>  }
>  
> -static void __init print_e820_memory_map(struct e820entry *map, unsigned int 
> entries)
> +void __init print_e820_memory_map(struct e820entry *map, unsigned int 
> entries)

While making this accessible from the outside, could you please
make "map" pointer-to-const?

Jan


Reply via email to