Messages in this thread | | | Date | Mon, 02 Jun 2008 14:38:55 -0400 | From | Jeff Garzik <> | Subject | Re: [PATCH] ata: ahci: power off unused ports |
| |
Kristen Carlson Accardi wrote: > On Mon, 02 Jun 2008 14:15:27 -0400 > Jeff Garzik <jeff@garzik.org> wrote: > >> Kristen Carlson Accardi wrote: >>> Here you can see that it would be very easy for another driver >>> to just add code to set the NO_HOTPLUG flag and then this code >>> will work for them as well, since we power off the phy using >>> DET which is specified by SATA. >>> here's that code: >> Agreed. The main discussion in this sub-thread is more about user >> interface. The user interface in this case, a module option, is >> specific to AHCI. >> >> Surely you can see how it is a bit silly to force each driver writer to >> create the same user interface in each driver, to support a generic concept. >> >> Jeff >> >> > Not all drivers will need a user interface to turn off hotplug > I would think. At any rate - I would think it'd be better to let > driver writers decide how they want their drivers to behave wrt > hotplug and power instead of forcing a generic policy on everyone. > > This patch would provide users of AHCI controllers a way to save > power now, while you work on the grand scheme for polling/turning on off > hotplug via sysfs. It's an interim solution that impacts nobody but > ahci users and is can be easily integrated into whatever solution you > eventually work out.
It's a one-off driver-specific interface that will have to be supported even after the same feature is available via libata core... that's not the path to scalable, sustainable engineering.
I know user interfaces are annoying because you have to think about chips other than your own, but that's life. Other hardware vendors have to do it too.
Letting each driver have a different user interface is /unfriendly/ to both developers users. It's easiest for Intel kernel developers, but that is not our target audience :)
The biggest power savings for the largest amount of users can be had if you take a moment and figure out what's best for Linux, rather than what is best for Intel.
Because you can be damned sure SATA users with non-AHCI chips want this power savings too... let's not put roadblocks to that in place in the beginning (by adding one-off interfaces).
Jeff
| |