Re: [RFC 0/3] sched: fix nr_busy_cpus

2012-11-19 Thread Viresh Kumar
On 19 November 2012 22:08, Vincent Guittot wrote: > The nr_busy_cpus field of the sched_group_power is sometime different from 0 > whereas the platform is fully idle. This serie fixes 3 use cases: > - when the SCHED softirq is raised on an idle core for idle load balance but >the platform doe

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Viresh Kumar
On 19 November 2012 22:44, Andrey Konovalov wrote: > I won't pull the big-LITTLE-MP-master-v12 into the > linux-linaro-core-tracking tree today due to the issues found by Tixy. > > Tomorrow evening I am going to pull this topic anyway - whether these issues > are resolved, or not. If the build err

[PATCH] dt: add helper function to read u8 & u16 variables & arrays

2012-11-19 Thread Viresh Kumar
This adds following helper routines: - of_property_read_u8_array() - of_property_read_u16_array() - of_property_read_u8() - of_property_read_u16() This expects arrays from DT to be passed as: - u8 array: property = /bits/ 8 <0x50 0x60 0x70>; - u16 array: property = /bits/ 16 <0x500

[PATCH] sched: Explicit division calls on 64-bit integers

2012-11-19 Thread Preeti U Murthy
Certain gcc tool chains convert the division on a 64-bit dividend into a __aeabi_uldivmod call which does unnecessary 64-bit by 64-bit divides although the divisor is 32-bit.This 64 by 64 bit division is not implemented in the kernel for reasons of efficiency,which results in undefined reference er

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Andrey Konovalov
On 11/19/2012 10:42 PM, Jon Medhurst (Tixy) wrote: On Mon, 2012-11-19 at 22:24 +0400, Andrey Konovalov wrote: On 11/19/2012 10:17 PM, Jon Medhurst (Tixy) wrote: On Mon, 2012-11-19 at 21:14 +0400, Andrey Konovalov wrote: Viresh, I won't pull the big-LITTLE-MP-master-v12 into the linux-linaro-c

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Jon Medhurst (Tixy)
On Mon, 2012-11-19 at 22:24 +0400, Andrey Konovalov wrote: > On 11/19/2012 10:17 PM, Jon Medhurst (Tixy) wrote: > > On Mon, 2012-11-19 at 21:14 +0400, Andrey Konovalov wrote: > >> Viresh, > >> > >> I won't pull the big-LITTLE-MP-master-v12 into the > >> linux-linaro-core-tracking tree today due to

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Andrey Konovalov
On 11/19/2012 10:17 PM, Jon Medhurst (Tixy) wrote: On Mon, 2012-11-19 at 21:14 +0400, Andrey Konovalov wrote: Viresh, I won't pull the big-LITTLE-MP-master-v12 into the linux-linaro-core-tracking tree today due to the issues found by Tixy. Tomorrow evening I am going to pull this topic anyway

RE: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Chris Redpath
Hi Tixy, I've got your kernel and the JB filesystem modified for USB booting here. There is no boot delay for me - it takes just over 70s from boot monitor to mouse pointer. Catch you in the morning :) Chris > -Original Message- > From: Jon Medhurst (Tixy) [mailto:t...@linaro.org] > S

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Jon Medhurst (Tixy)
On Mon, 2012-11-19 at 21:14 +0400, Andrey Konovalov wrote: > Viresh, > > I won't pull the big-LITTLE-MP-master-v12 into the > linux-linaro-core-tracking tree today due to the issues found by Tixy. > > Tomorrow evening I am going to pull this topic anyway - whether these > issues are resolved, o

Re: [PATCH Resend V2] dt: add helper function to read u8 & u16 variables & arrays

2012-11-19 Thread Viresh Kumar
On 19 November 2012 21:58, Stephen Warren wrote: > Support for byte- and word- properties is relatively recent I believe > (or at least, the /bits/ syntax is). Which dtc version are you using? Ok, i was on a older version. I just saw this patch now: commit cd296721a9645f9f28800a072490fa15458d1fb

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Andrey Konovalov
Viresh, I won't pull the big-LITTLE-MP-master-v12 into the linux-linaro-core-tracking tree today due to the issues found by Tixy. Tomorrow evening I am going to pull this topic anyway - whether these issues are resolved, or not. If the build error is not fixed by Thursday morning UTC, I'll m

[RFC 2/3] sched: fix init NOHZ_IDLE flag

2012-11-19 Thread Vincent Guittot
On my smp platform which is made of 5 cores in 2 clusters,I have the nr_busy_cpu field of sched_group_power struct that is not null when the platform is fully idle. The root cause seems to be: During the boot sequence, some CPUs reach the idle loop and set their NOHZ_IDLE flag while waiting for oth

[RFC 3/3] sched: fix update NOHZ_IDLE flag

2012-11-19 Thread Vincent Guittot
The function nohz_kick_needed modifies NOHZ_IDLE flag that is used to update the nr_busy_cpus of the sched_group. When the sched_domain are updated (because of the unplug of a CPUs as an example) a null_domain is attached to CPUs. We have to test likely(!on_null_domain(cpu) first in order to detect

[RFC 0/3] sched: fix nr_busy_cpus

2012-11-19 Thread Vincent Guittot
The nr_busy_cpus field of the sched_group_power is sometime different from 0 whereas the platform is fully idle. This serie fixes 3 use cases: - when the SCHED softirq is raised on an idle core for idle load balance but the platform doesn't go out of the cpuidle state - when some CPUs enter id

[RFC 1/3] sched: fix nr_busy_cpus with coupled cpuidle

2012-11-19 Thread Vincent Guittot
With the coupled cpuidle driver (but probably also with other drivers), a CPU loops in a temporary safe state while waiting for other CPUs of its cluster to be ready to enter the coupled C-state. If an IRQ or a softirq occurs, the CPU will stay in this internal loop if there is no need to resched.

Re: [PATCH Resend V2] dt: add helper function to read u8 & u16 variables & arrays

2012-11-19 Thread Stephen Warren
On 11/18/2012 11:41 PM, Viresh Kumar wrote: > On 19 November 2012 12:05, Rajanikanth HV wrote: >> On 19 November 2012 12:00, Viresh Kumar wrote: >>> Firstly you tried square braces [ ], I am not sure if that is allowed. >>> Can you point me to the specification? >> http://www.devicetree.org/Devic

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Jon Medhurst (Tixy)
On Mon, 2012-11-19 at 15:57 +, Chris Redpath wrote: > These patches are mine and Olivier's, can you tell me what your config is to > see even a 90s boot? I haven't noticed any boot-time extension on my system, > but I'm booting from the A15 cluster. I'd like to reproduce your system here. Th

RE: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Chris Redpath
Hi Tixy, These patches are mine and Olivier's, can you tell me what your config is to see even a 90s boot? I haven't noticed any boot-time extension on my system, but I'm booting from the A15 cluster. I'd like to reproduce your system here. The build error is my mistake, I need to know which CP

Re: [GIT PULL]; big LITTLE MP master v12

2012-11-19 Thread Jon Medhurst (Tixy)
On Sun, 2012-11-18 at 10:40 +0530, Viresh Kumar wrote: > Hi Andrey, > > Please pull big-LITTLE-MP-master-v12 with following updates: > > - Based on v3.7-rc5 > - Stats: > - Total Patches: 62 > - New Patches: 1 >- genirq: Add default affinity mask command line option in >

Re: [PATCH] genirq: Add default affinity mask command line option

2012-11-19 Thread Punit Agrawal
Hi Francesco, Thanks for your comments on the patch. On 18/11/12 10:41, Francesco Lavra wrote: Hi, On 11/12/2012 05:57 PM, Punit Agrawal wrote: I am attaching a patch by Thomas Gleixner which adds a kernel command line parameter to set the defauilt IRQ affinity mask. Could you please integrat

Re: [HMP tunables v2][PATCH 3/7] ARM: TC2: Re-enable SD_SHARE_POWERLINE

2012-11-19 Thread Morten Rasmussen
On 19/11/12 14:09, Vincent Guittot wrote: On 19 November 2012 14:36, Morten Rasmussen wrote: On 19/11/12 12:23, Vincent Guittot wrote: On 19 November 2012 13:08, Morten Rasmussen wrote: Hi Vincent, On 19/11/12 09:20, Vincent Guittot wrote: Hi, On 16 November 2012 19:32, Liviu Dudau

Re: [HMP tunables v2][PATCH 3/7] ARM: TC2: Re-enable SD_SHARE_POWERLINE

2012-11-19 Thread Vincent Guittot
On 19 November 2012 14:36, Morten Rasmussen wrote: > On 19/11/12 12:23, Vincent Guittot wrote: >> >> On 19 November 2012 13:08, Morten Rasmussen >> wrote: >>> >>> Hi Vincent, >>> >>> >>> On 19/11/12 09:20, Vincent Guittot wrote: Hi, On 16 November 2012 19:32, Liviu Dudau

Re: [HMP tunables v2][PATCH 3/7] ARM: TC2: Re-enable SD_SHARE_POWERLINE

2012-11-19 Thread Morten Rasmussen
On 19/11/12 12:23, Vincent Guittot wrote: On 19 November 2012 13:08, Morten Rasmussen wrote: Hi Vincent, On 19/11/12 09:20, Vincent Guittot wrote: Hi, On 16 November 2012 19:32, Liviu Dudau wrote: From: Morten Rasmussen Re-enable SD_SHARE_POWERLINE to reflect the power domains of TC2.

Re: [HMP tunables v2][PATCH 3/7] ARM: TC2: Re-enable SD_SHARE_POWERLINE

2012-11-19 Thread Vincent Guittot
On 19 November 2012 13:08, Morten Rasmussen wrote: > Hi Vincent, > > > On 19/11/12 09:20, Vincent Guittot wrote: >> >> Hi, >> >> On 16 November 2012 19:32, Liviu Dudau wrote: >>> >>> From: Morten Rasmussen >>> >>> Re-enable SD_SHARE_POWERLINE to reflect the power domains of TC2. >>> --- >>> ar

Re: [HMP tunables v2][PATCH 3/7] ARM: TC2: Re-enable SD_SHARE_POWERLINE

2012-11-19 Thread Morten Rasmussen
Hi Vincent, On 19/11/12 09:20, Vincent Guittot wrote: Hi, On 16 November 2012 19:32, Liviu Dudau wrote: From: Morten Rasmussen Re-enable SD_SHARE_POWERLINE to reflect the power domains of TC2. --- arch/arm/kernel/topology.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

Re: [HMP tunables v2][PATCH 2/7] sched: SD_SHARE_POWERLINE buddy selection fix

2012-11-19 Thread Vincent Guittot
On 19 November 2012 10:10, Viresh Kumar wrote: > On 19 November 2012 14:33, Vincent Guittot wrote: >> I would prefer that you use the branch in the git tree below instead >> which is the final correction >> http://git.linaro.org/gitweb?p=people/vingu/kernel.git;a=shortlog;h=refs/heads/sched-pack-

Re: [HMP tunables v2][PATCH 3/7] ARM: TC2: Re-enable SD_SHARE_POWERLINE

2012-11-19 Thread Vincent Guittot
Hi, On 16 November 2012 19:32, Liviu Dudau wrote: > From: Morten Rasmussen > > Re-enable SD_SHARE_POWERLINE to reflect the power domains of TC2. > --- > arch/arm/kernel/topology.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/kernel/topology.c b/arch/arm/k

Re: [HMP tunables v2][PATCH 2/7] sched: SD_SHARE_POWERLINE buddy selection fix

2012-11-19 Thread Viresh Kumar
On 19 November 2012 14:33, Vincent Guittot wrote: > I would prefer that you use the branch in the git tree below instead > which is the final correction > http://git.linaro.org/gitweb?p=people/vingu/kernel.git;a=shortlog;h=refs/heads/sched-pack-small-task-v1-fixed Hi Vingu, I have applied 3 patc

Re: [HMP tunables v2][PATCH 2/7] sched: SD_SHARE_POWERLINE buddy selection fix

2012-11-19 Thread Vincent Guittot
Hi, I would prefer that you use the branch in the git tree below instead which is the final correction http://git.linaro.org/gitweb?p=people/vingu/kernel.git;a=shortlog;h=refs/heads/sched-pack-small-task-v1-fixed Regards Vincent On 16 November 2012 19:32, Liviu Dudau wrote: > From: Morten Rasmu