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, 9 Jan 2008 07:57:32 +0000 From: Christoph Hellwig <hch@...radead.org> To: Harvey Harrison <harvey.harrison@...il.com> Cc: Andrew Morton <akpm@...ux-foundation.org>, LKML <linux-kernel@...r.kernel.org>, Masami Hiramatsu <mhiramat@...hat.com>, Ananth N Mavinakayanahalli <ananth@...ibm.com>, David Miller <davem@...emloft.net>, hskinnemoen@...el.com, schwidefsky@...ibm.com, heiko.carstens@...ibm.com, tony.luck@...el.com, Ingo Molnar <mingo@...e.hu>, Paul Mackerras <paulus@...ba.org> Subject: Re: [PATCHv3] kprobes: Introduce kprobe_handle_fault() On Tue, Jan 08, 2008 at 08:19:20PM -0800, Harvey Harrison wrote: > Use a central kprobe_handle_fault() inline in kprobes.h to remove > all of the arch-dependant, practically identical implementations in > avr32, ia64, powerpc, s390, sparc64, and x86. > > avr32 was the only arch without the preempt_disable/enable pair > in its notify_page_fault implementation. > > This uncovered a possible bug in the s390 version as that purely > copied the x86 version unconditionally passing 14 as the trapnr > rather than the error_code parameter. This is not an equivalnet transformation. The old code always made sure to disable preemption and the new one just skips the whole kprobes handler if preemption is enabled. Please separate code consolidation from changing semantics _and_ describe the semantics changes in detail. -- 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