From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: libata git tree, mbox queue status and contents Date: Mon, 13 Aug 2007 14:09:31 +0200 Message-ID: <20070813120930.GQ23758@kernel.dk> References: <46B35344.3090208@garzik.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from brick.kernel.dk ([87.55.233.238]:18867 "EHLO kernel.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S945126AbXHMMJf (ORCPT ); Mon, 13 Aug 2007 08:09:35 -0400 Content-Disposition: inline In-Reply-To: <46B35344.3090208@garzik.org> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Jeff Garzik Cc: IDE/ATA development list , Linux Kernel Mailing List , Andrew Morton , Alan On Fri, Aug 03 2007, Jeff Garzik wrote: > * Kristen: ALPM patches. We definitely want them, as they save a ton of > power. The problem with ALPM, as I see it, is that it is way too aggressive. It really needs to be combined with a timer to be useful, it's really a huge shame that it doesn't come equipped with a timeout setting in hardware. Lacking that, we could punt to using a second aligned timer that just checks for activity in the last second, and if none was seen then enable ALPM. That should have absolutely minimal impact on CPU consumption. Likewise for when we see IO, when the rate/sec goes beyond a low threshold then disable ALPM again. In my testing on this notebook (x60), throughput was reduced to about 30% when using ALPM. So while it does save a good amount of power, it also makes the disk a slow dog if you are actually using it. -- Jens Axboe