Re: [Xen-devel] [PATCH 22/34] x86/mm: put HVM only code under CONFIG_HVM

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Tue, Aug 21, 2018 at 02:07:22AM -0600, Jan Beulich wrote:
> #ifdef (also elsewhere; did I overlook similar issues in earlier patches?)
> 
> > --- a/xen/common/vm_event.c
> > +++ b/xen/common/vm_event.c
> > @@ -429,9 +429,11 @@ void vm_event_resume(struct domain *d, struct 
> > vm_event_domain *ved)
> >               */
> >              vm_event_toggle_singlestep(d, v, &rsp);
> >  
> > +#if CONFIG_HVM
> >              /* Check for altp2m switch */
> >              if ( rsp.flags & VM_EVENT_FLAG_ALTERNATE_P2M )
> >                  p2m_altp2m_check(v, rsp.altp2m_idx);
> > +#endif
> 
> Perhaps again better to have a static inline stub?

I will leave this as-is. The code churn would be the same.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel