linux-kernel - Re: [BUG] Oops on boot (probably ACPI related)

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 for Android: free password hash cracker in your pocket

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

Message-ID: <Pine.LNX.4.64.0609271436310.3952@g5.osdl.org>
Date:	Wed, 27 Sep 2006 14:38:36 -0700 (PDT)
From:	Linus Torvalds <torvalds@...l.org>
To:	Andi Kleen <ak@...e.de>
cc:	Kyle McMartin <kyle@...isc-linux.org>,
	linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
	akpm@...l.org, jbeulich@...ell.com
Subject: Re: [BUG] Oops on boot (probably ACPI related)



On Wed, 27 Sep 2006, Andi Kleen wrote:
> 
> It doesn't matter much because these days this stuff is all out of lined
> anyways and in a single function. And the dynamic branch predictor
> in all modern CPUs will usually cache the decision (unlocked) there.

Ahh, good point. Once there's only one copy, the branch predictor will get 
it right (and the code size won't much matter)

> > Are there any _real_ advantages to this broken unwinding code that has had 
> > more bugs that Windows XP?
> 
> I thought for a long time we didn't need it either, but these days with all 
> these callbacks in some parts of the kernel (driver model, others) and you 
> get a oops with 60+ entries it is just too much trouble to figure it out manually.

Ok, fair enough. I'll apply your fix (which in itself is obviously 
correct).

I just wanted to bring up the possibility that we should just remove the 
(fragile) unwinder.

But let's leave it for another day, if it keeps being problematic.

		Linus
-
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/