-------- Forwarded Message --------
From: Andrew Morton <a...@linux-foundation.org>
To: Greg KH <gre...@suse.de>
Cc: linux-ker...@vger.kernel.org, sta...@kernel.org, Xiaotian Feng 
<df...@redhat.com>, Thomas Gleixner <t...@linutronix.de>, 
torva...@linux-foundation.org, stable-rev...@kernel.org, 
Subject: Re: [Stable-review] [stable] [00/29] stable review
Date: Sat, 23 Jan 2010 23:21:52 -0800

On Fri, 22 Jan 2010 16:11:45 -0800 Greg KH <gre...@suse.de> wrote:

> This is the start of the stable review cycle for the release.
> There are 29 patches in this series, all will be posted as a response to
> this one.  If anyone has any issues with these being applied, please let
> us know.  If anyone is a maintainer of the proper subsystem, and wants
> to add a Signed-off-by: line to the patch, please respond with it.
> Responses should be made by Monday, January 24, 00:00:00 UTC.
> Anything received after that time might be too late. will still contain the regression described in (for example)


It's fixed with the below revert which has been in -mm (and only in
-mm) for ten days.

I don't what's going on - perhaps we're waiting for Thomas, and he's
otherwise engaged.  There's a fix in the lkml thread "Re: [RFC PATCH
0/4] clockevents: fix clockevent_devices list corruption after cpu
hotplug" to which Thomas replied

    I just applied your patch, but kept the cpuweight check.  This
    is the least intrusive solution for now.  The logic needs an
    overhaul, but thats neither rc4 nor stable material"

but no such patch is present in linux-next.

From: Andrew Morton <a...@linux-foundation.org>


: commit bb6eddf7676e1c1f3e637aa93c5224488d99036f
: Author:     Thomas Gleixner <t...@linutronix.de>
: AuthorDate: Thu Dec 10 15:35:10 2009 +0100
: Commit:     Thomas Gleixner <t...@linutronix.de>
: CommitDate: Fri Dec 11 10:28:08 2009 +0100
:     clockevents: Prevent clockevent_devices list corruption on cpu hotplug

due to the regression reported in

Cc: Xiaotian Feng <df...@redhat.com>
Cc: Thomas Gleixner <t...@linutronix.de>
Cc: "Rafael J. Wysocki" <r...@sisk.pl>
Reported-by: Martin Bammer <mr...@gmx.at>
Cc: <sta...@kernel.org>
Signed-off-by: Andrew Morton <a...@linux-foundation.org>

 kernel/time/clockevents.c |   18 +++---------------
 1 file changed, 3 insertions(+), 15 deletions(-)

diff -puN 
+++ a/kernel/time/clockevents.c
@@ -238,9 +238,8 @@ void clockevents_exchange_device(struct 
 void clockevents_notify(unsigned long reason, void *arg)
-       struct clock_event_device *dev, *tmp;
+       struct list_head *node, *tmp;
        unsigned long flags;
-       int cpu;
        raw_spin_lock_irqsave(&clockevents_lock, flags);
        clockevents_do_notify(reason, arg);
@@ -251,19 +250,8 @@ void clockevents_notify(unsigned long re
                 * Unregister the clock event devices which were
                 * released from the users in the notify chain.
-               list_for_each_entry_safe(dev, tmp, &clockevents_released, list)
-                       list_del(&dev->list);
-               /*
-                * Now check whether the CPU has left unused per cpu devices
-                */
-               cpu = *((int *)arg);
-               list_for_each_entry_safe(dev, tmp, &clockevent_devices, list) {
-                       if (cpumask_test_cpu(cpu, dev->cpumask) &&
-                           cpumask_weight(dev->cpumask) == 1) {
-                               BUG_ON(dev->mode != CLOCK_EVT_MODE_UNUSED);
-                               list_del(&dev->list);
-                       }
-               }
+               list_for_each_safe(node, tmp, &clockevents_released)
+                       list_del(node);

Stable-review mailing list

To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to