Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753712AbbKQNsY (ORCPT ); Tue, 17 Nov 2015 08:48:24 -0500 Received: from mga14.intel.com ([192.55.52.115]:17008 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752509AbbKQNsW (ORCPT ); Tue, 17 Nov 2015 08:48:22 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,307,1444719600"; d="scan'208";a="852925661" Message-ID: <564B3020.5060104@linux.intel.com> Date: Tue, 17 Nov 2015 21:48:16 +0800 From: "Fu, Zhonghui" User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Ulf Hansson CC: Adrian Hunter , Neil Brown , Jaehoon Chung , Andreas Fenkart , Joe Perches , linux-mmc , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v3] MMC/SDIO: enable SDIO device to suspend/resume asynchronously References: <56488E6F.3090109@linux.intel.com> In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3382 Lines: 79 On 11/16/2015 7:30 PM, Ulf Hansson wrote: > On 15 November 2015 at 14:53, Fu, Zhonghui wrote: >> Now, PM core supports asynchronous suspend/resume mode for devices >> during system suspend/resume, and the power state transition of one >> device may be completed in separate kernel thread. PM core ensures >> all power state transition timing dependency between devices. This >> patch enables SDIO card and function devices to suspend/resume >> asynchronously. This will take advantage of multicore and improve >> system suspend/resume speed. After enabling the SDIO devices and all >> their child devices to suspend/resume asynchronously on ASUS T100TA, >> the system suspend-to-idle time is reduced from 1645ms to 1119ms, and >> the system resume time is reduced from 940ms to 918ms. >> >> Signed-off-by: Zhonghui Fu > I think this is an interesting change, but I wonder if you really > understand how this affects the order of how devices may be > suspended/resumed? > > Also, I believe you didn't answer my question for the earlier version > of the patch, so let me try again. > > There are a strict dependency chain when suspending/resuming devices > that must be maintained. Currently this is controlled via device > registration/probe order. > > An SDIO func driver/device must always be suspended *before* the SDIO > card device. Additionally the corresponding MMC host, must be > suspended after the SDIO card device. Vice verse applies to the resume > sequence. > > As this patch enables asynchronous suspend, I am worried that it will > break this dependency chain. What do you think? After enabling asynchronous suspend/resume, PM core still ensures the strict suspend/resume dependency between child and parent devices - child must be suspended before its parent, and parent must be resumed before its child. SDIO function is child of SDIO card, and SDIO card is child of MMC host, and MMC host is child of MMC controller. So the dependency chain is not broken. Actually, many devices have been using asynchronous suspend/resume mode now. Thanks, Zhonghui > > Kind regards > Ulf Hansson > >> --- >> Changes in v3: >> - Add test result in commit message >> >> drivers/mmc/core/sdio.c | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c >> index 16d838e..530ce88 100644 >> --- a/drivers/mmc/core/sdio.c >> +++ b/drivers/mmc/core/sdio.c >> @@ -1113,6 +1113,8 @@ int mmc_attach_sdio(struct mmc_host *host) >> pm_runtime_enable(&card->dev); >> } >> >> + device_enable_async_suspend(&card->dev); >> + >> /* >> * The number of functions on the card is encoded inside >> * the ocr. >> @@ -1133,6 +1135,8 @@ int mmc_attach_sdio(struct mmc_host *host) >> */ >> if (host->caps & MMC_CAP_POWER_OFF_CARD) >> pm_runtime_enable(&card->sdio_func[i]->dev); >> + >> + device_enable_async_suspend(&card->sdio_func[i]->dev); >> } >> >> /* >> -- 1.7.1 >> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/