diff options
| author | Christian König <christian.koenig@amd.com> | 2023-11-14 13:37:09 +0100 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2023-11-27 20:00:47 +0100 |
| commit | 95ba893c9f4feb836ddce627efd0bb6af6667031 (patch) | |
| tree | efef508d34b320cc030705ba7aa24009bff905ea /drivers/dma-buf | |
| parent | 1d5e8f4bf06da86b71cc9169110d1a0e1e7af337 (diff) | |
| download | lwn-95ba893c9f4feb836ddce627efd0bb6af6667031.tar.gz lwn-95ba893c9f4feb836ddce627efd0bb6af6667031.zip | |
dma-buf: fix check in dma_resv_add_fence
It's valid to add the same fence multiple times to a dma-resv object and
we shouldn't need one extra slot for each.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Fixes: a3f7c10a269d5 ("dma-buf/dma-resv: check if the new fence is really later")
Cc: stable@vger.kernel.org # v5.19+
Link: https://patchwork.freedesktop.org/patch/msgid/20231115093035.1889-1-christian.koenig@amd.com
Diffstat (limited to 'drivers/dma-buf')
| -rw-r--r-- | drivers/dma-buf/dma-resv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c index 38b4110378de..eb8b733065b2 100644 --- a/drivers/dma-buf/dma-resv.c +++ b/drivers/dma-buf/dma-resv.c @@ -301,7 +301,7 @@ void dma_resv_add_fence(struct dma_resv *obj, struct dma_fence *fence, dma_resv_list_entry(fobj, i, obj, &old, &old_usage); if ((old->context == fence->context && old_usage >= usage && - dma_fence_is_later(fence, old)) || + dma_fence_is_later_or_same(fence, old)) || dma_fence_is_signaled(old)) { dma_resv_list_set(fobj, i, fence, usage); dma_fence_put(old); |