From patchwork Wed Feb 13 12:01:37 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sudeep Holla X-Patchwork-Id: 158203 Delivered-To: patch@linaro.org Received: by 2002:a02:48:0:0:0:0:0 with SMTP id 69csp4971257jaa; Wed, 13 Feb 2019 04:01:55 -0800 (PST) X-Google-Smtp-Source: AHgI3IawiOqMfLsgiMaL9JiU7jKz1k4MnRm10F8lM2XGxK37oAcIJ2oAZQLR5yOTyGdhWUf1UJaO X-Received: by 2002:a65:628e:: with SMTP id f14mr111142pgv.193.1550059315173; Wed, 13 Feb 2019 04:01:55 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1550059315; cv=none; d=google.com; s=arc-20160816; b=XPYOrB/julurlkmHjFPLipn0S1VToRcZcbxtKGV4t1rw1FT0WYXBJu4ZqDUtXddteW YX+NEhJZWPBDmh5wTwbjPhkaPcm9cujSiKsauyt1VO35MlP143uuWvmLABDXH1acw4aX Ekbf7/BT9zQ4e1tJhDem1rrRTRw+O75ind4ctjIqY9gl54kVPFli5K769M6xEzSpSyLn 399E64v5ajTE4cnLAdzdmaFw5oH8eFXq9IiRT/o/vp5Fwat5Pq2wxxCmya7KcuNfrD+R sFnPG/+LlEyLGk54VuPIZbsRiw7045m+y5ix4hbHqNVU4Zq8j6SaesMzliNumbF1cH6l 6t+g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from; bh=41ppITyOGunCkoEY7Q1TvLYWNNV++7u4dhGhmc5QU6I=; b=w4zBUQc7sJopgcPD13NbjLZywx2ejCv7BJ2q8mml7Bmrjc7nQorgQJfjYwWnN5yCAM u4hWwTyLozBSdtu6U5bZBNMI9Pno0JUoL/lC4J8WQeMjn/oR6NmFolxh45MgbmvoJMJk jwwFrfzTWAD+8XgIaQMR4uOlYg2EbItgnTyhZvANshaYw02lTND2HOT21nVGI0+fDi/C 5zXfe/2fjOtS3dAyI+Va5yqdfNWft86XIhawDqj9omrDljyRXa1GMR9NorQ6qC1JPg0j mwzKzKxJKy+6vTQFhykwjeB8HR/nIp3QSiaH3Q6zphAbibEqxWbocHaGZSZc5ulvgiBY 85tA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g1si15526680plo.406.2019.02.13.04.01.54; Wed, 13 Feb 2019 04:01:55 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-pm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-pm-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728062AbfBMMBx (ORCPT + 11 others); Wed, 13 Feb 2019 07:01:53 -0500 Received: from foss.arm.com ([217.140.101.70]:52638 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727568AbfBMMBx (ORCPT ); Wed, 13 Feb 2019 07:01:53 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DC989A78; Wed, 13 Feb 2019 04:01:47 -0800 (PST) Received: from usa.arm.com (e107155-lin.cambridge.arm.com [10.1.196.42]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id C8BF93F557; Wed, 13 Feb 2019 04:01:45 -0800 (PST) From: Sudeep Holla To: "Rafael J . Wysocki" , Linux Kernel Mailing List , Linux PM Cc: Sudeep Holla , Greg Kroah-Hartman , Jisheng Zhang , Ulf Hansson , Steve Longerbeam , Eugeniu Rosca , Joshua Frkuska , Eugeniu Rosca , "Rafael J. Wysocki" Subject: [PATCH v2] drivers: base: add support to skip power management in device/driver model Date: Wed, 13 Feb 2019 12:01:37 +0000 Message-Id: <20190213120137.23354-1-sudeep.holla@arm.com> X-Mailer: git-send-email 2.17.1 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org All device objects in the driver model contain fields that control the handling of various power management activities. However, it's not always useful. There are few instances where pseudo devices are added to the model just to take advantage of many other features like kobjects, udev events, and so on. One such example is cpu devices and their caches. The sysfs for the cpu caches are managed by adding devices with cpu as the parent in cpu_device_create() when secondary cpu is brought online. Generally when the secondary CPUs are hotplugged back in as part of resume from suspend-to-ram, we call cpu_device_create() from the cpu hotplug state machine while the cpu device associated with that CPU is not yet ready to be resumed as the device_resume() call happens bit later. It's not really needed to set the flag is_prepared for cpu devices as they are mostly pseudo device and hotplug framework deals with state machine and not managed through the cpu device. This often results in annoying warning when resuming: Enabling non-boot CPUs ... CPU1: Booted secondary processor cache: parent cpu1 should not be sleeping CPU1 is up CPU2: Booted secondary processor cache: parent cpu2 should not be sleeping CPU2 is up .... and so on. So in order to fix these kind of errors, we could just completely avoid doing any power management related initialisations and operations if they are not used by these devices. Lets add no_pm_required flags to indicate that the device doesn't require any sort of pm activities and all of them can be completely skipped. We can use the same flag to also avoid adding not used *power* sysfs entries for these devices. For now, lets use this for cpu cache devices. Cc: Greg Kroah-Hartman Cc: "Rafael J. Wysocki" Signed-off-by: Sudeep Holla --- drivers/base/cpu.c | 1 + drivers/base/power/main.c | 7 +++++++ drivers/base/power/sysfs.c | 4 ++++ include/linux/device.h | 10 ++++++++++ include/linux/pm.h | 1 + 5 files changed, 23 insertions(+) v1->v2: - dropped setting the flag for cpu devices, for now just cpu caches will make use of this new flag. RFC->v1: - dropped the idea of adding cpu hotplug callback to deal just with cpu devices, instead add a new flag in the device pm_info structure [RFC] : https://marc.info/?l=linux-pm&m=154842896407904&w=2 [v1] : https://marc.info/?l=linux-pm&m=154946578717730&w=2 -- 2.17.1 Reviewed-by: Ulf Hansson diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c index eb9443d5bae1..6ce93a52bf3f 100644 --- a/drivers/base/cpu.c +++ b/drivers/base/cpu.c @@ -427,6 +427,7 @@ __cpu_device_create(struct device *parent, void *drvdata, dev->parent = parent; dev->groups = groups; dev->release = device_create_release; + device_set_pm_not_required(dev); dev_set_drvdata(dev, drvdata); retval = kobject_set_name_vargs(&dev->kobj, fmt, args); diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c index 0992e67e862b..2a29c3d4e240 100644 --- a/drivers/base/power/main.c +++ b/drivers/base/power/main.c @@ -124,6 +124,10 @@ void device_pm_unlock(void) */ void device_pm_add(struct device *dev) { + /* No need to create pm sysfs if explicitly specified as not required */ + if (device_pm_not_required(dev)) + return; + pr_debug("PM: Adding info for %s:%s\n", dev->bus ? dev->bus->name : "No Bus", dev_name(dev)); device_pm_check_callbacks(dev); @@ -142,6 +146,9 @@ void device_pm_add(struct device *dev) */ void device_pm_remove(struct device *dev) { + if (device_pm_not_required(dev)) + return; + pr_debug("PM: Removing info for %s:%s\n", dev->bus ? dev->bus->name : "No Bus", dev_name(dev)); complete_all(&dev->power.completion); diff --git a/drivers/base/power/sysfs.c b/drivers/base/power/sysfs.c index d713738ce796..6cd159b51114 100644 --- a/drivers/base/power/sysfs.c +++ b/drivers/base/power/sysfs.c @@ -648,6 +648,10 @@ int dpm_sysfs_add(struct device *dev) { int rc; + /* No need to create pm sysfs if explicitly disabled */ + if (device_pm_not_required(dev)) + return 0; + rc = sysfs_create_group(&dev->kobj, &pm_attr_group); if (rc) return rc; diff --git a/include/linux/device.h b/include/linux/device.h index 6cb4640b6160..739d0b62e4d4 100644 --- a/include/linux/device.h +++ b/include/linux/device.h @@ -1165,6 +1165,16 @@ static inline bool device_async_suspend_enabled(struct device *dev) return !!dev->power.async_suspend; } +static inline bool device_pm_not_required(struct device *dev) +{ + return dev->power.no_pm_required; +} + +static inline void device_set_pm_not_required(struct device *dev) +{ + dev->power.no_pm_required = true; +} + static inline void dev_pm_syscore_device(struct device *dev, bool val) { #ifdef CONFIG_PM_SLEEP diff --git a/include/linux/pm.h b/include/linux/pm.h index 0bd9de116826..300ab9f0b858 100644 --- a/include/linux/pm.h +++ b/include/linux/pm.h @@ -592,6 +592,7 @@ struct dev_pm_info { bool is_suspended:1; /* Ditto */ bool is_noirq_suspended:1; bool is_late_suspended:1; + bool no_pm_required:1; bool early_init:1; /* Owned by the PM core */ bool direct_complete:1; /* Owned by the PM core */ u32 driver_flags;