Re: [PATCH] dma-buf/sync_file: only enable fence signalling during wait

2016-07-12 Thread Daniel Vetter
On Tue, Jul 12, 2016 at 10:46:45AM +0200, Maarten Lankhorst wrote: > Op 11-07-16 om 22:27 schreef Gustavo Padovan: > > 2016-07-10 Maarten Lankhorst : > > > >> Op 08-07-16 om 17:44 schreef Gustavo Padovan: > >>> From: Gustavo Padovan > >>> > >>> Signalling doesn't need to be enabled at sync_file cr

Re: [PATCH] dma-buf/sync_file: only enable fence signalling during wait

2016-07-12 Thread Gustavo Padovan
2016-07-12 Maarten Lankhorst : > Op 11-07-16 om 22:27 schreef Gustavo Padovan: > > 2016-07-10 Maarten Lankhorst : > > > >> Op 08-07-16 om 17:44 schreef Gustavo Padovan: > >>> From: Gustavo Padovan > >>> > >>> Signalling doesn't need to be enabled at sync_file creation, it is only > >>> required i

Re: [PATCH] dma-buf/sync_file: only enable fence signalling during wait

2016-07-12 Thread Maarten Lankhorst
Op 11-07-16 om 22:27 schreef Gustavo Padovan: > 2016-07-10 Maarten Lankhorst : > >> Op 08-07-16 om 17:44 schreef Gustavo Padovan: >>> From: Gustavo Padovan >>> >>> Signalling doesn't need to be enabled at sync_file creation, it is only >>> required if userspace waiting the fence to signal through

Re: [PATCH] dma-buf/sync_file: only enable fence signalling during wait

2016-07-11 Thread Gustavo Padovan
2016-07-10 Maarten Lankhorst : > Op 08-07-16 om 17:44 schreef Gustavo Padovan: > > From: Gustavo Padovan > > > > Signalling doesn't need to be enabled at sync_file creation, it is only > > required if userspace waiting the fence to signal through poll(). > > > > Thus we delay fence_add_callback()

Re: [PATCH] dma-buf/sync_file: only enable fence signalling during wait

2016-07-10 Thread Maarten Lankhorst
Op 08-07-16 om 17:44 schreef Gustavo Padovan: > From: Gustavo Padovan > > Signalling doesn't need to be enabled at sync_file creation, it is only > required if userspace waiting the fence to signal through poll(). > > Thus we delay fence_add_callback() until poll is called. It only adds the > call

[PATCH] dma-buf/sync_file: only enable fence signalling during wait

2016-07-08 Thread Gustavo Padovan
From: Gustavo Padovan Signalling doesn't need to be enabled at sync_file creation, it is only required if userspace waiting the fence to signal through poll(). Thus we delay fence_add_callback() until poll is called. It only adds the callback the first time poll() is called. This avoid re-adding