x86/amd-iommu: Add per IOMMU reference counting

This patch adds reference counting for protection domains
per IOMMU. This allows a smarter TLB flushing strategy.

Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
This commit is contained in:
Joerg Roedel 2009-11-20 14:57:32 +01:00
parent bb52777ec4
commit c459611424
2 changed files with 11 additions and 3 deletions

View file

@ -238,7 +238,9 @@ struct protection_domain {
unsigned long flags; /* flags to find out type of domain */ unsigned long flags; /* flags to find out type of domain */
bool updated; /* complete domain flush required */ bool updated; /* complete domain flush required */
unsigned dev_cnt; /* devices assigned to this domain */ unsigned dev_cnt; /* devices assigned to this domain */
unsigned dev_iommu[MAX_IOMMUS]; /* per-IOMMU reference count */
void *priv; /* private data */ void *priv; /* private data */
}; };
/* /*

View file

@ -1175,7 +1175,9 @@ static void __attach_device(struct amd_iommu *iommu,
/* update DTE entry */ /* update DTE entry */
set_dte_entry(devid, domain); set_dte_entry(devid, domain);
domain->dev_cnt += 1; /* Do reference counting */
domain->dev_iommu[iommu->index] += 1;
domain->dev_cnt += 1;
/* ready */ /* ready */
spin_unlock(&domain->lock); spin_unlock(&domain->lock);
@ -1209,6 +1211,9 @@ static void attach_device(struct amd_iommu *iommu,
*/ */
static void __detach_device(struct protection_domain *domain, u16 devid) static void __detach_device(struct protection_domain *domain, u16 devid)
{ {
struct amd_iommu *iommu = amd_iommu_rlookup_table[devid];
BUG_ON(!iommu);
/* lock domain */ /* lock domain */
spin_lock(&domain->lock); spin_lock(&domain->lock);
@ -1223,8 +1228,9 @@ static void __detach_device(struct protection_domain *domain, u16 devid)
amd_iommu_apply_erratum_63(devid); amd_iommu_apply_erratum_63(devid);
/* decrease reference counter */ /* decrease reference counters */
domain->dev_cnt -= 1; domain->dev_iommu[iommu->index] -= 1;
domain->dev_cnt -= 1;
/* ready */ /* ready */
spin_unlock(&domain->lock); spin_unlock(&domain->lock);