Re: [Xen-devel] [PATCH 01/19] xen: sched: leave CPUs doing tasklet work alone.

2016-07-07 Thread Dario Faggioli
On Wed, 2016-07-06 at 16:41 +0100, George Dunlap wrote: > On Sat, Jun 18, 2016 at 12:11 AM, Dario Faggioli >  > > diff --git a/xen/common/sched_credit.c b/xen/common/sched_credit.c > > index a38a63d..a6645a2 100644 > > --- a/xen/common/sched_credit.c > > +++ b/xen/common/sched_credit.c > > @@ -1819

Re: [Xen-devel] [PATCH 01/19] xen: sched: leave CPUs doing tasklet work alone.

2016-07-07 Thread Dario Faggioli
On Mon, 2016-06-20 at 01:48 -0600, Jan Beulich wrote: > > > > On 18.06.16 at 01:11, wrote: > > --- a/xen/common/sched_credit.c > > +++ b/xen/common/sched_credit.c > > @@ -1819,24 +1819,24 @@ csched_schedule( > >  else > >  snext = csched_load_balance(prv, cpu, snext, > > &ret.migrated)

Re: [Xen-devel] [PATCH 01/19] xen: sched: leave CPUs doing tasklet work alone.

2016-07-06 Thread George Dunlap
On Sat, Jun 18, 2016 at 12:11 AM, Dario Faggioli wrote: > In both Credit1 and Credit2, stop considering a pCPU idle, > if the reason why the idle vCPU is being selected, is to > do tasklet work. > > Not doing so means that the tickling and load balancing > logic, seeing the pCPU as idle, considers

Re: [Xen-devel] [PATCH 01/19] xen: sched: leave CPUs doing tasklet work alone.

2016-06-21 Thread anshul makkar
On 18/06/16 00:11, Dario Faggioli wrote: In both Credit1 and Credit2, stop considering a pCPU idle, if the reason why the idle vCPU is being selected, is to do tasklet work. Not doing so means that the tickling and load balancing logic, seeing the pCPU as idle, considers it a candidate for picki

Re: [Xen-devel] [PATCH 01/19] xen: sched: leave CPUs doing tasklet work alone.

2016-06-20 Thread Jan Beulich
>>> On 18.06.16 at 01:11, wrote: > --- a/xen/common/sched_credit.c > +++ b/xen/common/sched_credit.c > @@ -1819,24 +1819,24 @@ csched_schedule( > else > snext = csched_load_balance(prv, cpu, snext, &ret.migrated); > > + out: > /* > * Update idlers mask if necessary. When

[Xen-devel] [PATCH 01/19] xen: sched: leave CPUs doing tasklet work alone.

2016-06-17 Thread Dario Faggioli
In both Credit1 and Credit2, stop considering a pCPU idle, if the reason why the idle vCPU is being selected, is to do tasklet work. Not doing so means that the tickling and load balancing logic, seeing the pCPU as idle, considers it a candidate for picking up vCPUs. But the pCPU won't actually pi