From: Harvey Harrison Date: Wed, 30 Jan 2008 12:34:09 +0000 (+0100) Subject: x86: add is_errata100 helper to fault_32|64.c X-Git-Tag: v2.6.25-rc1~1143^2~22 X-Git-Url: http://pilppa.com/gitweb/?a=commitdiff_plain;h=35f3266ffbee7ff8ca0474dc8dfa3b73d5a1d1ef;p=linux-2.6-omap-h63xx.git x86: add is_errata100 helper to fault_32|64.c Further towards unifying these files, add another helper in same spirit as is_errata93. 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 75ae8c498d6..0bd241729aa 100644 --- a/arch/x86/mm/fault_32.c +++ b/arch/x86/mm/fault_32.c @@ -286,6 +286,22 @@ static int is_errata93(struct pt_regs *regs, unsigned long address) return 0; } +/* + * Work around K8 erratum #100 K8 in compat mode occasionally jumps to illegal + * addresses >4GB. We catch this in the page fault handler because these + * addresses are not reachable. Just detect this case and return. Any code + * segment in LDT is compatibility mode. + */ +static int is_errata100(struct pt_regs *regs, unsigned long address) +{ +#ifdef CONFIG_X86_64 + if ((regs->cs == __USER32_CS || (regs->cs & (1<<2))) && + (address >> 32)) + return 1; +#endif + return 0; +} + void do_invalid_op(struct pt_regs *, unsigned long); static int is_f00f_bug(struct pt_regs *regs, unsigned long address) @@ -566,6 +582,9 @@ bad_area_nosemaphore: if (is_prefetch(regs, address, error_code)) return; + if (is_errata100(regs, address)) + return; + if (show_unhandled_signals && unhandled_signal(tsk, SIGSEGV) && printk_ratelimit()) { printk( diff --git a/arch/x86/mm/fault_64.c b/arch/x86/mm/fault_64.c index df13487aa83..ccbb8e30d9f 100644 --- a/arch/x86/mm/fault_64.c +++ b/arch/x86/mm/fault_64.c @@ -256,6 +256,22 @@ static int is_errata93(struct pt_regs *regs, unsigned long address) return 0; } +/* + * Work around K8 erratum #100 K8 in compat mode occasionally jumps to illegal + * addresses >4GB. We catch this in the page fault handler because these + * addresses are not reachable. Just detect this case and return. Any code + * segment in LDT is compatibility mode. + */ +static int is_errata100(struct pt_regs *regs, unsigned long address) +{ +#ifdef CONFIG_X86_64 + if ((regs->cs == __USER32_CS || (regs->cs & (1<<2))) && + (address >> 32)) + return 1; +#endif + return 0; +} + void do_invalid_op(struct pt_regs *, unsigned long); static int is_f00f_bug(struct pt_regs *regs, unsigned long address) @@ -568,14 +584,7 @@ bad_area_nosemaphore: if (is_prefetch(regs, address, error_code)) return; - /* Work around K8 erratum #100 K8 in compat mode - occasionally jumps to illegal addresses >4GB. We - catch this here in the page fault handler because - these addresses are not reachable. Just detect this - case and return. Any code segment in LDT is - compatibility mode. */ - if ((regs->cs == __USER32_CS || (regs->cs & (1<<2))) && - (address >> 32)) + if (is_errata100(regs, address)) return; if (show_unhandled_signals && unhandled_signal(tsk, SIGSEGV) &&