diff options
author | Rob Clark <robdclark@gmail.com> | 2014-02-06 19:19:20 -0500 |
---|---|---|
committer | Rob Clark <robdclark@gmail.com> | 2014-02-07 10:26:25 -0500 |
commit | c2703b13a63da335053a2f3cb560bb806bdda0ae (patch) | |
tree | d7f4bfd1dac7d4c0a71b6823e2e29365a73fb2cb /drivers/gpu/drm/msm/msm_gpu.c | |
parent | 9999f105e76977d97304d7cb2030a52d96800d69 (diff) | |
download | linux-c2703b13a63da335053a2f3cb560bb806bdda0ae.tar.gz linux-c2703b13a63da335053a2f3cb560bb806bdda0ae.tar.bz2 linux-c2703b13a63da335053a2f3cb560bb806bdda0ae.zip |
drm/msm: bigger synchronization hammer
Because we use a list_head in the bo to track it's position in a submit,
we need to serialize at a higher layer. Otherwise there are problems
when multiple contexts are SUBMIT'ing in parallel cmdstreams referencing
a shared bo.
Signed-off-by: Rob Clark <robdclark@gmail.com>
Diffstat (limited to 'drivers/gpu/drm/msm/msm_gpu.c')
-rw-r--r-- | drivers/gpu/drm/msm/msm_gpu.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c index 4ebce8be489d..0cfe3f426ee4 100644 --- a/drivers/gpu/drm/msm/msm_gpu.c +++ b/drivers/gpu/drm/msm/msm_gpu.c @@ -298,8 +298,6 @@ int msm_gpu_submit(struct msm_gpu *gpu, struct msm_gem_submit *submit, struct msm_drm_private *priv = dev->dev_private; int i, ret; - mutex_lock(&dev->struct_mutex); - submit->fence = ++priv->next_fence; gpu->submitted_fence = submit->fence; @@ -331,7 +329,6 @@ int msm_gpu_submit(struct msm_gpu *gpu, struct msm_gem_submit *submit, msm_gem_move_to_active(&msm_obj->base, gpu, true, submit->fence); } hangcheck_timer_reset(gpu); - mutex_unlock(&dev->struct_mutex); return ret; } |