linux-kernel - Re: [2.6 patch] drivers/scsi/nsp32.c: remove kernel 2.4 code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <wtw7irya1dx.wl%gotom@sanori.org>
Date:	Fri, 27 Apr 2007 13:26:34 +0900
From:	GOTO Masanori <gotom@...ori.org>
To:	James Bottomley <James.Bottomley@...elEye.com>
Cc:	Adrian Bunk <bunk@...sta.de>,
	"Robert P. J. Day" <rpjday@...dspring.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
	Yokota Hiroshi <yokota@...lab.cs.tsukuba.ac.jp>,
	GOTO Masanori <gotom@...ori.org>
Subject: Re: [2.6 patch] drivers/scsi/nsp32.c: remove kernel 2.4 code

At Thu, 26 Apr 2007 20:30:54 -0400,
James Bottomley wrote:
> On Fri, 2007-04-27 at 02:13 +0200, Adrian Bunk wrote:
> > On Thu, Apr 26, 2007 at 07:59:57PM -0400, Robert P. J. Day wrote:
> > > On Fri, 27 Apr 2007, Adrian Bunk wrote:
> > > 
> > > > This patch removes kernel 2.4 code.
> > > >
> > > > Signed-off-by: Adrian Bunk <bunk@...sta.de>
> > > >
> > > > ---
> > > >
> > > > This patch has been sent on:
> > > > - 26 Mar 2007
> > > >
> > > >  drivers/scsi/nsp32.c |  109 +++++--------------------------------------
> > > >  1 file changed, 13 insertions(+), 96 deletions(-)
> > > >
> > > > --- linux-2.6.21-rc4-mm1/drivers/scsi/nsp32.c.old	2007-03-25 20:27:34.000000000 +0200
> > > > +++ linux-2.6.21-rc4-mm1/drivers/scsi/nsp32.c	2007-03-25 20:31:59.000000000 +0200
> > > > @@ -49,10 +49,6 @@
> > > >  #include <scsi/scsi_host.h>
> > > >  #include <scsi/scsi_ioctl.h>
> > > >
> > > > -#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,6,0))
> > > > -# include <linux/blk.h>
> > > > -#endif
> > > > -
> > > 
> > > i'm curious about the rules for removing code like this.  in the case
> > > of drivers, isn't it possible that some driver source could be
> > > relevant for both the 2.4 and 2.6 kernel source tree, and simply uses
> > > that kind of preprocessor check to make sure it's being compiled
> > > appropriately?
> > 
> > That's what it was for.
> > 
> > > or are you doing something more sophisticated than simply checking the
> > > kernel version being tested?
> > 
> > No.
> > 
> > The point is:
> > 
> > It seems this driver was once maintained for both 2.4 and 2.6 in one 
> > file.
> > 
> > As long as this is done, such version checks are OK.
> 
> Personally, I don't like to see 2.4 and 2.6 in a new driver, and will
> tend to try to force it to be 2.6  only.  For an existing driver, I tend
> to be much more tolerant: removing the huge gobs of code to achieve 2.6
> only is usually a bit disruptive on both the driver and the maintainer
> 
> > But if a driver is no longer actually maintained for both kernels these 
> > checks become useless (and there quickly arised unconditional 2.6-only 
> > code in such a driver) and can be removed.
> 
> This driver is maintained by 
> 
> Yokota Hiroshi <yokota@...lab.cs.tsukuba.ac.jp>
> GOTO Masanori <gotom@...ori.org>
> 
> As it says in the header.  It was last modified in May 2006, so it is
> maintained under the somewhat elastic standards of SCSI.  I've cc'd them
> to see what they think.

Thanks forwarding the mail.

Personally I don't mind to remove 2.4 compatibility code.

Historically nsp32 was introduced in 2.4.20 and 2.5.40 so that we
needed to maintain both kernel serieses at that time.  However, these
days 2.4 kernel is not the mainline code, and 2.6 is the standard one.
We don't need to keep paying attention for 2.4 compatibility issue.
Moreover, some other compatibility code were already removed from 2.6
nsp32 driver by many cleaning up patches (thanks!).  This removal
makes sense to me.

Regards,
-- gotom
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/