Re: [Xen-devel] [PATCH v2 3/3] xen/sched: add minimalistic idle scheduler for free cpus

2019-09-02 Thread Juergen Gross
On 02.09.19 13:06, Jan Beulich wrote: On 27.08.2019 14:40, Juergen Gross wrote: On 27.08.19 14:37, Andrew Cooper wrote: On 27/08/2019 11:59, Juergen Gross wrote: +static void * +sched_idle_alloc_vdata(const struct scheduler *ops, struct vcpu *v, + void *dd) +{ +/*

Re: [Xen-devel] [PATCH v2 3/3] xen/sched: add minimalistic idle scheduler for free cpus

2019-09-02 Thread Jan Beulich
On 27.08.2019 14:40, Juergen Gross wrote: > On 27.08.19 14:37, Andrew Cooper wrote: >> On 27/08/2019 11:59, Juergen Gross wrote: >>> +static void * >>> +sched_idle_alloc_vdata(const struct scheduler *ops, struct vcpu *v, >>> + void *dd) >>> +{ >>> +/* Any non-NULL pointer

Re: [Xen-devel] [PATCH v2 3/3] xen/sched: add minimalistic idle scheduler for free cpus

2019-08-27 Thread Juergen Gross
On 27.08.19 14:37, Andrew Cooper wrote: On 27/08/2019 11:59, Juergen Gross wrote: +static void * +sched_idle_alloc_vdata(const struct scheduler *ops, struct vcpu *v, + void *dd) +{ +/* Any non-NULL pointer is fine here. */ +return (void *)1UL; As an observation,

Re: [Xen-devel] [PATCH v2 3/3] xen/sched: add minimalistic idle scheduler for free cpus

2019-08-27 Thread Andrew Cooper
On 27/08/2019 11:59, Juergen Gross wrote: > +static void * > +sched_idle_alloc_vdata(const struct scheduler *ops, struct vcpu *v, > + void *dd) > +{ > +/* Any non-NULL pointer is fine here. */ > +return (void *)1UL; As an observation, the vdata interface (and others,

[Xen-devel] [PATCH v2 3/3] xen/sched: add minimalistic idle scheduler for free cpus

2019-08-27 Thread Juergen Gross
Instead of having a full blown scheduler running for the free cpus add a very minimalistic scheduler for that purpose only ever scheduling the related idle vcpu. This has the big advantage of not needing any per-cpu, per-domain or per-scheduling unit data for free cpus and in turn simplifying