All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
To: Greg KH <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
Cc: Mauro Carvalho Chehab
	<mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>,
	Linux Media Mailing List
	<linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Mauro Carvalho Chehab
	<mchehab-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
	linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Hans Verkuil <hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>,
	Laurent Pinchart
	<laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>,
	Javier Martinez Canillas
	<javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>,
	Sakari Ailus <sakari.ailus-X3B1VOXEql0@public.gmane.org>,
	Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
Subject: Re: [PATCH RFC] [media] Postpone the addition of MEDIA_IOC_G_TOPOLOGY
Date: Mon, 28 Dec 2015 09:05:34 -0700	[thread overview]
Message-ID: <56815DCE.6040304@osg.samsung.com> (raw)
In-Reply-To: <20151228154429.GA27560-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>

On 12/28/2015 08:44 AM, Greg KH wrote:
> On Mon, Dec 28, 2015 at 08:37:19AM -0700, Shuah Khan wrote:
>> On 12/28/2015 07:03 AM, Mauro Carvalho Chehab wrote:
>>> There are a few discussions left with regards to this ioctl:
>>>
>>> 1) the name of the new structs will contain _v2_ on it?
>>> 2) what's the best alternative to avoid compat32 issues?
>>>
>>> Due to that, let's postpone the addition of this new ioctl to
>>> the next Kernel version, to give people more time to discuss it.
>>
>> I thought we discussed this in our irc meeting and
>> arrived at a good solution for compat32 issue
>>
>> My recommendation is getting this ioctl into 4.5 with
>> a warning that it could change. The reason for that is
>> that this ioctl helps with testing the media controller
>> v2 api. Without this API, we won't see much testing from
>> userspace in 4.5
> 
> People will ignore the warning, that never works :(
> 

Yeah. If people do ignore warnings, it could become a
problem if we have to make changes.

-- Shuah


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org | (970) 217-8978

WARNING: multiple messages have this Message-ID (diff)
From: Shuah Khan <shuahkh@osg.samsung.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
	Linux Media Mailing List <linux-media@vger.kernel.org>,
	Mauro Carvalho Chehab <mchehab@infradead.org>,
	linux-api@vger.kernel.org, Hans Verkuil <hverkuil@xs4all.nl>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Javier Martinez Canillas <javier@osg.samsung.com>,
	Sakari Ailus <sakari.ailus@iki.fi>,
	Shuah Khan <shuahkh@osg.samsung.com>
Subject: Re: [PATCH RFC] [media] Postpone the addition of MEDIA_IOC_G_TOPOLOGY
Date: Mon, 28 Dec 2015 09:05:34 -0700	[thread overview]
Message-ID: <56815DCE.6040304@osg.samsung.com> (raw)
In-Reply-To: <20151228154429.GA27560@kroah.com>

On 12/28/2015 08:44 AM, Greg KH wrote:
> On Mon, Dec 28, 2015 at 08:37:19AM -0700, Shuah Khan wrote:
>> On 12/28/2015 07:03 AM, Mauro Carvalho Chehab wrote:
>>> There are a few discussions left with regards to this ioctl:
>>>
>>> 1) the name of the new structs will contain _v2_ on it?
>>> 2) what's the best alternative to avoid compat32 issues?
>>>
>>> Due to that, let's postpone the addition of this new ioctl to
>>> the next Kernel version, to give people more time to discuss it.
>>
>> I thought we discussed this in our irc meeting and
>> arrived at a good solution for compat32 issue
>>
>> My recommendation is getting this ioctl into 4.5 with
>> a warning that it could change. The reason for that is
>> that this ioctl helps with testing the media controller
>> v2 api. Without this API, we won't see much testing from
>> userspace in 4.5
> 
> People will ignore the warning, that never works :(
> 

Yeah. If people do ignore warnings, it could become a
problem if we have to make changes.

-- Shuah


-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978

  parent reply	other threads:[~2015-12-28 16:05 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-28 14:03 [PATCH RFC] [media] Postpone the addition of MEDIA_IOC_G_TOPOLOGY Mauro Carvalho Chehab
2015-12-28 14:03 ` Mauro Carvalho Chehab
     [not found] ` <d029047c76d6d3e5e6a531080ede83f6e063f7db.1451311244.git.mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-12-28 14:15   ` Javier Martinez Canillas
2015-12-28 14:15     ` Javier Martinez Canillas
2015-12-28 15:37 ` Shuah Khan
     [not found]   ` <5681572F.601-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-12-28 15:44     ` Greg KH
2015-12-28 15:44       ` Greg KH
     [not found]       ` <20151228154429.GA27560-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2015-12-28 16:05         ` Shuah Khan [this message]
2015-12-28 16:05           ` Shuah Khan
  -- strict thread matches above, loose matches on Subject: below --
2015-12-28 13:57 Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56815DCE.6040304@osg.samsung.com \
    --to=shuahkh-jph+aebz4p+uejcrhfaqsw@public.gmane.org \
    --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
    --cc=hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org \
    --cc=javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org \
    --cc=laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
    --cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org \
    --cc=mchehab-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
    --cc=sakari.ailus-X3B1VOXEql0@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.