[igt-dev] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[igt-dev] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

For this test, we need a laptop running on battery power so that we can
read the battery charge level before and after suspend. And then wait
long enough for a reliable measure.

References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
 tests/i915/gem_exec_suspend.c | 37 ++++++++++++++++++++++++++++++++++-
 1 file changed, 36 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
index af6190ddd..21cfa888a 100644
--- a/tests/i915/gem_exec_suspend.c
+++ b/tests/i915/gem_exec_suspend.c
@@ -27,9 +27,13 @@
  * Exercise executing batches across suspend before checking the results.
  */
 
+#include <fcntl.h>
+#include <unistd.h>
+
 #include "igt.h"
-#include "igt_gt.h"
 #include "igt_dummyload.h"
+#include "igt_gt.h"
+#include "igt_sysfs.h"
 
 #define NOSLEEP 0
 #define IDLE 1
@@ -232,6 +236,32 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 		test_all(fd, flags);
 }
 
+static void power_test(int i915, unsigned engine, unsigned flags)
+{
+	uint64_t before, after;
+	char *status;
+	int dir;
+
+	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
+	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
+
+	status = igt_sysfs_get(dir, "status");
+	igt_require_f(status && strcmp(status, "Discharging") == 0,
+		      "power test needs to be on battery, not mains, power\n");
+	free(status);
+
+	igt_set_autoresume_delay(30 * 60); /* 30 minutes */
+
+	igt_sysfs_scanf(dir, "energy_now", "%"PRIu64, &before);
+	run_test(i915, engine, flags);
+	igt_sysfs_scanf(dir, "energy_now", "%"PRIu64, &after);
+
+	igt_set_autoresume_delay(0);
+
+	igt_info("Power consumed while suspended: %.3fmWh\n",
+		 (before - after) * 1e-3);
+}
+
 igt_main
 {
 	const struct {
@@ -289,6 +319,11 @@ igt_main
 	igt_subtest("hang-S4")
 		run_test(fd, 0, HIBERNATE | HANG);
 
+	igt_subtest("power-S0")
+		power_test(fd, 0, IDLE);
+	igt_subtest("power-S3")
+		power_test(fd, 0, SUSPEND);
+
 	igt_fixture {
 		igt_disallow_hang(fd, hang);
 		close(fd);
-- 
2.24.0.rc1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[Intel-gfx] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

For this test, we need a laptop running on battery power so that we can
read the battery charge level before and after suspend. And then wait
long enough for a reliable measure.

References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
 tests/i915/gem_exec_suspend.c | 37 ++++++++++++++++++++++++++++++++++-
 1 file changed, 36 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
index af6190ddd..21cfa888a 100644
--- a/tests/i915/gem_exec_suspend.c
+++ b/tests/i915/gem_exec_suspend.c
@@ -27,9 +27,13 @@
  * Exercise executing batches across suspend before checking the results.
  */
 
+#include <fcntl.h>
+#include <unistd.h>
+
 #include "igt.h"
-#include "igt_gt.h"
 #include "igt_dummyload.h"
+#include "igt_gt.h"
+#include "igt_sysfs.h"
 
 #define NOSLEEP 0
 #define IDLE 1
@@ -232,6 +236,32 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 		test_all(fd, flags);
 }
 
+static void power_test(int i915, unsigned engine, unsigned flags)
+{
+	uint64_t before, after;
+	char *status;
+	int dir;
+
+	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
+	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
+
+	status = igt_sysfs_get(dir, "status");
+	igt_require_f(status && strcmp(status, "Discharging") == 0,
+		      "power test needs to be on battery, not mains, power\n");
+	free(status);
+
+	igt_set_autoresume_delay(30 * 60); /* 30 minutes */
+
+	igt_sysfs_scanf(dir, "energy_now", "%"PRIu64, &before);
+	run_test(i915, engine, flags);
+	igt_sysfs_scanf(dir, "energy_now", "%"PRIu64, &after);
+
+	igt_set_autoresume_delay(0);
+
+	igt_info("Power consumed while suspended: %.3fmWh\n",
+		 (before - after) * 1e-3);
+}
+
 igt_main
 {
 	const struct {
@@ -289,6 +319,11 @@ igt_main
 	igt_subtest("hang-S4")
 		run_test(fd, 0, HIBERNATE | HANG);
 
+	igt_subtest("power-S0")
+		power_test(fd, 0, IDLE);
+	igt_subtest("power-S3")
+		power_test(fd, 0, SUSPEND);
+
 	igt_fixture {
 		igt_disallow_hang(fd, hang);
 		close(fd);
-- 
2.24.0.rc1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

For this test, we need a laptop running on battery power so that we can
read the battery charge level before and after suspend. And then wait
long enough for a reliable measure.

References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
 tests/i915/gem_exec_suspend.c | 37 ++++++++++++++++++++++++++++++++++-
 1 file changed, 36 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
index af6190ddd..21cfa888a 100644
--- a/tests/i915/gem_exec_suspend.c
+++ b/tests/i915/gem_exec_suspend.c
@@ -27,9 +27,13 @@
  * Exercise executing batches across suspend before checking the results.
  */
 
+#include <fcntl.h>
+#include <unistd.h>
+
 #include "igt.h"
-#include "igt_gt.h"
 #include "igt_dummyload.h"
+#include "igt_gt.h"
+#include "igt_sysfs.h"
 
 #define NOSLEEP 0
 #define IDLE 1
@@ -232,6 +236,32 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 		test_all(fd, flags);
 }
 
+static void power_test(int i915, unsigned engine, unsigned flags)
+{
+	uint64_t before, after;
+	char *status;
+	int dir;
+
+	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
+	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
+
+	status = igt_sysfs_get(dir, "status");
+	igt_require_f(status && strcmp(status, "Discharging") == 0,
+		      "power test needs to be on battery, not mains, power\n");
+	free(status);
+
+	igt_set_autoresume_delay(30 * 60); /* 30 minutes */
+
+	igt_sysfs_scanf(dir, "energy_now", "%"PRIu64, &before);
+	run_test(i915, engine, flags);
+	igt_sysfs_scanf(dir, "energy_now", "%"PRIu64, &after);
+
+	igt_set_autoresume_delay(0);
+
+	igt_info("Power consumed while suspended: %.3fmWh\n",
+		 (before - after) * 1e-3);
+}
+
 igt_main
 {
 	const struct {
@@ -289,6 +319,11 @@ igt_main
 	igt_subtest("hang-S4")
 		run_test(fd, 0, HIBERNATE | HANG);
 
+	igt_subtest("power-S0")
+		power_test(fd, 0, IDLE);
+	igt_subtest("power-S3")
+		power_test(fd, 0, SUSPEND);
+
 	igt_fixture {
 		igt_disallow_hang(fd, hang);
 		close(fd);
-- 
2.24.0.rc1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[igt-dev] ✗ GitLab.Pipeline: warning for i915/gem_exec_suspend: Measure power consumption during suspend

[Patchwork]

== Series Details ==

Series: i915/gem_exec_suspend: Measure power consumption during suspend
URL   : https://patchwork.freedesktop.org/series/68613/
State : warning

== Summary ==

ERROR! This series introduces new undocumented tests:

gem_exec_suspend@power-S0
gem_exec_suspend@power-S3

Can you document them as per the requirement in the [CONTRIBUTING.md]?

[Documentation] has more details on how to do this.

Here are few examples:
https://gitlab.freedesktop.org/drm/igt-gpu-tools/commit/0316695d03aa46108296b27f3982ec93200c7a6e
https://gitlab.freedesktop.org/drm/igt-gpu-tools/commit/443cc658e1e6b492ee17bf4f4d891029eb7a205d

Thanks in advance!

[CONTRIBUTING.md]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/blob/master/CONTRIBUTING.md#L19
[Documentation]: https://drm.pages.freedesktop.org/igt-gpu-tools/igt-gpu-tools-Core.html#igt-describe

Other than that, pipeline status: SUCCESS.

see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/73814 for more details

== Logs ==

For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/73814
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✓ Fi.CI.BAT: success for i915/gem_exec_suspend: Measure power consumption during suspend

[Patchwork]

== Series Details ==

Series: i915/gem_exec_suspend: Measure power consumption during suspend
URL   : https://patchwork.freedesktop.org/series/68613/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_7194 -> IGTPW_3620
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/index.html

Known issues
------------

  Here are the changes found in IGTPW_3620 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       [PASS][1] -> [FAIL][2] ([fdo#111407])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@prime_self_import@basic-with_one_bo:
    - fi-icl-u3:          [PASS][3] -> [DMESG-WARN][4] ([fdo#107724]) +1 similar issue
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/fi-icl-u3/igt@prime_self_import@basic-with_one_bo.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/fi-icl-u3/igt@prime_self_import@basic-with_one_bo.html

  
#### Possible fixes ####

  * igt@gem_exec_create@basic:
    - {fi-tgl-u2}:        [INCOMPLETE][5] ([fdo#111736]) -> [PASS][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/fi-tgl-u2/igt@gem_exec_create@basic.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/fi-tgl-u2/igt@gem_exec_create@basic.html

  * igt@gem_exec_fence@basic-wait-default:
    - fi-icl-u3:          [DMESG-WARN][7] ([fdo#107724]) -> [PASS][8] +1 similar issue
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/fi-icl-u3/igt@gem_exec_fence@basic-wait-default.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/fi-icl-u3/igt@gem_exec_fence@basic-wait-default.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
  [fdo#111736]: https://bugs.freedesktop.org/show_bug.cgi?id=111736


Participating hosts (49 -> 41)
------------------------------

  Missing    (8): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-snb-2520m fi-icl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5244 -> IGTPW_3620

  CI-20190529: 20190529
  CI_DRM_7194: 5d2161b339033a7eb2d11eef17c5780b44edd996 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3620: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/index.html
  IGT_5244: 5eef4d167c00031709751f12bd77a42a1b74ac67 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools



== Testlist changes ==

+igt@gem_exec_suspend@power-s0
+igt@gem_exec_suspend@power-s3

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[Intel-gfx] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

For this test, we need a laptop running on battery power so that we can
read the battery charge level before and after suspend. And then wait
long enough for a reliable measure.

References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
 tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
 1 file changed, 66 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
index af6190ddd..ee0d0719f 100644
--- a/tests/i915/gem_exec_suspend.c
+++ b/tests/i915/gem_exec_suspend.c
@@ -27,9 +27,13 @@
  * Exercise executing batches across suspend before checking the results.
  */
 
+#include <fcntl.h>
+#include <unistd.h>
+
 #include "igt.h"
-#include "igt_gt.h"
 #include "igt_dummyload.h"
+#include "igt_gt.h"
+#include "igt_sysfs.h"
 
 #define NOSLEEP 0
 #define IDLE 1
@@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 		test_all(fd, flags);
 }
 
+struct battery_sample {
+	struct timespec tv;
+	uint64_t charge;
+};
+
+static bool get_power(int dir, struct battery_sample *s)
+{
+	return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
+		igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
+}
+
+static double d_charge(const struct battery_sample *after,
+		       const struct battery_sample *before)
+{
+	return (before->charge - after->charge) * 1e-3; /* mWh */
+}
+
+static double d_time(const struct battery_sample *after,
+		     const struct battery_sample *before)
+{
+	return ((after->tv.tv_sec - before->tv.tv_sec) +
+		(after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
+}
+
+static void power_test(int i915, unsigned engine, unsigned flags)
+{
+	struct battery_sample before, after;
+	char *status;
+	int dir;
+
+	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
+	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
+
+	igt_require_f(get_power(dir, &before),
+		      "power test needs reported energy level\n");
+	free(status);
+
+	status = igt_sysfs_get(dir, "status");
+	igt_require_f(status && strcmp(status, "Discharging") == 0,
+		      "power test needs to be on battery, not mains, power\n");
+	free(status);
+
+	igt_set_autoresume_delay(30 * 60); /* 30 minutes */
+
+	igt_assert(get_power(dir, &before));
+	run_test(i915, engine, flags);
+	igt_assert(get_power(dir, &after));
+
+	igt_set_autoresume_delay(0);
+
+	igt_info("Power consumed while suspended: %.3fmWh\n",
+		 d_charge(&after, &before));
+	igt_info("Discharge rate while suspended: %.3fmW\n",
+		 d_charge(&after, &before) * 3600 / d_time(&after, &before));
+}
+
 igt_main
 {
 	const struct {
@@ -289,6 +349,11 @@ igt_main
 	igt_subtest("hang-S4")
 		run_test(fd, 0, HIBERNATE | HANG);
 
+	igt_subtest("power-S0")
+		power_test(fd, 0, IDLE);
+	igt_subtest("power-S3")
+		power_test(fd, 0, SUSPEND);
+
 	igt_fixture {
 		igt_disallow_hang(fd, hang);
 		close(fd);
-- 
2.24.0.rc1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

For this test, we need a laptop running on battery power so that we can
read the battery charge level before and after suspend. And then wait
long enough for a reliable measure.

References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
 tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
 1 file changed, 66 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
index af6190ddd..ee0d0719f 100644
--- a/tests/i915/gem_exec_suspend.c
+++ b/tests/i915/gem_exec_suspend.c
@@ -27,9 +27,13 @@
  * Exercise executing batches across suspend before checking the results.
  */
 
+#include <fcntl.h>
+#include <unistd.h>
+
 #include "igt.h"
-#include "igt_gt.h"
 #include "igt_dummyload.h"
+#include "igt_gt.h"
+#include "igt_sysfs.h"
 
 #define NOSLEEP 0
 #define IDLE 1
@@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
 		test_all(fd, flags);
 }
 
+struct battery_sample {
+	struct timespec tv;
+	uint64_t charge;
+};
+
+static bool get_power(int dir, struct battery_sample *s)
+{
+	return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
+		igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
+}
+
+static double d_charge(const struct battery_sample *after,
+		       const struct battery_sample *before)
+{
+	return (before->charge - after->charge) * 1e-3; /* mWh */
+}
+
+static double d_time(const struct battery_sample *after,
+		     const struct battery_sample *before)
+{
+	return ((after->tv.tv_sec - before->tv.tv_sec) +
+		(after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
+}
+
+static void power_test(int i915, unsigned engine, unsigned flags)
+{
+	struct battery_sample before, after;
+	char *status;
+	int dir;
+
+	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
+	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
+
+	igt_require_f(get_power(dir, &before),
+		      "power test needs reported energy level\n");
+	free(status);
+
+	status = igt_sysfs_get(dir, "status");
+	igt_require_f(status && strcmp(status, "Discharging") == 0,
+		      "power test needs to be on battery, not mains, power\n");
+	free(status);
+
+	igt_set_autoresume_delay(30 * 60); /* 30 minutes */
+
+	igt_assert(get_power(dir, &before));
+	run_test(i915, engine, flags);
+	igt_assert(get_power(dir, &after));
+
+	igt_set_autoresume_delay(0);
+
+	igt_info("Power consumed while suspended: %.3fmWh\n",
+		 d_charge(&after, &before));
+	igt_info("Discharge rate while suspended: %.3fmW\n",
+		 d_charge(&after, &before) * 3600 / d_time(&after, &before));
+}
+
 igt_main
 {
 	const struct {
@@ -289,6 +349,11 @@ igt_main
 	igt_subtest("hang-S4")
 		run_test(fd, 0, HIBERNATE | HANG);
 
+	igt_subtest("power-S0")
+		power_test(fd, 0, IDLE);
+	igt_subtest("power-S3")
+		power_test(fd, 0, SUSPEND);
+
 	igt_fixture {
 		igt_disallow_hang(fd, hang);
 		close(fd);
-- 
2.24.0.rc1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [igt-dev] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Mika Kuoppala]

Chris Wilson <chris@chris-wilson.co.uk> writes:

> For this test, we need a laptop running on battery power so that we can
> read the battery charge level before and after suspend. And then wait
> long enough for a reliable measure.
>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> ---
>  tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
>  1 file changed, 66 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> index af6190ddd..ee0d0719f 100644
> --- a/tests/i915/gem_exec_suspend.c
> +++ b/tests/i915/gem_exec_suspend.c
> @@ -27,9 +27,13 @@
>   * Exercise executing batches across suspend before checking the results.
>   */
>  
> +#include <fcntl.h>
> +#include <unistd.h>
> +
>  #include "igt.h"
> -#include "igt_gt.h"
>  #include "igt_dummyload.h"
> +#include "igt_gt.h"
> +#include "igt_sysfs.h"
>  
>  #define NOSLEEP 0
>  #define IDLE 1
> @@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
>  		test_all(fd, flags);
>  }
>  
> +struct battery_sample {
> +	struct timespec tv;
> +	uint64_t charge;
> +};
> +
> +static bool get_power(int dir, struct battery_sample *s)
> +{
> +	return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
> +		igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
> +}
> +
> +static double d_charge(const struct battery_sample *after,
> +		       const struct battery_sample *before)
> +{
> +	return (before->charge - after->charge) * 1e-3; /* mWh */
> +}
> +
> +static double d_time(const struct battery_sample *after,
> +		     const struct battery_sample *before)
> +{
> +	return ((after->tv.tv_sec - before->tv.tv_sec) +
> +		(after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
> +}
> +
> +static void power_test(int i915, unsigned engine, unsigned flags)
> +{
> +	struct battery_sample before, after;
> +	char *status;
> +	int dir;
> +
> +	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
> +	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
> +
> +	igt_require_f(get_power(dir, &before),
> +		      "power test needs reported energy level\n");
> +	free(status);
This looks bogus.

> +
> +	status = igt_sysfs_get(dir, "status");
> +	igt_require_f(status && strcmp(status, "Discharging") == 0,
> +		      "power test needs to be on battery, not mains, power\n");
> +	free(status);
> +
> +	igt_set_autoresume_delay(30 * 60); /* 30 minutes */

As you said, this is quite a long time. Even with the suspend and modern
laptop it should show meaningful data quicker. But we need moar
data on multiple specimens to find a good spot.

> +
> +	igt_assert(get_power(dir, &before));
> +	run_test(i915, engine, flags);
> +	igt_assert(get_power(dir, &after));
> +
> +	igt_set_autoresume_delay(0);
> +
> +	igt_info("Power consumed while suspended: %.3fmWh\n",
> +		 d_charge(&after, &before));
> +	igt_info("Discharge rate while suspended: %.3fmW\n",
> +		 d_charge(&after, &before) * 3600 / d_time(&after, &before));

I dunno what is the go-to unit in here, but mA would be
as comparable and time independant. Perhaps show both.

There are lot of potential in here to automate against pm
regressions. Plugging this into the apc and gathering
baseline consumptions.


This is a neat start and I aimed at t-b too.
Didn't happen so I can offer only,

Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

> +}
> +
>  igt_main
>  {
>  	const struct {
> @@ -289,6 +349,11 @@ igt_main
>  	igt_subtest("hang-S4")
>  		run_test(fd, 0, HIBERNATE | HANG);
>  
> +	igt_subtest("power-S0")
> +		power_test(fd, 0, IDLE);
> +	igt_subtest("power-S3")
> +		power_test(fd, 0, SUSPEND);
> +
>  	igt_fixture {
>  		igt_disallow_hang(fd, hang);
>  		close(fd);
> -- 
> 2.24.0.rc1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [Intel-gfx] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Mika Kuoppala]

Chris Wilson <chris@chris-wilson.co.uk> writes:

> For this test, we need a laptop running on battery power so that we can
> read the battery charge level before and after suspend. And then wait
> long enough for a reliable measure.
>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> ---
>  tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
>  1 file changed, 66 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> index af6190ddd..ee0d0719f 100644
> --- a/tests/i915/gem_exec_suspend.c
> +++ b/tests/i915/gem_exec_suspend.c
> @@ -27,9 +27,13 @@
>   * Exercise executing batches across suspend before checking the results.
>   */
>  
> +#include <fcntl.h>
> +#include <unistd.h>
> +
>  #include "igt.h"
> -#include "igt_gt.h"
>  #include "igt_dummyload.h"
> +#include "igt_gt.h"
> +#include "igt_sysfs.h"
>  
>  #define NOSLEEP 0
>  #define IDLE 1
> @@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
>  		test_all(fd, flags);
>  }
>  
> +struct battery_sample {
> +	struct timespec tv;
> +	uint64_t charge;
> +};
> +
> +static bool get_power(int dir, struct battery_sample *s)
> +{
> +	return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
> +		igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
> +}
> +
> +static double d_charge(const struct battery_sample *after,
> +		       const struct battery_sample *before)
> +{
> +	return (before->charge - after->charge) * 1e-3; /* mWh */
> +}
> +
> +static double d_time(const struct battery_sample *after,
> +		     const struct battery_sample *before)
> +{
> +	return ((after->tv.tv_sec - before->tv.tv_sec) +
> +		(after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
> +}
> +
> +static void power_test(int i915, unsigned engine, unsigned flags)
> +{
> +	struct battery_sample before, after;
> +	char *status;
> +	int dir;
> +
> +	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
> +	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
> +
> +	igt_require_f(get_power(dir, &before),
> +		      "power test needs reported energy level\n");
> +	free(status);
This looks bogus.

> +
> +	status = igt_sysfs_get(dir, "status");
> +	igt_require_f(status && strcmp(status, "Discharging") == 0,
> +		      "power test needs to be on battery, not mains, power\n");
> +	free(status);
> +
> +	igt_set_autoresume_delay(30 * 60); /* 30 minutes */

As you said, this is quite a long time. Even with the suspend and modern
laptop it should show meaningful data quicker. But we need moar
data on multiple specimens to find a good spot.

> +
> +	igt_assert(get_power(dir, &before));
> +	run_test(i915, engine, flags);
> +	igt_assert(get_power(dir, &after));
> +
> +	igt_set_autoresume_delay(0);
> +
> +	igt_info("Power consumed while suspended: %.3fmWh\n",
> +		 d_charge(&after, &before));
> +	igt_info("Discharge rate while suspended: %.3fmW\n",
> +		 d_charge(&after, &before) * 3600 / d_time(&after, &before));

I dunno what is the go-to unit in here, but mA would be
as comparable and time independant. Perhaps show both.

There are lot of potential in here to automate against pm
regressions. Plugging this into the apc and gathering
baseline consumptions.


This is a neat start and I aimed at t-b too.
Didn't happen so I can offer only,

Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

> +}
> +
>  igt_main
>  {
>  	const struct {
> @@ -289,6 +349,11 @@ igt_main
>  	igt_subtest("hang-S4")
>  		run_test(fd, 0, HIBERNATE | HANG);
>  
> +	igt_subtest("power-S0")
> +		power_test(fd, 0, IDLE);
> +	igt_subtest("power-S3")
> +		power_test(fd, 0, SUSPEND);
> +
>  	igt_fixture {
>  		igt_disallow_hang(fd, hang);
>  		close(fd);
> -- 
> 2.24.0.rc1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Mika Kuoppala]

Chris Wilson <chris@chris-wilson.co.uk> writes:

> For this test, we need a laptop running on battery power so that we can
> read the battery charge level before and after suspend. And then wait
> long enough for a reliable measure.
>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> ---
>  tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
>  1 file changed, 66 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> index af6190ddd..ee0d0719f 100644
> --- a/tests/i915/gem_exec_suspend.c
> +++ b/tests/i915/gem_exec_suspend.c
> @@ -27,9 +27,13 @@
>   * Exercise executing batches across suspend before checking the results.
>   */
>  
> +#include <fcntl.h>
> +#include <unistd.h>
> +
>  #include "igt.h"
> -#include "igt_gt.h"
>  #include "igt_dummyload.h"
> +#include "igt_gt.h"
> +#include "igt_sysfs.h"
>  
>  #define NOSLEEP 0
>  #define IDLE 1
> @@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
>  		test_all(fd, flags);
>  }
>  
> +struct battery_sample {
> +	struct timespec tv;
> +	uint64_t charge;
> +};
> +
> +static bool get_power(int dir, struct battery_sample *s)
> +{
> +	return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
> +		igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
> +}
> +
> +static double d_charge(const struct battery_sample *after,
> +		       const struct battery_sample *before)
> +{
> +	return (before->charge - after->charge) * 1e-3; /* mWh */
> +}
> +
> +static double d_time(const struct battery_sample *after,
> +		     const struct battery_sample *before)
> +{
> +	return ((after->tv.tv_sec - before->tv.tv_sec) +
> +		(after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
> +}
> +
> +static void power_test(int i915, unsigned engine, unsigned flags)
> +{
> +	struct battery_sample before, after;
> +	char *status;
> +	int dir;
> +
> +	dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
> +	igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
> +
> +	igt_require_f(get_power(dir, &before),
> +		      "power test needs reported energy level\n");
> +	free(status);
This looks bogus.

> +
> +	status = igt_sysfs_get(dir, "status");
> +	igt_require_f(status && strcmp(status, "Discharging") == 0,
> +		      "power test needs to be on battery, not mains, power\n");
> +	free(status);
> +
> +	igt_set_autoresume_delay(30 * 60); /* 30 minutes */

As you said, this is quite a long time. Even with the suspend and modern
laptop it should show meaningful data quicker. But we need moar
data on multiple specimens to find a good spot.

> +
> +	igt_assert(get_power(dir, &before));
> +	run_test(i915, engine, flags);
> +	igt_assert(get_power(dir, &after));
> +
> +	igt_set_autoresume_delay(0);
> +
> +	igt_info("Power consumed while suspended: %.3fmWh\n",
> +		 d_charge(&after, &before));
> +	igt_info("Discharge rate while suspended: %.3fmW\n",
> +		 d_charge(&after, &before) * 3600 / d_time(&after, &before));

I dunno what is the go-to unit in here, but mA would be
as comparable and time independant. Perhaps show both.

There are lot of potential in here to automate against pm
regressions. Plugging this into the apc and gathering
baseline consumptions.


This is a neat start and I aimed at t-b too.
Didn't happen so I can offer only,

Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

> +}
> +
>  igt_main
>  {
>  	const struct {
> @@ -289,6 +349,11 @@ igt_main
>  	igt_subtest("hang-S4")
>  		run_test(fd, 0, HIBERNATE | HANG);
>  
> +	igt_subtest("power-S0")
> +		power_test(fd, 0, IDLE);
> +	igt_subtest("power-S3")
> +		power_test(fd, 0, SUSPEND);
> +
>  	igt_fixture {
>  		igt_disallow_hang(fd, hang);
>  		close(fd);
> -- 
> 2.24.0.rc1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [igt-dev] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

Quoting Mika Kuoppala (2019-10-30 12:30:47)
> Chris Wilson <chris@chris-wilson.co.uk> writes:
> 
> > For this test, we need a laptop running on battery power so that we can
> > read the battery charge level before and after suspend. And then wait
> > long enough for a reliable measure.
> >
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> > ---
> >  tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
> >  1 file changed, 66 insertions(+), 1 deletion(-)
> >
> > diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> > index af6190ddd..ee0d0719f 100644
> > --- a/tests/i915/gem_exec_suspend.c
> > +++ b/tests/i915/gem_exec_suspend.c
> > @@ -27,9 +27,13 @@
> >   * Exercise executing batches across suspend before checking the results.
> >   */
> >  
> > +#include <fcntl.h>
> > +#include <unistd.h>
> > +
> >  #include "igt.h"
> > -#include "igt_gt.h"
> >  #include "igt_dummyload.h"
> > +#include "igt_gt.h"
> > +#include "igt_sysfs.h"
> >  
> >  #define NOSLEEP 0
> >  #define IDLE 1
> > @@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
> >               test_all(fd, flags);
> >  }
> >  
> > +struct battery_sample {
> > +     struct timespec tv;
> > +     uint64_t charge;
> > +};
> > +
> > +static bool get_power(int dir, struct battery_sample *s)
> > +{
> > +     return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
> > +             igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
> > +}
> > +
> > +static double d_charge(const struct battery_sample *after,
> > +                    const struct battery_sample *before)
> > +{
> > +     return (before->charge - after->charge) * 1e-3; /* mWh */
> > +}
> > +
> > +static double d_time(const struct battery_sample *after,
> > +                  const struct battery_sample *before)
> > +{
> > +     return ((after->tv.tv_sec - before->tv.tv_sec) +
> > +             (after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
> > +}
> > +
> > +static void power_test(int i915, unsigned engine, unsigned flags)
> > +{
> > +     struct battery_sample before, after;
> > +     char *status;
> > +     int dir;
> > +
> > +     dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
> > +     igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
> > +
> > +     igt_require_f(get_power(dir, &before),
> > +                   "power test needs reported energy level\n");
> > +     free(status);
> This looks bogus.
> 
> > +
> > +     status = igt_sysfs_get(dir, "status");
> > +     igt_require_f(status && strcmp(status, "Discharging") == 0,
> > +                   "power test needs to be on battery, not mains, power\n");
> > +     free(status);
> > +
> > +     igt_set_autoresume_delay(30 * 60); /* 30 minutes */
> 
> As you said, this is quite a long time. Even with the suspend and modern
> laptop it should show meaningful data quicker. But we need moar
> data on multiple specimens to find a good spot.
> 
> > +
> > +     igt_assert(get_power(dir, &before));
> > +     run_test(i915, engine, flags);
> > +     igt_assert(get_power(dir, &after));
> > +
> > +     igt_set_autoresume_delay(0);
> > +
> > +     igt_info("Power consumed while suspended: %.3fmWh\n",
> > +              d_charge(&after, &before));
> > +     igt_info("Discharge rate while suspended: %.3fmW\n",
> > +              d_charge(&after, &before) * 3600 / d_time(&after, &before));
> 
> I dunno what is the go-to unit in here, but mA would be
> as comparable and time independant. Perhaps show both.

/sys/devices/power_supply/BAT0/ gives mWh and voltage iirc. I went with
mWh/mW as that seems the easiest with less modelling required.

> There are lot of potential in here to automate against pm
> regressions. Plugging this into the apc and gathering
> baseline consumptions.

Yup, fingers crossed that may happen. :)

> This is a neat start and I aimed at t-b too.
> Didn't happen so I can offer only,
> 
> Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

Ta.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [Intel-gfx] [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

Quoting Mika Kuoppala (2019-10-30 12:30:47)
> Chris Wilson <chris@chris-wilson.co.uk> writes:
> 
> > For this test, we need a laptop running on battery power so that we can
> > read the battery charge level before and after suspend. And then wait
> > long enough for a reliable measure.
> >
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> > ---
> >  tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
> >  1 file changed, 66 insertions(+), 1 deletion(-)
> >
> > diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> > index af6190ddd..ee0d0719f 100644
> > --- a/tests/i915/gem_exec_suspend.c
> > +++ b/tests/i915/gem_exec_suspend.c
> > @@ -27,9 +27,13 @@
> >   * Exercise executing batches across suspend before checking the results.
> >   */
> >  
> > +#include <fcntl.h>
> > +#include <unistd.h>
> > +
> >  #include "igt.h"
> > -#include "igt_gt.h"
> >  #include "igt_dummyload.h"
> > +#include "igt_gt.h"
> > +#include "igt_sysfs.h"
> >  
> >  #define NOSLEEP 0
> >  #define IDLE 1
> > @@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
> >               test_all(fd, flags);
> >  }
> >  
> > +struct battery_sample {
> > +     struct timespec tv;
> > +     uint64_t charge;
> > +};
> > +
> > +static bool get_power(int dir, struct battery_sample *s)
> > +{
> > +     return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
> > +             igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
> > +}
> > +
> > +static double d_charge(const struct battery_sample *after,
> > +                    const struct battery_sample *before)
> > +{
> > +     return (before->charge - after->charge) * 1e-3; /* mWh */
> > +}
> > +
> > +static double d_time(const struct battery_sample *after,
> > +                  const struct battery_sample *before)
> > +{
> > +     return ((after->tv.tv_sec - before->tv.tv_sec) +
> > +             (after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
> > +}
> > +
> > +static void power_test(int i915, unsigned engine, unsigned flags)
> > +{
> > +     struct battery_sample before, after;
> > +     char *status;
> > +     int dir;
> > +
> > +     dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
> > +     igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
> > +
> > +     igt_require_f(get_power(dir, &before),
> > +                   "power test needs reported energy level\n");
> > +     free(status);
> This looks bogus.
> 
> > +
> > +     status = igt_sysfs_get(dir, "status");
> > +     igt_require_f(status && strcmp(status, "Discharging") == 0,
> > +                   "power test needs to be on battery, not mains, power\n");
> > +     free(status);
> > +
> > +     igt_set_autoresume_delay(30 * 60); /* 30 minutes */
> 
> As you said, this is quite a long time. Even with the suspend and modern
> laptop it should show meaningful data quicker. But we need moar
> data on multiple specimens to find a good spot.
> 
> > +
> > +     igt_assert(get_power(dir, &before));
> > +     run_test(i915, engine, flags);
> > +     igt_assert(get_power(dir, &after));
> > +
> > +     igt_set_autoresume_delay(0);
> > +
> > +     igt_info("Power consumed while suspended: %.3fmWh\n",
> > +              d_charge(&after, &before));
> > +     igt_info("Discharge rate while suspended: %.3fmW\n",
> > +              d_charge(&after, &before) * 3600 / d_time(&after, &before));
> 
> I dunno what is the go-to unit in here, but mA would be
> as comparable and time independant. Perhaps show both.

/sys/devices/power_supply/BAT0/ gives mWh and voltage iirc. I went with
mWh/mW as that seems the easiest with less modelling required.

> There are lot of potential in here to automate against pm
> regressions. Plugging this into the apc and gathering
> baseline consumptions.

Yup, fingers crossed that may happen. :)

> This is a neat start and I aimed at t-b too.
> Didn't happen so I can offer only,
> 
> Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

Ta.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH i-g-t] i915/gem_exec_suspend: Measure power consumption during suspend

[Chris Wilson]

Quoting Mika Kuoppala (2019-10-30 12:30:47)
> Chris Wilson <chris@chris-wilson.co.uk> writes:
> 
> > For this test, we need a laptop running on battery power so that we can
> > read the battery charge level before and after suspend. And then wait
> > long enough for a reliable measure.
> >
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=111909
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> > ---
> >  tests/i915/gem_exec_suspend.c | 67 ++++++++++++++++++++++++++++++++++-
> >  1 file changed, 66 insertions(+), 1 deletion(-)
> >
> > diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
> > index af6190ddd..ee0d0719f 100644
> > --- a/tests/i915/gem_exec_suspend.c
> > +++ b/tests/i915/gem_exec_suspend.c
> > @@ -27,9 +27,13 @@
> >   * Exercise executing batches across suspend before checking the results.
> >   */
> >  
> > +#include <fcntl.h>
> > +#include <unistd.h>
> > +
> >  #include "igt.h"
> > -#include "igt_gt.h"
> >  #include "igt_dummyload.h"
> > +#include "igt_gt.h"
> > +#include "igt_sysfs.h"
> >  
> >  #define NOSLEEP 0
> >  #define IDLE 1
> > @@ -232,6 +236,62 @@ static void run_test(int fd, unsigned engine, unsigned flags)
> >               test_all(fd, flags);
> >  }
> >  
> > +struct battery_sample {
> > +     struct timespec tv;
> > +     uint64_t charge;
> > +};
> > +
> > +static bool get_power(int dir, struct battery_sample *s)
> > +{
> > +     return (clock_gettime(CLOCK_REALTIME, &s->tv) == 0 &&
> > +             igt_sysfs_scanf(dir, "charge_now", "%"PRIu64, &s->charge) == 1);
> > +}
> > +
> > +static double d_charge(const struct battery_sample *after,
> > +                    const struct battery_sample *before)
> > +{
> > +     return (before->charge - after->charge) * 1e-3; /* mWh */
> > +}
> > +
> > +static double d_time(const struct battery_sample *after,
> > +                  const struct battery_sample *before)
> > +{
> > +     return ((after->tv.tv_sec - before->tv.tv_sec) +
> > +             (after->tv.tv_nsec - before->tv.tv_nsec) * 1e-9);
> > +}
> > +
> > +static void power_test(int i915, unsigned engine, unsigned flags)
> > +{
> > +     struct battery_sample before, after;
> > +     char *status;
> > +     int dir;
> > +
> > +     dir = open("/sys/class/power_supply/BAT0", O_RDONLY);
> > +     igt_require_f(dir != -1, "/sys/class/power_supply/BAT0 not available\n");
> > +
> > +     igt_require_f(get_power(dir, &before),
> > +                   "power test needs reported energy level\n");
> > +     free(status);
> This looks bogus.
> 
> > +
> > +     status = igt_sysfs_get(dir, "status");
> > +     igt_require_f(status && strcmp(status, "Discharging") == 0,
> > +                   "power test needs to be on battery, not mains, power\n");
> > +     free(status);
> > +
> > +     igt_set_autoresume_delay(30 * 60); /* 30 minutes */
> 
> As you said, this is quite a long time. Even with the suspend and modern
> laptop it should show meaningful data quicker. But we need moar
> data on multiple specimens to find a good spot.
> 
> > +
> > +     igt_assert(get_power(dir, &before));
> > +     run_test(i915, engine, flags);
> > +     igt_assert(get_power(dir, &after));
> > +
> > +     igt_set_autoresume_delay(0);
> > +
> > +     igt_info("Power consumed while suspended: %.3fmWh\n",
> > +              d_charge(&after, &before));
> > +     igt_info("Discharge rate while suspended: %.3fmW\n",
> > +              d_charge(&after, &before) * 3600 / d_time(&after, &before));
> 
> I dunno what is the go-to unit in here, but mA would be
> as comparable and time independant. Perhaps show both.

/sys/devices/power_supply/BAT0/ gives mWh and voltage iirc. I went with
mWh/mW as that seems the easiest with less modelling required.

> There are lot of potential in here to automate against pm
> regressions. Plugging this into the apc and gathering
> baseline consumptions.

Yup, fingers crossed that may happen. :)

> This is a neat start and I aimed at t-b too.
> Didn't happen so I can offer only,
> 
> Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>

Ta.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[igt-dev] ✗ GitLab.Pipeline: warning for i915/gem_exec_suspend: Measure power consumption during suspend (rev2)

[Patchwork]

== Series Details ==

Series: i915/gem_exec_suspend: Measure power consumption during suspend (rev2)
URL   : https://patchwork.freedesktop.org/series/68613/
State : warning

== Summary ==

ERROR! This series introduces new undocumented tests:

gem_exec_suspend@power-S0
gem_exec_suspend@power-S3

Can you document them as per the requirement in the [CONTRIBUTING.md]?

[Documentation] has more details on how to do this.

Here are few examples:
https://gitlab.freedesktop.org/drm/igt-gpu-tools/commit/0316695d03aa46108296b27f3982ec93200c7a6e
https://gitlab.freedesktop.org/drm/igt-gpu-tools/commit/443cc658e1e6b492ee17bf4f4d891029eb7a205d

Thanks in advance!

[CONTRIBUTING.md]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/blob/master/CONTRIBUTING.md#L19
[Documentation]: https://drm.pages.freedesktop.org/igt-gpu-tools/igt-gpu-tools-Core.html#igt-describe

Other than that, pipeline status: SUCCESS.

see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/73819 for more details

== Logs ==

For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/73819
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✗ Fi.CI.BAT: failure for i915/gem_exec_suspend: Measure power consumption during suspend (rev2)

[Patchwork]

== Series Details ==

Series: i915/gem_exec_suspend: Measure power consumption during suspend (rev2)
URL   : https://patchwork.freedesktop.org/series/68613/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_7195 -> IGTPW_3621
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_3621 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_3621, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_3621:

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live_blt:
    - fi-bsw-n3050:       NOTRUN -> [DMESG-FAIL][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-bsw-n3050/igt@i915_selftest@live_blt.html

  
Known issues
------------

  Here are the changes found in IGTPW_3621 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_mmap@basic:
    - fi-icl-u3:          [PASS][2] -> [DMESG-WARN][3] ([fdo#107724])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7195/fi-icl-u3/igt@gem_mmap@basic.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-icl-u3/igt@gem_mmap@basic.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       [PASS][4] -> [FAIL][5] ([fdo#111407])
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7195/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@kms_frontbuffer_tracking@basic:
    - fi-icl-u3:          [PASS][6] -> [FAIL][7] ([fdo#103167])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7195/fi-icl-u3/igt@kms_frontbuffer_tracking@basic.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-icl-u3/igt@kms_frontbuffer_tracking@basic.html

  
#### Possible fixes ####

  * igt@gem_ctx_switch@rcs0:
    - fi-bxt-dsi:         [INCOMPLETE][8] ([fdo#103927]) -> [PASS][9]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7195/fi-bxt-dsi/igt@gem_ctx_switch@rcs0.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-bxt-dsi/igt@gem_ctx_switch@rcs0.html

  * igt@gem_exec_fence@basic-wait-default:
    - fi-icl-u3:          [DMESG-WARN][10] ([fdo#107724]) -> [PASS][11] +2 similar issues
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7195/fi-icl-u3/igt@gem_exec_fence@basic-wait-default.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-icl-u3/igt@gem_exec_fence@basic-wait-default.html

  * igt@kms_chamelium@hdmi-crc-fast:
    - fi-icl-u2:          [FAIL][12] ([fdo#109635 ]) -> [PASS][13]
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7195/fi-icl-u2/igt@kms_chamelium@hdmi-crc-fast.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/fi-icl-u2/igt@kms_chamelium@hdmi-crc-fast.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#109635 ]: https://bugs.freedesktop.org/show_bug.cgi?id=109635 
  [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
  [fdo#111747]: https://bugs.freedesktop.org/show_bug.cgi?id=111747


Participating hosts (49 -> 41)
------------------------------

  Missing    (8): fi-ilk-m540 fi-hsw-4200u fi-tgl-u2 fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5244 -> IGTPW_3621

  CI-20190529: 20190529
  CI_DRM_7195: 6be7daade9b9cf1c8e7caa865b840e5e544e5ce3 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3621: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/index.html
  IGT_5244: 5eef4d167c00031709751f12bd77a42a1b74ac67 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools



== Testlist changes ==

+igt@gem_exec_suspend@power-s0
+igt@gem_exec_suspend@power-s3

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3621/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✓ Fi.CI.IGT: success for i915/gem_exec_suspend: Measure power consumption during suspend

[Patchwork]

== Series Details ==

Series: i915/gem_exec_suspend: Measure power consumption during suspend
URL   : https://patchwork.freedesktop.org/series/68613/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_7194_full -> IGTPW_3620_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/index.html

Known issues
------------

  Here are the changes found in IGTPW_3620_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_isolation@vcs1-dirty-create:
    - shard-iclb:         [PASS][1] -> [SKIP][2] ([fdo#109276] / [fdo#112080]) +1 similar issue
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb4/igt@gem_ctx_isolation@vcs1-dirty-create.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb3/igt@gem_ctx_isolation@vcs1-dirty-create.html

  * igt@gem_eio@unwedge-stress:
    - shard-snb:          [PASS][3] -> [FAIL][4] ([fdo#109661])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-snb1/igt@gem_eio@unwedge-stress.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-snb2/igt@gem_eio@unwedge-stress.html

  * igt@gem_exec_parallel@vcs1-contexts:
    - shard-iclb:         [PASS][5] -> [SKIP][6] ([fdo#112080]) +6 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb4/igt@gem_exec_parallel@vcs1-contexts.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb5/igt@gem_exec_parallel@vcs1-contexts.html

  * igt@gem_exec_schedule@preempt-other-chain-bsd:
    - shard-iclb:         [PASS][7] -> [SKIP][8] ([fdo#112146]) +4 similar issues
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb5/igt@gem_exec_schedule@preempt-other-chain-bsd.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb2/igt@gem_exec_schedule@preempt-other-chain-bsd.html

  * igt@gem_exec_suspend@basic-s3:
    - shard-kbl:          [PASS][9] -> [DMESG-WARN][10] ([fdo#108566]) +3 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-kbl4/igt@gem_exec_suspend@basic-s3.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-kbl1/igt@gem_exec_suspend@basic-s3.html

  * igt@gem_userptr_blits@map-fixed-invalidate-busy:
    - shard-snb:          [PASS][11] -> [DMESG-WARN][12] ([fdo#111870]) +1 similar issue
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-snb2/igt@gem_userptr_blits@map-fixed-invalidate-busy.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-snb4/igt@gem_userptr_blits@map-fixed-invalidate-busy.html

  * igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy:
    - shard-hsw:          [PASS][13] -> [DMESG-WARN][14] ([fdo#111870]) +2 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-hsw2/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-hsw6/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render:
    - shard-iclb:         [PASS][15] -> [FAIL][16] ([fdo#103167]) +5 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-msflip-blt:
    - shard-glk:          [PASS][17] -> [FAIL][18] ([fdo#103167])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-glk7/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-msflip-blt.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-glk9/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-msflip-blt.html

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes:
    - shard-apl:          [PASS][19] -> [DMESG-WARN][20] ([fdo#108566]) +2 similar issues
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-apl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-apl1/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html

  * igt@kms_psr2_su@page_flip:
    - shard-iclb:         [PASS][21] -> [SKIP][22] ([fdo#109642] / [fdo#111068])
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb2/igt@kms_psr2_su@page_flip.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb5/igt@kms_psr2_su@page_flip.html

  * igt@kms_psr@psr2_primary_page_flip:
    - shard-iclb:         [PASS][23] -> [SKIP][24] ([fdo#109441]) +2 similar issues
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb4/igt@kms_psr@psr2_primary_page_flip.html

  * igt@prime_vgem@fence-wait-bsd2:
    - shard-iclb:         [PASS][25] -> [SKIP][26] ([fdo#109276]) +11 similar issues
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb2/igt@prime_vgem@fence-wait-bsd2.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb6/igt@prime_vgem@fence-wait-bsd2.html

  
#### Possible fixes ####

  * igt@gem_busy@busy-vcs1:
    - shard-iclb:         [SKIP][27] ([fdo#112080]) -> [PASS][28] +9 similar issues
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb8/igt@gem_busy@busy-vcs1.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb2/igt@gem_busy@busy-vcs1.html

  * {igt@gem_ctx_exec@basic-nohangcheck}:
    - shard-glk:          [FAIL][29] ([fdo#112142]) -> [PASS][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-glk7/igt@gem_ctx_exec@basic-nohangcheck.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-glk6/igt@gem_ctx_exec@basic-nohangcheck.html
    - {shard-tglb}:       [FAIL][31] ([fdo#112142]) -> [PASS][32]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-tglb7/igt@gem_ctx_exec@basic-nohangcheck.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-tglb7/igt@gem_ctx_exec@basic-nohangcheck.html
    - shard-iclb:         [FAIL][33] ([fdo#112142]) -> [PASS][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb1/igt@gem_ctx_exec@basic-nohangcheck.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb2/igt@gem_ctx_exec@basic-nohangcheck.html
    - shard-kbl:          [FAIL][35] ([fdo#112142]) -> [PASS][36]
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-kbl4/igt@gem_ctx_exec@basic-nohangcheck.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-kbl2/igt@gem_ctx_exec@basic-nohangcheck.html
    - shard-apl:          [FAIL][37] ([fdo#112142]) -> [PASS][38]
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-apl4/igt@gem_ctx_exec@basic-nohangcheck.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-apl7/igt@gem_ctx_exec@basic-nohangcheck.html

  * igt@gem_exec_schedule@preempt-queue-bsd2:
    - shard-iclb:         [SKIP][39] ([fdo#109276]) -> [PASS][40] +11 similar issues
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb7/igt@gem_exec_schedule@preempt-queue-bsd2.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb1/igt@gem_exec_schedule@preempt-queue-bsd2.html

  * igt@gem_exec_schedule@preemptive-hang-bsd:
    - shard-iclb:         [SKIP][41] ([fdo#112146]) -> [PASS][42] +5 similar issues
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb1/igt@gem_exec_schedule@preemptive-hang-bsd.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb8/igt@gem_exec_schedule@preemptive-hang-bsd.html

  * igt@gem_persistent_relocs@forked-interruptible-thrashing:
    - shard-kbl:          [FAIL][43] ([fdo#112037]) -> [PASS][44]
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-kbl1/igt@gem_persistent_relocs@forked-interruptible-thrashing.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-kbl3/igt@gem_persistent_relocs@forked-interruptible-thrashing.html

  * igt@gem_sync@basic-all:
    - {shard-tglb}:       [INCOMPLETE][45] ([fdo#111647]) -> [PASS][46] +1 similar issue
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-tglb6/igt@gem_sync@basic-all.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-tglb1/igt@gem_sync@basic-all.html

  * igt@gem_userptr_blits@sync-unmap:
    - shard-hsw:          [DMESG-WARN][47] ([fdo#111870]) -> [PASS][48]
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-hsw6/igt@gem_userptr_blits@sync-unmap.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-hsw2/igt@gem_userptr_blits@sync-unmap.html

  * igt@gem_userptr_blits@sync-unmap-cycles:
    - shard-snb:          [DMESG-WARN][49] ([fdo#111870]) -> [PASS][50] +1 similar issue
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-snb2/igt@gem_userptr_blits@sync-unmap-cycles.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-snb1/igt@gem_userptr_blits@sync-unmap-cycles.html

  * igt@gem_workarounds@suspend-resume-context:
    - shard-apl:          [DMESG-WARN][51] ([fdo#108566]) -> [PASS][52] +1 similar issue
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-apl1/igt@gem_workarounds@suspend-resume-context.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-apl1/igt@gem_workarounds@suspend-resume-context.html
    - {shard-tglb}:       [INCOMPLETE][53] ([fdo#111832] / [fdo#111850]) -> [PASS][54] +2 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-tglb4/igt@gem_workarounds@suspend-resume-context.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-tglb6/igt@gem_workarounds@suspend-resume-context.html

  * igt@i915_pm_rc6_residency@rc6-accuracy:
    - shard-snb:          [SKIP][55] ([fdo#109271]) -> [PASS][56]
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-snb6/igt@i915_pm_rc6_residency@rc6-accuracy.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-snb7/igt@i915_pm_rc6_residency@rc6-accuracy.html

  * igt@i915_selftest@live_hangcheck:
    - shard-hsw:          [DMESG-FAIL][57] ([fdo#111991]) -> [PASS][58]
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-hsw6/igt@i915_selftest@live_hangcheck.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-hsw1/igt@i915_selftest@live_hangcheck.html

  * igt@i915_selftest@mock_requests:
    - shard-glk:          [DMESG-WARN][59] ([fdo#112158]) -> [PASS][60]
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-glk3/igt@i915_selftest@mock_requests.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-glk8/igt@i915_selftest@mock_requests.html

  * igt@kms_atomic_interruptible@atomic-setmode:
    - shard-iclb:         [INCOMPLETE][61] ([fdo#107713]) -> [PASS][62]
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb7/igt@kms_atomic_interruptible@atomic-setmode.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb4/igt@kms_atomic_interruptible@atomic-setmode.html

  * igt@kms_busy@extended-modeset-hang-oldfb-render-d:
    - {shard-tglb}:       [INCOMPLETE][63] ([fdo#111747]) -> [PASS][64]
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-tglb3/igt@kms_busy@extended-modeset-hang-oldfb-render-d.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-tglb8/igt@kms_busy@extended-modeset-hang-oldfb-render-d.html

  * igt@kms_cursor_crc@pipe-c-cursor-128x128-random:
    - shard-apl:          [INCOMPLETE][65] ([fdo#103927]) -> [PASS][66] +3 similar issues
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-apl6/igt@kms_cursor_crc@pipe-c-cursor-128x128-random.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-apl8/igt@kms_cursor_crc@pipe-c-cursor-128x128-random.html

  * igt@kms_flip@2x-flip-vs-suspend:
    - shard-hsw:          [INCOMPLETE][67] ([fdo#103540]) -> [PASS][68]
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-hsw2/igt@kms_flip@2x-flip-vs-suspend.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-hsw8/igt@kms_flip@2x-flip-vs-suspend.html

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-pwrite:
    - shard-iclb:         [FAIL][69] ([fdo#103167]) -> [PASS][70] +4 similar issues
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-pwrite.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-pwrite.html

  * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-mmap-wc:
    - {shard-tglb}:       [FAIL][71] ([fdo#103167]) -> [PASS][72] +2 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-tglb1/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-mmap-wc.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-tglb7/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-mmap-wc.html

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
    - shard-kbl:          [DMESG-WARN][73] ([fdo#108566]) -> [PASS][74] +4 similar issues
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-kbl4/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-kbl3/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html

  * igt@kms_psr@psr2_cursor_plane_move:
    - shard-iclb:         [SKIP][75] ([fdo#109441]) -> [PASS][76]
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb4/igt@kms_psr@psr2_cursor_plane_move.html
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb2/igt@kms_psr@psr2_cursor_plane_move.html

  
#### Warnings ####

  * igt@gem_mocs_settings@mocs-settings-bsd2:
    - shard-iclb:         [SKIP][77] ([fdo#109276]) -> [FAIL][78] ([fdo#111330]) +1 similar issue
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb8/igt@gem_mocs_settings@mocs-settings-bsd2.html
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb2/igt@gem_mocs_settings@mocs-settings-bsd2.html

  * igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy:
    - shard-snb:          [DMESG-WARN][79] ([fdo#111870]) -> [DMESG-WARN][80] ([fdo#110789] / [fdo#111870])
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-snb1/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-snb1/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html

  * igt@kms_dp_dsc@basic-dsc-enable-edp:
    - shard-iclb:         [SKIP][81] ([fdo#109349]) -> [DMESG-WARN][82] ([fdo#107724])
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7194/shard-iclb7/igt@kms_dp_dsc@basic-dsc-enable-edp.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/shard-iclb2/igt@kms_dp_dsc@basic-dsc-enable-edp.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo# 112000 ]: https://bugs.freedesktop.org/show_bug.cgi?id= 112000 
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
  [fdo#109349]: https://bugs.freedesktop.org/show_bug.cgi?id=109349
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
  [fdo#109661]: https://bugs.freedesktop.org/show_bug.cgi?id=109661
  [fdo#110789]: https://bugs.freedesktop.org/show_bug.cgi?id=110789
  [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
  [fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330
  [fdo#111606]: https://bugs.freedesktop.org/show_bug.cgi?id=111606
  [fdo#111646]: https://bugs.freedesktop.org/show_bug.cgi?id=111646
  [fdo#111647]: https://bugs.freedesktop.org/show_bug.cgi?id=111647
  [fdo#111671]: https://bugs.freedesktop.org/show_bug.cgi?id=111671
  [fdo#111677]: https://bugs.freedesktop.org/show_bug.cgi?id=111677
  [fdo#111703]: https://bugs.freedesktop.org/show_bug.cgi?id=111703
  [fdo#111747]: https://bugs.freedesktop.org/show_bug.cgi?id=111747
  [fdo#111781]: https://bugs.freedesktop.org/show_bug.cgi?id=111781
  [fdo#111830 ]: https://bugs.freedesktop.org/show_bug.cgi?id=111830 
  [fdo#111832]: https://bugs.freedesktop.org/show_bug.cgi?id=111832
  [fdo#111850]: https://bugs.freedesktop.org/show_bug.cgi?id=111850
  [fdo#111855]: https://bugs.freedesktop.org/show_bug.cgi?id=111855
  [fdo#111865]: https://bugs.freedesktop.org/show_bug.cgi?id=111865
  [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870
  [fdo#111991]: https://bugs.freedesktop.org/show_bug.cgi?id=111991
  [fdo#112037]: https://bugs.freedesktop.org/show_bug.cgi?id=112037
  [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
  [fdo#112142]: https://bugs.freedesktop.org/show_bug.cgi?id=112142
  [fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146
  [fdo#112158]: https://bugs.freedesktop.org/show_bug.cgi?id=112158


Participating hosts (11 -> 8)
------------------------------

  Missing    (3): pig-skl-6260u pig-glk-j5005 pig-hsw-4770r 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5244 -> IGTPW_3620
  * Piglit: piglit_4509 -> None

  CI-20190529: 20190529
  CI_DRM_7194: 5d2161b339033a7eb2d11eef17c5780b44edd996 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3620: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/index.html
  IGT_5244: 5eef4d167c00031709751f12bd77a42a1b74ac67 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3620/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev