From: Jeff Garzik <jgarzik@mandrakesoft.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Matt_Domsch@Dell.com, greg@kroah.com, linux-kernel@vger.kernel.org
Subject: Re: Removal of pci_find_* in 2.5
Date: Sat, 13 Jul 2002 11:37:36 -0400 [thread overview]
Message-ID: <3D304940.7020207@mandrakesoft.com> (raw)
In-Reply-To: 1026570939.9958.92.camel@irongate.swansea.linux.org.uk
Alan Cox wrote:
> On Sat, 2002-07-13 at 05:41, Jeff Garzik wrote:
>
>>ordering, is simply hard-coding something that should really be in
>>userspace. Depending on pci_find_device logic / link order to
>>still-boot-the-system after adding new hardware sounds like an
>>incredibly fragile hope, not a reliable system users can trust.
>
>
> For hot plugging obvious. At system boot time however the ordering and
> seeing the ordering is rather important because in many cases the
> ordering is what tells you about things like IDE controller pairing. It
> tells you what order to assign many scsi devices because the ordering is
> defined by their BIOS ROM.
>
> One way to handle this generically would be to use pci_register_device,
> but in the register function for such wacky devices during boot up we
> merely keep track of what we have to look into.
My point is that depending on any method of internal kernel ordering is
fragile.
I would rather have the kernel export which drives are listed in CMOS /
BIOS ROM, and let userspace say "my boot drive is the nth BIOS-listed
drive." For example, looking through the aic7xxx (or was it
ncr53c8xxx?) drive, it gets boot drive ordering from BIOS/CMOS. That
piece of info can either be exported by driverfs from the low-level SCSI
driver, or by a separate, tiny ncr53c8xxx_boot_drive driver.
Depending on pci_find_* ordering is very situation-dependent, and only
covers N cases. Then you have another N cases covered by the order in
which you modprobe key drivers. Then you have another N cases covered
by special case code somewhere. You'll never get all these cases right,
in the kernel, the way the user wants. That's why I say the
responsibility for figuring out the boot drive should be pushed to
initrd/initramfs.
Jeff
next prev parent reply other threads:[~2002-07-13 15:35 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-07-13 4:04 Removal of pci_find_* in 2.5 Matt_Domsch
2002-07-13 4:41 ` Jeff Garzik
2002-07-13 5:09 ` Greg KH
2002-07-13 14:35 ` Alan Cox
2002-07-13 15:37 ` Jeff Garzik [this message]
2002-07-13 17:06 ` Alan Cox
2002-07-13 19:28 ` Jeff Garzik
2002-07-13 19:33 ` Jeff Garzik
-- strict thread matches above, loose matches on Subject: below --
2002-07-16 19:23 Matt_Domsch
2002-07-13 13:13 Matt_Domsch
2002-07-13 0:36 Greg KH
2002-07-13 2:23 ` Alan Cox
2002-07-13 1:12 ` David S. Miller
2002-07-13 14:46 ` Alan Cox
2002-07-13 20:52 ` David S. Miller
2002-07-13 13:45 ` Benjamin Herrenschmidt
2002-07-15 5:25 ` David S. Miller
2002-07-16 0:26 ` Greg KH
2002-07-14 20:07 ` Eric W. Biederman
2002-07-16 0:25 ` Greg KH
2002-07-16 10:56 ` Eric W. Biederman
2002-07-16 17:33 ` Patrick Mochel
2002-07-17 12:41 ` Eric W. Biederman
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=3D304940.7020207@mandrakesoft.com \
--to=jgarzik@mandrakesoft.com \
--cc=Matt_Domsch@Dell.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=greg@kroah.com \
--cc=linux-kernel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.