Re: [PATCH] dma-buf/sw_sync: hold a fence reference when check if it signaled

2017-07-28 Thread Chris Wilson
Quoting Gustavo Padovan (2017-07-28 02:57:25) > 2017-07-27 Chris Wilson : > > > Quoting Gustavo Padovan (2017-07-27 20:03:53) > > > From: Gustavo Padovan > > > > > > If userspace already dropped its own reference by closing the sw_sync > > > fence fd we might end up in a deadlock where > > > dma

Re: [PATCH] dma-buf/sw_sync: hold a fence reference when check if it signaled

2017-07-28 Thread Daniel Vetter
On Thu, Jul 27, 2017 at 04:03:53PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan > > If userspace already dropped its own reference by closing the sw_sync > fence fd we might end up in a deadlock where > dma_fence_is_signaled_locked() will trigger the release of the fence a > thus try to

Re: [PATCH] dma-buf/sw_sync: hold a fence reference when check if it signaled

2017-07-27 Thread Gustavo Padovan
2017-07-27 Chris Wilson : > Quoting Gustavo Padovan (2017-07-27 20:03:53) > > From: Gustavo Padovan > > > > If userspace already dropped its own reference by closing the sw_sync > > fence fd we might end up in a deadlock where > > dma_fence_is_signaled_locked() will trigger the release of the fe

Re: [PATCH] dma-buf/sw_sync: hold a fence reference when check if it signaled

2017-07-27 Thread Chris Wilson
Quoting Gustavo Padovan (2017-07-27 20:03:53) > From: Gustavo Padovan > > If userspace already dropped its own reference by closing the sw_sync > fence fd we might end up in a deadlock where > dma_fence_is_signaled_locked() will trigger the release of the fence a > thus try to hold the lock to re

[PATCH] dma-buf/sw_sync: hold a fence reference when check if it signaled

2017-07-27 Thread Gustavo Padovan
From: Gustavo Padovan If userspace already dropped its own reference by closing the sw_sync fence fd we might end up in a deadlock where dma_fence_is_signaled_locked() will trigger the release of the fence a thus try to hold the lock to remove the fence from the list. We need to grab a reference