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: Mon, 31 Mar 2014 14:45:32 -0700 From: Linus Torvalds <torvalds@...ux-foundation.org> To: Martin Schwidefsky <schwidefsky@...ibm.com>, Ingo Molnar <mingo@...nel.org> Cc: linux-kernel <linux-kernel@...r.kernel.org>, linux-s390 <linux-s390@...r.kernel.org>, Heiko Carstens <heiko.carstens@...ibm.com> Subject: Re: [GIT PULL] s390 patches for the 3.15 merge window On Mon, Mar 31, 2014 at 12:24 AM, Martin Schwidefsky <schwidefsky@...ibm.com> wrote: > > There are two memory management related changes, the CMMA support for > KVM to avoid swap-in of freed pages and the split page table lock for > the PMD level. These two come with common code changes in mm/. Ugh. I pulled it, but things like this makes me want to dig my eyes out with a spoon: +#ifdef finish_arch_post_lock_switch + finish_arch_post_lock_switch(); +#endif when I think the proper thing to do would have been to move the #ifndef that creates an empty finish_arch_post_lock_switch() from kernel/sched/sched.h to some common file, or possibly even just duplicate it. I detest #ifdef's in the middle of code. Yes, we do have them, but we should try to avoid adding more of them. Maybe we could have a <linux/mmu_context.h> that includes the <asm/mmu_context.h> and then does that "let's add the dummy function for architectures that don't need it"? Added Ingo to the cc, since this ends up intersecting with the scheduler code that now does the wrapper on its own. 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/
Powered by blists - more mailing lists