From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frederic Weisbecker Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2 Date: Tue, 11 Sep 2012 14:35:44 +0200 Message-ID: <20120911123528.GG21277@somewhere.redhat.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> <20120910215541.GE21277@somewhere.redhat.com> <20120910225845.GC4257@linux.vnet.ibm.com> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=AMprI/Hcm7mGee5G8HDhAgGg8ExmhdWkhqS8mMkeJZQ=; b=z93G9eWSyKY0ptc27iVLZ4IeKV7GZWxUBJyPrY3dWwJC0vh5QqBJVyQVoMpmC41xrP OrjF0SoVFZc/NL7lMWl0M4LG2HPhVeRWRq5SIOzsqOYrfSyaCrSopF6fOZ41kmcE39Xg Vta1l5RYPNZyRTdBSSpA8Seb6BUX6GUwMEN+ygcsaYyrZk7yfJ8K1ZV8CrFGnFYBj1OA +sLUsYOpeuHaEGqoiIbIxRmgfcNU1FqSmuhKsq2+xODRgrsbDFc+hb7hJ9NbYkbEVQOh rsKbo+0wcinIPdLv4O6cW40v0uTTaHfwTJ/TmguqaHQBh+po3rQu9ieOAtZmRZNp5mmI KaIw== Content-Disposition: inline In-Reply-To: <20120910225845.GC4257@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Paul E. McKenney" Cc: 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 , Michael Cree <> On Mon, Sep 10, 2012 at 03:58:45PM -0700, Paul E. McKenney wrote: > > Anyway, should I resend the patchset with this thing fixed? > > Please feel free to do so. ;-) > > Thanx, Paul > So here it is. On the menu for the updated branch: - fixed stable tags (use # 3.3+) - not sure what to do with "alpha: Fix preemption handling in idle loop", tag as stable or not? - added tested-by Michael Cree for alpha patches - propagate rename to rcu_user_*_after_irq() in the commit changelog - remove trailing #endif in "rcu: New rcu_user_enter_after_irq() and rcu_user_exit_after_irq() APIs" and early rcu_user_hooks_switch() header as well (reported by WU Fenguang). - add rcu_user_hooks_switch() header in "rcu: Switch task's syscall hooks on context switch" I believe this all should also fix what WU Fenguang reported: [rcu:rcu:next 54/82] include/linux/rcupdate.h:202:20: error: redefinition of 'rcu_user_exit' [rcu:rcu:next 54/82] include/linux/rcupdate.h:401:19: error: redefinition of 'rcu_read_lock_bh_held' [rcu:rcu:idle 2/27] include/linux/rcupdate.h:969:2: error: #endif without #if I'm not reposting the patches because few things have actually changed, except a few neats and some stuff that have been moved from a patch to another to unbreak bisection. But the end result is about the same. You can check the difference with a git diff between our branches. If you want to, though, I can repost the patches. This is pullable from: git://github.com/fweisbec/linux-dynticks.git rcu/idle-for-v3.7 Thanks. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frederic Weisbecker Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2 Date: Tue, 11 Sep 2012 14:35:44 +0200 Message-ID: <20120911123528.GG21277@somewhere.redhat.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> <20120910215541.GE21277@somewhere.redhat.com> <20120910225845.GC4257@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: 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 , Michael Cree Return-path: In-Reply-To: <20120910225845.GC4257@linux.vnet.ibm.com> List-ID: List-Id: linux-parisc.vger.kernel.org On Mon, Sep 10, 2012 at 03:58:45PM -0700, Paul E. McKenney wrote: > > Anyway, should I resend the patchset with this thing fixed? > > Please feel free to do so. ;-) > > Thanx, Paul > So here it is. On the menu for the updated branch: - fixed stable tags (use # 3.3+) - not sure what to do with "alpha: Fix preemption handling in idle loop", tag as stable or not? - added tested-by Michael Cree for alpha patches - propagate rename to rcu_user_*_after_irq() in the commit changelog - remove trailing #endif in "rcu: New rcu_user_enter_after_irq() and rcu_user_exit_after_irq() APIs" and early rcu_user_hooks_switch() header as well (reported by WU Fenguang). - add rcu_user_hooks_switch() header in "rcu: Switch task's syscall hooks on context switch" I believe this all should also fix what WU Fenguang reported: [rcu:rcu:next 54/82] include/linux/rcupdate.h:202:20: error: redefinition of 'rcu_user_exit' [rcu:rcu:next 54/82] include/linux/rcupdate.h:401:19: error: redefinition of 'rcu_read_lock_bh_held' [rcu:rcu:idle 2/27] include/linux/rcupdate.h:969:2: error: #endif without #if I'm not reposting the patches because few things have actually changed, except a few neats and some stuff that have been moved from a patch to another to unbreak bisection. But the end result is about the same. You can check the difference with a git diff between our branches. If you want to, though, I can repost the patches. This is pullable from: git://github.com/fweisbec/linux-dynticks.git rcu/idle-for-v3.7 Thanks. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754900Ab2IKMfz (ORCPT ); Tue, 11 Sep 2012 08:35:55 -0400 Received: from mail-vb0-f46.google.com ([209.85.212.46]:40791 "EHLO mail-vb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752627Ab2IKMfw (ORCPT ); Tue, 11 Sep 2012 08:35:52 -0400 Date: Tue, 11 Sep 2012 14:35:44 +0200 From: Frederic Weisbecker To: "Paul E. McKenney" Cc: 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 , Michael Cree Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2 Message-ID: <20120911123528.GG21277@somewhere.redhat.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> <20120910215541.GE21277@somewhere.redhat.com> <20120910225845.GC4257@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120910225845.GC4257@linux.vnet.ibm.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 10, 2012 at 03:58:45PM -0700, Paul E. McKenney wrote: > > Anyway, should I resend the patchset with this thing fixed? > > Please feel free to do so. ;-) > > Thanx, Paul > So here it is. On the menu for the updated branch: - fixed stable tags (use # 3.3+) - not sure what to do with "alpha: Fix preemption handling in idle loop", tag as stable or not? - added tested-by Michael Cree for alpha patches - propagate rename to rcu_user_*_after_irq() in the commit changelog - remove trailing #endif in "rcu: New rcu_user_enter_after_irq() and rcu_user_exit_after_irq() APIs" and early rcu_user_hooks_switch() header as well (reported by WU Fenguang). - add rcu_user_hooks_switch() header in "rcu: Switch task's syscall hooks on context switch" I believe this all should also fix what WU Fenguang reported: [rcu:rcu:next 54/82] include/linux/rcupdate.h:202:20: error: redefinition of 'rcu_user_exit' [rcu:rcu:next 54/82] include/linux/rcupdate.h:401:19: error: redefinition of 'rcu_read_lock_bh_held' [rcu:rcu:idle 2/27] include/linux/rcupdate.h:969:2: error: #endif without #if I'm not reposting the patches because few things have actually changed, except a few neats and some stuff that have been moved from a patch to another to unbreak bisection. But the end result is about the same. You can check the difference with a git diff between our branches. If you want to, though, I can repost the patches. This is pullable from: git://github.com/fweisbec/linux-dynticks.git rcu/idle-for-v3.7 Thanks.