summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJoerg Roedel <jroedel@suse.de>2015-06-12 12:02:09 +0200
committerJoerg Roedel <jroedel@suse.de>2015-06-16 10:59:34 +0200
commitdbcd861f252d727ac856a3da2cba8a480a36ab2e (patch)
tree888c1b9cb3f9a39acc61886c97186698f85853b9
parent091d42e43d21b6ca7ec39bf5f9e17bc0bd8d4312 (diff)
downloadop-kernel-dev-dbcd861f252d727ac856a3da2cba8a480a36ab2e.zip
op-kernel-dev-dbcd861f252d727ac856a3da2cba8a480a36ab2e.tar.gz
iommu/vt-d: Do not re-use domain-ids from the old kernel
Mark all domain-ids we find as reserved, so that there could be no collision between domains from the previous kernel and our domains in the IOMMU TLB. Tested-by: ZhenHua Li <zhen-hual@hp.com> Tested-by: Baoquan He <bhe@redhat.com> Signed-off-by: Joerg Roedel <jroedel@suse.de>
-rw-r--r--drivers/iommu/intel-iommu.c11
1 files changed, 10 insertions, 1 deletions
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index 1eb743c..4318533 100644
--- a/drivers/iommu/intel-iommu.c
+++ b/drivers/iommu/intel-iommu.c
@@ -273,6 +273,11 @@ static inline void context_set_domain_id(struct context_entry *context,
context->hi |= (value & ((1 << 16) - 1)) << 8;
}
+static inline int context_domain_id(struct context_entry *c)
+{
+ return((c->hi >> 8) & 0xffff);
+}
+
static inline void context_clear_entry(struct context_entry *context)
{
context->lo = 0;
@@ -2802,7 +2807,7 @@ static int copy_context_table(struct intel_iommu *iommu,
int bus, bool ext)
{
struct context_entry *old_ce = NULL, *new_ce = NULL, ce;
- int tbl_idx, pos = 0, idx, devfn, ret = 0;
+ int tbl_idx, pos = 0, idx, devfn, ret = 0, did;
phys_addr_t old_ce_phys;
tbl_idx = ext ? bus * 2 : bus;
@@ -2857,6 +2862,10 @@ static int copy_context_table(struct intel_iommu *iommu,
if (!context_present(&ce))
continue;
+ did = context_domain_id(&ce);
+ if (did >= 0 && did < cap_ndoms(iommu->cap))
+ set_bit(did, iommu->domain_ids);
+
new_ce[idx] = ce;
}
OpenPOWER on IntegriCloud