On 05.03.2020 13:45, pdurr...@amzn.com wrote: > --- a/xen/arch/x86/pv/dom0_build.c > +++ b/xen/arch/x86/pv/dom0_build.c > @@ -792,6 +792,10 @@ int __init dom0_construct_pv(struct domain *d, > { > mfn = mfn_x(page_to_mfn(page)); > BUG_ON(SHARED_M2P(get_gpfn_from_mfn(mfn))); > + > + if ( page->count_info & PGC_extra ) > + continue;
This surely is a pattern, i.e. there are more similar changes to make: tboot_gen_domain_integrity() e.g. ignores d->xenpage_list, and hence with the goal of converting the shared info page would also want adjustment. For dump_numa() it may be less important, but it would still look more correct if it too got changed. audit_p2m() might apparently complain about such pages (and hence might be a problem with the one PGC_extra page VMX domains now have). And this is only from me looking at page_list_for_each(..., &d->page_list) constructs; who knows what else there is. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel