Re: [Xen-devel] [PATCH] xen/page_alloc: statically allocate bootmem_region_list

2019-12-17 Thread Julien Grall
Hi, On 17/12/2019 16:37, Jan Beulich wrote: I'm sorry for the non-threaded reply, but my mail client has mixed up this mail with another one, so I have nothing to properly reply to. With one stylistic issue taken care of (blanks around the binary operator / ) Reviewed-by: Jan Beulich The change

Re: [Xen-devel] [PATCH] xen/page_alloc: statically allocate bootmem_region_list

2019-12-17 Thread Jan Beulich
I'm sorry for the non-threaded reply, but my mail client has mixed up this mail with another one, so I have nothing to properly reply to. With one stylistic issue taken care of (blanks around the binary operator / ) Reviewed-by: Jan Beulich The change would be easy enough to do while committing, b

Re: [Xen-devel] [PATCH] xen/page_alloc: statically allocate bootmem_region_list

2019-12-17 Thread Julien Grall
Hi Hongyan, On 17/12/2019 14:33, Hongyan Xia wrote: The existing code assumes that the first mfn passed to the boot allocator is mapped, which creates problems when, e.g., we do not have a direct map, and may create other bootstrapping problems in the future. Make it static. The size is kept the

[Xen-devel] [PATCH] xen/page_alloc: statically allocate bootmem_region_list

2019-12-17 Thread Hongyan Xia
The existing code assumes that the first mfn passed to the boot allocator is mapped, which creates problems when, e.g., we do not have a direct map, and may create other bootstrapping problems in the future. Make it static. The size is kept the same as before (1 page). Signed-off-by: Hongyan Xia