|
 |
ac32bf |
From 349da4c3d63ec6dceef66a405561984561d31582 Mon Sep 17 00:00:00 2001
|
|
|
ac32bf |
From: Seiichi Ikarashi <s.ikarashi@jp.fujitsu.com>
|
|
|
ac32bf |
Date: Wed, 10 Jun 2015 20:49:55 -0300
|
|
|
ac32bf |
Subject: [PATCH 3/5] rasdaemon: unnecessary comma for empty mc_location string
|
|
|
ac32bf |
|
|
|
ac32bf |
Into the /var/log/messages, rasdaemon sometimes prints an unnecessary
|
|
|
ac32bf |
comma ", " between mca= and cpu_type= like below:
|
|
|
ac32bf |
|
|
|
ac32bf |
Jun 9 02:44:39 localhost rasdaemon: <...>-4585 [1638893312] 1031.109000: mce_record: 2015-06-08 10:07:28 +0900 bank=3, status= 9c0000000000017a, mci=Corrected_error Error_enabled, mca=Generic CACHE Level-2 Eviction Error, , cpu_type= Intel Xeon v3 (Haswell) EP/EX, cpu= 1, socketid= 0, misc= 4004000000000080, addr= 204fffffff, mcgstatus= 0, mcgcap= 7000c16, apicid= 2
|
|
|
ac32bf |
|
|
|
ac32bf |
That's the comma for mc_location which is printed even if mc_location is
|
|
|
ac32bf |
empty due to a wrong if condition.
|
|
|
ac32bf |
|
|
|
ac32bf |
Signed-off-by: Seiichi Ikarashi <s.ikarashi@jp.fujitsu.com>
|
|
|
ac32bf |
Acked-by: Aristeu Rozanski <aris@redhat.com>
|
|
|
ac32bf |
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
|
|
|
ac32bf |
---
|
|
|
ac32bf |
ras-mce-handler.c | 2 +-
|
|
|
ac32bf |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
ac32bf |
|
|
|
ac32bf |
diff --git a/ras-mce-handler.c b/ras-mce-handler.c
|
|
|
ac32bf |
index fb6db8a..07252a0 100644
|
|
|
ac32bf |
--- a/ras-mce-handler.c
|
|
|
ac32bf |
+++ b/ras-mce-handler.c
|
|
|
ac32bf |
@@ -278,7 +278,7 @@ static void report_mce_event(struct ras_events *ras,
|
|
|
ac32bf |
if (*e->user_action)
|
|
|
ac32bf |
trace_seq_printf(s, " %s", e->user_action);
|
|
|
ac32bf |
|
|
|
ac32bf |
- if (e->mc_location)
|
|
|
ac32bf |
+ if (*e->mc_location)
|
|
|
ac32bf |
trace_seq_printf(s, ", %s", e->mc_location);
|
|
|
ac32bf |
|
|
|
ac32bf |
#if 0
|
|
|
ac32bf |
--
|
|
|
ac32bf |
1.8.3.1
|
|
|
ac32bf |
|