[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1185891488.3468.2.camel@localhost.localdomain>
Date: Tue, 31 Jul 2007 09:18:08 -0500
From: James Bottomley <James.Bottomley@...elEye.com>
To: Tejun Heo <htejun@...il.com>
Cc: Jeff Garzik <jeff@...zik.org>,
Kristen Carlson Accardi <kristen.c.accardi@...el.com>,
linux-scsi@...r.kernel.org, akpm@...ux-foundation.org,
linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org,
edwintorok@...il.com, axboe@...nel.dk
Subject: Re: [patch 2/4] Expose Power Management Policy option to users
On Tue, 2007-07-31 at 15:27 +0900, Tejun Heo wrote:
> Jeff Garzik wrote:
> > Any chance the SCSI peeps could ACK this, and then let me include it in
> > the ALPM patchset in the libata tree?
>
> ATA link PS is pretty complex with HIPM, DIPM and AHCI ALPM. I'm not
> sure whether this three level knob would be sufficient. It might be
> good enough if we're gonna develop extensive in-kernel black/white list
> specifying which method works on which combination but my gut tells me
> that it's best left to userland (probably in the form of per-notebook PS
> profile).
>
> Adding to the fun, there are quite a few broken devices out there which
> act weirdly when link PS actions are taken.
>
> Also, I generally don't think AHCI ALPM is a good idea. It doesn't have
> 'cool down' period before entering PS state which unnecessarily hampers
> performance and might increase chance of device malfunction.
>
> So, mild NACK from me.
The other comment is that power saving seems to be a property of the
transport rather than the host. If you do it in the transport classes,
then you can expose all the knobs the actual transport possesses (which
is, unfortunately, none for quite a few SCSI transports).
James
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists