From: Paulo Zanoni Date: Thu, 21 Nov 2013 19:32:42 +0000 (-0200) Subject: tests/pm_pc8: try harder to reach PC8+ residencies X-Git-Tag: intel-gpu-tools-1.6~301 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=78227101e27471b81af69b54e2de203678243338;p=platform%2Fupstream%2Fintel-gpu-tools.git tests/pm_pc8: try harder to reach PC8+ residencies 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. Another reason for this commit is that I got tired of having to readjust the runtime PM policies every time I reboot my machine. Signed-off-by: Paulo Zanoni --- diff --git a/tests/pm_pc8.c b/tests/pm_pc8.c index 1929884..cabf005 100644 --- a/tests/pm_pc8.c +++ b/tests/pm_pc8.c @@ -659,6 +659,51 @@ 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; + + write(fd, "min_power\n", 10); + close(fd); + } + free(file_name); + + /* Audio runtime PM policies. */ + fd = open("/sys/module/snd_hda_intel/parameters/power_save", O_WRONLY); + if (fd >= 0) { + write(fd, "1\n", 2); + close(fd); + } + fd = open("/sys/bus/pci/devices/0000:00:03.0/power/control", O_WRONLY); + if (fd >= 0) { + write(fd, "auto\n", 5); + close(fd); + } + + /* For some yet unknown reason, it takes some time for the machine to + * reach PC8+ residencies after we do this. I don't really know how much + * we should wait, but this value seems to be working for me. */ + sleep(10); +} + static void setup_environment(void) { drm_fd = drm_open_any(); @@ -666,6 +711,8 @@ static void setup_environment(void) init_mode_set_data(&ms_data); + setup_non_graphics_runtime_pm(); + setup_runtime_pm(); setup_pc8();