All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frederic Weisbecker <fweisbec@gmail.com>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Ben Hutchings <ben@decadent.org.uk>,
	LKML <linux-kernel@vger.kernel.org>,
	Chris Zankel <chris@zankel.net>,
	"3.2.x.." <stable@vger.kernel.org>,
	Chen Liqin <liqin.chen@sunplusct.com>,
	Lennox Wu <lennox.wu@gmail.com>,
	"James E.J. Bottomley" <jejb@parisc-linux.org>,
	Helge Deller <deller@gmx.de>,
	Parisc <linux-parisc@vger.kernel.org>,
	David Howells <dhowells@redhat.com>,
	Koichi Yasutake <yasutake.koichi@jp.panasonic.com>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	m68k <linux-m68k@lists.linux-m68k.org>,
	Hirokazu Takata <takata@linux-m32r.org>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Mikael Starvik <starvik@axis.com>,
	Jesper Nilsson <jesper.nilsson@axis.com>,
	Cris <linux-cris-kernel@axis.com>,
	Richard Henderson <rth@twiddle.net>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
	Matt Turner <mattst88@gmail.com>,
	alpha <linux-alpha@vger.kernel.org>, Michael Cree <>
Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2
Date: Tue, 11 Sep 2012 14:35:44 +0200	[thread overview]
Message-ID: <20120911123528.GG21277@somewhere.redhat.com> (raw)
In-Reply-To: <20120910225845.GC4257@linux.vnet.ibm.com>

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 <mcree@orcon.net.nz> 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.

WARNING: multiple messages have this Message-ID (diff)
From: Frederic Weisbecker <fweisbec@gmail.com>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Ben Hutchings <ben@decadent.org.uk>,
	LKML <linux-kernel@vger.kernel.org>,
	Chris Zankel <chris@zankel.net>,
	"3.2.x.." <stable@vger.kernel.org>,
	Chen Liqin <liqin.chen@sunplusct.com>,
	Lennox Wu <lennox.wu@gmail.com>,
	"James E.J. Bottomley" <jejb@parisc-linux.org>,
	Helge Deller <deller@gmx.de>,
	Parisc <linux-parisc@vger.kernel.org>,
	David Howells <dhowells@redhat.com>,
	Koichi Yasutake <yasutake.koichi@jp.panasonic.com>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	m68k <linux-m68k@lists.linux-m68k.org>,
	Hirokazu Takata <takata@linux-m32r.org>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Mikael Starvik <starvik@axis.com>,
	Jesper Nilsson <jesper.nilsson@axis.com>,
	Cris <linux-cris-kernel@axis.com>,
	Richard Henderson <rth@twiddle.net>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
	Matt Turner <mattst88@gmail.com>,
	alpha <linux-alpha@vger.kernel.org>, Michael Cree <mcree@
Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2
Date: Tue, 11 Sep 2012 14:35:44 +0200	[thread overview]
Message-ID: <20120911123528.GG21277@somewhere.redhat.com> (raw)
In-Reply-To: <20120910225845.GC4257@linux.vnet.ibm.com>

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 <mcree@orcon.net.nz> 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.

WARNING: multiple messages have this Message-ID (diff)
From: Frederic Weisbecker <fweisbec@gmail.com>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Ben Hutchings <ben@decadent.org.uk>,
	LKML <linux-kernel@vger.kernel.org>,
	Chris Zankel <chris@zankel.net>,
	"3.2.x.." <stable@vger.kernel.org>,
	Chen Liqin <liqin.chen@sunplusct.com>,
	Lennox Wu <lennox.wu@gmail.com>,
	"James E.J. Bottomley" <jejb@parisc-linux.org>,
	Helge Deller <deller@gmx.de>,
	Parisc <linux-parisc@vger.kernel.org>,
	David Howells <dhowells@redhat.com>,
	Koichi Yasutake <yasutake.koichi@jp.panasonic.com>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	m68k <linux-m68k@vger.kernel.org>,
	Hirokazu Takata <takata@linux-m32r.org>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Mikael Starvik <starvik@axis.com>,
	Jesper Nilsson <jesper.nilsson@axis.com>,
	Cris <linux-cris-kernel@axis.com>,
	Richard Henderson <rth@twiddle.net>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
	Matt Turner <mattst88@gmail.com>,
	alpha <linux-alpha@vger.kernel.org>,
	Michael Cree <mcree@orcon.net.nz>
Subject: Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2
Date: Tue, 11 Sep 2012 14:35:44 +0200	[thread overview]
Message-ID: <20120911123528.GG21277@somewhere.redhat.com> (raw)
In-Reply-To: <20120910225845.GC4257@linux.vnet.ibm.com>

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 <mcree@orcon.net.nz> 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.

  parent reply	other threads:[~2012-09-11 12:35 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-23 14:58 [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2 Frederic Weisbecker
2012-08-23 14:58 ` Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 01/11] alpha: Fix preemption handling in idle loop Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 02/11] alpha: Add missing RCU idle APIs on " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 03/11] cris: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 04/11] frv: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 05/11] h8300: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 06/11] m32r: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 07/11] m68k: " Frederic Weisbecker
2012-08-23 14:58 ` Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 08/11] mn10300: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 09/11] parisc: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 10/11] score: " Frederic Weisbecker
2012-08-23 14:58 ` [PATCH 11/11] xtensa: " Frederic Weisbecker
2012-08-24 21:26 ` [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2 Paul E. McKenney
2012-08-24 21:26   ` Paul E. McKenney
2012-08-25  1:19   ` Ben Hutchings
2012-08-25  1:19     ` Ben Hutchings
2012-08-25  1:19     ` Ben Hutchings
2012-08-25  2:10     ` Michael Cree
2012-08-25  2:10     ` Michael Cree
2012-08-25  2:10       ` Michael Cree
2012-08-25  2:10       ` Michael Cree
2012-08-25  3:50     ` Paul E. McKenney
2012-08-25  3:50     ` Paul E. McKenney
2012-08-25  3:50       ` Paul E. McKenney
2012-08-25  3:50       ` Paul E. McKenney
2012-08-25 13:16       ` Frederic Weisbecker
2012-08-25 13:16         ` Frederic Weisbecker
2012-08-25 13:16         ` Frederic Weisbecker
2012-08-25 16:18         ` Paul E. McKenney
2012-08-25 16:18         ` Paul E. McKenney
2012-08-25 16:18           ` Paul E. McKenney
2012-08-25 16:18           ` Paul E. McKenney
2012-08-25 21:15           ` Michael Cree
2012-08-25 21:15           ` Michael Cree
2012-08-25 21:15             ` Michael Cree
2012-08-25 21:15             ` Michael Cree
2012-08-26  0:12             ` Paul E. McKenney
2012-08-26  0:12             ` Paul E. McKenney
2012-08-26  0:12               ` Paul E. McKenney
2012-08-26  0:12               ` Paul E. McKenney
2012-08-26  9:21               ` Tobias Klausmann
2012-08-26  9:21                 ` Tobias Klausmann
2012-08-26  9:21                 ` Tobias Klausmann
2012-09-10 21:53                 ` Frederic Weisbecker
2012-09-10 21:53                 ` Frederic Weisbecker
2012-09-10 21:53                   ` Frederic Weisbecker
2012-09-10 21:53                   ` Frederic Weisbecker
2012-09-12 18:01                   ` Tobias Klausmann
2012-08-26  9:21               ` Tobias Klausmann
2012-08-25 13:16       ` Frederic Weisbecker
2012-09-10 21:56     ` Frederic Weisbecker
2012-09-10 21:56       ` Frederic Weisbecker
2012-09-10 21:56       ` Frederic Weisbecker
2012-09-10 22:58       ` Paul E. McKenney
2012-09-10 22:58         ` Paul E. McKenney
2012-09-10 22:58         ` Paul E. McKenney
2012-09-10 22:58         ` Paul E. McKenney
2012-09-11 12:35         ` Frederic Weisbecker
2012-09-11 12:35         ` Frederic Weisbecker [this message]
2012-09-11 12:35           ` Frederic Weisbecker
2012-09-11 12:35           ` Frederic Weisbecker
2012-09-10 21:56     ` Frederic Weisbecker
2012-08-25  1:19   ` Ben Hutchings
  -- strict thread matches above, loose matches on Subject: below --
2012-08-23 14:58 Frederic Weisbecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120911123528.GG21277@somewhere.redhat.com \
    --to=fweisbec@gmail.com \
    --cc=ben@decadent.org.uk \
    --cc=chris@zankel.net \
    --cc=deller@gmx.de \
    --cc=dhowells@redhat.com \
    --cc=geert@linux-m68k.org \
    --cc=ink@jurassic.park.msu.ru \
    --cc=jejb@parisc-linux.org \
    --cc=jesper.nilsson@axis.com \
    --cc=lennox.wu@gmail.com \
    --cc=linux-alpha@vger.kernel.org \
    --cc=linux-cris-kernel@axis.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-m68k@lists.linux-m68k.org \
    --cc=linux-parisc@vger.kernel.org \
    --cc=liqin.chen@sunplusct.com \
    --cc=mattst88@gmail.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=rth@twiddle.net \
    --cc=stable@vger.kernel.org \
    --cc=starvik@axis.com \
    --cc=takata@linux-m32r.org \
    --cc=yasutake.koichi@jp.panasonic.com \
    --cc=ysato@users.sourceforge.jp \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.