Hi all, Today's linux-next merge of the kmemcheck tree got a conflict in arch/x86/mm/fault.c between commit do_page_fault ("x86: optimise x86's do_page_fault (C entry point for the page fault path)") from the x86 tree and commit 787ecfaa503dc63ff1831ddc74b15dad49bace1d ("x86: add hooks for kmemcheck") from the kmemcheck tree. I fixed it up (see below) but it is worth a check. I can carry the fix as necessary. -- Cheers, Stephen Rothwell sfr@xxxxxxxxxxxxxxxx http://www.canb.auug.org.au/~sfr/ diff --cc arch/x86/mm/fault.c index d3eee74,c73afbb..0000000 --- a/arch/x86/mm/fault.c +++ b/arch/x86/mm/fault.c @@@ -806,7 -602,16 +807,14 @@@ void __kprobes do_page_fault(struct pt_ /* get the address */ address = read_cr2(); - si_code = SEGV_MAPERR; - + /* + * Detect and handle instructions that would cause a page fault for + * both a tracked kernel page and a userspace page. + */ + if(kmemcheck_active(regs)) + kmemcheck_hide(regs); + - if (notify_page_fault(regs)) + if (unlikely(notify_page_fault(regs))) return; if (unlikely(kmmio_fault(regs, address))) return; @@@ -829,12 -634,16 +837,16 @@@ #else if (unlikely(address >= TASK_SIZE64)) { #endif - if (!(error_code & (PF_RSVD|PF_USER|PF_PROT)) && - vmalloc_fault(address) >= 0) - return; + if (!(error_code & (PF_RSVD | PF_USER | PF_PROT))) { + if (vmalloc_fault(address) >= 0) + return; + + if (kmemcheck_fault(regs, address, error_code)) + return; + } /* Can handle a stale RO->RW TLB */ - if (spurious_fault(address, error_code)) + if (spurious_fault(error_code, address)) return; /* -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html