From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Fri, 19 Dec 2014 00:46:37 +0000 Subject: Re: [PATCH] clocksource: sh_tmu: Set cpu_possible_mask to fix SMP broadcast Message-Id: <2833437.mUHC45LYXV@avalon> List-Id: References: <20141216094854.22639.69181.sendpatchset@w520> <2855304.l47b4KVhTf@avalon> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Magnus Damm Cc: Geert Uytterhoeven , Daniel Lezcano , linux-kernel , SH-Linux , "Simon Horman [Horms]" , Thomas Gleixner 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 tests with CONFIG_PREEMPT instead without noticing any > > regression I think we can merge your patch. Whatever breakage it has > > caused in the past is likely hidden somewhere beyond eyesight. > > I've now successfully boot tested the different PREEMPT settings on > r8a7779 Marzen multiplatform. Everything seems to work as expected. > > Compared to before I've been a bit more lazy, so I've kept the timer > config to default which means TWD and TMU enabled. If your test setup is still around could you quickly test PREEMPT + TMU without TWD ? > The kernel tree is renesas-devel-20141218-v3.18 which is v3.18 plus local > integration stuff including TWD DT. I've applied this patch too. -- Regards, Laurent Pinchart