diff options
| author | Christian König <christian.koenig@amd.com> | 2021-11-11 12:46:46 +0300 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2021-11-30 17:59:33 +0300 |
| commit | c99907c723c6f3f6cae63201ddfac610be13f997 (patch) | |
| tree | a61811da7cd2727f30f9424b873548d016a65b04 /drivers | |
| parent | f8be2c5971f48dacbd0e6daa44184dbb6f283dc4 (diff) | |
| download | linux-c99907c723c6f3f6cae63201ddfac610be13f997.tar.xz | |
dma-buf: make fence mandatory for dma_resv_add_excl_fence v2
Calling dma_resv_add_excl_fence() with the fence as NULL and expecting
that that this frees up the fences is simply abuse of the internals of
the dma_resv object.
v2: drop the fence pruning completely.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20211129120659.1815-4-christian.koenig@amd.com
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/dma-buf/dma-resv.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c index ff3c0558b3b8..4deea75c0b9c 100644 --- a/drivers/dma-buf/dma-resv.c +++ b/drivers/dma-buf/dma-resv.c @@ -305,8 +305,7 @@ void dma_resv_add_excl_fence(struct dma_resv *obj, struct dma_fence *fence) if (old) i = old->shared_count; - if (fence) - dma_fence_get(fence); + dma_fence_get(fence); write_seqcount_begin(&obj->seq); /* write_seqcount_begin provides the necessary memory barrier */ |