drm/amdkfd: fix ref count leak when pm_runtime_get_sync fails
authorAlex Deucher <alexander.deucher@amd.com>
Wed, 17 Jun 2020 18:52:07 +0000 (14:52 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 1 Jul 2020 05:59:23 +0000 (01:59 -0400)
The call to pm_runtime_get_sync increments the counter even in case of
failure, leading to incorrect ref count.
In case of failure, decrement the ref count before returning.

Reviewed-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdkfd/kfd_process.c

index a9a7f5a..0b4f409 100644 (file)
@@ -1116,8 +1116,10 @@ struct kfd_process_device *kfd_bind_process_to_device(struct kfd_dev *dev,
         */
        if (!pdd->runtime_inuse) {
                err = pm_runtime_get_sync(dev->ddev->dev);
-               if (err < 0)
+               if (err < 0) {
+                       pm_runtime_put_autosuspend(dev->ddev->dev);
                        return ERR_PTR(err);
+               }
        }
 
        err = kfd_iommu_bind_process_to_device(pdd);