[PATCH] dma-buf/sync_file: fix build warning with context format type

Sumit Semwal sumit.semwal at linaro.org
Fri Jun 3 17:57:31 UTC 2016


Hi Gustavo,

On 03-Jun-2016 9:16 PM, "Gustavo Padovan" <gustavo at padovan.org> wrote:
>
> From: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
>
> Christian König changed fence context to a u64 type, so we need to
> update all users accordingly.
>
> Signed-off-by: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
Thanks for the patch,
Acked-by: Sumit Semwal <sumit.semwal at linaro.org>
> ---
>
> Should we get this via drm-misc to silent the warning added by the change
> on the fence context?
Yes, please! :)

>
>  drivers/dma-buf/sync_file.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/dma-buf/sync_file.c b/drivers/dma-buf/sync_file.c
> index f08cf2d..9aaa608 100644
> --- a/drivers/dma-buf/sync_file.c
> +++ b/drivers/dma-buf/sync_file.c
> @@ -82,7 +82,7 @@ struct sync_file *sync_file_create(struct fence *fence)
>
>         sync_file->num_fences = 1;
>         atomic_set(&sync_file->status, 1);
> -       snprintf(sync_file->name, sizeof(sync_file->name), "%s-%s%d-%d",
> +       snprintf(sync_file->name, sizeof(sync_file->name), "%s-%s%llu-%d",
>                  fence->ops->get_driver_name(fence),
>                  fence->ops->get_timeline_name(fence), fence->context,
>                  fence->seqno);
> --
> 2.5.5
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160603/6e82f721/attachment.html>


More information about the dri-devel mailing list