From: Jeff Garzik <jgarzik@pobox.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: JBottomley@parallels.com, linux-ide@vger.kernel.org,
linux-scsi@vger.kernel.org
Subject: Re: [resend PATCH 2/5] libata: export ata_port suspend/resume infrastructure for sas
Date: Wed, 25 Jul 2012 01:21:47 -0400 [thread overview]
Message-ID: <500F826B.2080405@pobox.com> (raw)
In-Reply-To: <20120710040552.26529.47482.stgit@dwillia2-linux.jf.intel.com>
On 07/10/2012 12:05 AM, Dan Williams wrote:
> Reuse ata_port_{suspend|resume}_common for sas. This path is chosen
> over adding coordination between ata-tranport and sas-transport because
> libsas wants to revalidate the domain at resume-time at the host level.
> It can not validate links have resumed properly until libata has had a
> chance to perform its revalidation, and any sane placing of an ata_port
> in the sas-transport model would delay it's resumption until after the
> host.
>
> Export the common portion of port suspend/resume (bypass pm_runtime),
> and allow sas to perform these operations asynchronously (similar to the
> libsas async-ata probe implmentation). Async operation is determined by
> having an external, rather than stack based, location for storing the
> result of the operation.
>
> Reviewed-by: Jacek Danecki <jacek.danecki@intel.com>
> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Acked-by: Jeff Garzik <jgarzik@redhat.com>
next prev parent reply other threads:[~2012-07-25 5:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-10 4:05 [resend PATCH 0/5] libsas, libata: suspend / resume and "reset once" Dan Williams
2012-07-10 4:05 ` [resend PATCH 1/5] libata: reset once Dan Williams
2012-07-25 5:20 ` Jeff Garzik
2012-07-10 4:05 ` [resend PATCH 2/5] libata: export ata_port suspend/resume infrastructure for sas Dan Williams
2012-07-25 5:21 ` Jeff Garzik [this message]
2012-07-10 4:05 ` [resend PATCH 3/5] libsas: suspend / resume support Dan Williams
2012-07-10 4:06 ` [resend PATCH 4/5] isci: implement suspend/resume support Dan Williams
2012-07-10 4:06 ` [resend PATCH 5/5] libsas, ipr: cleanup ata_host flags initialization via ata_host_init Dan Williams
2012-07-25 18:42 ` Jeff Garzik
2012-07-25 20:01 ` Brian King
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=500F826B.2080405@pobox.com \
--to=jgarzik@pobox.com \
--cc=JBottomley@parallels.com \
--cc=dan.j.williams@intel.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-scsi@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 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.