Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754983AbaKEOn1 (ORCPT ); Wed, 5 Nov 2014 09:43:27 -0500 Received: from mail-wg0-f43.google.com ([74.125.82.43]:42217 "EHLO mail-wg0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754660AbaKEOn0 (ORCPT ); Wed, 5 Nov 2014 09:43:26 -0500 MIME-Version: 1.0 In-Reply-To: <20141105092707.7c9bb1ae@gandalf.local.home> References: <20141031170326.1491a63f@gandalf.local.home> <1414913427.5380.110.camel@marge.simpson.net> <20141105092707.7c9bb1ae@gandalf.local.home> Date: Wed, 5 Nov 2014 15:43:24 +0100 Message-ID: Subject: Re: [ANNOUNCE] 3.14.23-rt20 From: Juerg Haefliger To: Steven Rostedt Cc: Mike Galbraith , LKML , linux-rt-users , Thomas Gleixner , Carsten Emde , John Kacur , Sebastian Andrzej Siewior , Clark Williams Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 5, 2014 at 3:27 PM, Steven Rostedt wrote: > On Wed, 5 Nov 2014 14:50:41 +0100 > Juerg Haefliger wrote: > >> On Sun, Nov 2, 2014 at 8:30 AM, Mike Galbraith >> wrote: >> > >> > On Fri, 2014-10-31 at 17:03 -0400, Steven Rostedt wrote: >> > > Dear RT Folks, >> > > >> > > I'm pleased to announce the 3.14.23-rt20 stable release. >> > > >> > > This is the first 3.14-rt release in the stable-rt series. Normally I >> > > wait till the next development release is out before I pull in a new >> > > one. That is, I would pull in 3.14-rt when 3.16-rt or later was >> > > released. But because development is now moving at a "hobbyist rate" >> > > (read http://lwn.net/Articles/617140/ for details) >> > > and 3.14-rt is no longer being developed against, I figured it was >> > time >> > > to put it under the "stable-rt" umbrella. >> > >> > I piddled about with it yesterday, found that you can't change cpufreq >> > governor IFF the tree is configured as rt, but works fine as voluntary >> > preempt. >> >> The problem seems to be this patch: https://lkml.org/lkml/2014/4/8/584 >> >> The cpufreq code does nested down_read_trylocks and only the first one >> succeeds: >> >> drivers/cpufreq/cpufreq.c: >> store >> down_read_trylock(cpufreq_rwsem) <- succeeds >> store_scaling_governor >> cpufreq_get_policy >> cpufreq_cpu_get >> down_read_trylock(cpufreq_rwsem) <-- fails >> >> Reverting the above patch 'fixes' the problem. I don't understand Steven's >> commit comment that readers of rwsem are not allowed to nest in mainline >> since this works just fine in mainline. > > When we allow multiple readers, this will be allowed. But even in > mainline, if a writer were to come in and block between those two > down_read_trylocks(), the second trylock would fail. Thanks for the explanation. So is this considered a temporary failure until multiple readers are allowed or does cpufreq need fixing or something else? Just trying to figure out what to do next. ...Juerg > PREEMPT_RT just has that fail all the time as we only allow an rwsem to > be held by a single reader. > > > -- Steve > > > >> >> ...Juerg >> >> >> >> > I'll poke about for the entertainment value. Having no >> > personal need/use for rt detracts from its hobby value somewhat, but rt >> > problems do have a tendency to be 'entertaining'. >> > >> > I'll follow up with a few patches that folks can apply to their trees if >> > they so desire. There being no devel tree to submit against, I can't do >> > a proper submission (rules), and some of them you surely don't want :) >> > >> > -Mike >> > >> > >> > >> > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/