* [PATCH] dma-buf/sync_file: fix documentation error
@ 2016-09-19 4:21 Emilio López
2016-09-19 12:10 ` Gustavo Padovan
0 siblings, 1 reply; 3+ messages in thread
From: Emilio López @ 2016-09-19 4:21 UTC (permalink / raw)
To: sumit.semwal, gustavo.padovan; +Cc: dri-devel, linux-kernel, Emilio López
The ioctl name and description on the documentation block don't
match the ioctl being defined. This was probably overlooked while
renaming the ioctls during the sync file destaging. This patch
provides a more accurate description of what the ioctl actually does.
Signed-off-by: Emilio López <emilio.lopez@collabora.co.uk>
---
This is something I saw while refreshing my kselftest patches. Hopefully
this patch describes the new ioctl well enough, let me know if you
think it doesn't :)
Cheers,
Emilio
include/uapi/linux/sync_file.h | 13 +++++--------
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/include/uapi/linux/sync_file.h b/include/uapi/linux/sync_file.h
index 413303d..cdf8ec2 100644
--- a/include/uapi/linux/sync_file.h
+++ b/include/uapi/linux/sync_file.h
@@ -85,15 +85,12 @@ struct sync_file_info {
#define SYNC_IOC_MERGE _IOWR(SYNC_IOC_MAGIC, 3, struct sync_merge_data)
/**
- * DOC: SYNC_IOC_FENCE_INFO - get detailed information on a fence
+ * DOC: SYNC_IOC_FILE_INFO - get detailed information on a sync_file
*
- * Takes a struct sync_file_info_data with extra space allocated for pt_info.
- * Caller should write the size of the buffer into len. On return, len is
- * updated to reflect the total size of the sync_file_info_data including
- * pt_info.
- *
- * pt_info is a buffer containing sync_pt_infos for every sync_pt in the fence.
- * To iterate over the sync_pt_infos, use the sync_pt_info.len field.
+ * Takes a struct sync_file_info. If num_fences is 0, the field is updated
+ * with the actual number of fences. If num_fences is > 0, the system will
+ * use the pointer provided on sync_fence_info to return up to num_fences of
+ * struct sync_fence_info, with detailed fence information.
*/
#define SYNC_IOC_FILE_INFO _IOWR(SYNC_IOC_MAGIC, 4, struct sync_file_info)
--
2.9.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dma-buf/sync_file: fix documentation error
2016-09-19 4:21 [PATCH] dma-buf/sync_file: fix documentation error Emilio López
@ 2016-09-19 12:10 ` Gustavo Padovan
2016-09-20 12:57 ` Sumit Semwal
0 siblings, 1 reply; 3+ messages in thread
From: Gustavo Padovan @ 2016-09-19 12:10 UTC (permalink / raw)
To: Emilio López; +Cc: sumit.semwal, dri-devel, linux-kernel
Hi Emilio,
2016-09-19 Emilio López <emilio.lopez@collabora.co.uk>:
> The ioctl name and description on the documentation block don't
> match the ioctl being defined. This was probably overlooked while
> renaming the ioctls during the sync file destaging. This patch
> provides a more accurate description of what the ioctl actually does.
>
> Signed-off-by: Emilio López <emilio.lopez@collabora.co.uk>
> ---
>
> This is something I saw while refreshing my kselftest patches. Hopefully
> this patch describes the new ioctl well enough, let me know if you
> think it doesn't :)
>
> Cheers,
> Emilio
>
> include/uapi/linux/sync_file.h | 13 +++++--------
> 1 file changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/include/uapi/linux/sync_file.h b/include/uapi/linux/sync_file.h
> index 413303d..cdf8ec2 100644
> --- a/include/uapi/linux/sync_file.h
> +++ b/include/uapi/linux/sync_file.h
> @@ -85,15 +85,12 @@ struct sync_file_info {
> #define SYNC_IOC_MERGE _IOWR(SYNC_IOC_MAGIC, 3, struct sync_merge_data)
>
> /**
> - * DOC: SYNC_IOC_FENCE_INFO - get detailed information on a fence
> + * DOC: SYNC_IOC_FILE_INFO - get detailed information on a sync_file
> *
> - * Takes a struct sync_file_info_data with extra space allocated for pt_info.
> - * Caller should write the size of the buffer into len. On return, len is
> - * updated to reflect the total size of the sync_file_info_data including
> - * pt_info.
> - *
> - * pt_info is a buffer containing sync_pt_infos for every sync_pt in the fence.
> - * To iterate over the sync_pt_infos, use the sync_pt_info.len field.
> + * Takes a struct sync_file_info. If num_fences is 0, the field is updated
> + * with the actual number of fences. If num_fences is > 0, the system will
> + * use the pointer provided on sync_fence_info to return up to num_fences of
> + * struct sync_fence_info, with detailed fence information.
> */
Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Gustavo
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dma-buf/sync_file: fix documentation error
2016-09-19 12:10 ` Gustavo Padovan
@ 2016-09-20 12:57 ` Sumit Semwal
0 siblings, 0 replies; 3+ messages in thread
From: Sumit Semwal @ 2016-09-20 12:57 UTC (permalink / raw)
To: Gustavo Padovan; +Cc: Emilio López, DRI mailing list, LKML
Hi Emilio,
On 19 September 2016 at 17:40, Gustavo Padovan
<gustavo.padovan@collabora.com> wrote:
> Hi Emilio,
>
> 2016-09-19 Emilio López <emilio.lopez@collabora.co.uk>:
>
>> The ioctl name and description on the documentation block don't
>> match the ioctl being defined. This was probably overlooked while
>> renaming the ioctls during the sync file destaging. This patch
>> provides a more accurate description of what the ioctl actually does.
>>
>> Signed-off-by: Emilio López <emilio.lopez@collabora.co.uk>
>> ---
>
> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
>
Thanks; applied to drm-misc.
> Gustavo
>
Best,
Sumit.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-20 12:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-19 4:21 [PATCH] dma-buf/sync_file: fix documentation error Emilio López
2016-09-19 12:10 ` Gustavo Padovan
2016-09-20 12:57 ` Sumit Semwal
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.