Re: [PATCH 0/4] PCI legacy I/O port free driver (take 3)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 27, 2006 at 05:23:02PM -0500, Jeff Garzik wrote:
> Grant Grundler wrote:
> >On Mon, Feb 27, 2006 at 12:43:09PM -0500, Jeff Garzik wrote:
> >
> >>This series still leaves a lot to be desired, and creates unnecessary
> >>driver churn.
> >
> >
> >This is a pretty small change and is not necessary for every driver.
> 
> The latter is decidedly false.  The change makes no sense at all unless 
> you update every conceivable driver that will be used on the target 
> platform.  You will always be patching drivers as users stick new cards 
> in the target hardware.

It's sufficiently true if 90% of the cards plugged into the machine
are covered by 3 or 4 drivers. The remaining 10% can use IO port space
and everything will work fine.


> >If in #1 pci_enable_device() assigns I/O Port resources even though
> >the driver doesn't need it, PCI devices which _only_ support I/O Port
> >space will get screwed (depending on config). We are trying to avoid that.
> >Or do you have another way of avoiding unused resource allocation?
> 
> Fix the [firmware | device load order] to allocate I/O ports first to 
> the hardware that only supports IO port accesses.  Problem solved with 
> zero kernel mods...

After we (OS developers) have been telling firmware people to NOT
configure every device and run it's BIOS driver at power up?

OS "hotplug" now deals with resource assignment and device initialization
in most cases on these boxes. Running the BIOS/EFI driver to configure
each device just takes too long.

grant
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux