On Fri, 19 Feb 2016, Corneliu ZUZU wrote:
> On 2/19/2016 3:49 PM, Stefano Stabellini wrote:
> > On Thu, 18 Feb 2016, Corneliu ZUZU wrote:
> > > +
> > > +    if ( sync )
> > > +    {
> > > +        req->flags |= VM_EVENT_FLAG_VCPU_PAUSED;
> > > +        vm_event_vcpu_pause(v);
> > > +    }
> > > +
> > > +#if CONFIG_X86
> > > +    if ( altp2m_active(d) )
> > I would rather
> > 
> > #define altp2m_active(d) (0)
> > 
> > on ARM, removing the two ifdefs in this file.
> 
> Yeah, I actually wanted to get rid of that too at some point, the question is,
> what do I do with "req->altp2m_idx = vcpu_altp2m(v).p2midx"? I'm not familiar
> w/ altp2m design, maybe someone that knows more of the internals of that can
> give a suggestion.

If you #define altp2m_active to (0), gcc will automatically avoid the if
statement.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to