In the probe_acpi_namespace_devices function,when the physical
node of the acpi device is NULL,the unlock function is missing.
Add mutex_unlock(&adev->physical_node_lock).

Reported-by: Dan Carpenter <dan.carpen...@oracle.com>
Signed-off-by: FelixCuioc <felixcui...@zhaoxin.com>
---
 drivers/iommu/intel/iommu.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c
index b31f02f41c96..25e9853cba1b 100644
--- a/drivers/iommu/intel/iommu.c
+++ b/drivers/iommu/intel/iommu.c
@@ -4851,6 +4851,7 @@ static int __init probe_acpi_namespace_devices(void)
                        if (pn_dev == NULL) {
                                dev->bus->iommu_ops = &intel_iommu_ops;
                                ret = iommu_probe_device(dev);
+                               mutex_unlock(&adev->physical_node_lock);
                                if (ret) {
                                        pr_err("acpi_device probe fail! 
ret:%d\n", ret);
                                        return ret;
-- 
2.17.1

_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to