public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Tejun Heo <htejun@gmail.com>
Cc: kernel list <linux-kernel@vger.kernel.org>, axboe@suse.de
Subject: Re: SATA powersave patches
Date: Wed, 22 Nov 2006 00:07:44 +0100	[thread overview]
Message-ID: <20061121230744.GD1995@elf.ucw.cz> (raw)
In-Reply-To: <456280FE.70607@gmail.com>

Hi!

> >>If I understood correctly, the high power consumption of 
> >>ahci controller can be solved by dynamically turning off 
> >>command processing while the controller is idle, which 
> >>fits nicely into link powersaving, right?  So, I think 
> >>full-fledged leveled dynamic PM would be an overkill for 
> >>this particular problem, but then again, maybe the 
> >
> >It is single bit, and it should not even need a timeout, AFAICT, so
> >perhaps we should just fix it (no need for dynamic PM layers). It
> >probably does not even need to be configurable...
> 
> I think this has been discussed in linux-ide recently but just to add my 
> 2 cents.  ALPE and ASP can cause quite some problems.  Many devices 
> don't implement link powermanagement mode properly and some locks up 

Ok, so it needs to be optional :-). I played with the code a little
bit, but did not find combination that both worked and saved power :-(.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

      reply	other threads:[~2006-11-21 23:07 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20060908110346.GC920@elf.ucw.cz>
     [not found] ` <45015767.1090002@gmail.com>
     [not found]   ` <20060908123537.GB17640@elf.ucw.cz>
     [not found]     ` <4501655F.5000103@gmail.com>
2006-09-10 22:48       ` SATA powersave patches Pavel Machek
2006-09-11 10:24         ` Tejun Heo
2006-09-18 10:05           ` Pavel Machek
2006-09-18 10:38             ` Tejun Heo
2006-11-06 13:57               ` Pavel Machek
2006-11-06 17:44                 ` Tejun Heo
2006-11-12 18:39                   ` Pavel Machek
2006-11-21  4:30                     ` Tejun Heo
2006-11-21 23:07                       ` Pavel Machek [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20061121230744.GD1995@elf.ucw.cz \
    --to=pavel@ucw.cz \
    --cc=axboe@suse.de \
    --cc=htejun@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox