* [PATCH i-g-t v3] lib/igt_pm: Lib for power management
2016-01-14 11:30 ` [PATCH i-g-t v3] Add a lib for power management helpers David Weinehall
@ 2016-01-14 11:30 ` David Weinehall
0 siblings, 0 replies; 2+ messages in thread
From: David Weinehall @ 2016-01-14 11:30 UTC (permalink / raw)
To: intel-gfx; +Cc: David Weinehall
Move power management related code to a separate library.
Initially this is done only for workarounds that apply to external
components. Modify the users of such workarounds accordingly.
This currently involves HD audio and SATA link power management.
For SATA link PM there's also code to save the previous settings,
to allow for resetting the values after we've finished testing.
Signed-off-by: David Weinehall <david.weinehall@intel.com>
---
.../intel-gpu-tools/intel-gpu-tools-docs.xml | 1 +
lib/Makefile.sources | 2 +
lib/igt.h | 1 +
lib/igt_aux.c | 15 +-
lib/igt_pm.c | 233 +++++++++++++++++++++
lib/igt_pm.h | 31 +++
tests/pm_lpsp.c | 25 +--
tests/pm_rpm.c | 40 +---
8 files changed, 281 insertions(+), 67 deletions(-)
create mode 100644 lib/igt_pm.c
create mode 100644 lib/igt_pm.h
diff --git a/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml b/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml
index 618dc5fd7076..3ea3563a029a 100644
--- a/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml
+++ b/docs/reference/intel-gpu-tools/intel-gpu-tools-docs.xml
@@ -24,6 +24,7 @@
<xi:include href="xml/igt_fb.xml"/>
<xi:include href="xml/igt_aux.xml"/>
<xi:include href="xml/igt_gt.xml"/>
+ <xi:include href="xml/igt_pm.xml"/>
<xi:include href="xml/ioctl_wrappers.xml"/>
<xi:include href="xml/intel_batchbuffer.xml"/>
<xi:include href="xml/intel_chipset.xml"/>
diff --git a/lib/Makefile.sources b/lib/Makefile.sources
index 4999868052b1..2f0eb2075e14 100644
--- a/lib/Makefile.sources
+++ b/lib/Makefile.sources
@@ -60,6 +60,8 @@ libintel_tools_la_SOURCES = \
igt_core.h \
igt_draw.c \
igt_draw.h \
+ igt_pm.c \
+ igt_pm.h \
$(NULL)
.PHONY: version.h.tmp
diff --git a/lib/igt.h b/lib/igt.h
index 3be25511bb77..d751f2439de2 100644
--- a/lib/igt.h
+++ b/lib/igt.h
@@ -35,6 +35,7 @@
#include "igt_fb.h"
#include "igt_gt.h"
#include "igt_kms.h"
+#include "igt_pm.h"
#include "igt_stats.h"
#include "instdone.h"
#include "intel_batchbuffer.h"
diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 4d08d68bb932..cd7f14649fe2 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -59,6 +59,7 @@
#include "intel_reg.h"
#include "ioctl_wrappers.h"
#include "igt_kms.h"
+#include "igt_pm.h"
/**
* SECTION:igt_aux
@@ -531,19 +532,7 @@ bool igt_setup_runtime_pm(void)
if (pm_status_fd >= 0)
return true;
- /* The Audio driver can get runtime PM references, so we need to make
- * sure its runtime PM is enabled, so it can release the refs and
- * actually enable us to runtime suspend. */
- fd = open("/sys/module/snd_hda_intel/parameters/power_save", O_WRONLY);
- if (fd >= 0) {
- igt_assert(write(fd, "1\n", 2) == 2);
- close(fd);
- }
- fd = open("/sys/bus/pci/devices/0000:00:03.0/power/control", O_WRONLY);
- if (fd >= 0) {
- igt_assert(write(fd, "auto\n", 5) == 5);
- close(fd);
- }
+ igt_pm_enable_audio_runtime_pm();
/* Our implementation uses autosuspend. Try to set it to 0ms so the test
* suite goes faster and we have a higher probability of triggering race
diff --git a/lib/igt_pm.c b/lib/igt_pm.c
new file mode 100644
index 000000000000..2f574961d179
--- /dev/null
+++ b/lib/igt_pm.c
@@ -0,0 +1,233 @@
+/*
+ * Copyright © 2013, 2015 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ * Authors:
+ * Paulo Zanoni <paulo.r.zanoni@intel.com>
+ * David Weinehall <david.weinehall@intel.com>
+ *
+ */
+#include <fcntl.h>
+#include <stdio.h>
+#include <limits.h>
+#include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+
+#include "drmtest.h"
+#include "igt_pm.h"
+
+enum {
+ POLICY_UNKNOWN = -1,
+ POLICY_MAX_PERFORMANCE = 0,
+ POLICY_MEDIUM_POWER = 1,
+ POLICY_MIN_POWER = 2
+};
+
+#define MAX_PERFORMANCE_STR "max_performance\n"
+#define MEDIUM_POWER_STR "medium_power\n"
+#define MIN_POWER_STR "min_power\n"
+/* Remember to fix this if adding longer strings */
+#define MAX_POLICY_STRLEN strlen(MAX_PERFORMANCE_STR)
+
+/**
+ * SECTION:igt_pm
+ * @short_description: Power Management related helpers
+ * @title: Power Management
+ * @include: igt.h
+ *
+ * This library provides various helpers to enable power management for,
+ * and in some cases subsequently allow restoring the old behaviour of,
+ * various external components that by default are set up in a way
+ * that interferes with the testing of our power management functionality.
+ */
+/**
+ * igt_pm_enable_audio_runtime_pm:
+ *
+ * We know that if we don't enable audio runtime PM, snd_hda_intel will never
+ * release its power well refcount, and we'll never reach the LPSP sate.
+ * There's no guarantee that it will release the power well if we enable
+ * runtime PM, but at least we can try.
+ *
+ * We don't have any assertions on open since the user may not even have
+ * snd_hda_intel loaded, which is not a problem.
+ */
+void igt_pm_enable_audio_runtime_pm(void)
+{
+ int fd;
+
+ fd = open("/sys/module/snd_hda_intel/parameters/power_save", O_WRONLY);
+ if (fd >= 0) {
+ igt_assert_eq(write(fd, "1\n", 2), 2);
+ close(fd);
+ }
+ fd = open("/sys/bus/pci/devices/0000:00:03.0/power/control", O_WRONLY);
+ if (fd >= 0) {
+ igt_assert_eq(write(fd, "auto\n", 5), 5);
+ close(fd);
+ }
+ /* Give some time for it to react. */
+ sleep(1);
+}
+
+/**
+ * igt_pm_enable_sata_link_power_management:
+ *
+ * Enable the min_power policy for SATA link power management.
+ * Without this we cannot reach deep runtime power states.
+ *
+ * We don't have any assertions on open since the system might not have
+ * a SATA host.
+ *
+ * Returns:
+ * An opaque pointer to the data needed to restore the default values
+ * after the test has terminated, or NULL if SATA link power management
+ * is not supported. This pointer should be freed when no longer used
+ * (typically after having called restore_sata_link_power_management()).
+ */
+int8_t *igt_pm_enable_sata_link_power_management(void)
+{
+ int fd, i;
+ ssize_t len;
+ char *buf;
+ char *file_name;
+ int8_t *link_pm_policies = NULL;
+
+ file_name = malloc(PATH_MAX);
+ buf = malloc(MAX_POLICY_STRLEN + 1);
+
+ for (i = 0; ; i++) {
+ int8_t policy;
+
+ snprintf(file_name, PATH_MAX,
+ "/sys/class/scsi_host/host%d/link_power_management_policy",
+ i);
+
+ fd = open(file_name, O_RDWR);
+ if (fd < 0)
+ break;
+
+ len = read(fd, buf, MAX_POLICY_STRLEN);
+ buf[len] = '\0';
+
+ if (!strncmp(MAX_PERFORMANCE_STR, buf,
+ strlen(MAX_PERFORMANCE_STR)))
+ policy = POLICY_MAX_PERFORMANCE;
+ else if (!strncmp(MEDIUM_POWER_STR, buf,
+ strlen(MEDIUM_POWER_STR)))
+ policy = POLICY_MEDIUM_POWER;
+ else if (!strncmp(MIN_POWER_STR, buf,
+ strlen(MIN_POWER_STR)))
+ policy = POLICY_MIN_POWER;
+ else
+ policy = POLICY_UNKNOWN;
+
+ if (!(i % 256))
+ link_pm_policies = realloc(link_pm_policies,
+ (i / 256 + 1) * 256 + 1);
+
+ link_pm_policies[i] = policy;
+ link_pm_policies[i + 1] = 0;
+
+ /* If the policy is something we don't know about,
+ * don't touch it, since we might potentially break things.
+ * And we obviously don't need to touch anything if the
+ * setting is already correct...
+ */
+ if (policy != POLICY_UNKNOWN &&
+ policy != POLICY_MIN_POWER) {
+ lseek(fd, 0, SEEK_SET);
+ igt_assert_eq(write(fd, MIN_POWER_STR,
+ strlen(MIN_POWER_STR)),
+ strlen(MIN_POWER_STR));
+ }
+ close(fd);
+ }
+ free(buf);
+ free(file_name);
+
+ return link_pm_policies;
+}
+
+/**
+ * igt_pm_restore_sata_link_power_management:
+ * @pm_data: An opaque pointer with saved link PM policies;
+ * If NULL is passed we force enable the "max_performance" policy.
+ *
+ * Restore the link power management policies to the values
+ * prior to enabling min_power.
+ *
+ * Caveat: If the system supports hotplugging and hotplugging takes
+ * place during our testing so that the hosts change numbers
+ * we might restore the settings to the wrong hosts.
+ */
+void igt_pm_restore_sata_link_power_management(int8_t *pm_data)
+{
+ int fd, i;
+ char *file_name;
+
+ /* Disk runtime PM policies. */
+ file_name = malloc(PATH_MAX);
+ for (i = 0; ; i++) {
+ int8_t policy;
+
+ if (!pm_data)
+ policy = POLICY_MAX_PERFORMANCE;
+ else if (pm_data[i] == POLICY_UNKNOWN)
+ continue;
+ else
+ policy = pm_data[i];
+
+ snprintf(file_name, PATH_MAX,
+ "/sys/class/scsi_host/host%d/link_power_management_policy",
+ i);
+
+ fd = open(file_name, O_WRONLY);
+ if (fd < 0)
+ break;
+
+ switch (policy) {
+ default:
+ case POLICY_MAX_PERFORMANCE:
+ igt_assert_eq(write(fd, MAX_PERFORMANCE_STR,
+ strlen(MAX_PERFORMANCE_STR)),
+ strlen(MAX_PERFORMANCE_STR));
+ break;
+
+ case POLICY_MEDIUM_POWER:
+ igt_assert_eq(write(fd, MEDIUM_POWER_STR,
+ strlen(MEDIUM_POWER_STR)),
+ strlen(MEDIUM_POWER_STR));
+ break;
+
+ case POLICY_MIN_POWER:
+ igt_assert_eq(write(fd, MIN_POWER_STR,
+ strlen(MIN_POWER_STR)),
+ strlen(MIN_POWER_STR));
+ break;
+ }
+
+ close(fd);
+ }
+ free(file_name);
+}
diff --git a/lib/igt_pm.h b/lib/igt_pm.h
new file mode 100644
index 000000000000..c14ff1f7a0ef
--- /dev/null
+++ b/lib/igt_pm.h
@@ -0,0 +1,31 @@
+/*
+ * Copyright © 2015 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ */
+
+#ifndef IGT_PM_H
+#define IGT_PM_H
+
+void igt_pm_enable_audio_runtime_pm(void);
+int8_t *igt_pm_enable_sata_link_power_management(void);
+void igt_pm_restore_sata_link_power_management(int8_t *pm_data);
+
+#endif /* IGT_PM_H */
diff --git a/tests/pm_lpsp.c b/tests/pm_lpsp.c
index 257ae1b8b1d9..6cb697ed0bd5 100644
--- a/tests/pm_lpsp.c
+++ b/tests/pm_lpsp.c
@@ -31,29 +31,6 @@
#include <unistd.h>
-/* We know that if we don't enable audio runtime PM, snd_hda_intel will never
- * release its power well refcount, and we'll never reach the LPSP sate. OTOH
- * there's no guarantee that it will release the power well if we enable runtime
- * PM, but at least we can try. We don't have any assertions since the user may
- * not even have snd_hda_intel loaded, which is not a problem. */
-static void disable_audio_runtime_pm(void)
-{
- int fd;
-
- fd = open("/sys/module/snd_hda_intel/parameters/power_save", O_WRONLY);
- if (fd >= 0) {
- igt_assert_eq(write(fd, "1\n", 2), 2);
- close(fd);
- }
- fd = open("/sys/bus/pci/devices/0000:00:03.0/power/control", O_WRONLY);
- if (fd >= 0) {
- igt_assert_eq(write(fd, "auto\n", 5), 5);
- close(fd);
- }
- /* Give some time for it to react. */
- sleep(1);
-}
-
static bool supports_lpsp(uint32_t devid)
{
return IS_HASWELL(devid) || IS_BROADWELL(devid);
@@ -234,7 +211,7 @@ igt_main
drm_connectors[i] = drmModeGetConnectorCurrent(drm_fd,
drm_res->connectors[i]);
- disable_audio_runtime_pm();
+ igt_pm_enable_audio_runtime_pm();
igt_require(supports_lpsp(devid));
diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c
index 55fdb31cb723..13618a1b3246 100644
--- a/tests/pm_rpm.c
+++ b/tests/pm_rpm.c
@@ -1,5 +1,5 @@
/*
- * Copyright © 2013 Intel Corporation
+ * Copyright © 2013, 2015 Intel Corporation
*
* Permission is hereby granted, free of charge, to any person obtaining a
* copy of this software and associated documentation files (the "Software"),
@@ -111,6 +111,8 @@ struct modeset_params lpsp_mode_params;
struct modeset_params non_lpsp_mode_params;
struct modeset_params *default_mode_params;
+static int8_t *pm_data = NULL;
+
/* If the read fails, then the machine doesn't support PC8+ residencies. */
static bool supports_pc8_plus_residencies(void)
{
@@ -670,41 +672,13 @@ static void setup_pc8(void)
has_pc8 = true;
}
-/* If we want to actually reach PC8+ states, we need to properly configure all
- * the devices on the system to allow this. This function will try to setup the
- * things we know we need, but won't scream in case anything fails: we don't
- * know which devices are present on your machine, so we can't really expect
- * anything, just try to help with the more common problems. */
-static void setup_non_graphics_runtime_pm(void)
-{
- int fd, i;
- char *file_name;
-
- /* Disk runtime PM policies. */
- file_name = malloc(PATH_MAX);
- for (i = 0; ; i++) {
-
- snprintf(file_name, PATH_MAX,
- "/sys/class/scsi_host/host%d/link_power_management_policy",
- i);
-
- fd = open(file_name, O_WRONLY);
- if (fd < 0)
- break;
-
- igt_assert(write(fd, "min_power\n", 10) == 10);
- close(fd);
- }
- free(file_name);
-}
-
static void setup_environment(void)
{
drm_fd = drm_open_driver_master(DRIVER_INTEL);
init_mode_set_data(&ms_data);
- setup_non_graphics_runtime_pm();
+ pm_data = igt_pm_enable_sata_link_power_management();
has_runtime_pm = igt_setup_runtime_pm();
setup_pc8();
@@ -713,11 +687,17 @@ static void setup_environment(void)
igt_info("PC8 residency support: %d\n", has_pc8);
igt_require(has_runtime_pm);
+}
+static void restore_environment(void)
+{
+ igt_pm_restore_sata_link_power_management(pm_data);
+ free(pm_data);
}
static void teardown_environment(void)
{
+ restore_environment();
fini_mode_set_data(&ms_data);
drmClose(drm_fd);
close(msr_fd);
--
2.7.0.rc3
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 2+ messages in thread