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/
Home Products Support Community News


RE: [Xen-devel] [pvops-dom0] Adding MCA logging support in pv_ops

To: Jeremy Fitzhardinge <jeremy@xxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>, Christoph Egger <Christoph.Egger@xxxxxxx>, "Kleen, Andi" <andi.kleen@xxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [pvops-dom0] Adding MCA logging support in pv_ops
From: "Ke, Liping" <liping.ke@xxxxxxxxx>
Date: Wed, 5 Aug 2009 10:54:38 +0800
Accept-language: en-US
Acceptlanguage: en-US
Delivery-date: Tue, 04 Aug 2009 19:56:15 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
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: <E2263E4A5B2284449EEBD0AAB751098402CDF6599F@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4A7174D3020000780000D350@xxxxxxxxxxxxxxxxxx> <E2263E4A5B2284449EEBD0AAB751098402CDF65BEB@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <E2263E4A5B2284449EEBD0AAB751098402CDF65C35@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4A71D4F8.70804@xxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcoROTmG+c49Ab9RTWWb+TPvquK6RwEN7M6wAAG9MOA=
Thread-topic: [Xen-devel] [pvops-dom0] Adding MCA logging support in pv_ops
seems the mail is missing so resend it. Thanks!

Hi, all
I modified the patch according to all feedbacks.
Mainly changes inlcudes:
1. Rebased code to "rebase/master" branch 2.6.31-rc4. We fetch more
    useful physical cpu information which is useful to mcelog tools.
2. Modified the code for defects/ kernel conventions according to feedback.
3. Update out of date comments and add neccessary comments according
   to feedback.

This patch is only for your review. I will resend the formated patch later.
Also, I will re-sync the xen-mca.h head file changes back to XEN after review.

Thanks a lot for your help!

Jeremy Fitzhardinge wrote:
> On 07/30/09 02:23, Ke, Liping wrote:
>> Hi, Jan
>> I also noticed that in .31, several important missing field such as
>> apic_id, socket id is added. Those field is better to be filled too.
>> Since now pv_ops is based on .30, those changes haven't happen yet.
>> So what's your suggestion? Seems we have to keep the old code and
>> make 
>> a patch to it when .31 is supported in pv_ops?
> The "rebase/master" branch is based on 2.6.31-rc4.  It looks like your
> patch doesn't really have any other code-level dependencies on the
> dom0 changes, so you could just base it on upstream 2.6.31-rc4.
>     J

Xen-devel mailing list