From: Jeff Garzik <jeff@garzik.org>
To: Tejun Heo <tj@kernel.org>
Cc: linux-ide@vger.kernel.org
Subject: Re: [PATCHSET #upstream] libata: reimplement LPM support
Date: Fri, 17 Sep 2010 02:06:49 -0400 [thread overview]
Message-ID: <4C930579.3090507@garzik.org> (raw)
In-Reply-To: <1283356208-1884-1-git-send-email-tj@kernel.org>
On 09/01/2010 11:50 AM, Tejun Heo wrote:
> Hello,
>
> This patchset reimplement LPM support so that it's better integrated
> with the rest of libata and supports more than just ALPM. After the
> patchset, DIPM works on ahci w/o ALPM support, host link of PMP can
> use both H/DIPM and fan-out links supports DIPM, and ata_piix w/ SIDPR
> also supports DIPM.
>
> Adding DIPM support for other controllers should be easy and DIPM
> actually works better than ALPM because DIPM implements sane timeout
> before entering powersave mode and thus doesn't hurt throughput like
> ALPM does.
>
> Tested with intel and JMB ahci's, ich8 ata_piix, SIMG and marvell
> PMPs. SATA bus analyzer was used to verify links are actually
> entering and coming out of powersave mode.
>
> This patchset contains the following five patches
>
> 0001-libata-clean-up-lpm-related-symbols-and-sysfs-show-s.patch
> 0002-libata-implement-sata_link_scr_lpm-and-make-ata_dev_.patch
> 0003-libata-reimplement-link-power-management.patch
> 0004-libata-implement-LPM-support-for-port-multipliers.patch
> 0005-ata_piix-implement-LPM-support.patch
>
> and available in the folloing git tree
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tj/libata-dev.git lpm
Had to hunt a bit for a good DIPM local test.
Looks good, applied, thanks!
prev parent reply other threads:[~2010-09-17 6:06 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-01 15:50 [PATCHSET #upstream] libata: reimplement LPM support Tejun Heo
2010-09-01 15:50 ` [PATCH 1/5] libata: clean up lpm related symbols and sysfs show/store functions Tejun Heo
2010-09-01 15:50 ` [PATCH 2/5] libata: implement sata_link_scr_lpm() and make ata_dev_set_feature() global Tejun Heo
2010-09-01 15:50 ` [PATCH 3/5] libata: reimplement link power management Tejun Heo
2010-09-01 15:50 ` [PATCH 4/5] libata: implement LPM support for port multipliers Tejun Heo
2010-09-01 15:50 ` [PATCH 5/5] ata_piix: implement LPM support Tejun Heo
2010-09-17 6:06 ` Jeff Garzik [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=4C930579.3090507@garzik.org \
--to=jeff@garzik.org \
--cc=linux-ide@vger.kernel.org \
--cc=tj@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.