On 14.03.2025 18:24, Alejandro Vallejo wrote: > --- a/xen/common/page_alloc.c > +++ b/xen/common/page_alloc.c > @@ -523,7 +523,7 @@ out: > int domain_set_outstanding_pages(struct domain *d, unsigned long pages) > { > int ret = -ENOMEM; > - unsigned long claim, avail_pages; > + unsigned long avail_pages; > > /* > * take the domain's page_alloc_lock, else all d->tot_page adjustments > @@ -549,28 +549,21 @@ int domain_set_outstanding_pages(struct domain *d, > unsigned long pages) > goto out; > } > > - /* disallow a claim not exceeding domain_tot_pages() or above max_pages > */ > - if ( (pages <= domain_tot_pages(d)) || (pages > d->max_pages) ) > + /* Don't claim past max_pages */ > + if ( (domain_tot_pages(d) + pages) > d->max_pages )
In addition to what Roger has said, how can such a behavioral change come without any caller-side adjustment? Jan