]> xenbits.xensource.com Git - people/vhanquez/xen-unstable.git/commitdiff
x86-64: don't allow wrmsr to MSR_FAM10H_MMIO_CONF_BASE when Xen itself is using it
authorKeir Fraser <keir@xen.org>
Mon, 10 Jan 2011 08:42:32 +0000 (08:42 +0000)
committerKeir Fraser <keir@xen.org>
Mon, 10 Jan 2011 08:42:32 +0000 (08:42 +0000)
Signed-off-by: Jan Beulich <jbeulich@novell.com>
xen/arch/x86/traps.c

index f7d8f69c9c2e451a34d78d50001566d66d197324..6e2c5c7311b6440462e3420afac6f9be593bbd74 100644 (file)
@@ -1694,6 +1694,10 @@ static int is_cpufreq_controller(struct domain *d)
             (d->domain_id == 0));
 }
 
+#ifdef CONFIG_X86_64
+#include "x86_64/mmconfig.h"
+#endif
+
 static int emulate_privileged_op(struct cpu_user_regs *regs)
 {
     struct vcpu *v = current;
@@ -2288,7 +2292,14 @@ static int emulate_privileged_op(struct cpu_user_regs *regs)
                 goto fail;
             if ( !IS_PRIV(v->domain) )
                 break;
-            if ( (rdmsr_safe(MSR_FAM10H_MMIO_CONF_BASE, val) != 0) ||
+            if ( (rdmsr_safe(MSR_FAM10H_MMIO_CONF_BASE, val) != 0) )
+                goto fail;
+            if (
+#ifdef CONFIG_X86_64
+                 (pci_probe & PCI_PROBE_MMCONF) &&
+                 (pci_probe & PCI_CHECK_ENABLE_AMD_MMCONF) ?
+                 val != msr_content :
+#endif
                  ((val ^ msr_content) &
                   ~( FAM10H_MMIO_CONF_ENABLE |
                     (FAM10H_MMIO_CONF_BUSRANGE_MASK <<