Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-18 Thread Laurent Pinchart
Hi Magnus, On Friday 19 December 2014 09:03:00 Magnus Damm wrote: > On Wed, Dec 17, 2014 at 11:08 AM, Laurent Pinchart wrote: > > Magnus, you have told me that you've performed tests on Marzen with > > CONFIG_PREEMPT_NONE and with both TWD enabled and disabled. If you could > > perform the same te

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-18 Thread Magnus Damm
Hi Laurent, On Wed, Dec 17, 2014 at 11:08 AM, Laurent Pinchart wrote: > Magnus, you have told me that you've performed tests on Marzen with > CONFIG_PREEMPT_NONE and with both TWD enabled and disabled. If you could > perform the same tests with CONFIG_PREEMPT instead without noticing any > regres

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Magnus Damm
Hi Laurent, On Wed, Dec 17, 2014 at 11:08 AM, Laurent Pinchart wrote: > Hi Magnus and Geert, > > On Wednesday 17 December 2014 10:30:33 Magnus Damm wrote: >> On Tue, Dec 16, 2014 at 9:40 PM, Geert Uytterhoeven wrote: >> > On Tue, Dec 16, 2014 at 12:46 PM, Magnus Damm wrote: >> >>> Could you pleas

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Laurent Pinchart
Hi Geert, On Wednesday 17 December 2014 13:11:05 Geert Uytterhoeven wrote: > On Wed, Dec 17, 2014 at 1:04 PM, Laurent Pinchart wrote: > > On Wednesday 17 December 2014 10:42:52 Geert Uytterhoeven wrote: > >> On Wed, Dec 17, 2014 at 9:30 AM, Geert Uytterhoeven wrote: > >> > On Wed, Dec 17, 2014 at

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Geert Uytterhoeven
Hi Laurent, On Wed, Dec 17, 2014 at 1:04 PM, Laurent Pinchart wrote: > On Wednesday 17 December 2014 10:42:52 Geert Uytterhoeven wrote: >> On Wed, Dec 17, 2014 at 9:30 AM, Geert Uytterhoeven wrote: >> > On Wed, Dec 17, 2014 at 3:08 AM, Laurent Pinchart wrote: >> Kzm9g-reference still hangs a

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Laurent Pinchart
Hi Geert, On Wednesday 17 December 2014 10:42:52 Geert Uytterhoeven wrote: > On Wed, Dec 17, 2014 at 9:30 AM, Geert Uytterhoeven wrote: > > On Wed, Dec 17, 2014 at 3:08 AM, Laurent Pinchart wrote: > Kzm9g-reference still hangs at "Calibrating local timer..." (it did > work at some point

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Laurent Pinchart
Hi Geert, On Wednesday 17 December 2014 09:30:57 Geert Uytterhoeven wrote: > On Wed, Dec 17, 2014 at 3:08 AM, Laurent Pinchart wrote: > >> > Kzm9g-reference still hangs at "Calibrating local timer..." (it did > >> > work at some point in the past). > > > > kzm9g-reference boots for me with kzm9g_

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Geert Uytterhoeven
Hi Laurent, Magnus, On Wed, Dec 17, 2014 at 9:30 AM, Geert Uytterhoeven wrote: > On Wed, Dec 17, 2014 at 3:08 AM, Laurent Pinchart > wrote: >>> > Kzm9g-reference still hangs at "Calibrating local timer..." (it did work >>> > at some point in the past). >> >> kzm9g-reference boots for me with kzm

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Geert Uytterhoeven
Hi Laurent, On Wed, Dec 17, 2014 at 3:08 AM, Laurent Pinchart wrote: >> > Kzm9g-reference still hangs at "Calibrating local timer..." (it did work >> > at some point in the past). > > kzm9g-reference boots for me with kzm9g_defconfig on Simon's devel branch with OK. I had expected the breakage

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-17 Thread Geert Uytterhoeven
Hi Magnus, On Wed, Dec 17, 2014 at 2:30 AM, Magnus Damm wrote: >> Armadillo-legacy is broken, and probably won't come back (cfr. >> https://lkml.org/lkml/2014/12/2/339). >> >> Kzm9g-reference still hangs at "Calibrating local timer..." (it did work >> at some point in the past). > > When the time

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Laurent Pinchart
Hi Magnus and Geert, On Wednesday 17 December 2014 10:30:33 Magnus Damm wrote: > On Tue, Dec 16, 2014 at 9:40 PM, Geert Uytterhoeven wrote: > > On Tue, Dec 16, 2014 at 12:46 PM, Magnus Damm wrote: > >>> Could you please confirm that you've tested both CONFIG_PREEMPT_NONE and > >>> CONFIG_PREEMPT w

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Magnus Damm
Hi Geert, On Tue, Dec 16, 2014 at 9:40 PM, Geert Uytterhoeven wrote: > Hi Magnus, > > On Tue, Dec 16, 2014 at 12:46 PM, Magnus Damm wrote: >>> Could you please confirm that you've tested both CONFIG_PREEMPT_NONE and >>> CONFIG_PREEMPT with and without the ARM TWD times, and that you've booted to

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Laurent Pinchart
Hi Daniel, On Tuesday 16 December 2014 12:54:56 Daniel Lezcano wrote: > On 12/16/2014 12:46 PM, Magnus Damm wrote: > > On Tue, Dec 16, 2014 at 8:20 PM, Laurent Pinchart wrote: > >> On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: > >>> On 12/16/2014 10:48 AM, Magnus Damm wrote: > Fr

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Laurent Pinchart
Hi Magnus, On Tuesday 16 December 2014 20:46:33 Magnus Damm wrote: > On Tue, Dec 16, 2014 at 8:20 PM, Laurent Pinchart wrote: > > On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: > >> On 12/16/2014 10:48 AM, Magnus Damm wrote: > >> > From: Magnus Damm > >> > > >> > Update the TMU drive

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Geert Uytterhoeven
Hi Magnus, On Tue, Dec 16, 2014 at 12:46 PM, Magnus Damm wrote: >> Could you please confirm that you've tested both CONFIG_PREEMPT_NONE and >> CONFIG_PREEMPT with and without the ARM TWD times, and that you've booted to >> userspace and tested timer broadcast on all CPUs ? > > No I have not. I've

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Daniel Lezcano
On 12/16/2014 12:46 PM, Magnus Damm wrote: Hi Laurent, On Tue, Dec 16, 2014 at 8:20 PM, Laurent Pinchart wrote: Hi Daniel, On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: On 12/16/2014 10:48 AM, Magnus Damm wrote: From: Magnus Damm Update the TMU driver to use cpu_possible_mask

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Magnus Damm
Hi Laurent, On Tue, Dec 16, 2014 at 8:20 PM, Laurent Pinchart wrote: > Hi Daniel, > > On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: >> On 12/16/2014 10:48 AM, Magnus Damm wrote: >> > From: Magnus Damm >> > >> > Update the TMU driver to use cpu_possible_mask as cpumask to make >> > r

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Daniel Lezcano
On 12/16/2014 12:20 PM, Laurent Pinchart wrote: Hi Daniel, On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: On 12/16/2014 10:48 AM, Magnus Damm wrote: From: Magnus Damm Update the TMU driver to use cpu_possible_mask as cpumask to make r8a7779 SMP work as expected with or without th

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Laurent Pinchart
Hi Magnus, On Tuesday 16 December 2014 13:20:56 Laurent Pinchart wrote: > On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: > > On 12/16/2014 10:48 AM, Magnus Damm wrote: > > > From: Magnus Damm > > > > > > Update the TMU driver to use cpu_possible_mask as cpumask to make > > > r8a7779

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Laurent Pinchart
Hi Daniel, On Tuesday 16 December 2014 12:14:40 Daniel Lezcano wrote: > On 12/16/2014 10:48 AM, Magnus Damm wrote: > > From: Magnus Damm > > > > Update the TMU driver to use cpu_possible_mask as cpumask to make > > r8a7779 SMP work as expected with or without the ARM TWD timer. > > > > Signed-o

Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Daniel Lezcano
On 12/16/2014 10:48 AM, Magnus Damm wrote: From: Magnus Damm Update the TMU driver to use cpu_possible_mask as cpumask to make r8a7779 SMP work as expected with or without the ARM TWD timer. Signed-off-by: Magnus Damm Applied as a 3.18 fix. Thanks ! -- Daniel ps: May I suggest to use t

[PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast

2014-12-16 Thread Magnus Damm
From: Magnus Damm Update the TMU driver to use cpu_possible_mask as cpumask to make r8a7779 SMP work as expected with or without the ARM TWD timer. Signed-off-by: Magnus Damm --- drivers/clocksource/sh_tmu.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- 0001/drivers/clocksourc