On Sun, 23 Feb 2003, Russell King wrote:
> On Sun, Feb 23, 2003 at 04:01:10PM -0500, Scott Murray wrote:
> > Having beaten out something roughly similiar for the cPCI hotplug code,
> > I have a couple of comments:
> > 1) The description of pci_remove_bus_device says "informing the drivers
> > that the device has been removed", yet unless I'm missing some sysfs
> > wrinkle, no call will be made to an attached driver's remove callback.
>
> pci_remove_all_bus_devices => pci_remove_bus_device =>
> pci_remove_device => device_unregister => device_del =>
> bus_remove_device => device_release_driver => driver->remove
Okay, cool. My cursory browsing of drivers/base/core.c missed this,
my apologies.
> > 2) The recursive bus handling in pci_remove_bus_device should probably
> > call pci_proc_detach_bus
>
> Good catch - I'll create a new patch for Monday.
>
> > and potentially should also update the parent bridge's subordinate
> > field.
>
> Yes - I think this is something we may consider when sorting out the
> insertion.
Your previous example wouldn't, but depending on how the BIOS assigns bus
numbers at boot time, I could see some docking station hardware suffering
from the problems I see on cPCI. For example, if the BIOS does not leave
a hole in the bus numbers for the docking station's bridge, and there
happens to be another bridge on the laptop, then you would have the same
overlapping range problem after docking that I see after hot-inserting a
peripheral card with a bridge.
> However, whether x86 PCs will survive bus renumbering or not remains to
> be seen. We currently try to leave as much of the configuration intact
> from the BIOS.
I suspect I'm going to have to try and find out sooner or later, as all of
the x86 based cPCI system boards that I've seen use either Phoenix or
Award BIOSes that started their lives in desktop PCs.
Scott
-- Scott Murray SOMA Networks, Inc. Toronto, Ontario e-mail: scottm@somanetworks.com- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sun Feb 23 2003 - 22:00:40 EST