Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-11 Thread Vincent Guittot
On 11 September 2015 at 15:21, Dietmar Eggemann wrote: > On 04/09/15 08:26, Vincent Guittot wrote: >> On 3 September 2015 at 21:58, Dietmar Eggemann >> wrote: > > [...] > >> So, with the patch below that updates the arm definition of >> arch_scale_cpu_capacity, you can add my Acked-by: Vincent

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-11 Thread Dietmar Eggemann
On 04/09/15 08:26, Vincent Guittot wrote: > On 3 September 2015 at 21:58, Dietmar Eggemann > wrote: [...] > So, with the patch below that updates the arm definition of > arch_scale_cpu_capacity, you can add my Acked-by: Vincent Guittot > on this patch and the additional one > below My tests

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-11 Thread Dietmar Eggemann
On 04/09/15 08:26, Vincent Guittot wrote: > On 3 September 2015 at 21:58, Dietmar Eggemann > wrote: [...] > So, with the patch below that updates the arm definition of > arch_scale_cpu_capacity, you can add my Acked-by: Vincent Guittot >

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-11 Thread Vincent Guittot
On 11 September 2015 at 15:21, Dietmar Eggemann wrote: > On 04/09/15 08:26, Vincent Guittot wrote: >> On 3 September 2015 at 21:58, Dietmar Eggemann >> wrote: > > [...] > >> So, with the patch below that updates the arm definition of >>

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-07 Thread Dietmar Eggemann
On 04/09/15 08:26, Vincent Guittot wrote: > On 3 September 2015 at 21:58, Dietmar Eggemann > wrote: [...] >>> So you change the way to declare arch_scale_cpu_capacity but i don't >>> see the update of the arm arch which declare a >>> arch_scale_cpu_capacity to reflect this change in your

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-07 Thread Dietmar Eggemann
On 04/09/15 08:26, Vincent Guittot wrote: > On 3 September 2015 at 21:58, Dietmar Eggemann > wrote: [...] >>> So you change the way to declare arch_scale_cpu_capacity but i don't >>> see the update of the arm arch which declare a >>> arch_scale_cpu_capacity to reflect

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-04 Thread Vincent Guittot
On 3 September 2015 at 21:58, Dietmar Eggemann wrote: > Hi Vincent, > > On 02/09/15 10:31, Vincent Guittot wrote: >> Hi Morten, >> >> On 14 August 2015 at 18:23, Morten Rasmussen >> wrote: >>> Bring arch_scale_cpu_capacity() in line with the recent change of its >>> arch_scale_freq_capacity()

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-04 Thread Vincent Guittot
On 3 September 2015 at 21:58, Dietmar Eggemann wrote: > Hi Vincent, > > On 02/09/15 10:31, Vincent Guittot wrote: >> Hi Morten, >> >> On 14 August 2015 at 18:23, Morten Rasmussen >> wrote: >>> Bring arch_scale_cpu_capacity() in line with the

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-03 Thread Dietmar Eggemann
Hi Vincent, On 02/09/15 10:31, Vincent Guittot wrote: > Hi Morten, > > On 14 August 2015 at 18:23, Morten Rasmussen wrote: >> Bring arch_scale_cpu_capacity() in line with the recent change of its >> arch_scale_freq_capacity() sibling in commit dfbca41f3479 ("sched: >> Optimize freq invariant

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-03 Thread Dietmar Eggemann
Hi Vincent, On 02/09/15 10:31, Vincent Guittot wrote: > Hi Morten, > > On 14 August 2015 at 18:23, Morten Rasmussen wrote: >> Bring arch_scale_cpu_capacity() in line with the recent change of its >> arch_scale_freq_capacity() sibling in commit dfbca41f3479 ("sched: >>

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-02 Thread Vincent Guittot
On 2 September 2015 at 11:31, Vincent Guittot wrote: > Hi Morten, > > On 14 August 2015 at 18:23, Morten Rasmussen wrote: >> Bring arch_scale_cpu_capacity() in line with the recent change of its >> arch_scale_freq_capacity() sibling in commit dfbca41f3479 ("sched: >> Optimize freq invariant

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-02 Thread Vincent Guittot
Hi Morten, On 14 August 2015 at 18:23, Morten Rasmussen wrote: > Bring arch_scale_cpu_capacity() in line with the recent change of its > arch_scale_freq_capacity() sibling in commit dfbca41f3479 ("sched: > Optimize freq invariant accounting") from weak function to #define to > allow inlining of

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-02 Thread Vincent Guittot
Hi Morten, On 14 August 2015 at 18:23, Morten Rasmussen wrote: > Bring arch_scale_cpu_capacity() in line with the recent change of its > arch_scale_freq_capacity() sibling in commit dfbca41f3479 ("sched: > Optimize freq invariant accounting") from weak function to

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-02 Thread Vincent Guittot
On 2 September 2015 at 11:31, Vincent Guittot wrote: > Hi Morten, > > On 14 August 2015 at 18:23, Morten Rasmussen wrote: >> Bring arch_scale_cpu_capacity() in line with the recent change of its >> arch_scale_freq_capacity() sibling in commit

[PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-08-14 Thread Morten Rasmussen
Bring arch_scale_cpu_capacity() in line with the recent change of its arch_scale_freq_capacity() sibling in commit dfbca41f3479 ("sched: Optimize freq invariant accounting") from weak function to #define to allow inlining of the function. While at it, remove the ARCH_CAPACITY sched_feature as

[PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-08-14 Thread Morten Rasmussen
Bring arch_scale_cpu_capacity() in line with the recent change of its arch_scale_freq_capacity() sibling in commit dfbca41f3479 (sched: Optimize freq invariant accounting) from weak function to #define to allow inlining of the function. While at it, remove the ARCH_CAPACITY sched_feature as well.