# HG changeset patch
# User Tim Deegan <tim@xxxxxxx>
# Date 1314092605 -3600
# Node ID 9d2a8912597dba29e57a3e4db0a98f1f6c73168d
# Parent 41f00cf6b82205cab03a583356c6a64855539726
IOMMU: only try to share IOMMU and HAP tables for domains with P2M.
This makes the check more precise, and brings VTd in line with AMD code.
Signed-off-by: Tim Deegan <tim@xxxxxxx>
---
diff -r 41f00cf6b822 -r 9d2a8912597d xen/drivers/passthrough/amd/iommu_map.c
--- a/xen/drivers/passthrough/amd/iommu_map.c Tue Aug 23 10:43:20 2011 +0100
+++ b/xen/drivers/passthrough/amd/iommu_map.c Tue Aug 23 10:43:25 2011 +0100
@@ -758,7 +758,7 @@
BUG_ON( !hd->root_table );
- if ( iommu_hap_pt_share && is_hvm_domain(d) )
+ if ( iommu_use_hap_pt(d) )
return 0;
memset(pt_mfn, 0, sizeof(pt_mfn));
@@ -835,7 +835,7 @@
BUG_ON( !hd->root_table );
- if ( iommu_hap_pt_share && is_hvm_domain(d) )
+ if ( iommu_use_hap_pt(d) )
return 0;
memset(pt_mfn, 0, sizeof(pt_mfn));
@@ -935,7 +935,7 @@
ASSERT( is_hvm_domain(d) && d->arch.hvm_domain.hap_enabled );
- if ( !iommu_hap_pt_share )
+ if ( !iommu_use_hap_pt(d) )
return;
pgd_mfn = pagetable_get_mfn(p2m_get_pagetable(p2m_get_hostp2m(d)));
diff -r 41f00cf6b822 -r 9d2a8912597d xen/drivers/passthrough/amd/pci_amd_iommu.c
--- a/xen/drivers/passthrough/amd/pci_amd_iommu.c Tue Aug 23 10:43:20
2011 +0100
+++ b/xen/drivers/passthrough/amd/pci_amd_iommu.c Tue Aug 23 10:43:25
2011 +0100
@@ -21,6 +21,7 @@
#include <xen/sched.h>
#include <xen/pci.h>
#include <xen/pci_regs.h>
+#include <xen/paging.h>
#include <asm/hvm/iommu.h>
#include <asm/amd-iommu.h>
#include <asm/hvm/svm/amd-iommu-proto.h>
@@ -388,7 +389,7 @@
{
struct hvm_iommu *hd = domain_hvm_iommu(d);
- if ( iommu_hap_pt_share )
+ if ( iommu_use_hap_pt(d) )
return;
spin_lock(&hd->mapping_lock);
diff -r 41f00cf6b822 -r 9d2a8912597d xen/drivers/passthrough/iommu.c
--- a/xen/drivers/passthrough/iommu.c Tue Aug 23 10:43:20 2011 +0100
+++ b/xen/drivers/passthrough/iommu.c Tue Aug 23 10:43:25 2011 +0100
@@ -180,7 +180,7 @@
if ( has_arch_pdevs(d) && !need_iommu(d) )
{
d->need_iommu = 1;
- if ( !iommu_hap_pt_share )
+ if ( !iommu_use_hap_pt(d) )
rc = iommu_populate_page_table(d);
goto done;
}
diff -r 41f00cf6b822 -r 9d2a8912597d xen/drivers/passthrough/vtd/iommu.c
--- a/xen/drivers/passthrough/vtd/iommu.c Tue Aug 23 10:43:20 2011 +0100
+++ b/xen/drivers/passthrough/vtd/iommu.c Tue Aug 23 10:43:25 2011 +0100
@@ -1613,7 +1613,7 @@
if ( list_empty(&acpi_drhd_units) )
return;
- if ( iommu_hap_pt_share )
+ if ( iommu_use_hap_pt(d) )
return;
spin_lock(&hd->mapping_lock);
@@ -1635,7 +1635,7 @@
int iommu_domid;
/* Do nothing if VT-d shares EPT page table */
- if ( iommu_hap_pt_share )
+ if ( iommu_use_hap_pt(d) )
return 0;
/* do nothing if dom0 and iommu supports pass thru */
@@ -1760,7 +1760,7 @@
ASSERT( is_hvm_domain(d) && d->arch.hvm_domain.hap_enabled );
- if ( !iommu_hap_pt_share )
+ if ( !iommu_use_hap_pt(d) )
return;
pgd_mfn = pagetable_get_mfn(p2m_get_pagetable(p2m_get_hostp2m(d)));
diff -r 41f00cf6b822 -r 9d2a8912597d xen/include/xen/iommu.h
--- a/xen/include/xen/iommu.h Tue Aug 23 10:43:20 2011 +0100
+++ b/xen/include/xen/iommu.h Tue Aug 23 10:43:25 2011 +0100
@@ -34,6 +34,9 @@
extern bool_t iommu_debug;
extern bool_t amd_iommu_perdev_intremap;
+/* Does this domain have a P2M table we can use as its IOMMU pagetable? */
+#define iommu_use_hap_pt(d) (paging_mode_hap(d) && iommu_hap_pt_share)
+
extern struct rangeset *mmio_ro_ranges;
#define domain_hvm_iommu(d) (&d->arch.hvm_domain.hvm_iommu)
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog
|
Home