| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1964657.FmY68Z0miD@vostro.rjw.lan>
Date: Mon, 21 Jan 2013 01:53:54 +0100
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>,
"Kristen C. Accardi" <kristen.c.accardi@...el.com>,
Len Brown <lenb@...nel.org>
Subject: [RFC][Update][PATCH 3/3] ACPI / PM: Expose lists of device power resources to user space
From: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
Since ACPI power resources are going to be used more extensively on
new hardware platforms, it is necessary to allow user space (powertop
in particular) to look at the lists of power resources corresponding
to different power states of devices for diagnostics and control
purposes.
For this reason, for each power state of an ACPI device node using
power resources create a special attribute group under the device
node's directory in sysfs containing links to sysfs directories
representing the power resources in that list. The names of the
new attribute groups are "power_resources_<state>", where <state>
is the state name i.e. "D0", "D1", "D2", or "D3hot".
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
---
Documentation/ABI/testing/sysfs-devices-power_resources_D0 | 12 +
Documentation/ABI/testing/sysfs-devices-power_resources_D1 | 12 +
Documentation/ABI/testing/sysfs-devices-power_resources_D2 | 12 +
Documentation/ABI/testing/sysfs-devices-power_resources_D3hot | 12 +
drivers/acpi/power.c | 96 ++++++++--
5 files changed, 131 insertions(+), 13 deletions(-)
Index: linux-pm/drivers/acpi/power.c
===================================================================
--- linux-pm.orig/drivers/acpi/power.c
+++ linux-pm/drivers/acpi/power.c
@@ -417,24 +417,94 @@ static void acpi_power_remove_dependent(
}
}
-void acpi_power_add_remove_device(struct acpi_device *adev, bool add)
+static struct attribute *attrs[] = {
+ NULL,
+};
+
+static void acpi_power_hide_list(struct acpi_device *adev,
+ struct acpi_device_power_state *ps,
+ const char *group_name)
{
- if (adev->power.flags.power_resources) {
- struct acpi_device_power_state *ps;
- struct acpi_power_resource_entry *entry;
-
- ps = &adev->power.states[ACPI_STATE_D0];
- list_for_each_entry(entry, &ps->resources, node) {
- struct acpi_power_resource *resource = entry->resource;
-
- if (add)
- acpi_power_add_dependent(resource, adev);
- else
- acpi_power_remove_dependent(resource, adev);
+ struct attribute_group attr_group = {
+ .name = group_name,
+ .attrs = attrs,
+ };
+ struct acpi_power_resource_entry *entry;
+
+ list_for_each_entry_reverse(entry, &ps->resources, node) {
+ struct acpi_device *res_dev = &entry->resource->device;
+
+ sysfs_remove_link_from_group(&adev->dev.kobj, group_name,
+ dev_name(&res_dev->dev));
+ }
+ sysfs_remove_group(&adev->dev.kobj, &attr_group);
+}
+
+static void acpi_power_expose_list(struct acpi_device *adev,
+ struct acpi_device_power_state *ps,
+ const char *group_name)
+{
+ struct attribute_group attr_group = {
+ .name = group_name,
+ .attrs = attrs,
+ };
+ struct acpi_power_resource_entry *entry;
+ int ret;
+
+ ret = sysfs_create_group(&adev->dev.kobj, &attr_group);
+ if (ret)
+ return;
+
+ list_for_each_entry(entry, &ps->resources, node) {
+ struct acpi_device *res_dev = &entry->resource->device;
+
+ ret = sysfs_add_link_to_group(&adev->dev.kobj, group_name,
+ &res_dev->dev.kobj,
+ dev_name(&res_dev->dev));
+ if (ret) {
+ acpi_power_hide_list(adev, ps, group_name);
+ break;
}
}
}
+void acpi_power_add_remove_device(struct acpi_device *adev, bool add)
+{
+ static const char *group_names[ACPI_D_STATE_COUNT] = {
+ [ACPI_STATE_D0] = "power_resources_D0",
+ [ACPI_STATE_D1] = "power_resources_D1",
+ [ACPI_STATE_D2] = "power_resources_D2",
+ [ACPI_STATE_D3_HOT] = "power_resources_D3hot",
+ };
+ struct acpi_device_power_state *ps;
+ struct acpi_power_resource_entry *entry;
+ int state;
+
+ if (!adev->power.flags.power_resources)
+ return;
+
+ ps = &adev->power.states[ACPI_STATE_D0];
+ list_for_each_entry(entry, &ps->resources, node) {
+ struct acpi_power_resource *resource = entry->resource;
+
+ if (add)
+ acpi_power_add_dependent(resource, adev);
+ else
+ acpi_power_remove_dependent(resource, adev);
+ }
+
+ for (state = ACPI_STATE_D0; state <= ACPI_STATE_D3_HOT; state++) {
+ ps = &adev->power.states[state];
+ if (list_empty(&ps->resources))
+ continue;
+
+ if (add)
+ acpi_power_expose_list(adev, ps, group_names[state]);
+ else
+ acpi_power_hide_list(adev, ps, group_names[state]);
+ }
+}
+
int acpi_power_min_system_level(struct list_head *list)
{
struct acpi_power_resource_entry *entry;
Index: linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D0
===================================================================
--- /dev/null
+++ linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D0
@@ -0,0 +1,12 @@
+What: /sys/devices/.../power_resources_D0/
+Date: January 2012
+Contact: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
+Description:
+ The /sys/devices/.../power_resources_D0/ directory contains a
+ list of symbolic links to device object directories representing
+ ACPI power resources that need to be turned on for the given
+ device to be in ACPI power state D0.
+
+ This directory is only present for device objects representing
+ ACPI device nodes and only if there are power resources needed
+ by the given device node to be in power state D0.
Index: linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D1
===================================================================
--- /dev/null
+++ linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D1
@@ -0,0 +1,12 @@
+What: /sys/devices/.../power_resources_D1/
+Date: January 2012
+Contact: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
+Description:
+ The /sys/devices/.../power_resources_D1/ directory contains a
+ list of symbolic links to device object directories representing
+ ACPI power resources that need to be turned on for the given
+ device to be in ACPI power state D1.
+
+ This directory is only present for device objects representing
+ ACPI device nodes and only if there are power resources needed
+ by the given device node to be in power state D1.
Index: linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D2
===================================================================
--- /dev/null
+++ linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D2
@@ -0,0 +1,12 @@
+What: /sys/devices/.../power_resources_D2/
+Date: January 2012
+Contact: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
+Description:
+ The /sys/devices/.../power_resources_D2/ directory contains a
+ list of symbolic links to device object directories representing
+ ACPI power resources that need to be turned on for the given
+ device to be in ACPI power state D2.
+
+ This directory is only present for device objects representing
+ ACPI device nodes and only if there are power resources needed
+ by the given device node to be in power state D2.
Index: linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D3hot
===================================================================
--- /dev/null
+++ linux-pm/Documentation/ABI/testing/sysfs-devices-power_resources_D3hot
@@ -0,0 +1,12 @@
+What: /sys/devices/.../power_resources_D3hot/
+Date: January 2012
+Contact: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
+Description:
+ The /sys/devices/.../power_resources_D3hot/ directory contains a
+ list of symbolic links to device object directories representing
+ ACPI power resources that need to be turned on for the given
+ device to be in ACPI power state D3hot.
+
+ This directory is only present for device objects representing
+ ACPI device nodes and only if there are power resources needed
+ by the given device node to be in power state D3hot.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists