Hi, At 18:28 +0200 on 29 Jul (1469816936), Roger Pau Monne wrote: > --- a/xen/arch/x86/mm/shadow/common.c > +++ b/xen/arch/x86/mm/shadow/common.c > @@ -1604,9 +1604,8 @@ shadow_free_p2m_page(struct domain *d, struct page_info > *pg) > * Input will be rounded up to at least shadow_min_acceptable_pages(), > * plus space for the p2m table. > * Returns 0 for success, non-zero for failure. */ > -static unsigned int sh_set_allocation(struct domain *d, > - unsigned int pages, > - int *preempted) > +unsigned int sh_set_allocation(struct domain *d, unsigned int pages, > + int *preempted)
Making this non-static is fine by me. Since you'll be respinning anyway for Andrew's comments about can you please also move the full comment above it to shadow.h so callers can see the rounding of inputs &c. It might be good to repeat some of that in the comment for paging_set_allocation() too. With that, Acked-by: Tim Deegan <[email protected]> Cheers, Tim. _______________________________________________ Xen-devel mailing list [email protected] https://lists.xen.org/xen-devel
