Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1266412
| From | Borislav Petkov <bp@alien8.de> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas |
| Date | 2015-11-10 12:30 +0100 |
| Message-ID | <qtfhD-4pX-5@gated-at.bofh.it> (permalink) |
| References | <qsZFT-1HI-9@gated-at.bofh.it> <qsZFU-1HI-27@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Fri, Nov 06, 2015 at 01:01:55PM -0800, Tony Luck wrote:
> Extend the severity checking code to add a new context IN_KERN_RECOV
> which is used to indicate that the machine check was triggered by code
> in the kernel with a fixup entry.
>
> Add code to check for this situation and respond by altering the return
> IP to the fixup address and changing the regs->ax so that the recovery
> code knows the physical address of the error. Note that we also set bit
> 63 because 0x0 is a legal physical address.
>
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---
> arch/x86/kernel/cpu/mcheck/mce-severity.c | 19 +++++++++++++++++--
> arch/x86/kernel/cpu/mcheck/mce.c | 13 ++++++++++---
> 2 files changed, 27 insertions(+), 5 deletions(-)
...
> diff --git a/arch/x86/kernel/cpu/mcheck/mce.c b/arch/x86/kernel/cpu/mcheck/mce.c
> index 9d014b82a124..472d11150b7a 100644
> --- a/arch/x86/kernel/cpu/mcheck/mce.c
> +++ b/arch/x86/kernel/cpu/mcheck/mce.c
> @@ -31,6 +31,7 @@
> #include <linux/types.h>
> #include <linux/slab.h>
> #include <linux/init.h>
> +#include <linux/module.h>
> #include <linux/kmod.h>
> #include <linux/poll.h>
> #include <linux/nmi.h>
> @@ -1132,9 +1133,15 @@ void do_machine_check(struct pt_regs *regs, long error_code)
You could save a precious indentation level here:
if (cfg->tolerant == 3)
goto clear;
and add the "clear" label below.
clear:
if (worst > 0)
mce_report_event(regs);
mce_wrmsrl(MSR_IA32_MCG_STATUS, 0)
> if (no_way_out)
> mce_panic("Fatal machine check on current CPU", &m, msg);
> if (worst == MCE_AR_SEVERITY) {
> - recover_paddr = m.addr;
> - if (!(m.mcgstatus & MCG_STATUS_RIPV))
> - flags |= MF_MUST_KILL;
> + if ((m.cs & 3) == 3) {
> + recover_paddr = m.addr;
> + if (!(m.mcgstatus & MCG_STATUS_RIPV))
> + flags |= MF_MUST_KILL;
> + } else if (fixup_mcexception(regs)) {
> + regs->ax = BIT(63) | m.addr;
> + } else
> + mce_panic("Failed kernel mode recovery",
> + &m, NULL);
> } else if (kill_it) {
> force_sig(SIGBUS, current);
> }
> --
> 2.1.4
>
>
--
Regards/Gruss,
Boris.
ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RFC PATCH 0/3] Machine check recovery when kernel accesses poison Tony Luck <tony.luck@intel.com> - 2015-11-09 19:50 +0100
[PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas Tony Luck <tony.luck@intel.com> - 2015-11-09 19:50 +0100
Re: [PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas Borislav Petkov <bp@alien8.de> - 2015-11-10 12:30 +0100
Re: [PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas "Luck, Tony" <tony.luck@intel.com> - 2015-11-10 23:20 +0100
Re: [PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas Borislav Petkov <bp@alien8.de> - 2015-11-11 12:10 +0100
Re: [PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas Andy Lutomirski <luto@kernel.org> - 2015-11-12 05:30 +0100
Re: [PATCH 2/3] x86, ras: Extend machine check recovery code to annotated ring0 areas "Luck, Tony" <tony.luck@intel.com> - 2015-11-12 21:00 +0100
[PATCH 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks Tony Luck <tony.luck@intel.com> - 2015-11-09 19:50 +0100
Re: [PATCH 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks Ingo Molnar <mingo@kernel.org> - 2015-11-12 09:00 +0100
Re: [PATCH 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks "Luck, Tony" <tony.luck@intel.com> - 2015-11-12 21:10 +0100
Re: [RFC PATCH 0/3] Machine check recovery when kernel accesses poison Borislav Petkov <bp@alien8.de> - 2015-11-10 12:30 +0100
Re: [RFC PATCH 0/3] Machine check recovery when kernel accesses poison "Luck, Tony" <tony.luck@intel.com> - 2015-11-10 23:00 +0100
Re: [RFC PATCH 0/3] Machine check recovery when kernel accesses poison Borislav Petkov <bp@alien8.de> - 2015-11-11 21:50 +0100
Re: [RFC PATCH 0/3] Machine check recovery when kernel accesses poison "Luck, Tony" <tony.luck@intel.com> - 2015-11-11 22:50 +0100
Re: [RFC PATCH 0/3] Machine check recovery when kernel accesses poison Borislav Petkov <bp@alien8.de> - 2015-11-11 23:30 +0100
RE: [RFC PATCH 0/3] Machine check recovery when kernel accesses poison "Luck, Tony" <tony.luck@intel.com> - 2015-11-11 23:40 +0100
csiph-web