WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] 3/3: MCA/MCE correctable error handling

To: Christoph Egger <Christoph.Egger@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] 3/3: MCA/MCE correctable error handling
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: Thu, 23 Aug 2007 15:07:44 +0100
Cc: Gavin.Maltby@xxxxxxx, Jan Beulich <jbeulich@xxxxxxxxxx>
Delivery-date: Thu, 23 Aug 2007 07:08:26 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <200708230857.29134.Christoph.Egger@xxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcfljvmnODOWflGCEdy1JQAX8io7RQ==
Thread-topic: [Xen-devel] [PATCH] 3/3: MCA/MCE correctable error handling
User-agent: Microsoft-Entourage/11.3.6.070618
On 23/8/07 07:57, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

>> Oh, and is AMD-specific code really needed in non-fatal.c? I though the MCA
>> stuff was architectural now rather than vendor specific? If there are
>> vendor-specific extensions then they belong in the vendor's .c file.
> 
> AMD-specific is the use of the hw register code. Intel has some additional
> machine check MSR's containing the register set. Intel may add a structure
> to patch 2/3 that make use of them. Should I move the amd polling handler to
> amd.c ?

I think so.

 -- Keir


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel