On Thu, 2015-10-01 at 07:28 +0200, Juergen Gross wrote: > On 09/29/2015 06:56 PM, Dario Faggioli wrote:
> > --- a/xen/common/schedule.c > > +++ b/xen/common/schedule.c > > @@ -1501,8 +1501,8 @@ int schedule_cpu_switch(unsigned int cpu, > > struct cpupool *c) > > return 0; > > > > idle = idle_vcpu[cpu]; > > - ppriv = SCHED_OP(new_ops, alloc_pdata, cpu); > > - if ( ppriv == NULL ) > > + if ( (new_ops->alloc_pdata != NULL) && > > + ((ppriv = new_ops->alloc_pdata(new_ops, cpu)) == NULL) ) > > return -ENOMEM; > > vpriv = SCHED_OP(new_ops, alloc_vdata, idle, idle->domain > > ->sched_priv); > > if ( vpriv == NULL ) > > Just below this there are 2 SCHED_OP calls to free_pdata. You'll have > to > check whether the hook is present as you just have nuked it above. > I think this is fine, for the same reasons explained when replying to your comment to patch 4. Thanks and Regards, Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
