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 4/6] MCE: Fix the vMCE address translation for HV

To: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH 4/6] MCE: Fix the vMCE address translation for HVM guest.
From: Christoph Egger <Christoph.Egger@xxxxxxx>
Date: Thu, 28 Jan 2010 09:14:00 +0100
Cc: "Frank.Vanderlinden@xxxxxxx" <Frank.Vanderlinden@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Delivery-date: Thu, 28 Jan 2010 01:27:26 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C8EDE645B81E5141A8C6B2F73FD9265117C58A3368@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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: <C8EDE645B81E5141A8C6B2F73FD9265117C58A3368@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: KMail/1.9.10
On Thursday 28 January 2010 06:55:58 Jiang, Yunhong wrote:
> Fix the vMCE address translation for HVM guest.
>
> Fix address translation when we inject a virtual MCE to HVM guest.

IMO, the whole address translation should be x86 generic with hooks
to handle AMD/Intel specific registers.

Christoph

>
> Signed-off-by: Jiang, Yunhong <yunhong.jiang@xxxxxxxxx>
>
> diff -r ac2951705a86 xen/arch/x86/cpu/mcheck/mce_intel.c
> --- a/xen/arch/x86/cpu/mcheck/mce_intel.c     Tue Jan 26 20:04:51 2010 +0800
> +++ b/xen/arch/x86/cpu/mcheck/mce_intel.c     Tue Jan 26 22:49:11 2010 +0800
> @@ -371,8 +371,8 @@ static void intel_UCR_handler(struct mci
>
>                            gfn =
>                                mfn_to_gmfn(d, ((bank->mc_addr) >>
> PAGE_SHIFT)); -                          bank->mc_addr =
> -                              gfn << PAGE_SHIFT | (bank->mc_addr &
> PAGE_MASK); +                          bank->mc_addr =  gfn << PAGE_SHIFT |
> +                                        (bank->mc_addr & (PAGE_SIZE -1 ));
>                            if (fill_vmsr_data(bank, global->mc_gstatus) ==
> -1) {
>                                mce_printk(MCE_QUIET, "Fill vMCE# data for
> DOM%d "



-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Andrew Bowd, Thomas M. McCoy, Giuliano Meroni
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632


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