diff options
| author | Monk Liu <Monk.Liu@amd.com> | 2017-04-07 18:39:07 +0800 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2017-04-28 16:38:00 -0400 |
| commit | d7b1eeb2ca039d04f1a1fcb241920cb112b4b52a (patch) | |
| tree | 60d4a49eeb0baf5e3a3181cec53d5c4033af4c97 /drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | |
| parent | 5e78835abdabc303ba980f50e9f71039e7689cc9 (diff) | |
| download | talos-op-linux-d7b1eeb2ca039d04f1a1fcb241920cb112b4b52a.tar.gz talos-op-linux-d7b1eeb2ca039d04f1a1fcb241920cb112b4b52a.zip | |
drm/amdgpu:fix race condition
sequence is protected by spinlock so don't access sequence
in paramter seq when invoking this function.
~0 means to get the latest sequence number and 0 means none to
get.
Change-Id: Ib7a03f3cf5594deeb4ad333cc59b47a6bddfd1ad
Signed-off-by: Monk Liu <Monk.Liu@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c index cf0500671353..90d1ac8a80f8 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c @@ -273,6 +273,9 @@ struct dma_fence *amdgpu_ctx_get_fence(struct amdgpu_ctx *ctx, spin_lock(&ctx->ring_lock); + if (seq == ~0ull) + seq = ctx->rings[ring->idx].sequence - 1; + if (seq >= cring->sequence) { spin_unlock(&ctx->ring_lock); return ERR_PTR(-EINVAL); |
