public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: "Justin T. Gibbs" <gibbs@scsiguy.com>
To: Arjan van de Ven <arjanv@redhat.com>
Cc: Eric Kerin <eric@bootseg.com>, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] 2.6.0 aic7xxx and aic79xx stale pci_device list entry
Date: Thu, 29 Jan 2004 22:25:05 -0700	[thread overview]
Message-ID: <2228980000.1075440305@aslan.btc.adaptec.com> (raw)
In-Reply-To: <20040104100448.GA2416@devserv.devel.redhat.com>

> <snip the rest>
> -----------------
> 
> There are 2 reasons to work this way, and I would very much prefer the
> adaptec drivers to behave like this:
> 1) PCI Hotplugging
> 2) Adding PCI ID's at runtime
> 
> While you may not care about 1), a lot of IHV/distros care about 2); eg if
> Adaptec puts out a card that can be driven by aic79xx but with just a new
> PCI ID, hardware vendors can just add the ID at runtime and have it work in
> the (factory) installer etc but only *if the driver is loaded* ....

Well, if you look at the Adaptec drivers, they already attach to all
possible IDs for the chips that are supported.  They do this by registering
a very generic PCI ID table and filtering all requests through their
own tables.  Each of these tables is terminated with a generic entry
that will attach to any of the supported chipsets regardless of the
"fungible bits" in the chip's PCI IDs.  The only exception to this rule is
for chips that are explicitly labeled as owned by a RAID controller or are
in HostRAID mode.  This latter restriction is going away too since
Adaptec is migrating to an MD based solution for its HostRAID products.

In other words, #2 doesn't require these drivers to stay loaded.  For
#1 to work generically, something should load drivers since it isn't
practical to have every driver that could possibly be needed to service
a hot-plug event in loaded in your kernel.

--
Justin


  reply	other threads:[~2004-01-30  5:18 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-01-02 20:50 [PATCH] 2.6.0 aic7xxx and aic79xx stale pci_device list entry Eric Kerin
2004-01-03  9:01 ` Arjan van de Ven
2004-01-03 11:12   ` Eric Kerin
2004-01-03 13:59     ` Jan-Benedict Glaw
2004-01-24 16:09     ` Eric Kerin
2004-01-03 22:24   ` Justin T. Gibbs
2004-01-04 10:04     ` Arjan van de Ven
2004-01-30  5:25       ` Justin T. Gibbs [this message]
2004-01-30 14:41         ` Arjan van de Ven
2004-01-30 15:03           ` Justin T. Gibbs
2004-01-03 22:24 ` Justin T. Gibbs

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=2228980000.1075440305@aslan.btc.adaptec.com \
    --to=gibbs@scsiguy.com \
    --cc=arjanv@redhat.com \
    --cc=eric@bootseg.com \
    --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