From: Harvey Harrison Date: Wed, 30 Jan 2008 12:32:59 +0000 (+0100) Subject: x86: more users of PF_ constants in fault_32|64.c X-Git-Tag: v2.6.25-rc1~1143^2~344 X-Git-Url: http://pilppa.com/gitweb/?a=commitdiff_plain;h=318aa296c329625916c47703eab906d6ebf13930;p=linux-2.6-omap-h63xx.git x86: more users of PF_ constants in fault_32|64.c Should be the last of the error_code tests that could use the PF_ defines. Makes X86_32|64 a little closer. Signed-off-by: Harvey Harrison Signed-off-by: Ingo Molnar Signed-off-by: Thomas Gleixner --- diff --git a/arch/x86/mm/fault_32.c b/arch/x86/mm/fault_32.c index b1893ebf645..421e276770a 100644 --- a/arch/x86/mm/fault_32.c +++ b/arch/x86/mm/fault_32.c @@ -404,7 +404,8 @@ void __kprobes do_page_fault(struct pt_regs *regs, unsigned long error_code) * protection error (error_code & 9) == 0. */ if (unlikely(address >= TASK_SIZE)) { - if (!(error_code & 0x0000000d) && vmalloc_fault(address) >= 0) + if (!(error_code & (PF_RSVD|PF_USER|PF_PROT)) && + vmalloc_fault(address) >= 0) return; if (notify_page_fault(regs)) return; @@ -603,7 +604,7 @@ no_context: __typeof__(pte_val(__pte(0))) page; #ifdef CONFIG_X86_PAE - if (error_code & 16) { + if (error_code & PF_INSTR) { pte_t *pte = lookup_address(address); if (pte && pte_present(*pte) && !pte_exec_kernel(*pte)) @@ -674,7 +675,7 @@ out_of_memory: goto survive; } printk("VM: killing process %s\n", tsk->comm); - if (error_code & 4) + if (error_code & PF_USER) do_group_exit(SIGKILL); goto no_context; diff --git a/arch/x86/mm/fault_64.c b/arch/x86/mm/fault_64.c index 058b04cb96d..95f142f5b5c 100644 --- a/arch/x86/mm/fault_64.c +++ b/arch/x86/mm/fault_64.c @@ -677,7 +677,7 @@ out_of_memory: goto again; } printk("VM: killing process %s\n", tsk->comm); - if (error_code & 4) + if (error_code & PF_USER) do_group_exit(SIGKILL); goto no_context;