diff options
| author | Christian König <christian.koenig@amd.com> | 2020-02-10 16:41:39 +0300 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2020-02-11 16:31:05 +0300 |
| commit | e81a2557e0650ee0ce37ba14476ddb709d6daf57 (patch) | |
| tree | 4fbb542169fbd0b5559b92bfb8975bd50328abd1 | |
| parent | 832c90df89f21c2ad1b3374a274fbd954bcac2ba (diff) | |
| download | linux-e81a2557e0650ee0ce37ba14476ddb709d6daf57.tar.xz | |
drm/ttm: use RCU in ttm_bo_flush_all_fences
This allows it to call the function without the lock held.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: xinhui pan <xinhui.pan@amd.com>
Link: https://patchwork.freedesktop.org/patch/352742/
| -rw-r--r-- | drivers/gpu/drm/ttm/ttm_bo.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c index c1104c8857b7..e12fc2c2d165 100644 --- a/drivers/gpu/drm/ttm/ttm_bo.c +++ b/drivers/gpu/drm/ttm/ttm_bo.c @@ -429,22 +429,24 @@ static int ttm_bo_individualize_resv(struct ttm_buffer_object *bo) static void ttm_bo_flush_all_fences(struct ttm_buffer_object *bo) { + struct dma_resv *resv = &bo->base._resv; struct dma_resv_list *fobj; struct dma_fence *fence; int i; - fobj = dma_resv_get_list(&bo->base._resv); - fence = dma_resv_get_excl(&bo->base._resv); + rcu_read_lock(); + fobj = rcu_dereference(resv->fence); + fence = rcu_dereference(resv->fence_excl); if (fence && !fence->ops->signaled) dma_fence_enable_sw_signaling(fence); for (i = 0; fobj && i < fobj->shared_count; ++i) { - fence = rcu_dereference_protected(fobj->shared[i], - dma_resv_held(bo->base.resv)); + fence = rcu_dereference(fobj->shared[i]); if (!fence->ops->signaled) dma_fence_enable_sw_signaling(fence); } + rcu_read_unlock(); } static void ttm_bo_cleanup_refs_or_queue(struct ttm_buffer_object *bo) |