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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 03, 2016 at 11:27:31PM +0530, Sumit Semwal wrote:
> Hi Gustavo,
> 
> On 03-Jun-2016 9:16 PM, "Gustavo Padovan" <gustavo@xxxxxxxxxxx> wrote:
> >
> > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
> >
> > 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@xxxxxxxxxxxxxxx>
> Thanks for the patch,
> Acked-by: Sumit Semwal <sumit.semwal@xxxxxxxxxx>
> > ---
> >
> > Should we get this via drm-misc to silent the warning added by the change
> > on the fence context?
> Yes, please! :)

Done, thanks.
-Daniel

> 
> >
> >  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
> >

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux