lkml.org 
[lkml]   [2016]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.13.y-ckt 078/138] drm/radeon: hold reference to fences in radeon_sa_bo_new
    Date
    3.13.11-ckt36 -stable review patch.  If anyone has any objections, please let me know.

    ---8<------------------------------------------------------------

    From: =?UTF-8?q?Nicolai=20H=C3=A4hnle?= <nicolai.haehnle@amd.com>

    commit f6ff4f67cdf8455d0a4226eeeaf5af17c37d05eb upstream.

    An arbitrary amount of time can pass between spin_unlock and
    radeon_fence_wait_any, so we need to ensure that nobody frees the
    fences from under us.

    Based on the analogous fix for amdgpu.

    Signed-off-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
    Reviewed-by: Christian König <christian.koenig@amd.com>
    Signed-off-by: Kamal Mostafa <kamal@canonical.com>
    ---
    drivers/gpu/drm/radeon/radeon_sa.c | 5 +++++
    1 file changed, 5 insertions(+)

    diff --git a/drivers/gpu/drm/radeon/radeon_sa.c b/drivers/gpu/drm/radeon/radeon_sa.c
    index f0bac68..bb16684 100644
    --- a/drivers/gpu/drm/radeon/radeon_sa.c
    +++ b/drivers/gpu/drm/radeon/radeon_sa.c
    @@ -349,8 +349,13 @@ int radeon_sa_bo_new(struct radeon_device *rdev,
    /* see if we can skip over some allocations */
    } while (radeon_sa_bo_next_hole(sa_manager, fences, tries));

    + for (i = 0; i < RADEON_NUM_RINGS; ++i)
    + radeon_fence_ref(fences[i]);
    +
    spin_unlock(&sa_manager->wq.lock);
    r = radeon_fence_wait_any(rdev, fences, false);
    + for (i = 0; i < RADEON_NUM_RINGS; ++i)
    + radeon_fence_unref(&fences[i]);
    spin_lock(&sa_manager->wq.lock);
    /* if we have nothing to wait for block */
    if (r == -ENOENT && block) {
    --
    2.7.0
    \
     
     \ /
      Last update: 2016-03-10 01:01    [W:4.042 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site