Re: [PATCH 07/10] dma-buf/resv: add new fences container implementation

2019-08-22 Thread Koenig, Christian
Am 22.08.19 um 15:02 schrieb Daniel Vetter: > On Thu, Aug 22, 2019 at 10:23:29AM +0200, Christian König wrote: >> Am 21.08.19 um 18:04 schrieb Daniel Vetter: >>> On Wed, Aug 21, 2019 at 02:31:44PM +0200, Christian König wrote: [SNIP] + /* Try to drop the last reference */ + if (!dm

Re: [PATCH 07/10] dma-buf/resv: add new fences container implementation

2019-08-22 Thread Daniel Vetter
On Thu, Aug 22, 2019 at 10:23:29AM +0200, Christian König wrote: > Am 21.08.19 um 18:04 schrieb Daniel Vetter: > > On Wed, Aug 21, 2019 at 02:31:44PM +0200, Christian König wrote: > > > [SNIP] > > > + /* Try to drop the last reference */ > > > + if (!dma_fence_array_recycle(staged)) > > Without an

Re: [PATCH 07/10] dma-buf/resv: add new fences container implementation

2019-08-22 Thread Christian König
Am 21.08.19 um 18:04 schrieb Daniel Vetter: On Wed, Aug 21, 2019 at 02:31:44PM +0200, Christian König wrote: [SNIP] + /* Try to drop the last reference */ + if (!dma_fence_array_recycle(staged)) Without an rcu barrier here you're not syncing to new clients at all. I don't think this

Re: [PATCH 07/10] dma-buf/resv: add new fences container implementation

2019-08-21 Thread Daniel Vetter
On Wed, Aug 21, 2019 at 02:31:44PM +0200, Christian König wrote: > Add a new container for fences which internally uses > dma_fence_array's to store the fences. > > Signed-off-by: Christian König > --- > drivers/dma-buf/dma-resv.c | 221 + > include/linux/dma-

[PATCH 07/10] dma-buf/resv: add new fences container implementation

2019-08-21 Thread Christian König
Add a new container for fences which internally uses dma_fence_array's to store the fences. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 221 + include/linux/dma-resv.h | 49 2 files changed, 270 insertions(+) diff --git a/drive