From patchwork Fri Oct 2 07:14:30 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tomeu Vizoso X-Patchwork-Id: 7313671 Return-Path: X-Original-To: patchwork-linux-pm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 035D6BEEA4 for ; Fri, 2 Oct 2015 07:18:29 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 070AE2085B for ; Fri, 2 Oct 2015 07:18:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id ED248207B6 for ; Fri, 2 Oct 2015 07:18:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752269AbbJBHSE (ORCPT ); Fri, 2 Oct 2015 03:18:04 -0400 Received: from mail-wi0-f181.google.com ([209.85.212.181]:36225 "EHLO mail-wi0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752028AbbJBHQq (ORCPT ); Fri, 2 Oct 2015 03:16:46 -0400 Received: by wicgb1 with SMTP id gb1so19828060wic.1; Fri, 02 Oct 2015 00:16:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=br+bzfGo0rC71dp0cywcHysgc+BEU29oiWpB6GK7DA4=; b=F1XB4wd0Q0YYk8ajy2fFQc6eHfOvMvl7dDqHURzn/Q1Iw6wXNfNUQ5ig0LwCfA9Kdx wz400LApLyb4apqgbIMlcWMP7l5DSqee8AHomKw2mH58kY2KtOQ2HY5O/MjaU/O5/zDl Ax6gplwajzQKC34bmv5zu6/zY1WV8kSeOqVI/D0VRgV9xT4t5EEBCiOaLGHqRyrxDgIY pTh8PGnGhEa4wz52yp0KWqBlPdv2v+Mj6tboCg2SkrNZ6dHPt/RTGi1+LitjrkVm9eFg hs9GZv1Wne2LzZkwwqi+bU736JXtzD69PVhXP3zFOpiqyXA0BXi9f+b/dDAi9BxiW+b1 aCvw== X-Received: by 10.180.75.38 with SMTP id z6mr2466986wiv.36.1443770205272; Fri, 02 Oct 2015 00:16:45 -0700 (PDT) Received: from cizrna.lan ([109.72.12.253]) by smtp.gmail.com with ESMTPSA id ry1sm9840153wjb.24.2015.10.02.00.16.43 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 02 Oct 2015 00:16:44 -0700 (PDT) From: Tomeu Vizoso To: linux-pm@vger.kernel.org, Alan Stern , "Rafael J. Wysocki" , martyn.welch@collabora.co.uk, Ulf Hansson Cc: Tomeu Vizoso , linux-kernel@vger.kernel.org, Len Brown , Kevin Hilman , Greg Kroah-Hartman , Pavel Machek Subject: [PATCH v8 3/4] PM / sleep: Go direct_complete if driver has no callbacks Date: Fri, 2 Oct 2015 09:14:30 +0200 Message-Id: <1443770071-2570-4-git-send-email-tomeu.vizoso@collabora.com> X-Mailer: git-send-email 2.4.3 In-Reply-To: <1443770071-2570-1-git-send-email-tomeu.vizoso@collabora.com> References: <1443770071-2570-1-git-send-email-tomeu.vizoso@collabora.com> Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID,T_RP_MATCHES_RCVD,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP If a suitable prepare callback cannot be found for a given device and its driver has no PM callbacks at all, assume that it can go direct to complete when the system goes to sleep. The reason for this is that there's lots of devices in a system that do no PM at all and there's no reason for them to prevent their ancestors to do direct_complete if they can support it. Signed-off-by: Tomeu Vizoso --- Changes in v8: - Move no_pm_callbacks field into CONFIG_PM_SLEEP - Call device_check_pm_callbacks only after a device is bound or unbound Changes in v7: - Reduce indentation by adding a label in device_prepare() Changes in v6: - Add stub for !CONFIG_PM. - Move implementation of device_check_pm_callbacks to power/main.c as it doesn't belong to CONFIG_PM_SLEEP. - Take dev->power.lock before modifying flag. Changes in v5: - Check for all dev_pm_ops instances associated to a device, updating a no_pm_callbacks flag at the times when that could change. drivers/base/dd.c | 3 +++ drivers/base/power/domain.c | 2 ++ drivers/base/power/main.c | 33 +++++++++++++++++++++++++++++++++ drivers/base/power/power.h | 6 ++++++ include/linux/pm.h | 1 + 5 files changed, 45 insertions(+) diff --git a/drivers/base/dd.c b/drivers/base/dd.c index f2007fa7316f..2e3809affb06 100644 --- a/drivers/base/dd.c +++ b/drivers/base/dd.c @@ -210,6 +210,8 @@ static void driver_bound(struct device *dev) klist_add_tail(&dev->p->knode_driver, &dev->driver->p->klist_devices); + device_check_pm_callbacks(dev); + /* * Make sure the device is no longer in one of the deferred lists and * kick off retrying all pending devices @@ -700,6 +702,7 @@ static void __device_release_driver(struct device *dev) dev->pm_domain->dismiss(dev); klist_remove(&dev->p->knode_driver); + device_check_pm_callbacks(dev); if (dev->bus) blocking_notifier_call_chain(&dev->bus->p->bus_notifier, BUS_NOTIFY_UNBOUND_DRIVER, diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index b75d02aa8d93..9dc1ce9c573b 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -20,6 +20,8 @@ #include #include +#include "power.h" + #define GENPD_RETRY_MAX_MS 250 /* Approximate */ #define GENPD_DEV_CALLBACK(genpd, type, callback, dev) \ diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c index 1710c26ba097..43b44f8251c7 100644 --- a/drivers/base/power/main.c +++ b/drivers/base/power/main.c @@ -1569,6 +1569,11 @@ static int device_prepare(struct device *dev, pm_message_t state) dev->power.wakeup_path = device_may_wakeup(dev); + if (dev->power.no_pm_callbacks) { + ret = 1; /* Let device go direct_complete */ + goto unlock; + } + if (dev->pm_domain) { info = "preparing power domain "; callback = dev->pm_domain->ops.prepare; @@ -1591,6 +1596,7 @@ static int device_prepare(struct device *dev, pm_message_t state) if (callback) ret = callback(dev); +unlock: device_unlock(dev); if (ret < 0) { @@ -1719,3 +1725,30 @@ void dpm_for_each_dev(void *data, void (*fn)(struct device *, void *)) device_pm_unlock(); } EXPORT_SYMBOL_GPL(dpm_for_each_dev); + +static bool pm_ops_is_empty(const struct dev_pm_ops *ops) +{ + if (!ops) + return true; + + return !ops->prepare && + !ops->suspend && + !ops->suspend_late && + !ops->suspend_noirq && + !ops->resume_noirq && + !ops->resume_early && + !ops->resume && + !ops->complete; +} + +void device_check_pm_callbacks(struct device *dev) +{ + spin_lock_irq(&dev->power.lock); + dev->power.no_pm_callbacks = + (!dev->bus || pm_ops_is_empty(dev->bus->pm)) && + (!dev->class || pm_ops_is_empty(dev->class->pm)) && + (!dev->type || pm_ops_is_empty(dev->type->pm)) && + (!dev->pm_domain || pm_ops_is_empty(&dev->pm_domain->ops)) && + (!dev->driver || pm_ops_is_empty(dev->driver->pm)); + spin_unlock_irq(&dev->power.lock); +} diff --git a/drivers/base/power/power.h b/drivers/base/power/power.h index 998fa6b23084..b3e35e825cd8 100644 --- a/drivers/base/power/power.h +++ b/drivers/base/power/power.h @@ -37,6 +37,8 @@ extern void device_wakeup_detach_irq(struct device *dev); extern void device_wakeup_arm_wake_irqs(void); extern void device_wakeup_disarm_wake_irqs(void); +extern void device_check_pm_callbacks(struct device *dev); + #else static inline int @@ -58,6 +60,10 @@ static inline void device_wakeup_disarm_wake_irqs(void) { } +static inline void device_check_pm_callbacks(struct device *dev) +{ +} + #endif /* CONFIG_PM_SLEEP */ /* diff --git a/include/linux/pm.h b/include/linux/pm.h index 35d599e7250d..7943acb74ae4 100644 --- a/include/linux/pm.h +++ b/include/linux/pm.h @@ -573,6 +573,7 @@ struct dev_pm_info { struct wakeup_source *wakeup; bool wakeup_path:1; bool syscore:1; + bool no_pm_callbacks:1; /* Owned by the PM core */ #else unsigned int should_wakeup:1; #endif