public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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.



  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