From: Christoph Hellwig <hch@infradead.org>
To: Linus Torvalds <torvalds@transmeta.com>, pam.delaney@lsil.com
Cc: Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Linux 2.5.63
Date: Mon, 24 Feb 2003 20:10:30 +0000 [thread overview]
Message-ID: <20030224201030.A13503@infradead.org> (raw)
In-Reply-To: <Pine.LNX.4.44.0302241127050.13335-100000@penguin.transmeta.com>; from torvalds@transmeta.com on Mon, Feb 24, 2003 at 11:32:07AM -0800
> <pam.delaney@lsil.com>:
> o Fusion Driver 2.05.00.03 against 2.5.62bk3
This update is broken and strange in many ways, it would have been
nice if you actually sent this to some list for review before submitting..
(1) there's stuff like:
#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,5,58)
#define mpt_inc_use_count()
#define mpt_dec_use_count()
#else
#define mpt_inc_use_count() MOD_INC_USE_COUNT
#define mpt_dec_use_count() MOD_DEC_USE_COUNT
#endif
but even if the old-style refcounting is deprecated now you can't
just simply remove it! you need to rearchitecture your code to
work with try_module_get/module_put
(2) and like this:
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,59)
MODULE_PARM(PortIo, "0-1i");
MODULE_PARM_DESC(PortIo, "[0]=Use mmap, 1=Use port io");
+#endif
Again. just because old-style module paramters are deprecated you can't
just remove them without replacement, use module_param() instead.
(3) you remove backward copatiblity code in one place but add lots more
in other places. this doesn't make sense - either you have a nice or
all releases driver, but not one that doesn't work everywhere _and_ looks
horrible.
next prev parent reply other threads:[~2003-02-24 20:00 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-02-24 19:32 Linux 2.5.63 Linus Torvalds
2003-02-24 20:10 ` Christoph Hellwig [this message]
2003-02-24 20:30 ` Christoph Hellwig
2003-02-26 2:08 ` george anzinger
2003-02-24 22:11 ` Linux 2.5.63 [kernel BUG at drivers/net/tulip/de2104x.c:925!] Remco Post
2003-02-24 22:32 ` Linux 2.5.63 John Cherry
2003-02-25 13:15 ` 2.5.63: fore200e.c doesn't compile Adrian Bunk
2003-02-25 14:45 ` chas williams
[not found] ` <200302251453.h1PErvGi030679@locutus.cmf.nrl.navy.mil>
2003-02-25 15:44 ` Adrian Bunk
2003-02-25 14:09 ` Linux 2.5.63 - nfs mount fails Helge Hafting
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=20030224201030.A13503@infradead.org \
--to=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pam.delaney@lsil.com \
--cc=torvalds@transmeta.com \
/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