linux-kernel - [PATCHv2] x86, mce: Kernel does full decoding for AMD, others still need /dev/mcelog reports

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <3d6a39202cff6415eeb0af7bff58f36ce879a253.1422641617.git.tony.luck@intel.com>
Date:	Fri, 30 Jan 2015 10:16:48 -0800
From:	Tony Luck <tony.luck@...el.com>
To:	linux-kernel@...r.kernel.org, linux-edac@...r.kernel.org
Cc:	Borislav Petkov <bp@...e.de>, Chen Gong <gong.chen@...ux.intel.com>
Subject: [PATCHv2] x86, mce: Kernel does full decoding for AMD, others still need /dev/mcelog reports

We should only believe a NOTIFY_STOP on an AMD system. For others
we need to provide the error record out to a user level decoder
via /dev/mcelog.

Signed-off-by: Tony Luck <tony.luck@...el.com>
---
v2: Boris says: "on AMD, mcelog is not needed"

 arch/x86/kernel/cpu/mcheck/mce.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/x86/kernel/cpu/mcheck/mce.c b/arch/x86/kernel/cpu/mcheck/mce.c
index d2c611699cd9..ea1bc1a05c07 100644
--- a/arch/x86/kernel/cpu/mcheck/mce.c
+++ b/arch/x86/kernel/cpu/mcheck/mce.c
@@ -151,12 +151,13 @@ void mce_log(struct mce *mce)
 {
 	unsigned next, entry;
 	int ret = 0;
+	struct cpuinfo_x86 *c = &boot_cpu_data;
 
 	/* Emit the trace record: */
 	trace_mce_record(mce);
 
 	ret = atomic_notifier_call_chain(&x86_mce_decoder_chain, 0, mce);
-	if (ret == NOTIFY_STOP)
+	if (c->x86_vendor == X86_VENDOR_AMD && ret == NOTIFY_STOP)
 		return;
 
 	mce->finished = 0;
-- 
2.1.0

--
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/