From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>
Cc: Linux PM list <linux-pm@vger.kernel.org>,
Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
Chris Ball <cjb@laptop.org>,
Ulf Hansson <ulf.hansson@stericsson.com>,
Magnus Damm <magnus.damm@gmail.com>,
Linus Walleij <linus.walleij@linaro.org>
Subject: [PATCH 1/3] MMC / PM: Make it possible to use PM QoS latency constraints
Date: Sun, 4 Mar 2012 01:02:39 +0100 [thread overview]
Message-ID: <201203040102.39253.rjw@sisk.pl> (raw)
In-Reply-To: <201203040101.53177.rjw@sisk.pl>
From: Rafael J. Wysocki <rjw@sisk.pl>
A runtime suspend of an MMC controller belonging to a power domain
or, in a more complicated scenario, a runtime suspend of another
device in the same power domain, may cause power to be removed from
the entire domain. In that case, the amount of time necessary to
runtime-resume the MMC controller is often substantially greater
than the time needed to enable its clock. That may hurt performance
in some situations, because user data may need to wait for the
controller to become operational, so we should make it possible to
prevent that from happening.
For this reason, introduce a new sysfs attribute for MMC hosts,
pm_latency_limit_ms, allowing user space to specify the upper bound
of the time necessary to bring the host up after it has been
runtime-suspended. However, make it appear ony for the hosts whose
drivers declare support for PM QoS by populating the pm_qos_req
member of struct mmc_host before registering the host.
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
---
Documentation/mmc/mmc-dev-attrs.txt | 22 +++++++++++
drivers/mmc/core/host.c | 67 ++++++++++++++++++++++++++++++++++++
include/linux/mmc/host.h | 5 ++
3 files changed, 94 insertions(+)
Index: linux/drivers/mmc/core/host.c
===================================================================
--- linux.orig/drivers/mmc/core/host.c
+++ linux/drivers/mmc/core/host.c
@@ -293,6 +293,68 @@ static inline void mmc_host_clk_sysfs_in
#endif
+static ssize_t pm_qos_val_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
+{
+ struct mmc_host *host = cls_dev_to_mmc_host(dev);
+ return snprintf(buf, PAGE_SIZE, "%d\n", host->pm_qos_val);
+}
+
+static ssize_t pm_qos_val_store(struct device *dev,
+ struct device_attribute *attr, const char *buf, size_t count)
+{
+ struct mmc_host *host = cls_dev_to_mmc_host(dev);
+ s32 value;
+ int ret;
+
+ if (kstrtos32(buf, 0, &value))
+ return -EINVAL;
+
+ if (value < 0)
+ return -EINVAL;
+
+ host->pm_qos_val = value;
+ ret = dev_pm_qos_update_request(host->pm_qos_req, host->pm_qos_val);
+ return ret < 0 ? ret : count;
+}
+
+static void mmc_host_pm_qos_init(struct mmc_host *host)
+{
+ int ret;
+
+ if (!host->pm_qos_req)
+ return;
+
+ ret = dev_pm_qos_add_request(host->parent, host->pm_qos_req,
+ host->pm_qos_val);
+ if (ret < 0) {
+ dev_err(host->parent, "Unable to use PM QoS: %d\n", ret);
+ return;
+ }
+
+ host->pm_qos_val_attr.show = pm_qos_val_show;
+ host->pm_qos_val_attr.store = pm_qos_val_store;
+ sysfs_attr_init(&host->pm_qos_val_attr.attr);
+ host->pm_qos_val_attr.attr.name = "pm_latency_limit_ms";
+ host->pm_qos_val_attr.attr.mode = S_IRUGO | S_IWUSR;
+ if (device_create_file(&host->class_dev, &host->pm_qos_val_attr)) {
+ pr_err("%s: Failed to create PM latency limit sysfs entry\n",
+ mmc_hostname(host));
+ host->pm_qos_val_attr.attr.name = NULL;
+ }
+}
+
+static void mmc_host_pm_qos_exit(struct mmc_host *host)
+{
+ if (!host->pm_qos_req)
+ return;
+
+ if (host->pm_qos_val_attr.attr.name)
+ device_remove_file(&host->class_dev, &host->pm_qos_val_attr);
+
+ dev_pm_qos_remove_request(host->pm_qos_req);
+}
+
/**
* mmc_alloc_host - initialise the per-host structure.
* @extra: sizeof private data structure
@@ -381,6 +443,8 @@ int mmc_add_host(struct mmc_host *host)
#endif
mmc_host_clk_sysfs_init(host);
+ mmc_host_pm_qos_init(host);
+
mmc_start_host(host);
register_pm_notifier(&host->pm_notify);
@@ -400,8 +464,11 @@ EXPORT_SYMBOL(mmc_add_host);
void mmc_remove_host(struct mmc_host *host)
{
unregister_pm_notifier(&host->pm_notify);
+
mmc_stop_host(host);
+ mmc_host_pm_qos_exit(host);
+
#ifdef CONFIG_DEBUG_FS
mmc_remove_host_debugfs(host);
#endif
Index: linux/include/linux/mmc/host.h
===================================================================
--- linux.orig/include/linux/mmc/host.h
+++ linux/include/linux/mmc/host.h
@@ -13,6 +13,7 @@
#include <linux/leds.h>
#include <linux/sched.h>
#include <linux/fault-inject.h>
+#include <linux/pm_qos.h>
#include <linux/mmc/core.h>
#include <linux/mmc/pm.h>
@@ -191,6 +192,10 @@ struct mmc_host {
u32 ocr_avail_mmc; /* MMC-specific OCR */
struct notifier_block pm_notify;
+ struct dev_pm_qos_request *pm_qos_req;
+ s32 pm_qos_val;
+ struct device_attribute pm_qos_val_attr;
+
#define MMC_VDD_165_195 0x00000080 /* VDD voltage 1.65 - 1.95 */
#define MMC_VDD_20_21 0x00000100 /* VDD voltage 2.0 ~ 2.1 */
#define MMC_VDD_21_22 0x00000200 /* VDD voltage 2.1 ~ 2.2 */
Index: linux/Documentation/mmc/mmc-dev-attrs.txt
===================================================================
--- linux.orig/Documentation/mmc/mmc-dev-attrs.txt
+++ linux/Documentation/mmc/mmc-dev-attrs.txt
@@ -74,3 +74,25 @@ This attribute appears only if CONFIG_MM
clkgate_delay Tune the clock gating delay with desired value in milliseconds.
echo <desired delay> > /sys/class/mmc_host/mmcX/clkgate_delay
+
+MMC PM QoS Support
+==================
+
+If the MMC host driver supports PM QoS, it should populate the pm_qos_req member
+of struct mmc_host before registering the host. In that case, read and write
+access if provided to the following attribute:
+
+ pm_latency_limit_ms Specify the upper bound of the time necessary
+ to bring the host up after it has been
+ runtime-suspended (in milliseconds).
+
+echo <desired latency limit> > /sys/class/mmc_host/mmcX/pm_latency_limit_ms
+
+The precise meaning of this attribute is "whenever the host is runtime-suspended,
+make sure that it will be possible to bring it up within the given time" (e.g.
+if the limit is set to 10 ms, it should always be possible to make the host
+operational at most 10 ms after it has been runtime-suspended). Note, however,
+that if the limit is set too small, the only physically possible way to respect
+it may be to keep the host permanently operational (i.e. never suspend it).
+
+If 0 is written to pm_latency_limit_ms, it means that there is no limit.
next prev parent reply other threads:[~2012-03-04 0:00 UTC|newest]
Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-04 0:01 [PATCH 0/3] MMC / PM: Make it possible to use PM QoS latency constraints Rafael J. Wysocki
2012-03-04 0:02 ` Rafael J. Wysocki [this message]
2012-03-04 10:59 ` [PATCH 1/3] " Linus Walleij
2012-03-04 19:47 ` Rafael J. Wysocki
2012-03-04 0:04 ` [PATCH 2/3] tmio_mmc / PM: Use PM QoS requests Rafael J. Wysocki
2012-03-04 0:04 ` [PATCH 3/3] sh_mmcif " Rafael J. Wysocki
2012-03-04 19:53 ` [PATCH 0/3] MMC / PM: Make it possible to use PM QoS latency constraints Rafael J. Wysocki
2012-03-04 19:55 ` [PATCH 1/3] MMC / PM: Make it possible to use PM QoS latency constraints, v2 Rafael J. Wysocki
2012-03-05 7:02 ` Linus Walleij
2012-03-06 9:34 ` Guennadi Liakhovetski
2012-03-06 21:06 ` Rafael J. Wysocki
2012-03-04 19:56 ` [PATCH 2/3] tmio_mmc / PM: Use PM QoS requests, v2 Rafael J. Wysocki
2012-03-06 9:40 ` Guennadi Liakhovetski
2012-03-06 21:07 ` Rafael J. Wysocki
2012-03-06 22:33 ` Guennadi Liakhovetski
2012-03-06 23:41 ` Rafael J. Wysocki
2012-03-04 19:56 ` [PATCH 3/3] sh_mmcif " Rafael J. Wysocki
2012-03-06 9:40 ` Guennadi Liakhovetski
2012-03-06 21:09 ` Rafael J. Wysocki
2012-03-07 23:27 ` [PATCH 0/3] PM: Make it possible to expose PM QoS latency constraints Rafael J. Wysocki
2012-03-07 23:28 ` [PATCH 1/3] PM / QoS: " Rafael J. Wysocki
2012-03-08 17:49 ` Kevin Hilman
2012-03-08 18:01 ` Mark Brown
2012-03-08 21:28 ` Rafael J. Wysocki
2012-03-08 21:23 ` Guennadi Liakhovetski
2012-03-08 21:27 ` Rafael J. Wysocki
2012-03-08 22:05 ` Kevin Hilman
2012-03-08 22:37 ` Rafael J. Wysocki
2012-03-08 23:18 ` Kevin Hilman
2012-03-08 23:30 ` Rafael J. Wysocki
2012-03-09 1:02 ` Kevin Hilman
2012-03-09 15:17 ` Alan Stern
2012-03-09 17:10 ` Kevin Hilman
2012-03-09 20:59 ` Rafael J. Wysocki
2012-03-09 21:34 ` Kevin Hilman
2012-03-07 23:29 ` [PATCH 2/3] tmio_mmc / PM: Use PM QoS latency constraint Rafael J. Wysocki
2012-03-08 8:02 ` Adrian Hunter
2012-03-08 21:29 ` Rafael J. Wysocki
2012-03-07 23:30 ` [PATCH 3/3] sh_mmcif " Rafael J. Wysocki
2012-03-08 11:03 ` Mark Brown
2012-03-08 21:29 ` Rafael J. Wysocki
2012-03-08 23:00 ` [Update][PATCH 0/3] PM: Make it possible to expose PM QoS latency constraints Rafael J. Wysocki
2012-03-08 23:01 ` [Update][PATCH 1/3] PM / QoS: " Rafael J. Wysocki
2012-03-12 19:32 ` Linus Walleij
2012-03-13 0:02 ` Rafael J. Wysocki
2012-03-08 23:03 ` [Update][PATCH 2/3] tmio_mmc / PM: Use PM QoS latency constraint Rafael J. Wysocki
2012-03-08 23:03 ` [Update][PATCH 3/3] sh_mmcif " Rafael J. Wysocki
2012-03-10 21:14 ` [Update][PATCH 0/3] PM: Make it possible to expose PM QoS latency constraints Rafael J. Wysocki
2012-03-06 10:30 ` [PATCH 0/3] MMC / PM: Make it possible to use " Adrian Hunter
2012-03-06 13:39 ` Guennadi Liakhovetski
2012-03-06 21:14 ` Rafael J. Wysocki
2012-03-07 8:31 ` Adrian Hunter
2012-03-07 9:05 ` Rafael J. Wysocki
2012-03-07 19:38 ` Mark Brown
2012-03-07 20:38 ` Kevin Hilman
2012-03-07 20:51 ` Rafael J. Wysocki
2012-03-07 20:54 ` Mark Brown
2012-03-07 21:31 ` Rafael J. Wysocki
2012-03-06 21:47 ` Rafael J. Wysocki
2012-03-07 7:06 ` Adrian Hunter
2012-03-07 9:05 ` Rafael J. Wysocki
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=201203040102.39253.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=cjb@laptop.org \
--cc=g.liakhovetski@gmx.de \
--cc=linus.walleij@linaro.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=ulf.hansson@stericsson.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).