Commit d66ce54b authored by Joerg Roedel's avatar Joerg Roedel

iommu/vt-d: Split iommu_prepare_identity_map

Split the part of the function that fetches the domain out
and put the rest into into a domain_prepare_identity_map, so
that the code can also be used with when the domain is
already known.
Signed-off-by: default avatarJoerg Roedel <jroedel@suse.de>
parent 9ffecb10
...@@ -2429,17 +2429,11 @@ static int iommu_domain_identity_map(struct dmar_domain *domain, ...@@ -2429,17 +2429,11 @@ static int iommu_domain_identity_map(struct dmar_domain *domain,
DMA_PTE_READ|DMA_PTE_WRITE); DMA_PTE_READ|DMA_PTE_WRITE);
} }
static int iommu_prepare_identity_map(struct device *dev, static int domain_prepare_identity_map(struct device *dev,
unsigned long long start, struct dmar_domain *domain,
unsigned long long end) unsigned long long start,
unsigned long long end)
{ {
struct dmar_domain *domain;
int ret;
domain = get_domain_for_dev(dev, DEFAULT_DOMAIN_ADDRESS_WIDTH);
if (!domain)
return -ENOMEM;
/* For _hardware_ passthrough, don't bother. But for software /* For _hardware_ passthrough, don't bother. But for software
passthrough, we do it anyway -- it may indicate a memory passthrough, we do it anyway -- it may indicate a memory
range which is reserved in E820, so which didn't get set range which is reserved in E820, so which didn't get set
...@@ -2459,8 +2453,7 @@ static int iommu_prepare_identity_map(struct device *dev, ...@@ -2459,8 +2453,7 @@ static int iommu_prepare_identity_map(struct device *dev,
dmi_get_system_info(DMI_BIOS_VENDOR), dmi_get_system_info(DMI_BIOS_VENDOR),
dmi_get_system_info(DMI_BIOS_VERSION), dmi_get_system_info(DMI_BIOS_VERSION),
dmi_get_system_info(DMI_PRODUCT_VERSION)); dmi_get_system_info(DMI_PRODUCT_VERSION));
ret = -EIO; return -EIO;
goto error;
} }
if (end >> agaw_to_width(domain->agaw)) { if (end >> agaw_to_width(domain->agaw)) {
...@@ -2470,18 +2463,27 @@ static int iommu_prepare_identity_map(struct device *dev, ...@@ -2470,18 +2463,27 @@ static int iommu_prepare_identity_map(struct device *dev,
dmi_get_system_info(DMI_BIOS_VENDOR), dmi_get_system_info(DMI_BIOS_VENDOR),
dmi_get_system_info(DMI_BIOS_VERSION), dmi_get_system_info(DMI_BIOS_VERSION),
dmi_get_system_info(DMI_PRODUCT_VERSION)); dmi_get_system_info(DMI_PRODUCT_VERSION));
ret = -EIO; return -EIO;
goto error;
} }
ret = iommu_domain_identity_map(domain, start, end); return iommu_domain_identity_map(domain, start, end);
if (ret) }
goto error;
return 0; static int iommu_prepare_identity_map(struct device *dev,
unsigned long long start,
unsigned long long end)
{
struct dmar_domain *domain;
int ret;
domain = get_domain_for_dev(dev, DEFAULT_DOMAIN_ADDRESS_WIDTH);
if (!domain)
return -ENOMEM;
ret = domain_prepare_identity_map(dev, domain, start, end);
if (ret)
domain_exit(domain);
error:
domain_exit(domain);
return ret; return ret;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment