Writing to MCG_STATUS register

Writing to MCG_STATUS register

[Kiselev, Sergey]

[-- Attachment #1.1: Type: text/plain, Size: 901 bytes --]

Hi,
 
Some older (~2.6.7) kernels write MCG_STATUS register during kernel boot
(mce_clear_all() function, called from mce_init()). It's not currently
handled by kvm and will cause it to inject a GPF.
Following patch adds a "nop" handler for this:
 
--- kvm-17/kernel/kvm_main.c.orig       2007-03-20 15:12:42.000000000
+0200
+++ kvm-17/kernel/kvm_main.c    2007-03-22 13:37:34.000000000 +0200
@@ -1498,6 +1498,10 @@
                printk(KERN_WARNING "%s: MSR_IA32_MC0_STATUS 0x%llx,
nop\n",
                       __FUNCTION__, data);
                break;
+       case MSR_IA32_MCG_STATUS:
+               printk(KERN_WARNING "%s: MSR_IA32_MCG_STATUS 0x%llx,
nop\n",
+                       __FUNCTION__, data);
+               break;
        case MSR_IA32_UCODE_REV:
        case MSR_IA32_UCODE_WRITE:
        case 0x200 ... 0x2ff: /* MTRRs */

Thanks,
Sergey Kiselev


[-- Attachment #1.2: Type: text/html, Size: 2543 bytes --]

[-- Attachment #2: Type: text/plain, Size: 345 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #3: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Re: Writing to MCG_STATUS register

[Avi Kivity]

Kiselev, Sergey wrote:
> Hi,
>  
> Some older (~2.6.7) kernels write MCG_STATUS register during kernel 
> boot (mce_clear_all() function, called from mce_init()). It's not 
> currently handled by kvm and will cause it to inject a GPF.
> Following patch adds a "nop" handler for this:
>  

Your email client mangles whitespace.  Please resend as attachment.

Also, please add a Signed-off-by: line.

-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[PATCH] RE: Writing to MCG_STATUS register

[Kiselev, Sergey]

[-- Attachment #1: Type: text/plain, Size: 780 bytes --]

Resending as attachment...

-----Original Message-----
From: Avi Kivity [mailto:avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org] 
Sent: 22 March 2007 13:47
To: Kiselev, Sergey
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: [kvm-devel] Writing to MCG_STATUS register

Kiselev, Sergey wrote:
> Hi,
>  
> Some older (~2.6.7) kernels write MCG_STATUS register during kernel 
> boot (mce_clear_all() function, called from mce_init()). It's not 
> currently handled by kvm and will cause it to inject a GPF.
> Following patch adds a "nop" handler for this:
>  

Your email client mangles whitespace.  Please resend as attachment.

Also, please add a Signed-off-by: line.

-- 
error compiling committee.c: too many arguments to function

[-- Attachment #2: kvm-17-mcg_status.patch --]
[-- Type: application/octet-stream, Size: 540 bytes --]

Signed-off-by: Sergey Kiselev <sergey.kiselev@intel.com>

--- kvm-17/kernel/kvm_main.c.orig	2007-03-20 15:12:42.000000000 +0200
+++ kvm-17/kernel/kvm_main.c	2007-03-22 13:37:34.000000000 +0200
@@ -1498,6 +1498,10 @@
 		printk(KERN_WARNING "%s: MSR_IA32_MC0_STATUS 0x%llx, nop\n",
 		       __FUNCTION__, data);
 		break;
+	case MSR_IA32_MCG_STATUS:
+		printk(KERN_WARNING "%s: MSR_IA32_MCG_STATUS 0x%llx, nop\n",
+			__FUNCTION__, data);
+		break;
 	case MSR_IA32_UCODE_REV:
 	case MSR_IA32_UCODE_WRITE:
 	case 0x200 ... 0x2ff: /* MTRRs */

[-- Attachment #3: Type: text/plain, Size: 345 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Re: [PATCH] RE: Writing to MCG_STATUS register

[Avi Kivity]

Kiselev, Sergey wrote:
> Resending as attachment...
>   

Applied, thanks.


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV