Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1404666

Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4

Show all headers | View raw

Am 20.05.2016 um 16:47 schrieb Gustavo Padovan:
> 2016-05-20 Christian König <deathsimple@vodafone.de>:
>
>> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
>>
>> struct fence_collection inherits from struct fence and carries a
>> collection of fences that needs to be waited together.
>>
>> It is useful to translate a sync_file to a fence to remove the complexity
>> of dealing with sync_files on DRM drivers. So even if there are many
>> fences in the sync_file that needs to waited for a commit to happen,
>> they all get added to the fence_collection and passed for DRM use as
>> a standard struct fence.
>>
>> That means that no changes needed to any driver besides supporting fences.
>>
>> fence_collection's fence doesn't belong to any timeline context, so
>> fence_is_later() and fence_later() are not meant to be called with
>> fence_collections fences.
>>
>> v2: Comments by Daniel Vetter:
>> 	- merge fence_collection_init() and fence_collection_add()
>> 	- only add callbacks at ->enable_signalling()
>> 	- remove fence_collection_put()
>> 	- check for type on to_fence_collection()
>> 	- adjust fence_is_later() and fence_later() to WARN_ON() if they
>> 	are used with collection fences.
>>
>> v3: - Initialize fence_cb.node at fence init.
>>
>>      Comments by Chris Wilson:
>> 	- return "unbound" on fence_collection_get_timeline_name()
>> 	- don't stop adding callbacks if one fails
>> 	- remove redundant !! on fence_collection_enable_signaling()
>> 	- remove redundant () on fence_collection_signaled
>> 	- use fence_default_wait() instead
>>
>> v4 (chk): Rework, simplification and cleanup:
>> 	- Drop FENCE_NO_CONTEXT handling, always allocate a context.
>> 	- Rename to fence_array.
>> 	- Return fixed driver name.
>> 	- Register only one callback at a time.
>> 	- Document that create function takes ownership of array.
> This looks good to me. Dropping NO_CONTEXT was a good idea, also
> registering only one callback makes it looks better.

Thinking about it a bit more I think we need to avoid removing the 
callback when the fence is released as well.

That stuff is just a bit to racy (see the comment on the 
fence_remove_callback function as well).

I will just grab a reference to the fence while there is any callback 
registered.

Also please note that this is only compile tested at the moment. I'm 
still working on integrating it into my code.

Regards,
Christian.

>
> 	Gustavo

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] dma-buf/fence: make fence context 64 bit v2 Christian König <deathsimple@vodafone.de> - 2016-05-20 16:10 +0200
  [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Christian König <deathsimple@vodafone.de> - 2016-05-20 16:10 +0200
    Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Gustavo Padovan <gustavo@padovan.org> - 2016-05-20 16:50 +0200
      Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Christian König <deathsimple@vodafone.de> - 2016-05-20 20:00 +0200
      Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Daniel Vetter <daniel@ffwll.ch> - 2016-05-23 09:50 +0200
        Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Christian König <deathsimple@vodafone.de> - 2016-05-23 13:30 +0200
          Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Daniel Vetter <daniel@ffwll.ch> - 2016-05-23 16:10 +0200
    Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Chris Wilson <chris@chris-wilson.co.uk> - 2016-05-20 16:50 +0200
      Re: [PATCH 2/2] dma-buf/fence: add fence_array fences v4 Christian König <deathsimple@vodafone.de> - 2016-05-23 09:40 +0200

csiph-web