From: Patrick Mansfield <patmans@us.ibm.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: linux-scsi@vger.kernel.org,
James Bottomley <James.Bottomley@steeleye.com>
Subject: Re: [PATCH] convert scsi core to use module_param interfaces
Date: Fri, 6 Jun 2003 08:17:01 -0700 [thread overview]
Message-ID: <20030606081701.A30221@beaverton.ibm.com> (raw)
In-Reply-To: <20030606073732.B13259@infradead.org>; from hch@infradead.org on Fri, Jun 06, 2003 at 07:37:32AM +0100
On Fri, Jun 06, 2003 at 07:37:32AM +0100, Christoph Hellwig wrote:
> I don't think we can just change the name of the scsi core module -
> to much userspace stuff is relying on that. Changing the prefix would
> be nice but I have no idea how we can do that.
Is it that bad?
Do all distros that build with scsi build it into the kernel by default?
The only option (ignoring logging) we have in 2.4 is max_scsi_luns,
changing its name from max_scsi_luns to max_luns means users already have
to modify their modules.conf. For boot command line, users have to change
to scsi[_mod].whatever no matter what.
If not changing the module name:
Rusty had suggested undef/define KBUILD_MODNAME, that might not be very
nice for use across files, and/or we could get a patch into moduleparam.h
that gives a cleaner define name.
But it is cleaner to have the modprobe name option look like the boot
command line, i.e.:
modprobe scsi max_luns=1
scsi.max_luns=1
> We should also kill the scsi substrings in the individual options now
> that we have a regular prefix, I'll submit a patch.
The patch looks good.
-- Patrick Mansfield
prev parent reply other threads:[~2003-06-06 15:08 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-06-04 21:57 [PATCH] convert scsi core to use module_param interfaces Patrick Mansfield
2003-06-06 6:37 ` Christoph Hellwig
2003-06-06 15:17 ` Patrick Mansfield [this message]
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=20030606081701.A30221@beaverton.ibm.com \
--to=patmans@us.ibm.com \
--cc=James.Bottomley@steeleye.com \
--cc=hch@infradead.org \
--cc=linux-scsi@vger.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