Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-18 Thread Morten Rasmussen
On Mon, Jul 18, 2016 at 02:48:42PM +0200, Vincent Guittot wrote: > On 15 July 2016 at 18:02, Morten Rasmussen wrote: > > On Fri, Jul 15, 2016 at 03:39:05PM +0200, Vincent Guittot wrote: > >> On 15 July 2016 at 13:46, Morten Rasmussen > >>

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-18 Thread Morten Rasmussen
On Mon, Jul 18, 2016 at 02:48:42PM +0200, Vincent Guittot wrote: > On 15 July 2016 at 18:02, Morten Rasmussen wrote: > > On Fri, Jul 15, 2016 at 03:39:05PM +0200, Vincent Guittot wrote: > >> On 15 July 2016 at 13:46, Morten Rasmussen > >> wrote: > >> > On Thu, Jul 14, 2016 at 04:15:20PM +0100,

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-18 Thread Vincent Guittot
On 15 July 2016 at 18:02, Morten Rasmussen wrote: > On Fri, Jul 15, 2016 at 03:39:05PM +0200, Vincent Guittot wrote: >> On 15 July 2016 at 13:46, Morten Rasmussen wrote: >> > On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: >>

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-18 Thread Vincent Guittot
On 15 July 2016 at 18:02, Morten Rasmussen wrote: > On Fri, Jul 15, 2016 at 03:39:05PM +0200, Vincent Guittot wrote: >> On 15 July 2016 at 13:46, Morten Rasmussen wrote: >> > On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: >> >> On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-15 Thread Morten Rasmussen
On Fri, Jul 15, 2016 at 03:39:05PM +0200, Vincent Guittot wrote: > On 15 July 2016 at 13:46, Morten Rasmussen wrote: > > On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: > >> On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: > >> > On 13

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-15 Thread Morten Rasmussen
On Fri, Jul 15, 2016 at 03:39:05PM +0200, Vincent Guittot wrote: > On 15 July 2016 at 13:46, Morten Rasmussen wrote: > > On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: > >> On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: > >> > On 13 July 2016 at 18:37, Morten

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-15 Thread Vincent Guittot
On 15 July 2016 at 13:46, Morten Rasmussen wrote: > On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: >> On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: >> > On 13 July 2016 at 18:37, Morten Rasmussen >> >

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-15 Thread Vincent Guittot
On 15 July 2016 at 13:46, Morten Rasmussen wrote: > On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: >> On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: >> > On 13 July 2016 at 18:37, Morten Rasmussen >> > wrote: >> > > Also, for SMT max capacity is less than

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-15 Thread Morten Rasmussen
On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: > On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: > > On 13 July 2016 at 18:37, Morten Rasmussen wrote: > > > Also, for SMT max capacity is less than 1024 already. No? > > > > Yes, it is.

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-15 Thread Morten Rasmussen
On Thu, Jul 14, 2016 at 04:15:20PM +0100, Morten Rasmussen wrote: > On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: > > On 13 July 2016 at 18:37, Morten Rasmussen wrote: > > > Also, for SMT max capacity is less than 1024 already. No? > > > > Yes, it is. I haven't looked in

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-14 Thread Morten Rasmussen
On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: > On 13 July 2016 at 18:37, Morten Rasmussen wrote: > > On Wed, Jul 13, 2016 at 02:48:24PM +0100, Dietmar Eggemann wrote: > >> On 13/07/16 13:40, Vincent Guittot wrote: > >> > On 22 June 2016 at 19:03,

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-14 Thread Morten Rasmussen
On Thu, Jul 14, 2016 at 03:25:36PM +0200, Vincent Guittot wrote: > On 13 July 2016 at 18:37, Morten Rasmussen wrote: > > On Wed, Jul 13, 2016 at 02:48:24PM +0100, Dietmar Eggemann wrote: > >> On 13/07/16 13:40, Vincent Guittot wrote: > >> > On 22 June 2016 at 19:03, Morten Rasmussen > >> >

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-14 Thread Vincent Guittot
On 13 July 2016 at 18:37, Morten Rasmussen wrote: > On Wed, Jul 13, 2016 at 02:48:24PM +0100, Dietmar Eggemann wrote: >> On 13/07/16 13:40, Vincent Guittot wrote: >> > On 22 June 2016 at 19:03, Morten Rasmussen >> > wrote: >> >> From: Dietmar

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-14 Thread Vincent Guittot
On 13 July 2016 at 18:37, Morten Rasmussen wrote: > On Wed, Jul 13, 2016 at 02:48:24PM +0100, Dietmar Eggemann wrote: >> On 13/07/16 13:40, Vincent Guittot wrote: >> > On 22 June 2016 at 19:03, Morten Rasmussen >> > wrote: >> >> From: Dietmar Eggemann >> >> >> >> To be able to compare the

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Morten Rasmussen
On Wed, Jul 13, 2016 at 02:48:24PM +0100, Dietmar Eggemann wrote: > On 13/07/16 13:40, Vincent Guittot wrote: > > On 22 June 2016 at 19:03, Morten Rasmussen wrote: > >> From: Dietmar Eggemann > >> > >> To be able to compare the capacity of the

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Morten Rasmussen
On Wed, Jul 13, 2016 at 02:48:24PM +0100, Dietmar Eggemann wrote: > On 13/07/16 13:40, Vincent Guittot wrote: > > On 22 June 2016 at 19:03, Morten Rasmussen wrote: > >> From: Dietmar Eggemann > >> > >> To be able to compare the capacity of the target cpu with the highest > >> available cpu

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Dietmar Eggemann
On 13/07/16 13:40, Vincent Guittot wrote: > On 22 June 2016 at 19:03, Morten Rasmussen wrote: >> From: Dietmar Eggemann >> >> To be able to compare the capacity of the target cpu with the highest >> available cpu capacity, store the maximum

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Dietmar Eggemann
On 13/07/16 13:40, Vincent Guittot wrote: > On 22 June 2016 at 19:03, Morten Rasmussen wrote: >> From: Dietmar Eggemann >> >> To be able to compare the capacity of the target cpu with the highest >> available cpu capacity, store the maximum per-cpu capacity in the root >> domain. > > I thought

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Vincent Guittot
On 22 June 2016 at 19:03, Morten Rasmussen wrote: > From: Dietmar Eggemann > > To be able to compare the capacity of the target cpu with the highest > available cpu capacity, store the maximum per-cpu capacity in the root > domain. I thought

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Vincent Guittot
On 22 June 2016 at 19:03, Morten Rasmussen wrote: > From: Dietmar Eggemann > > To be able to compare the capacity of the target cpu with the highest > available cpu capacity, store the maximum per-cpu capacity in the root > domain. I thought that the capacity of all CPUS were built so the

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Dietmar Eggemann
On 12/07/16 12:42, Peter Zijlstra wrote: > On Mon, Jul 11, 2016 at 05:16:06PM +0100, Dietmar Eggemann wrote: >> On 11/07/16 11:18, Peter Zijlstra wrote: >>> On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-13 Thread Dietmar Eggemann
On 12/07/16 12:42, Peter Zijlstra wrote: > On Mon, Jul 11, 2016 at 05:16:06PM +0100, Dietmar Eggemann wrote: >> On 11/07/16 11:18, Peter Zijlstra wrote: >>> On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-12 Thread Peter Zijlstra
On Mon, Jul 11, 2016 at 05:16:06PM +0100, Dietmar Eggemann wrote: > On 11/07/16 11:18, Peter Zijlstra wrote: > > On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: > >> @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct > >> cpumask *cpu_map, > >>/* Attach the

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-12 Thread Peter Zijlstra
On Mon, Jul 11, 2016 at 05:16:06PM +0100, Dietmar Eggemann wrote: > On 11/07/16 11:18, Peter Zijlstra wrote: > > On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: > >> @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct > >> cpumask *cpu_map, > >>/* Attach the

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-11 Thread Dietmar Eggemann
On 11/07/16 11:18, Peter Zijlstra wrote: > On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: >> @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct cpumask >> *cpu_map, >> /* Attach the domains */ >> rcu_read_lock(); >> for_each_cpu(i, cpu_map) { >>

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-11 Thread Dietmar Eggemann
On 11/07/16 11:18, Peter Zijlstra wrote: > On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: >> @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct cpumask >> *cpu_map, >> /* Attach the domains */ >> rcu_read_lock(); >> for_each_cpu(i, cpu_map) { >>

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-11 Thread Peter Zijlstra
On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: > @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct cpumask > *cpu_map, > /* Attach the domains */ > rcu_read_lock(); > for_each_cpu(i, cpu_map) { > + rq = cpu_rq(i); >

Re: [PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-07-11 Thread Peter Zijlstra
On Wed, Jun 22, 2016 at 06:03:17PM +0100, Morten Rasmussen wrote: > @@ -6905,11 +6906,19 @@ static int build_sched_domains(const struct cpumask > *cpu_map, > /* Attach the domains */ > rcu_read_lock(); > for_each_cpu(i, cpu_map) { > + rq = cpu_rq(i); >

[PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-06-22 Thread Morten Rasmussen
From: Dietmar Eggemann To be able to compare the capacity of the target cpu with the highest available cpu capacity, store the maximum per-cpu capacity in the root domain. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by:

[PATCH v2 06/13] sched: Store maximum per-cpu capacity in root domain

2016-06-22 Thread Morten Rasmussen
From: Dietmar Eggemann To be able to compare the capacity of the target cpu with the highest available cpu capacity, store the maximum per-cpu capacity in the root domain. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Dietmar Eggemann Signed-off-by: Morten Rasmussen ---