From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S967771AbcA0U0U (ORCPT <rfc822;w@1wt.eu>);
	Wed, 27 Jan 2016 15:26:20 -0500
Received: from mail-yk0-f174.google.com ([209.85.160.174]:35814 "EHLO
	mail-yk0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S967714AbcA0UZp (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 27 Jan 2016 15:25:45 -0500
Date: Wed, 27 Jan 2016 18:25:40 -0200
From: Gustavo Padovan <gustavo@padovan.org>
To: Emil Velikov <emil.l.velikov@gmail.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>,
        devel@driverdev.osuosl.org,
        ML dri-devel <dri-devel@lists.freedesktop.org>,
        Daniel Stone <daniels@collabora.com>,
        Arve =?iso-8859-1?B?SGr4bm5lduVn?= <arve@android.com>,
        Riley Andrews <riandrews@android.com>,
        Daniel Vetter <daniel.vetter@ffwll.ch>,
        Rob Clark <robdclark@gmail.com>, Greg Hackmann <ghackmann@google.com>,
        John Harrison <John.C.Harrison@intel.com>,
        Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Subject: Re: [PATCH v2 01/11] dma-buf/sync_file: de-stage sync_file
Message-ID: <20160127202540.GD3773@joana>
Mail-Followup-To: Gustavo Padovan <gustavo@padovan.org>,
	Emil Velikov <emil.l.velikov@gmail.com>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>,
	devel@driverdev.osuosl.org,
	ML dri-devel <dri-devel@lists.freedesktop.org>,
	Daniel Stone <daniels@collabora.com>,
	Arve =?iso-8859-1?B?SGr4bm5lduVn?= <arve@android.com>,
	Riley Andrews <riandrews@android.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	Rob Clark <robdclark@gmail.com>,
	Greg Hackmann <ghackmann@google.com>,
	John Harrison <John.C.Harrison@intel.com>,
	Gustavo Padovan <gustavo.padovan@collabora.co.uk>
References: <1453901439-19467-1-git-send-email-gustavo@padovan.org>
 <1453901439-19467-2-git-send-email-gustavo@padovan.org>
 <56A8D4A7.1070409@linux.intel.com>
 <20160127170313.GC3773@joana>
 <CACvgo52bp9FQO50REHMzqaqXnwdkTx2yjx_xiet2V_wQscuu1Q@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CACvgo52bp9FQO50REHMzqaqXnwdkTx2yjx_xiet2V_wQscuu1Q@mail.gmail.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Emil,

2016-01-27 Emil Velikov <emil.l.velikov@gmail.com>:

> Hi Gustavo,
> 
> On 27 January 2016 at 17:03, Gustavo Padovan <gustavo@padovan.org> wrote:
> > Hi Maarten,
> >
> > 2016-01-27 Maarten Lankhorst <maarten.lankhorst@linux.intel.com>:
> >
> >> Hey,
> >>
> >> Op 27-01-16 om 14:30 schreef Gustavo Padovan:
> >> > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> >> >
> >> > sync_file is useful to connect one or more fences to the file. The file is
> >> > used by userspace to track fences.
> >> >
> >> > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> >> >
> >> 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.
> >
> Speaking of ABI... there are a couple of things that rang bells here:
>  - In most/all of the kernel a len/size named member variable
> indicates the length of the extra data (zero sized array). While here
> it includes the size of the struct as well.

len in this case is the size of the buffer sent to the kernel,
the total length (including the whole struct) is returned in the ioctl.

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