mirrors/linux-stable
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git synced 2024-09-12 21:57:43 +00:00
Code Issues Releases Wiki Activity

x86/amd-iommu: Allow dev_data->alias to be NULL

Browse source 
Let dev_data->alias be just NULL if the device has no alias.

Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
This commit is contained in:
Joerg Roedel 2011-06-09 17:48:39 +02:00
parent ec9e79ef06
commit 2b02b091ab
1 changed files with 41 additions and 36 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

77
arch/x86/kernel/amd_iommu.c
View file

@ -183,12 +183,14 @@ static int iommu_init_device(struct device *dev)
return -ENOMEM; return -ENOMEM;
alias = amd_iommu_alias_table[dev_data->devid]; alias = amd_iommu_alias_table[dev_data->devid];
pdev = pci_get_bus_and_slot(PCI_BUS(alias), alias & 0xff); if (alias != dev_data->devid) {
if (pdev) pdev = pci_get_bus_and_slot(PCI_BUS(alias), alias & 0xff);
dev_data->alias = &pdev->dev; if (pdev)
else { dev_data->alias = &pdev->dev;
free_dev_data(dev_data); else {
return -ENOTSUPP; free_dev_data(dev_data);
return -ENOTSUPP;
}
} }
dev->archdata.iommu = dev_data; dev->archdata.iommu = dev_data;
@ -1604,29 +1606,31 @@ static void do_detach(struct iommu_dev_data *dev_data)
static int __attach_device(struct iommu_dev_data *dev_data, static int __attach_device(struct iommu_dev_data *dev_data,
struct protection_domain *domain) struct protection_domain *domain)
{ {
struct iommu_dev_data *alias_data;
int ret; int ret;
alias_data = get_dev_data(dev_data->alias);
if (!alias_data)
return -EINVAL;
/* lock domain */ /* lock domain */
spin_lock(&domain->lock); spin_lock(&domain->lock);
/* Some sanity checks */ if (dev_data->alias != NULL) {
ret = -EBUSY; struct iommu_dev_data *alias_data;
if (alias_data->domain != NULL &&
alias_data->domain != domain)
goto out_unlock;
if (dev_data->domain != NULL && alias_data = get_dev_data(dev_data->alias);
dev_data->domain != domain)
goto out_unlock;
/* Do real assignment */ ret = -EINVAL;
if (dev_data->devid != alias_data->devid) { if (!alias_data)
goto out_unlock;
/* Some sanity checks */
ret = -EBUSY;
if (alias_data->domain != NULL &&
alias_data->domain != domain)
goto out_unlock;
if (dev_data->domain != NULL &&
dev_data->domain != domain)
goto out_unlock;
/* Do real assignment */
if (alias_data->domain == NULL) if (alias_data->domain == NULL)
do_attach(alias_data, domain); do_attach(alias_data, domain);
@ -1696,8 +1700,8 @@ static void __detach_device(struct iommu_dev_data *dev_data)
spin_lock_irqsave(&domain->lock, flags); spin_lock_irqsave(&domain->lock, flags);
alias_data = get_dev_data(dev_data->alias); if (dev_data->alias) {
if (dev_data->devid != alias_data->devid) { alias_data = get_dev_data(dev_data->alias);
if (atomic_dec_and_test(&alias_data->bind)) if (atomic_dec_and_test(&alias_data->bind))
do_detach(alias_data); do_detach(alias_data);
} }
@ -1744,25 +1748,26 @@ static void detach_device(struct device *dev)
*/ */
static struct protection_domain *domain_for_device(struct device *dev) static struct protection_domain *domain_for_device(struct device *dev)
{ {
struct protection_domain *dom;
struct iommu_dev_data *dev_data, *alias_data; struct iommu_dev_data *dev_data, *alias_data;
struct protection_domain *dom = NULL;
unsigned long flags; unsigned long flags;
dev_data = get_dev_data(dev); dev_data = get_dev_data(dev);
alias_data = get_dev_data(dev_data->alias);
if (!alias_data)
return NULL;
read_lock_irqsave(&amd_iommu_devtable_lock, flags); if (dev_data->domain)
dom = dev_data->domain; return dev_data->domain;
if (dom == NULL &&
alias_data->domain != NULL) { if (dev_data->alias != NULL) {
__attach_device(dev_data, alias_data->domain); alias_data = get_dev_data(dev_data->alias);
dom = alias_data->domain;
read_lock_irqsave(&amd_iommu_devtable_lock, flags);
if (alias_data->domain != NULL) {
__attach_device(dev_data, alias_data->domain);
dom = alias_data->domain;
}
read_unlock_irqrestore(&amd_iommu_devtable_lock, flags);
} }
read_unlock_irqrestore(&amd_iommu_devtable_lock, flags);
return dom; return dom;
} }