Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755930Ab2HZAMr (ORCPT ); Sat, 25 Aug 2012 20:12:47 -0400 Received: from e32.co.us.ibm.com ([32.97.110.150]:36715 "EHLO e32.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754773Ab2HZAMn (ORCPT ); Sat, 25 Aug 2012 20:12:43 -0400 Date: Sat, 25 Aug 2012 17:12:33 -0700 From: "Paul E. McKenney" To: Michael Cree Cc: Frederic Weisbecker , Ben Hutchings , LKML , Chris Zankel , "3.2.x.." , Chen Liqin , Lennox Wu , "James E.J. Bottomley" , Helge Deller , Parisc , David Howells , Koichi Yasutake , Geert Uytterhoeven , m68k , Hirokazu Takata , Yoshinori Sato , Mikael Starvik , Jesper Nilsson , Cris , Richard Henderson , Ivan Kokshaysky , Matt Turner , alpha Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2 Message-ID: <20120826001233.GH3436@linux.vnet.ibm.com> Reply-To: paulmck@linux.vnet.ibm.com References: <1345733915-20040-1-git-send-email-fweisbec@gmail.com> <20120824212647.GU2472@linux.vnet.ibm.com> <1345857554.4840.43.camel@deadeye.wl.decadent.org.uk> <20120825035047.GB3436@linux.vnet.ibm.com> <20120825131647.GA1994@somewhere> <20120825161806.GD3436@linux.vnet.ibm.com> <5039406F.9040604@orcon.net.nz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5039406F.9040604@orcon.net.nz> User-Agent: Mutt/1.5.21 (2010-09-15) X-Content-Scanned: Fidelis XPS MAILER x-cbid: 12082600-2356-0000-0000-00000197F915 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3081 Lines: 74 On Sun, Aug 26, 2012 at 09:15:27AM +1200, Michael Cree wrote: > On 26/08/12 04:18, Paul E. McKenney wrote: > > On Sat, Aug 25, 2012 at 03:16:49PM +0200, Frederic Weisbecker wrote: > >> On Fri, Aug 24, 2012 at 08:50:47PM -0700, Paul E. McKenney wrote: > >>> On Sat, Aug 25, 2012 at 02:19:14AM +0100, Ben Hutchings wrote: > >>>> On Fri, 2012-08-24 at 14:26 -0700, Paul E. McKenney wrote: > >>>>> On Thu, Aug 23, 2012 at 04:58:24PM +0200, Frederic Weisbecker wrote: > >>>>>> Hi, > >>>>>> > >>>>>> Changes since v1: > >>>>>> > >>>>>> - Fixed preempt handling in alpha idle loop > >>>>>> - added ack from Geert > >>>>>> - fixed stable email address, sorry :-/ > >>>>>> > >>>>>> This time I built tested everywhere but: h8300 (compiler internal error), > >>>>>> and mn10300, parisc, score (cross compilers not available in > >>>>>> ftp://ftp.kernel.org/pub/tools/crosstool/files/bin/x86_64/4.6.3/) > >>>>>> > >>>>>> For testing, you can pull from: > >>>>>> > >>>>>> git://github.com/fweisbec/linux-dynticks.git > >>>>>> rcu/idle-fix-v2 > >>>>>> > >>>>>> Thanks. > >>>>> > >>>>> I have queued these on -rcu branch rcu/idle: > >>>>> > >>>>> git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git > >>>>> > >>>>> This problem has been in place since 3.3, so it is hard to argue that > >>>>> it is a regression for this merge window. I have therefore queued it > >>>>> for 3.7. > >>>> > >>>> I don't follow that; I would expect any serious bug fix (serious enough > >>>> for a stable update) to be acceptable for 3.6 at this point. > >>> > >>> OK, if any of the arch maintainers wishes to submit the patch to 3.6, > >>> they are free to do so -- just let me know and I will drop the patch from > >>> my tree. > >>> > >>> That said, all this does is cause spurious warnings to be printed, so > >>> not sure it really qualifies as serious. But I am happy to leave that > >>> decision with the individual arch maintainers -- it is their arch, > >>> after all, so their decision. > >> > >> Couldn't that cause hung tasks due to long lasting synchronize_rcu() ? > > > > In theory, definitely. In practice, they haven't been running into it, > > or they would be reporting hangs. > > I am hereby reporting that RCU CPU stall warnings and hung tasks are > being experienced on SMP kernels built for generic Alpha. This problem > dates back quite a few kernel releases. The discussed patches appear to > fix the problem. OK, fair enough. > A backport to the 3.2 kernel, of at least the Alpha patches, would be > very much appreciated! :-) Both Alpha patches should apply as-is back to 3.3, and should also fix the problem. Could you please check this on the versions of interest? If you are seeing something like this on 3.2 or earlier, you have some other problem that needs to be tracked down separately. Thanx, Paul -- 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/