Groups > linux.kernel > #1319956

Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file

From	Greg Hackmann <ghackmann@google.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file
Date	2016-01-27 22:50 +0100
Message-ID	<qVG8q-1wI-3@gated-at.bofh.it> (permalink)
References	(1 earlier) <qVyue-49M-5@gated-at.bofh.it> <qVzqj-4RW-37@gated-at.bofh.it> <qVBLs-6NS-17@gated-at.bofh.it> <qVCRd-7CM-43@gated-at.bofh.it> <qVESZ-FR-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 01/27/2016 12:25 PM, Gustavo Padovan wrote:
>>>> Is there a value in keeping the abi unchanged?
>>>> If not, then Documentation/ioctl/botching-up-ioctls.txt is worth a read.
>>>
>>> None from me. I'll look where we can improve the ABI.

Android has existing clients of the current ABI.  Thankfully they're all 
contained in system services like SurfaceFlinger, since end-user apps 
don't get direct access to fence fds.

As long the ABI breaks don't remove functionality we depend on, we can 
wrap around them in our userspace libsync.  I'd rather not have to do 
that, but it's a price I'm willing to pay to get this moved out of staging.

>>   - struct sync_file_info_data::fence_info is of type __u8 yet it is "a
>> fence_info struct for every fence in the sync_file". Thus shouldn't
>> one use "struct fence_info" as the type ?
>
> Agreed. But I'm currently thinking if we really should keep this ioctl.
>
> 	Gustavo
>

I'm not seeing any consumers of driver_data in our tree.  OTOH 
completely getting rid of the ioctl would be a problem, since 
SurfaceFlinger depends on the timestamp information for its own bookkeeping.

Thread

[PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Gustavo Padovan <gustavo@padovan.org> - 2016-01-27 14:40 +0100
  Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Maarten Lankhorst <maarten.lankhorst@linux.intel.com> - 2016-01-27 15:40 +0100
    Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Gustavo Padovan <gustavo@padovan.org> - 2016-01-27 18:10 +0100
      Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Emil Velikov <emil.l.velikov@gmail.com> - 2016-01-27 19:20 +0100
        Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Gustavo Padovan <gustavo@padovan.org> - 2016-01-27 21:30 +0100
          Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Greg Hackmann <ghackmann@google.com> - 2016-01-27 22:50 +0100
            Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Daniel Vetter <daniel@ffwll.ch> - 2016-01-28 10:30 +0100
              Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file Greg Hackmann <ghackmann@google.com> - 2016-01-29 18:50 +0100

csiph-web