* [PATCH] EDAC, MCE, AMD: Don't emit 'CE' for Deferred error
@ 2015-07-13 11:53 Aravind Gopalakrishnan
2015-07-14 4:34 ` Borislav Petkov
0 siblings, 1 reply; 2+ messages in thread
From: Aravind Gopalakrishnan @ 2015-07-13 11:53 UTC (permalink / raw)
To: bp, dougthompson, mchehab; +Cc: linux-edac, linux-kernel
Currently, we also display 'CE' for a Deferred error.
like so-
[Hardware Error]: CPU:0 (15:2:0)
MC4_STATUS[Over|CE|MiscV|-|AddrV|Deferred|-|UECC]: 0xdc04b00095080813
When the 'UC' bit on MCx_STATUS register is clear, the error status
is either a Corrected error or Deferred error as determined by the
'Deferred' bit.
Refer AMD Error Scope Hierarchy table in a newer BKDG
(example: 49125_15h_Models_30h-3Fh_BKDG.pdf, section "RAS Features")
for clarification.
Therefore, fix the output to not disply 'CE'
Signed-off-by: Aravind Gopalakrishnan <aravind.gopalakrishnan@amd.com>
---
drivers/edac/mce_amd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/edac/mce_amd.c b/drivers/edac/mce_amd.c
index 58586d5..e3a945c 100644
--- a/drivers/edac/mce_amd.c
+++ b/drivers/edac/mce_amd.c
@@ -763,7 +763,8 @@ int amd_decode_mce(struct notifier_block *nb, unsigned long val, void *data)
c->x86, c->x86_model, c->x86_mask,
m->bank,
((m->status & MCI_STATUS_OVER) ? "Over" : "-"),
- ((m->status & MCI_STATUS_UC) ? "UE" : "CE"),
+ ((m->status & MCI_STATUS_UC) ? "UE" :
+ (m->status & MCI_STATUS_DEFERRED) ? "-" : "CE"),
((m->status & MCI_STATUS_MISCV) ? "MiscV" : "-"),
((m->status & MCI_STATUS_PCC) ? "PCC" : "-"),
((m->status & MCI_STATUS_ADDRV) ? "AddrV" : "-"));
--
2.4.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] EDAC, MCE, AMD: Don't emit 'CE' for Deferred error
2015-07-13 11:53 [PATCH] EDAC, MCE, AMD: Don't emit 'CE' for Deferred error Aravind Gopalakrishnan
@ 2015-07-14 4:34 ` Borislav Petkov
0 siblings, 0 replies; 2+ messages in thread
From: Borislav Petkov @ 2015-07-14 4:34 UTC (permalink / raw)
To: Aravind Gopalakrishnan; +Cc: dougthompson, mchehab, linux-edac, linux-kernel
On Mon, Jul 13, 2015 at 06:53:02AM -0500, Aravind Gopalakrishnan wrote:
> Currently, we also display 'CE' for a Deferred error.
> like so-
> [Hardware Error]: CPU:0 (15:2:0)
> MC4_STATUS[Over|CE|MiscV|-|AddrV|Deferred|-|UECC]: 0xdc04b00095080813
>
> When the 'UC' bit on MCx_STATUS register is clear, the error status
> is either a Corrected error or Deferred error as determined by the
> 'Deferred' bit.
>
> Refer AMD Error Scope Hierarchy table in a newer BKDG
> (example: 49125_15h_Models_30h-3Fh_BKDG.pdf, section "RAS Features")
> for clarification.
>
> Therefore, fix the output to not disply 'CE'
>
> Signed-off-by: Aravind Gopalakrishnan <aravind.gopalakrishnan@amd.com>
> ---
> drivers/edac/mce_amd.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Applied, thanks.
--
Regards/Gruss,
Boris.
ECO tip #101: Trim your mails when you reply.
--
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-07-14 4:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-13 11:53 [PATCH] EDAC, MCE, AMD: Don't emit 'CE' for Deferred error Aravind Gopalakrishnan
2015-07-14 4:34 ` Borislav Petkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®