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
| ||
|
Date: Thu, 01 Apr 2010 09:35:58 +1100 From: Benjamin Herrenschmidt <benh@...nel.crashing.org> To: Andrew Morton <akpm@...ux-foundation.org> Cc: "H. Peter Anvin" <hpa@...or.com>, Yinghai Lu <yinghai@...nel.org>, Rabin Vincent <rabin@....in>, lkml <linux-kernel@...r.kernel.org>, penberg@...helsinki.fi, cl@...ux-foundation.org, linux-arch@...r.kernel.org, David Howells <dhowells@...hat.com>, Linus Torvalds <torvalds@...ux-foundation.org> Subject: Re: start_kernel(): bug: interrupts were enabled early On Wed, 2010-03-31 at 14:28 -0700, Andrew Morton wrote: > > Well, it's all a bit nasty. kmem_cache_create() does a lot of stuff, > including calling into the page allocator with GFP_KERNEL - expecting > kmem_cache_create() to preserve local_irq_disable() is a bit > optimistic. Well, the sl*b allocator -has- been modified to avoid enabling IRQs early, at least I remember we did that when we moved it to be initialized earlier. > radix_tree_init() calls hotcpu_notifier() which also does > mutex_lock(&cpu_add_remove_lock); > > The easiest fix is to reposition the interrutps-are-now-enabled point > in start_kernel(). But I have a feeling that some versions of > early_irq_init() won't like that. Yeah that won't work. Interrupts must not be enabled before at least init_IRQ() and time_init(). The problem is that until all these guys have gone through their initializations, there may be pending spurrious crap coming from the HW (timers, external IRQs, profile IRQs) due to such HW not yet properly "sanitized" by the kernel. Plenty of archs have those assumptions wired in. I don't think moving the IRQ enable point earlier is the right approach. Cheers, Ben. -- 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/
Powered by blists - more mailing lists