[PATCH] Allow Dom0 to read patchlevel MSR

[PATCH] Allow Dom0 to read patchlevel MSR

[Christoph Egger]

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


Hi,

Allow Dom0 to read microcode patchlevel MSR.
Solaris needs this.

Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>

-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy

[-- Attachment #2: xen_microcode_msr.diff --]
[-- Type: text/x-diff, Size: 665 bytes --]

diff -r 087008dfb005 xen/arch/x86/traps.c
--- a/xen/arch/x86/traps.c	Mon Sep 15 17:10:43 2008 +0100
+++ b/xen/arch/x86/traps.c	Tue Sep 16 13:57:31 2008 +0200
@@ -2242,6 +2242,12 @@ static int emulate_privileged_op(struct 
             if ( rdmsr_safe(regs->ecx, regs->eax, regs->edx) )
                 goto fail;
             break;
+        case MSR_AMD_PATCHLEVEL:
+            if ( boot_cpu_data.x86_vendor != X86_VENDOR_AMD )
+                goto fail;
+            if ( rdmsr_safe(regs->ecx, regs->eax, regs->edx) )
+                goto fail;
+            break;
         default:
             if ( rdmsr_hypervisor_regs(regs->ecx, &l, &h) )
             {

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

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

Re: [PATCH] Allow Dom0 to read patchlevel MSR

[Keir Fraser]

On 16/9/08 13:06, "Christoph Egger" <Christoph.Egger@amd.com> wrote:

> 
> Hi,
> 
> Allow Dom0 to read microcode patchlevel MSR.
> Solaris needs this.
> 
> Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>

A pointless patch. The default case of the switch statement has equivalent
effect.

I guess I'll keep the MSR labelled there just for documentation in case we
tighten up RDMSR/WRMSR emulation in future.

 -- Keir

Re: [PATCH] Allow Dom0 to read patchlevel MSR

[Christoph Egger]

On Tuesday 16 September 2008 14:15:10 Keir Fraser wrote:
> On 16/9/08 13:06, "Christoph Egger" <Christoph.Egger@amd.com> wrote:
> > Hi,
> >
> > Allow Dom0 to read microcode patchlevel MSR.
> > Solaris needs this.
> >
> > Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>
>
> A pointless patch. The default case of the switch statement has equivalent
> effect.
>
> I guess I'll keep the MSR labelled there just for documentation in case we
> tighten up RDMSR/WRMSR emulation in future.

Tnx.


-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy