On 30.07.2025 12:38, Petr Beneš wrote: > --- a/xen/arch/x86/include/asm/p2m.h > +++ b/xen/arch/x86/include/asm/p2m.h > @@ -889,6 +889,8 @@ void shadow_p2m_init(struct p2m_domain *p2m); > void cf_check nestedp2m_write_p2m_entry_post( > struct p2m_domain *p2m, unsigned int oflags); > > +#ifdef CONFIG_ALTP2M > + > /* > * Alternate p2m: shadow p2m tables used for alternate memory views > */ > @@ -965,6 +967,16 @@ int p2m_set_altp2m_view_visibility(struct domain *d, > unsigned int altp2m_idx, > > #else /* !CONFIG_HVM */ > struct p2m_domain *p2m_get_altp2m(struct vcpu *v); > +bool p2m_set_altp2m(struct vcpu *v, unsigned int idx); > +bool p2m_altp2m_get_or_propagate(struct p2m_domain *ap2m, unsigned long > gfn_l, > + mfn_t *mfn, p2m_type_t *p2mt, > + p2m_access_t *p2ma, unsigned int > *page_order); > +int p2m_altp2m_propagate_change(struct domain *d, gfn_t gfn, > + mfn_t mfn, unsigned int page_order, > + p2m_type_t p2mt, p2m_access_t p2ma);
These latter two already have declarations; we don't want to have two of them, as they may then go out of sync. They simply want to move out of the #ifdef, imo. Jan
