lkml.org 
[lkml]   [2008]   [Jan]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 1/3] ide: use MODULE_VERSION()
Date
On Tuesday 01 January 2008, Adrian Bunk wrote:
> On Tue, Jan 01, 2008 at 06:40:38PM +0100, Bartlomiej Zolnierkiewicz wrote:
> >
> > * Replace driver versions in comments by DRV_VERSION define.
> >
> > * Add MODULE_VERSION(DRV_VERSION) to drivers defining DRV_VERSION.
> >...
>
> Could you limit this to actively maintained version numbers and also
> remove the others from .version, printk's,...?
>
> An example:
>
> > --- a/drivers/ide/legacy/umc8672.c
> > +++ b/drivers/ide/legacy/umc8672.c
> > @@ -1,5 +1,5 @@
> > /*
> > - * linux/drivers/ide/legacy/umc8672.c Version 0.05 Jul 31, 1996
> > + * linux/drivers/ide/legacy/umc8672.c
> > *
> > * Copyright (C) 1995-1996 Linus Torvalds & author (see below)
> > */
> > @@ -53,6 +53,8 @@
> >
> > #include <asm/io.h>
> >
> > +#define DRV_VERSION "0.05"
> > +
> > /*
> > * Default speeds. These can be changed with "auto-tune" and/or hdparm.
> > */
> > @@ -185,3 +187,4 @@ module_init(umc8672_init);
> > MODULE_AUTHOR("Wolfram Podien");
> > MODULE_DESCRIPTION("Support for UMC 8672 IDE chipset");
> > MODULE_LICENSE("GPL");
> > +MODULE_VERSION(DRV_VERSION);
> >...
>
> I do not see any value in printing a version number from 1996 (sic),
> and the driver has for sure changed without any update of the
> version number.
>
> For actively maintained version numbers (and even more when
> distributions backport more recent versions of a driver) printing a
> (maintained) driver version number makes sense, but otherwise the kernel
> version contains more information and a stale version number only gives
> the false impression the driver hadn't changed.

Good point.

On the second thought: maybe we will be better off with limiting
MODULE_VERSION() to the device drivers and the IDE core module for now,
and just removing all these private version numbers from host drivers
(with one or two exceptions they are not printed or exported currently,
moreover exceptions are the cases like stale version numbers from 199x)?

Bart


\
 
 \ /
  Last update: 2008-01-01 19:25    [W:0.183 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site