Hi Jan,
> On 17 Sep 2021, at 10:58, Jan Beulich <[email protected]> wrote:
>
> The on-commit editing of 5260e8fb93f0 ("xen: re-define assign_pages and
> introduce a new function assign_page") didn't go quite far enough: A
> local variable and a function argument also would have wanted adjusting.
>
> Signed-off-by: Jan Beulich <[email protected]>
Reviewed-by: Bertrand Marquis <[email protected]>
Cheers
Bertrand
>
> --- a/xen/common/page_alloc.c
> +++ b/xen/common/page_alloc.c
> @@ -2269,7 +2269,7 @@ int assign_pages(
> unsigned int memflags)
> {
> int rc = 0;
> - unsigned long i;
> + unsigned int i;
>
> spin_lock(&d->page_alloc_lock);
>
> @@ -2339,7 +2339,7 @@ int assign_pages(
> int assign_page(struct page_info *pg, unsigned int order, struct domain *d,
> unsigned int memflags)
> {
> - return assign_pages(pg, 1UL << order, d, memflags);
> + return assign_pages(pg, 1U << order, d, memflags);
> }
>
> struct page_info *alloc_domheap_pages(
>
>
