linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: jeff@garzik.org, linux-ide@vger.kernel.org
Subject: [PATCHSET #upstream] libata: reimplement LPM support
Date: Wed,  1 Sep 2010 17:50:03 +0200	[thread overview]
Message-ID: <1283356208-1884-1-git-send-email-tj@kernel.org> (raw)

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

with the following changes.

 drivers/ata/ahci.c          |    3 
 drivers/ata/ahci.h          |    3 
 drivers/ata/ahci_platform.c |    3 
 drivers/ata/ata_piix.c      |   24 +++
 drivers/ata/libahci.c       |  160 ++++++-------------------
 drivers/ata/libata-core.c   |  276 +++++++++++---------------------------------
 drivers/ata/libata-eh.c     |  173 ++++++++++++++++++++++++---
 drivers/ata/libata-pmp.c    |   48 +++++++
 drivers/ata/libata-scsi.c   |   73 ++++-------
 drivers/ata/libata.h        |   12 +
 include/linux/libata.h      |   42 +++---
 11 files changed, 397 insertions(+), 420 deletions(-)

Thanks.

--
tejun

             reply	other threads:[~2010-09-01 15:50 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-01 15:50 Tejun Heo [this message]
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 ` [PATCHSET #upstream] libata: reimplement " Jeff Garzik

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=1283356208-1884-1-git-send-email-tj@kernel.org \
    --to=tj@kernel.org \
    --cc=jeff@garzik.org \
    --cc=linux-ide@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;
as well as URLs for NNTP newsgroup(s).