linux-kernel - Re: [patch 00/47] Sparse irq rework

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] [thread-next>] [day] [month] [year] [list]

Date:	Sun, 3 Oct 2010 05:57:51 -0600
From:	Grant Likely <grant.likely@...retlab.ca>
To:	Russell King - ARM Linux <linux@....linux.org.uk>
Cc:	Thomas Gleixner <tglx@...utronix.de>,
	LKML <linux-kernel@...r.kernel.org>, linux-arch@...r.kernel.org,
	Linus Torvalds <torvalds@...l.org>,
	Andrew Morton <akpm@...ux-foundation.org>, x86@...nel.org,
	Peter Zijlstra <peterz@...radead.org>,
	Benjamin Herrenschmidt <benh@...nel.crashing.org>,
	Paul Mundt <lethal@...ux-sh.org>,
	David Woodhouse <dwmw2@...radead.org>,
	Jesse Barnes <jbarnes@...tuousgeek.org>,
	Yinghai Lu <yinghai@...nel.org>,
	"Eric W. Biederman" <ebiederm@...ssion.com>
Subject: Re: [patch 00/47] Sparse irq rework

On Sun, Oct 3, 2010 at 5:29 AM, Russell King - ARM Linux
<linux@....linux.org.uk> wrote:
> On Sun, Oct 03, 2010 at 05:23:12AM -0600, Grant Likely wrote:
>> On Thu, Sep 30, 2010 at 11:14:27PM -0000, Thomas Gleixner wrote:
>> > The following patch series cleans up and mostly reimplements the core
>> > sparse irq implementation and sanitizes the most complex (ab)user:
>> > arch/x86
>>
>> Hoy Thomas,
>>
>> patches 1-13 & 15 all look fine to me.  Feel free to add my Acked-by:
>> line.
>>
>> Patch 14 seems to be missing, and by inference it appears that the
>> all important new allocator is added by patch 14.  All of the rest of
>> the series seems to be fine, but I'm forced to make assumptions about
>> what the new allocator looks like, so I cannot be positive.
>
> Patch 14 was present, you were even CC'd on it.

Hmmm... found it now.  I'm not sure what happened there, but I
couldn't find it the first time I looked.  Most likely it is a PEBKAC
failure.  Sorry for the noise.

Okay, patch 14 looks good to me too (including Yinghai's comment).
The new allocator seems sane, and I didn't see any obvious errors in
patches 16-47.  I've not tested any of this yet.  Hopefully I'll be
able to carve out some time to do so early this week.

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
--
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/