public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Devin Heitmueller <dheitmueller@kernellabs.com>,
	Hans Verkuil <hverkuil@xs4all.nl>, Jesper Juhl <jj@chaosbits.net>,
	LKML <linux-kernel@vger.kernel.org>,
	trivial@kernel.org, linux-media@vger.kernel.org,
	ceph-devel@vger.kernel.org, Sage Weil <sage@newdream.net>
Subject: Re: [RFC] Don't use linux/version.h anymore to indicate a per-driver version - Was: Re: [PATCH 03/37] Remove unneeded version.h includes from include/
Date: Fri, 24 Jun 2011 18:52:58 -0300	[thread overview]
Message-ID: <4E05073A.3030209@infradead.org> (raw)
In-Reply-To: <20110624231028.7f03dcae@stein>

Em 24-06-2011 18:10, Stefan Richter escreveu:
> On Jun 24 Devin Heitmueller wrote:
>> Really, this is all about applications being able to jam a hack into
>> their code that translates to "don't call this ioctl() with some
>> particular argument if it's driver W less than version X, because the
>> driver had a bug that is likely to panic the guy's PC".  Sure, it's a
>> crummy solution, but at this point it's the best that we have got.
> 
> The second best.  The best that we have got is that the user runs a fixed
> kernel.
> 
> Anyway; if this is the only purpose that this interface version¹ serves,
> then Mauro's subsystem-centralized solution has the benefit that it
> eliminates mistakes due to oversight by individual driver authors.
> Especially because the kind of implementation behavior changes that are
> tracked by this type of version datum are sometimes just discovered or
> documented in hindsight.  On the other hand, Mauro's solution is redundant
> to the uname(2) syscall.

Yes. That's why my initial proposal were to add some value to it by not associating
it with the kernel version, but with a number that will be incremented only if
the V4L2 API changes.

> 
> ¹) Yes, it is still an ABI version, nothing less.  With all its backwards
> and forwards compatibility ramifications.


  reply	other threads:[~2011-06-24 21:53 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <alpine.LNX.2.00.1106232344480.17688@swampdragon.chaosbits.net>
2011-06-23 21:58 ` [PATCH 03/37] Remove unneeded version.h includes from include/ Jesper Juhl
2011-06-23 22:15   ` Sage Weil
2011-06-24 11:21   ` [RFC] Don't use linux/version.h anymore to indicate a per-driver version - Was: " Mauro Carvalho Chehab
2011-06-24 11:26     ` Hans Verkuil
2011-06-24 12:20       ` Devin Heitmueller
2011-06-24 13:29         ` Mauro Carvalho Chehab
2011-06-24 13:45           ` Devin Heitmueller
2011-06-24 13:54             ` Hans Verkuil
2011-06-24 14:37               ` Mauro Carvalho Chehab
2011-06-24 18:25                 ` [PATCH] [media] Stop using linux/version.h on most drivers Mauro Carvalho Chehab
2011-06-25 10:09                   ` Hans Verkuil
2011-06-25 12:14                     ` Mauro Carvalho Chehab
2011-06-27  0:59                       ` Laurent Pinchart
2011-06-24 18:34                 ` [RFC] Don't use linux/version.h anymore to indicate a per-driver version - Was: Re: [PATCH 03/37] Remove unneeded version.h includes from include/ Stefan Richter
2011-06-24 18:48                   ` Devin Heitmueller
2011-06-24 21:04                     ` Andy Walls
2011-06-24 21:20                       ` Stefan Richter
2011-06-24 21:22                         ` Devin Heitmueller
2011-06-24 21:49                           ` Mauro Carvalho Chehab
2011-06-24 22:39                           ` Stefan Richter
2011-06-24 23:02                             ` Mauro Carvalho Chehab
2011-06-24 22:16                       ` Mauro Carvalho Chehab
2011-06-24 22:57                         ` Andy Walls
2011-06-24 21:10                     ` Stefan Richter
2011-06-24 21:52                       ` Mauro Carvalho Chehab [this message]
2011-06-24 21:44                   ` Mauro Carvalho Chehab
2011-06-29 21:54         ` Mauro Carvalho Chehab
2011-06-23 22:11 ` [PATCH 10/37] Remove unneeded version.h includes from drivers/media/dvb/ Jesper Juhl
2011-06-23 22:14 ` [PATCH 11/37] Remove unneeded version.h includes (and add where needed) for drivers/media/radio/ Jesper Juhl
2011-06-23 22:17 ` [PATCH 12/37] Remove unneeded version.h includes (and add where needed) for drivers/media/video/ Jesper Juhl
2011-06-24  8:52   ` Laurent Pinchart

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=4E05073A.3030209@infradead.org \
    --to=mchehab@infradead.org \
    --cc=ceph-devel@vger.kernel.org \
    --cc=dheitmueller@kernellabs.com \
    --cc=hverkuil@xs4all.nl \
    --cc=jj@chaosbits.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=sage@newdream.net \
    --cc=stefanr@s5r6.in-berlin.de \
    --cc=trivial@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox