On Thu, Sep 20, 2007 at 06:04:22PM -0400, Jeff Garzik wrote: > the code looks correct. I have one main reservation. > > how can we be sure that this is active only where other hand-programmed > hotplug code is absent? Yes, that's difficult. As Tejun pointed out, there's missing locking here at the moment - if I add that, am I right in thinking that the worst case scenario is that the hotplugging path will be called twice? One option would be to limit this to PATA-style controllers - I'm not aware of any mobile hardware shipping with natively hotplug-capable controllers, so that would probably do for the moment. If (when) they move to AHCI, with luck the native hotplugging will work and we won't have to worry about this so much. The alternative would be to add a flag to the ap structure indicating whether the hotplugging is handled by the firmware or not. If we find a reference to a controller or port in the firmware tables, it probably indicates that the hardware has opinions about how this should be handled. We might be safer leaving it to the firmware in those cases, and using that flag to skip the controller-specific hotplug code. -- Matthew Garrett | mjg59@xxxxxxxxxxxxx - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
