lkml.org 
[lkml]   [2020]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    SubjectRE: [PATCH] drm/radeon: Fix reference count leaks caused by pm_runtime_get_sync
    Date
    [AMD Official Use Only - Internal Distribution Only]

    Acked-by: Evan Quan <evan.quan@amd.com>

    -----Original Message-----
    From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of Aditya Pakki
    Sent: Sunday, June 14, 2020 10:21 AM
    To: pakki001@umn.edu
    Cc: wu000273@umn.edu; David Airlie <airlied@linux.ie>; kjlu@umn.edu; linux-kernel@vger.kernel.org; amd-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; Daniel Vetter <daniel@ffwll.ch>; Deucher, Alexander <Alexander.Deucher@amd.com>; Koenig, Christian <Christian.Koenig@amd.com>
    Subject: [PATCH] drm/radeon: Fix reference count leaks caused by pm_runtime_get_sync

    On calling pm_runtime_get_sync() the reference count of the device is incremented. In case of failure, decrement the reference count before returning the error.

    Signed-off-by: Aditya Pakki <pakki001@umn.edu>
    ---
    drivers/gpu/drm/radeon/radeon_display.c | 4 +++-
    drivers/gpu/drm/radeon/radeon_drv.c | 4 +++-
    drivers/gpu/drm/radeon/radeon_kms.c | 4 +++-
    3 files changed, 9 insertions(+), 3 deletions(-)

    diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c
    index 35db79a168bf..df1a7eb73651 100644
    --- a/drivers/gpu/drm/radeon/radeon_display.c
    +++ b/drivers/gpu/drm/radeon/radeon_display.c
    @@ -635,8 +635,10 @@ radeon_crtc_set_config(struct drm_mode_set *set,
    dev = set->crtc->dev;

    ret = pm_runtime_get_sync(dev->dev);
    -if (ret < 0)
    +if (ret < 0) {
    +pm_runtime_put_autosuspend(dev->dev);
    return ret;
    +}

    ret = drm_crtc_helper_set_config(set, ctx);

    diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
    index bbb0883e8ce6..62b5069122cc 100644
    --- a/drivers/gpu/drm/radeon/radeon_drv.c
    +++ b/drivers/gpu/drm/radeon/radeon_drv.c
    @@ -549,8 +549,10 @@ long radeon_drm_ioctl(struct file *filp,
    long ret;
    dev = file_priv->minor->dev;
    ret = pm_runtime_get_sync(dev->dev);
    -if (ret < 0)
    +if (ret < 0) {
    +pm_runtime_put_autosuspend(dev->dev);
    return ret;
    +}

    ret = drm_ioctl(filp, cmd, arg);

    diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c
    index c5d1dc9618a4..99ee60f8b604 100644
    --- a/drivers/gpu/drm/radeon/radeon_kms.c
    +++ b/drivers/gpu/drm/radeon/radeon_kms.c
    @@ -638,8 +638,10 @@ int radeon_driver_open_kms(struct drm_device *dev, struct drm_file *file_priv)
    file_priv->driver_priv = NULL;

    r = pm_runtime_get_sync(dev->dev);
    -if (r < 0)
    +if (r < 0) {
    +pm_runtime_put_autosuspend(dev->dev);
    return r;
    +}

    /* new gpu have virtual address space support */
    if (rdev->family >= CHIP_CAYMAN) {
    --
    2.25.1
    _______________________________________________
    amd-gfx mailing list
    amd-gfx@lists.freedesktop.org
    https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&amp;data=02%7C01%7Cevan.quan%40amd.com%7Cc86101e02ef24c52b36408d810fdcc14%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637278029582429567&amp;sdata=qtKTCV33q8l2GTxMUX0nlJ4fV32dXaLH7y6hymksQEo%3D&amp;reserved=0
    \
     
     \ /
      Last update: 2020-06-16 05:27    [W:2.741 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site