Re: [PATCH v5 4/5] Documentation: add doc for sync_file_get_fence()

From	Chris Wilson <chris@chris-wilson.co.uk>
Newsgroups	linux.kernel
Subject	Re: [PATCH v5 4/5] Documentation: add doc for sync_file_get_fence()
Date	2016-08-05 09:30 +0200
Message-ID	<s2HJT-4rz-5@gated-at.bofh.it> (permalink)
References	<s2D3z-1uF-1@gated-at.bofh.it> <s2D3z-1uF-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Aug 04, 2016 at 11:24:13PM -0300, Gustavo Padovan wrote:
> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> 
> Document the new function added to sync_file.c
> 
> v2: Adapt to fence_array
> 
> Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> Acked-by: Christian König <christian.koenig@amd.com>
> ---
>  Documentation/sync_file.txt | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/Documentation/sync_file.txt b/Documentation/sync_file.txt
> index e8e2eba..ae2dbad1 100644
> --- a/Documentation/sync_file.txt
> +++ b/Documentation/sync_file.txt
> @@ -64,6 +64,21 @@ The sync_file fd now can be sent to userspace.
>  If the creation process fail, or the sync_file needs to be released by any
>  other reason fput(sync_file->file) should be used.
>  
> +Receiving Sync Files from Userspace
> +-----------------------------------
> +
> +When userspace needs to send an in-fence to the driver it pass file descriptor

s/pass/passes/

> +of the Sync File to the kernel. The kernel then can retrieve the fences

/then can/can then/

> +from it.
> +
> +Interface:
> +	struct fence *sync_file_get_fence(int fd);
> +
> +
> +The function return a struct fence pointer referencing the fence(s) in the Sync
> +File.

+ The returned reference is owned by the caller and must be disposed of
afterwards using fence_put(). In case of error, a NULL is returned
instead.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre

Thread

[PATCH v5 1/5] dma-buf/fence-array: add fence_is_array() Gustavo Padovan <gustavo@padovan.org> - 2016-08-05 04:30 +0200
  [PATCH v5 4/5] Documentation: add doc for sync_file_get_fence() Gustavo Padovan <gustavo@padovan.org> - 2016-08-05 04:30 +0200
    Re: [PATCH v5 4/5] Documentation: add doc for sync_file_get_fence() Chris Wilson <chris@chris-wilson.co.uk> - 2016-08-05 09:30 +0200
  [PATCH v5 5/5] dma-buf/sync_file: only enable fence signalling on poll() Gustavo Padovan <gustavo@padovan.org> - 2016-08-05 04:30 +0200
    Re: [PATCH v5 5/5] dma-buf/sync_file: only enable fence signalling  on poll() Chris Wilson <chris@chris-wilson.co.uk> - 2016-08-05 09:30 +0200
      Re: [PATCH v5 5/5] dma-buf/sync_file: only enable fence signalling  on poll() Chris Wilson <chris@chris-wilson.co.uk> - 2016-08-05 09:50 +0200
  [PATCH v5 2/5] dma-buf/sync_file: refactor fence storage in struct sync_file Gustavo Padovan <gustavo@padovan.org> - 2016-08-05 04:30 +0200
  [PATCH v5 3/5] dma-buf/sync_file: add sync_file_get_fence() Gustavo Padovan <gustavo@padovan.org> - 2016-08-05 04:40 +0200

csiph-web