Re: [PATCH 1/7] sync_file: get rid of internal reference count.

2017-04-18 Thread Gustavo Padovan 
2017-04-17 Sumit Semwal :

> On 17 April 2017 at 18:43, Gustavo Padovan  wrote:
> > 2017-04-13 Dave Airlie :
> >
> >> From: Dave Airlie 
> >>
> >> sync_file uses the reference count of the file, the internal
> >> kref was never getting moved past 1.
> >>
> >> We can reintroduce this if we decide we need it later.
> >>
> >> [airlied: fix buildbot warnings]
> >>
> >> Reviewed-by: Chris Wilson 
> >> Signed-off-by: Dave Airlie 
> >> ---
> >>  drivers/dma-buf/sync_file.c | 13 ++---
> >>  include/linux/sync_file.h   |  3 ---
> >>  2 files changed, 2 insertions(+), 14 deletions(-)
> >
> > Reviewed-by: Gustavo Padovan 
> Acked-by: Sumit Semwal 
> >
> > Or should I just push this one?
> Of course, go right ahead, Gustavo :)

pushed to drm-misc-next.

Gustavo
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 1/7] sync_file: get rid of internal reference count.

2017-04-17 Thread Sumit Semwal 
On 17 April 2017 at 18:43, Gustavo Padovan  wrote:
> 2017-04-13 Dave Airlie :
>
>> From: Dave Airlie 
>>
>> sync_file uses the reference count of the file, the internal
>> kref was never getting moved past 1.
>>
>> We can reintroduce this if we decide we need it later.
>>
>> [airlied: fix buildbot warnings]
>>
>> Reviewed-by: Chris Wilson 
>> Signed-off-by: Dave Airlie 
>> ---
>>  drivers/dma-buf/sync_file.c | 13 ++---
>>  include/linux/sync_file.h   |  3 ---
>>  2 files changed, 2 insertions(+), 14 deletions(-)
>
> Reviewed-by: Gustavo Padovan 
Acked-by: Sumit Semwal 
>
> Or should I just push this one?
Of course, go right ahead, Gustavo :)
>
> Gustavo
> ___
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 1/7] sync_file: get rid of internal reference count.

2017-04-17 Thread Gustavo Padovan 
2017-04-13 Dave Airlie :

> From: Dave Airlie 
> 
> sync_file uses the reference count of the file, the internal
> kref was never getting moved past 1.
> 
> We can reintroduce this if we decide we need it later.
> 
> [airlied: fix buildbot warnings]
> 
> Reviewed-by: Chris Wilson 
> Signed-off-by: Dave Airlie 
> ---
>  drivers/dma-buf/sync_file.c | 13 ++---
>  include/linux/sync_file.h   |  3 ---
>  2 files changed, 2 insertions(+), 14 deletions(-)

Reviewed-by: Gustavo Padovan 

Or should I just push this one?

Gustavo
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel