diff options
| author | Christian König <christian.koenig@amd.com> | 2022-01-19 14:39:39 +0100 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2022-02-08 09:21:10 +0100 |
| commit | 68129f431faab376c1dd1c701f2fb999eea53383 (patch) | |
| tree | a417f9632bc5f399449ea172c217723c14f26666 | |
| parent | 270b48bb8da7452b4357d8726933beba72652310 (diff) | |
dma-buf: warn about containers in dma_resv object
Drivers should not add containers as shared fences to the dma_resv
object, instead each fence should be added individually.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220204100429.2049-5-christian.koenig@amd.com
| -rw-r--r-- | drivers/dma-buf/dma-resv.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c index ee31f15d633a..b51416405e86 100644 --- a/drivers/dma-buf/dma-resv.c +++ b/drivers/dma-buf/dma-resv.c @@ -256,6 +256,11 @@ void dma_resv_add_shared_fence(struct dma_resv *obj, struct dma_fence *fence) dma_resv_assert_held(obj); + /* Drivers should not add containers here, instead add each fence + * individually. + */ + WARN_ON(dma_fence_is_container(fence)); + fobj = dma_resv_shared_list(obj); count = fobj->shared_count; |