Re: [PATCH 00/12] x86: minor cleanups/fixes for MCE codes (v2)

[Borislav Petkov <bp@amd64.org>]

On Tue, Jun 07, 2011 at 09:48:57PM -0400, Hidetoshi Seto wrote:
> This is a set of minor changes for MCE codes.
> 
> Though I know there are many people expecting new RAS architecture
> such as one that integrates RAS/EDAC with MCE, I hope these small
> steps will be a help for future works.
> 
> v2:
>  reflect comments, update patch 3,4,5,7,10,12.
>  rebased on 3.0-rc2.
> 
> Thanks,
> H.Seto
> 
>  arch/x86/include/asm/entry_arch.h         |    4 -
>  arch/x86/include/asm/hw_irq.h             |    1 -
>  arch/x86/include/asm/irq_vectors.h        |    5 -
>  arch/x86/include/asm/mce.h                |   19 ++-
>  arch/x86/kernel/cpu/mcheck/mce-severity.c |  152 ++++++++++------
>  arch/x86/kernel/cpu/mcheck/mce.c          |  288 ++++++++++++++---------------
>  arch/x86/kernel/cpu/mcheck/mce_amd.c      |   10 +-
>  arch/x86/kernel/entry_64.S                |    5 -
>  arch/x86/kernel/irqinit.c                 |    3 -
>  9 files changed, 251 insertions(+), 236 deletions(-)
> 
> Hidetoshi Seto (11):
>       mce-severity: cleanup severity table, prep
>       mce-severity: cleanup severity table
>       mce-severity: trivial cleanups
>       x86, mce: replace MCE_SELF_VECTOR by irq_work
>       x86, mce: replace MCM_ to MCI_MISC_
>       x86, mce: introduce mce_gather_info()
>       x86, mce: check the result of ancient_init()
>       x86, mce: cleanup mce_create/remove_device
>       x86, mce: cleanup mce_read
>       x86, mce: use prefix mce_chrdev_ to group functions
>       x86, mce: use prefix mce_sysdev_ to group functions
> 
> Tony Luck (1):
>       mce-severity: fixes for mce severity table

Well, I can't say I'm crazy about the severities mechanism but I guess
this is fine for a cleanup patchset. Btw, the problem of making the
severities readable will come up again when we implement this into perf
filters and the complexity will be moved into the filter function...

Other than that,

Reviewed-by: Borislav Petkov <bp@amd64.org>

Thanks.

-- 
Regards/Gruss,
Boris.

Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632