From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: "cpufreq: fix serialization issues with freq change notifiers" breaks cpufreq too Date: Wed, 11 Sep 2013 15:28:26 +0200 Message-ID: <11617025.s66NV700Er@vostro.rjw.lan> References: Mime-Version: 1.0 Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: cpufreq-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Guennadi Liakhovetski Cc: Viresh Kumar , Greg KH , "Rafael J. Wysocki" , Linux Kernel Mailing List , "linux-arm-kernel@lists.infradead.org" , "linux-pm@vger.kernel.org" , "cpufreq@vger.kernel.org" , SH-Linux , Magnus Damm On Wednesday, September 11, 2013 10:15:53 AM Guennadi Liakhovetski wrote: > On Wed, 11 Sep 2013, Viresh Kumar wrote: > > > On 10 September 2013 22:37, Guennadi Liakhovetski wrote: > > > On Tue, 10 Sep 2013, Viresh Kumar wrote: > > >> Quite straight forward actually.. > > > > > > Apparently, not quite. > > > > I overlooked the situation where we return early from ->target() routines.. :( > > > > Please try attached patches, I will repost them later (once I am able to > > convince Rafael that these are really important :) ) > > I'd rather wait until Rafael is convinced, then we'll see. I'm going to revert the commit that was the source of these issues, but that doesn't mean that everything is now rosy. We need to fix the concurrency problems that Viresh was trying to fix in that commit, so it would be nice if you could verify whether or not he is on the right track. Thanks, Rafael