All of lore.kernel.org
 help / color / mirror / Atom feed
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



  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.