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

[Xen-devel] RE: [patch 0/3]Enable CMCI (Corrected Machine Check Error In

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: [Xen-devel] RE: [patch 0/3]Enable CMCI (Corrected Machine Check Error Interrupt) for Intel CPUs
From: "Ke, Liping" <liping.ke@xxxxxxxxx>
Date: Tue, 23 Dec 2008 13:17:51 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Mon, 22 Dec 2008 21:18:49 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C5753357.11E5%keir.fraser@xxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <E2263E4A5B2284449EEBD0AAB751098401C4C335E3@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C5753357.11E5%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Aclj6QtzedLMaThJSm+cYEhexWDApwARLffQACOX8KA=
Thread-topic: [patch 0/3]Enable CMCI (Corrected Machine Check Error Interrupt) for Intel CPUs
Hi, Keir

Thanks a lot for the clean-up and format refine! I will pay more attention to 
this too.And also I tested on the related platform. It's fine.

As for moving *cmci_owner_set* out of stopmachine_run is basically ok for us. 
Just one thing:  
CMCI might happen and lost during the very small window (old owner is cleared 
while new owner is not set). In order to make sure that CMCI could be triggered 
an on the new owner, we need to clear MSR Bank(i) status register [Corrected 
Error Counter] field ( We normally do this @ CMCI interrupt handler, according 
to spec, if the counter is not cleared, CMCI will not be triggered any more).
I made a small patch for it in the attachment. How do you think?

Thanks a lot!
Criping

-----Original Message-----
From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx] 
Sent: 2008年12月22日 20:07
To: Ke, Liping
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [patch 0/3]Enable CMCI (Corrected Machine Check Error Interrupt) 
for Intel CPUs

On 22/12/2008 03:54, "Ke, Liping" <liping.ke@xxxxxxxxx> wrote:

> Hi, All
> 
> According to Keir's suggestion, I re-organize those patches and make sure that
> each intermediate patch could be compiled and running.
> 
> Since remove old p4 and p6 files are hard to be split, I now put the remove
> and replace in the one patch. Is it ok?
> 
> Any comment, just let me know.

I checked in the patches as c/s 18938 and then cleaned up significantly as
c/s 18939. Let me know if this stops things working for you. Notice I
deleted more than I added. ;-)

 -- Keir


Attachment: clear_error_count.patch
Description: clear_error_count.patch

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>