Re: [PATCH 00/11] rcu: Add missing RCU idle APIs on idle loop v2

From: Frederic Weisbecker
Date: Mon Sep 10 2012 - 17:56:07 EST


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.
>
> If the regression occurred in 3.3, then the cc lines should be something
> like:
>
> Cc: <stable@xxxxxxxxxxxxxxx> # 3.3+
>
> and not the current:
>
> Cc: 3.2.x.. <stable@xxxxxxxxxxxxxxx>
>
> (Note, version annotations should be on the right of the address, not in
> the 'real name' position on the left.)

Ah indeed.

Hmm, I don't understand why we took that convention while we can reuse the name
that precedes the email to put that information.

Anyway, should I resend the patchset with this thing fixed?

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/