lkml.org 
[lkml]   [2015]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [GIT PULL] RAS update for 3.20 (one more thing)

* Luck, Tony <tony.luck@intel.com> wrote:

> The following changes since commit 26bc420b59a38e4e6685a73345a0def461136dce:
>
> Linux 3.19-rc6 (2015-01-25 20:04:41 -0800)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/ras/ras.git tags/please-pull-fixmcelog
>
> for you to fetch changes up to 728b6f14abaa7f36a8c6d41c6d6fe0320d32d5e9:
>
> x86, mce: Kernel does full decoding for AMD, others still need /dev/mcelog reports (2015-01-30 11:25:46 -0800)
>
> ----------------------------------------------------------------
> Long standing regression - functions registered on the mce decoder
> chain can declare that they have completely dealt with an event.
> True for AMD (says Boris), since the kernel fully decodes the
> machine check bank information. Not true for Intel processors. Full
> decode is done in user space, so we need to make the log visible
> via /dev/mcelog

So I'm having second thoughts about this:

This kind of vendor specific hard coding is really ugly:

if (c->x86_vendor == X86_VENDOR_AMD && ret == NOTIFY_STOP)
return;

Instead we should fix the Intel side to do a proper decode
as well - by the time mcelog is running it might be too
late, attempting an intelligent printk is way better...

So what would we need to make the Intel side just as good
as the AMD side?

I've undone the pull for now.

Thanks,

Ingo


\
 
 \ /
  Last update: 2015-02-09 11:21    [W:0.115 / U:0.532 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site