linux-kernel - Re: [stable] Regression in 2.6.32.2: segfault on halt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4B2E41BD.20708@googlemail.com>
Date:	Sun, 20 Dec 2009 16:24:45 +0100
From:	Holger Hoffstätte 
	<holger.hoffstaette@...glemail.com>
To:	Mike Galbraith <efault@....de>
CC:	Willy Tarreau <w@....eu>, linux-kernel@...r.kernel.org,
	stable@...r.kernel.org
Subject: Re: [stable] Regression in 2.6.32.2: segfault on halt

(sorry for the dupes)

Mike Galbraith wrote:
> On Sun, 2009-12-20 at 14:27 +0100, Holger Hoffstätte wrote:
> 
>> Took me some time (still learning git - I usually use hg) but I just
>> managed to fix it by reverting not the bisected revision (won't compile
>> any longer), but the follow-up "cleanup & fix":
>>
>> >From 35c1ee3e78766d5666f418af638def9c67e63ecb Mon Sep 17 00:00:00 2001
>> From: Mike Galbraith <efault@....de>
>> Date: Tue, 10 Nov 2009 03:50:02 +0100
>> Subject: [PATCH] sched: Fix and clean up rate-limit newidle code
>>
>> commit eae0c9dfb534cb3449888b9601228efa6480fdb5 upstream.
>>
>> Commit 1b9508f, "Rate-limit newidle" has been confirmed to fix
>> the netperf UDP loopback regression reported by Alex Shi.
>>
>> This is a cleanup and a fix:
>>
>>  - moved to a more out of the way spot
>>
>>  - fix to ensure that balancing doesn't try to balance
>>    runqueues which haven't gone online yet, which can
>>    mess up CPU enumeration during boot.
>>
>> Reported-by: Alex Shi <alex.shi@...el.com>
>> Reported-by: Zhang, Yanmin <yanmin_zhang@...ux.intel.com>
>> Signed-off-by: Mike Galbraith <efault@....de>
>> Acked-by: Peter Zijlstra <a.p.zijlstra@...llo.nl>
>> LKML-Reference: <1257821402.5648.17.camel@...ge.simson.net>
>> Signed-off-by: Ingo Molnar <mingo@...e.hu>
>> Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
>>
>> aka:
>> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.32.y.git;a=commit;h=35c1ee3e78766d5666f418af638def9c67e63ecb
>>
>> Reverting this from a clean 32.2 tree results in a kernel with newidle
>> fix, but still working halt/reboot. The only difference between this and
>> the bisected one is the additional change in cpumask handling.
>>
>> That was more fun than expected :)
> 
> Egad.  Reverting the cpumask bit alone cures the problem?

Yup. I manually changed both cpumask_copy(..) lines back to
cpumask_setall(..) in a clean .32.2 tree and can halt the resulting kernel
without problems.

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