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: Wed, 11 Nov 2009 04:50:26 +0900 From: Tejun Heo <tj@...nel.org> To: Ingo Molnar <mingo@...e.hu> CC: Linus Torvalds <torvalds@...ux-foundation.org>, Linux Kernel <linux-kernel@...r.kernel.org>, Yinghai Lu <yhlu.kernel@...il.com> Subject: Re: [GIT PULL] percpu fixes for 2.6.32-rc6 Ingo Molnar wrote: > * Tejun Heo <tj@...nel.org> wrote: > >> In this case, as the second function needs to release to free the old >> map, the extra unlock/lock pair is actually necessary. Splitting into >> two functions is fine but I think it would be better to fix it first >> and then split them in following patches so that it can be bisected if >> I screw up while splitting, right? > > i think Linus's point was that this hack was the last straw that broke > the camel's back, and that we are better off with a clean solution > straight away. If you send the clean approach i can help test it on a > good number of boxes. If you're talking about the locking itself, there really is no clean solution at this point. Until vmalloc locking is updated, we'll have to do locking impedance matching in percpu code. I'm still not quite sure whether we really need to update vmalloc code to be irqsafe. If you're talking about the three way return value, which I do agree to be quite ugly, I think it will be a lot safer to have three patches - one to fix the deadlock, another to fix the return value and the final one to de-uglify the function, especially as we're pretty late in the release cycle. Thanks. -- tejun -- 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