From: Todd Brandt <todd.e.bra...@linux.intel.com> Improve overall system resume time by making libata link recovery actions asynchronous relative to other resume events.
Link resume operations are performed using the scsi_eh thread, so commands, particularly the sd resume start/stop command, will be held off until the device exits error handling. Libata already flushes eh with ata_port_wait_eh() in the port teardown paths, so there are no concerns with async operation colliding with the end-of-life of the ata_port object. Also, libata-core is already careful to flush in-flight pm operations before another round of pm starts on the given ata_port. Reference: https://01.org/suspendresume/blogs/tebrandt/2013/hard-disk-resume-optimization-simpler-approach Cc: Len Brown <len.br...@intel.com> Cc: Phillip Susi <ps...@ubuntu.com> Cc: Alan Stern <st...@rowland.harvard.edu> Signed-off-by: Todd Brandt <todd.e.bra...@linux.intel.com> [djbw: rebase on cleanup patch, changelog wordsmithing] Signed-off-by: Dan Williams <dan.j.willi...@intel.com> --- drivers/ata/libata-core.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index 0f47436c714c..7719ec7d9df9 100644 --- a/drivers/ata/libata-core.c +++ b/drivers/ata/libata-core.c @@ -5444,7 +5444,7 @@ static int ata_port_resume(struct device *dev) { int rc; - rc = ata_port_resume_sync(to_ata_port(dev), PMSG_RESUME); + rc = queue_ata_port_resume(to_ata_port(dev), PMSG_RESUME); if (!rc) { pm_runtime_disable(dev); pm_runtime_set_active(dev); -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
