On 31.07.2025 17:58, Oleksii Kurochko wrote: > @@ -30,3 +34,18 @@ int p2m_init(struct domain *d) > > return 0; > } > + > +/* > + * Set the pool of pages to the required number of pages. > + * Returns 0 for success, non-zero for failure. > + * Call with d->arch.paging.lock held. > + */ > +int p2m_set_allocation(struct domain *d, unsigned long pages, bool > *preempted)
Noticed only when looking at the subsequent patch: With this being ... > +{ > + int rc; > + > + if ( (rc = paging_freelist_init(d, pages, preempted)) ) ... a caller of this function, the "init" in the name feels wrong. Jan