>>> On 27.09.16 at 17:56, <roger....@citrix.com> wrote: > --- a/xen/arch/x86/mm/hap/hap.c > +++ b/xen/arch/x86/mm/hap/hap.c > @@ -379,7 +379,9 @@ hap_set_allocation(struct domain *d, unsigned long pages, > int *preempted) > break; > > /* Check to see if we need to yield and try again */ > - if ( preempted && hypercall_preempt_check() ) > + if ( preempted && > + (is_idle_vcpu(current) ? softirq_pending(smp_processor_id()) : > + hypercall_preempt_check()) )
So what is the supposed action for the caller to take in this case? I think this should at least be spelled out in the commit message. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel