All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915
@ 2022-04-25 12:21 Riana Tauro
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915 Riana Tauro
                   ` (4 more replies)
  0 siblings, 5 replies; 13+ messages in thread
From: Riana Tauro @ 2022-04-25 12:21 UTC (permalink / raw)
  To: igt-dev

From: "Tauro, Riana" <riana.tauro@intel.com>

Add basic-s2idle-without-i915 and basic-s3-without-i915 to test 
s2idle and s3 cycles without i915 module.

This test will be part of CI fast-feedback test list

Rev2: Add subtests for s2idle and s3 (Anshuman)

Tauro, Riana (2):
  tests/i915/i915_suspend: Add s2idle and s3 subtests without i915
  tests/intel-ci/fast-feedback: Add suspend tests without i915 to
    fast-feedback list

 tests/i915/i915_suspend.c             | 14 +++++++++-----
 tests/intel-ci/fast-feedback.testlist |  3 ++-
 2 files changed, 11 insertions(+), 6 deletions(-)

-- 
2.25.1

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915
  2022-04-25 12:21 [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915 Riana Tauro
@ 2022-04-25 12:21 ` Riana Tauro
  2022-04-29  4:40   ` Gupta, Anshuman
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list Riana Tauro
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 13+ messages in thread
From: Riana Tauro @ 2022-04-25 12:21 UTC (permalink / raw)
  To: igt-dev

From: "Tauro, Riana" <riana.tauro@intel.com>

Add new tests basic-s2idle-without-i915 and basic-s3-without-i915
to exercise s2idle and s3 cycles to know the system wide
health of suspend/resume

Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
---
 tests/i915/i915_suspend.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
index d34b7a5d..8203b806 100644
--- a/tests/i915/i915_suspend.c
+++ b/tests/i915/i915_suspend.c
@@ -204,12 +204,12 @@ test_forcewake(int fd, bool hibernate)
 }
 
 static void
-test_suspend_without_i915(void)
+test_suspend_without_i915(int state)
 {
 	igt_kmsg(KMSG_INFO "Unloading i915\n");
 	igt_assert_eq(igt_i915_driver_unload(),0);
 
-	igt_system_suspend_autoresume(SUSPEND_STATE_MEM, SUSPEND_TEST_NONE);
+	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
 
 	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
 	igt_assert_eq(igt_i915_driver_load(NULL), 0);
@@ -219,9 +219,13 @@ int fd;
 
 igt_main
 {
-	igt_describe("Validate system suspend cycle without i915 module");
-	igt_subtest("system-suspend-without-i915")
-		test_suspend_without_i915();
+	igt_describe("Validate suspend-to-idle without i915 module");
+	igt_subtest("basic-s2idle-without-i915")
+		test_suspend_without_i915(SUSPEND_STATE_FREEZE);
+
+	igt_describe("Validate S3 without i915 module");
+	igt_subtest("basic-s3-without-i915")
+		test_suspend_without_i915(SUSPEND_STATE_MEM);
 
 	igt_fixture
 		fd = drm_open_driver(DRIVER_INTEL);
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
  2022-04-25 12:21 [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915 Riana Tauro
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915 Riana Tauro
@ 2022-04-25 12:21 ` Riana Tauro
  2022-04-25 13:28   ` [igt-dev] [PATCH i-g-t v2 " Riana Tauro
  2022-04-25 12:33 ` [igt-dev] ✗ Fi.CI.BUILD: failure for Extend system-suspend-without-i915 (rev2) Patchwork
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 13+ messages in thread
From: Riana Tauro @ 2022-04-25 12:21 UTC (permalink / raw)
  To: igt-dev

From: "Tauro, Riana" <riana.tauro@intel.com>

Add basic-s2idle-without-i915 and basic-s3-without-i915
to fast-feedback list.

Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
---
 tests/intel-ci/fast-feedback.testlist | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index d075aa31..b579c20a 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -176,7 +176,8 @@ igt@i915_selftest@live
 igt@dmabuf@all
 
 # System wide suspend tests
-igt@i915_suspend@system-suspend-without-i915
+igt@i915_suspend@basic-s2idle-without-i915
+igt@i915_suspend@basic-s3-without-i915
 igt@gem_exec_suspend@basic-s0
 igt@gem_exec_suspend@basic-s3
 igt@kms_chamelium@common-hpd-after-suspend
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [igt-dev] ✗ Fi.CI.BUILD: failure for Extend system-suspend-without-i915 (rev2)
  2022-04-25 12:21 [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915 Riana Tauro
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915 Riana Tauro
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list Riana Tauro
@ 2022-04-25 12:33 ` Patchwork
  2022-04-25 14:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev3) Patchwork
  2022-04-29 10:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev4) Patchwork
  4 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-04-25 12:33 UTC (permalink / raw)
  To: Riana Tauro; +Cc: igt-dev

== Series Details ==

Series: Extend system-suspend-without-i915 (rev2)
URL   : https://patchwork.freedesktop.org/series/103053/
State : failure

== Summary ==

Applying: tests/i915/i915_suspend: Add s2idle and s3 subtests without i915
Applying: tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
Using index info to reconstruct a base tree...
M	tests/intel-ci/fast-feedback.testlist
Falling back to patching base and 3-way merge...
Auto-merging tests/intel-ci/fast-feedback.testlist
CONFLICT (content): Merge conflict in tests/intel-ci/fast-feedback.testlist
Patch failed at 0002 tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [igt-dev] [PATCH i-g-t v2 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list Riana Tauro
@ 2022-04-25 13:28   ` Riana Tauro
  2022-04-29  4:49     ` Gupta, Anshuman
  2022-04-29  9:54     ` [igt-dev] [PATCH i-g-t v3 " Riana Tauro
  0 siblings, 2 replies; 13+ messages in thread
From: Riana Tauro @ 2022-04-25 13:28 UTC (permalink / raw)
  To: igt-dev

From: "Tauro, Riana" <riana.tauro@intel.com>

Add basic-s2idle-without-i915 and basic-s3-without-i915
to fast-feedback list.

Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
---
 tests/intel-ci/fast-feedback.testlist | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index f6329708..6c39df0c 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -174,7 +174,8 @@ igt@gem_lmem_swapping@parallel-random-engines
 igt@gem_lmem_swapping@random-engines
 igt@gem_lmem_swapping@verify-random
 igt@i915_pm_rpm@module-reload
-igt@i915_suspend@system-suspend-without-i915
+igt@i915_suspend@basic-s2idle-without-i915
+igt@i915_suspend@basic-s3-without-i915
 
 # Kernel selftests
 igt@i915_selftest@live
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev3)
  2022-04-25 12:21 [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915 Riana Tauro
                   ` (2 preceding siblings ...)
  2022-04-25 12:33 ` [igt-dev] ✗ Fi.CI.BUILD: failure for Extend system-suspend-without-i915 (rev2) Patchwork
@ 2022-04-25 14:17 ` Patchwork
  2022-04-29 10:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev4) Patchwork
  4 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-04-25 14:17 UTC (permalink / raw)
  To: Riana Tauro; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 5617 bytes --]

== Series Details ==

Series: Extend system-suspend-without-i915 (rev3)
URL   : https://patchwork.freedesktop.org/series/103053/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_11550 -> IGTPW_6995
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_6995 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_6995, 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_6995/index.html

Participating hosts (43 -> 42)
------------------------------

  Additional (2): bat-dg2-8 bat-dg1-6 
  Missing    (3): fi-kbl-soraka fi-bsw-cyan fi-bdw-5557u 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@gem_exec_suspend@basic-s0@smem:
    - bat-dg1-6:          NOTRUN -> [INCOMPLETE][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6995/bat-dg1-6/igt@gem_exec_suspend@basic-s0@smem.html

  
New tests
---------

  New tests have been introduced between CI_DRM_11550 and IGTPW_6995:

### New IGT tests (2) ###

  * igt@i915_suspend@basic-s2idle-without-i915:
    - Statuses : 35 pass(s)
    - Exec time: [0.86, 19.84] s

  * igt@i915_suspend@basic-s3-without-i915:
    - Statuses : 35 pass(s)
    - Exec time: [1.57, 18.64] s

  

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

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

### IGT changes ###

#### Possible fixes ####

  * igt@i915_selftest@live@gt_heartbeat:
    - fi-cfl-guc:         [DMESG-FAIL][2] ([i915#5334]) -> [PASS][3]
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11550/fi-cfl-guc/igt@i915_selftest@live@gt_heartbeat.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6995/fi-cfl-guc/igt@i915_selftest@live@gt_heartbeat.html

  * igt@i915_selftest@live@hangcheck:
    - fi-hsw-4770:        [INCOMPLETE][4] ([i915#4785]) -> [PASS][5]
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11550/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6995/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html

  * igt@kms_busy@basic@flip:
    - {bat-adlp-6}:       [DMESG-WARN][6] ([i915#3576]) -> [PASS][7]
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11550/bat-adlp-6/igt@kms_busy@basic@flip.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6995/bat-adlp-6/igt@kms_busy@basic@flip.html

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

  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
  [i915#3595]: https://gitlab.freedesktop.org/drm/intel/issues/3595
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
  [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
  [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
  [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
  [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
  [i915#4873]: https://gitlab.freedesktop.org/drm/intel/issues/4873
  [i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
  [i915#5270]: https://gitlab.freedesktop.org/drm/intel/issues/5270
  [i915#5274]: https://gitlab.freedesktop.org/drm/intel/issues/5274
  [i915#5275]: https://gitlab.freedesktop.org/drm/intel/issues/5275
  [i915#5329]: https://gitlab.freedesktop.org/drm/intel/issues/5329
  [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
  [i915#5341]: https://gitlab.freedesktop.org/drm/intel/issues/5341
  [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
  [i915#5356]: https://gitlab.freedesktop.org/drm/intel/issues/5356
  [i915#5537]: https://gitlab.freedesktop.org/drm/intel/issues/5537
  [i915#5763]: https://gitlab.freedesktop.org/drm/intel/issues/5763


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

  * CI: CI-20190529 -> None
  * IGT: IGT_6451 -> IGTPW_6995

  CI-20190529: 20190529
  CI_DRM_11550: 56b089ae03ef8ea8ab7f474eaa70367898891ef0 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_6995: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6995/index.html
  IGT_6451: f055bd83bd831a938d639718c2359516224f15f9 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git


Testlist changes
----------------

+igt@i915_suspend@basic-s2idle-without-i915
+igt@i915_suspend@basic-s3-without-i915
-igt@i915_suspend@system-suspend-without-i915

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6995/index.html

[-- Attachment #2: Type: text/html, Size: 4499 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915
  2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915 Riana Tauro
@ 2022-04-29  4:40   ` Gupta, Anshuman
  2022-04-29 10:58     ` Gupta, Anshuman
  0 siblings, 1 reply; 13+ messages in thread
From: Gupta, Anshuman @ 2022-04-29  4:40 UTC (permalink / raw)
  To: Tauro, Riana, igt-dev



> -----Original Message-----
> From: Tauro, Riana <riana.tauro@intel.com>
> Sent: Monday, April 25, 2022 5:52 PM
> To: igt-dev@lists.freedesktop.org
> Cc: Tauro, Riana <riana.tauro@intel.com>; Gupta, Anshuman
> <anshuman.gupta@intel.com>
> Subject: [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests
> without i915
> 
> From: "Tauro, Riana" <riana.tauro@intel.com>
> 
> Add new tests basic-s2idle-without-i915 and basic-s3-without-i915 to exercise
> s2idle and s3 cycles to know the system wide health of suspend/resume
> 
> Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
> ---
>  tests/i915/i915_suspend.c | 14 +++++++++-----
>  1 file changed, 9 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c index
> d34b7a5d..8203b806 100644
> --- a/tests/i915/i915_suspend.c
> +++ b/tests/i915/i915_suspend.c
> @@ -204,12 +204,12 @@ test_forcewake(int fd, bool hibernate)  }
> 
>  static void
> -test_suspend_without_i915(void)
> +test_suspend_without_i915(int state)
>  {
>  	igt_kmsg(KMSG_INFO "Unloading i915\n");
>  	igt_assert_eq(igt_i915_driver_unload(),0);
> 
> -	igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
> SUSPEND_TEST_NONE);
> +	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
> 
>  	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
>  	igt_assert_eq(igt_i915_driver_load(NULL), 0); @@ -219,9 +219,13 @@
> int fd;
> 
>  igt_main
>  {
> -	igt_describe("Validate system suspend cycle without i915 module");
> -	igt_subtest("system-suspend-without-i915")
> -		test_suspend_without_i915();
> +	igt_describe("Validate suspend-to-idle without i915 module");
> +	igt_subtest("basic-s2idle-without-i915")
> +		test_suspend_without_i915(SUSPEND_STATE_FREEZE);
> +
> +	igt_describe("Validate S3 without i915 module");
> +	igt_subtest("basic-s3-without-i915")
> +		test_suspend_without_i915(SUSPEND_STATE_MEM);
Looks Good to me.
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> 
>  	igt_fixture
>  		fd = drm_open_driver(DRIVER_INTEL);
> --
> 2.25.1

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v2 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
  2022-04-25 13:28   ` [igt-dev] [PATCH i-g-t v2 " Riana Tauro
@ 2022-04-29  4:49     ` Gupta, Anshuman
  2022-04-29  9:54     ` [igt-dev] [PATCH i-g-t v3 " Riana Tauro
  1 sibling, 0 replies; 13+ messages in thread
From: Gupta, Anshuman @ 2022-04-29  4:49 UTC (permalink / raw)
  To: Tauro, Riana, igt-dev



> -----Original Message-----
> From: Tauro, Riana <riana.tauro@intel.com>
> Sent: Monday, April 25, 2022 6:58 PM
> To: igt-dev@lists.freedesktop.org
> Cc: Tauro, Riana <riana.tauro@intel.com>; Gupta, Anshuman
> <anshuman.gupta@intel.com>
> Subject: [PATCH i-g-t v2 2/2] tests/intel-ci/fast-feedback: Add suspend tests
> without i915 to fast-feedback list
> 
> From: "Tauro, Riana" <riana.tauro@intel.com>
> 
> Add basic-s2idle-without-i915 and basic-s3-without-i915 to fast-feedback list.
> 
> Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
> ---
>  tests/intel-ci/fast-feedback.testlist | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-
> feedback.testlist
> index f6329708..6c39df0c 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -174,7 +174,8 @@ igt@gem_lmem_swapping@parallel-random-engines
>  igt@gem_lmem_swapping@random-engines
>  igt@gem_lmem_swapping@verify-random
>  igt@i915_pm_rpm@module-reload
> -igt@i915_suspend@system-suspend-without-i915
> +igt@i915_suspend@basic-s2idle-without-i915
> +igt@i915_suspend@basic-s3-without-i915
Please rebase this patch.
Thanks,
Anshuman Gupta.
> 
>  # Kernel selftests
>  igt@i915_selftest@live
> --
> 2.25.1

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [igt-dev] [PATCH i-g-t v3 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
  2022-04-25 13:28   ` [igt-dev] [PATCH i-g-t v2 " Riana Tauro
  2022-04-29  4:49     ` Gupta, Anshuman
@ 2022-04-29  9:54     ` Riana Tauro
  2022-04-29 10:58       ` Gupta, Anshuman
  1 sibling, 1 reply; 13+ messages in thread
From: Riana Tauro @ 2022-04-29  9:54 UTC (permalink / raw)
  To: igt-dev

From: "Tauro, Riana" <riana.tauro@intel.com>

Add basic-s2idle-without-i915 and basic-s3-without-i915
to fast-feedback list.

v2: rebase
v3: rebase

Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
---
 tests/intel-ci/fast-feedback.testlist | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index d075aa31..b579c20a 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -176,7 +176,8 @@ igt@i915_selftest@live
 igt@dmabuf@all
 
 # System wide suspend tests
-igt@i915_suspend@system-suspend-without-i915
+igt@i915_suspend@basic-s2idle-without-i915
+igt@i915_suspend@basic-s3-without-i915
 igt@gem_exec_suspend@basic-s0
 igt@gem_exec_suspend@basic-s3
 igt@kms_chamelium@common-hpd-after-suspend
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev4)
  2022-04-25 12:21 [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915 Riana Tauro
                   ` (3 preceding siblings ...)
  2022-04-25 14:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev3) Patchwork
@ 2022-04-29 10:55 ` Patchwork
  4 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-04-29 10:55 UTC (permalink / raw)
  To: Riana Tauro; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 12009 bytes --]

== Series Details ==

Series: Extend system-suspend-without-i915 (rev4)
URL   : https://patchwork.freedesktop.org/series/103053/
State : failure

== Summary ==

CI Bug Log - changes from IGT_6462 -> IGTPW_7027
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_7027 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_7027, 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_7027/index.html

Participating hosts (44 -> 45)
------------------------------

  Additional (4): fi-cml-u2 bat-adln-1 fi-rkl-11600 bat-adlm-1 
  Missing    (3): fi-bsw-cyan bat-rpls-1 fi-icl-u2 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live@active:
    - fi-bdw-5557u:       [PASS][1] -> [INCOMPLETE][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6462/fi-bdw-5557u/igt@i915_selftest@live@active.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-bdw-5557u/igt@i915_selftest@live@active.html

  * {igt@i915_suspend@basic-s2idle-without-i915} (NEW):
    - {bat-adln-1}:       NOTRUN -> [DMESG-WARN][3] +1 similar issue
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-adln-1/igt@i915_suspend@basic-s2idle-without-i915.html
    - fi-bdw-gvtdvm:      NOTRUN -> [INCOMPLETE][4]
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-bdw-gvtdvm/igt@i915_suspend@basic-s2idle-without-i915.html

  * {igt@i915_suspend@basic-s3-without-i915} (NEW):
    - fi-cml-u2:          NOTRUN -> [INCOMPLETE][5]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@i915_suspend@basic-s3-without-i915.html
    - fi-rkl-11600:       NOTRUN -> [INCOMPLETE][6]
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_pm_rpm@module-reload:
    - {bat-adln-1}:       NOTRUN -> [DMESG-WARN][7] +37 similar issues
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-adln-1/igt@i915_pm_rpm@module-reload.html

  * igt@i915_selftest@live@gt_lrc:
    - {bat-adlm-1}:       NOTRUN -> [INCOMPLETE][8]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-adlm-1/igt@i915_selftest@live@gt_lrc.html

  * igt@kms_psr@primary_page_flip:
    - {bat-adln-1}:       NOTRUN -> [SKIP][9] +18 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-adln-1/igt@kms_psr@primary_page_flip.html

  
New tests
---------

  New tests have been introduced between IGT_6462 and IGTPW_7027:

### New IGT tests (2) ###

  * igt@i915_suspend@basic-s2idle-without-i915:
    - Statuses : 1 dmesg-warn(s) 1 incomplete(s) 35 pass(s)
    - Exec time: [0.0, 17.69] s

  * igt@i915_suspend@basic-s3-without-i915:
    - Statuses : 1 dmesg-warn(s) 2 incomplete(s) 33 pass(s)
    - Exec time: [0.0, 18.64] s

  

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_fence@basic-busy@bcs0:
    - fi-cml-u2:          NOTRUN -> [SKIP][10] ([i915#1208]) +1 similar issue
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@gem_exec_fence@basic-busy@bcs0.html

  * igt@gem_huc_copy@huc-copy:
    - fi-cml-u2:          NOTRUN -> [SKIP][11] ([i915#2190])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@gem_huc_copy@huc-copy.html
    - fi-rkl-11600:       NOTRUN -> [SKIP][12] ([i915#2190])
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@gem_huc_copy@huc-copy.html

  * igt@gem_lmem_swapping@basic:
    - fi-rkl-11600:       NOTRUN -> [SKIP][13] ([i915#4613]) +3 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@gem_lmem_swapping@basic.html

  * igt@gem_lmem_swapping@parallel-random-engines:
    - fi-cml-u2:          NOTRUN -> [SKIP][14] ([i915#4613]) +3 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@gem_lmem_swapping@parallel-random-engines.html

  * igt@gem_tiled_pread_basic:
    - fi-rkl-11600:       NOTRUN -> [SKIP][15] ([i915#3282])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@gem_tiled_pread_basic.html

  * igt@i915_pm_backlight@basic-brightness:
    - fi-rkl-11600:       NOTRUN -> [SKIP][16] ([i915#3012])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@i915_pm_backlight@basic-brightness.html

  * igt@i915_selftest@live@objects:
    - fi-bdw-5557u:       [PASS][17] -> [INCOMPLETE][18] ([i915#5801])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6462/fi-bdw-5557u/igt@i915_selftest@live@objects.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-bdw-5557u/igt@i915_selftest@live@objects.html

  * igt@kms_busy@basic@modeset:
    - bat-adlp-4:         [PASS][19] -> [DMESG-WARN][20] ([i915#3576])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6462/bat-adlp-4/igt@kms_busy@basic@modeset.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-adlp-4/igt@kms_busy@basic@modeset.html

  * igt@kms_chamelium@dp-crc-fast:
    - fi-rkl-11600:       NOTRUN -> [SKIP][21] ([fdo#111827]) +7 similar issues
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@kms_chamelium@dp-crc-fast.html

  * igt@kms_chamelium@dp-hpd-fast:
    - fi-cml-u2:          NOTRUN -> [SKIP][22] ([fdo#109284] / [fdo#111827]) +7 similar issues
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@kms_chamelium@dp-hpd-fast.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
    - fi-rkl-11600:       NOTRUN -> [SKIP][23] ([i915#4070] / [i915#4103]) +1 similar issue
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
    - fi-cml-u2:          NOTRUN -> [SKIP][24] ([fdo#109278]) +1 similar issue
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html

  * igt@kms_force_connector_basic@force-load-detect:
    - fi-cml-u2:          NOTRUN -> [SKIP][25] ([fdo#109285])
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@kms_force_connector_basic@force-load-detect.html
    - fi-rkl-11600:       NOTRUN -> [SKIP][26] ([fdo#109285] / [i915#4098])
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_frontbuffer_tracking@basic:
    - fi-cml-u2:          NOTRUN -> [DMESG-WARN][27] ([i915#4269])
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html

  * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
    - fi-rkl-11600:       NOTRUN -> [SKIP][28] ([i915#4070] / [i915#533])
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
    - fi-cml-u2:          NOTRUN -> [SKIP][29] ([fdo#109278] / [i915#533])
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html

  * igt@kms_psr@primary_mmap_gtt:
    - fi-rkl-11600:       NOTRUN -> [SKIP][30] ([i915#1072]) +3 similar issues
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@kms_psr@primary_mmap_gtt.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - fi-cml-u2:          NOTRUN -> [SKIP][31] ([i915#3555])
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-rkl-11600:       NOTRUN -> [SKIP][32] ([i915#3555] / [i915#4098])
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@prime_vgem@basic-userptr:
    - fi-rkl-11600:       NOTRUN -> [SKIP][33] ([i915#3301] / [i915#3708])
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@prime_vgem@basic-userptr.html
    - fi-cml-u2:          NOTRUN -> [SKIP][34] ([i915#3301])
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-cml-u2/igt@prime_vgem@basic-userptr.html

  * igt@prime_vgem@basic-write:
    - fi-rkl-11600:       NOTRUN -> [SKIP][35] ([i915#3291] / [i915#3708]) +2 similar issues
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/fi-rkl-11600/igt@prime_vgem@basic-write.html

  
#### Possible fixes ####

  * igt@gem_ringfill@basic-all:
    - {bat-dg2-9}:        [TIMEOUT][36] ([i915#5647]) -> [PASS][37]
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6462/bat-dg2-9/igt@gem_ringfill@basic-all.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-dg2-9/igt@gem_ringfill@basic-all.html

  * igt@kms_flip@basic-plain-flip@a-edp1:
    - bat-adlp-4:         [DMESG-WARN][38] ([i915#3576]) -> [PASS][39] +1 similar issue
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6462/bat-adlp-4/igt@kms_flip@basic-plain-flip@a-edp1.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/bat-adlp-4/igt@kms_flip@basic-plain-flip@a-edp1.html

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

  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155
  [i915#1208]: https://gitlab.freedesktop.org/drm/intel/issues/1208
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#3012]: https://gitlab.freedesktop.org/drm/intel/issues/3012
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070
  [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4269]: https://gitlab.freedesktop.org/drm/intel/issues/4269
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
  [i915#5647]: https://gitlab.freedesktop.org/drm/intel/issues/5647
  [i915#5801]: https://gitlab.freedesktop.org/drm/intel/issues/5801


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

  * CI: CI-20190529 -> None
  * IGT: IGT_6462 -> IGTPW_7027

  CI-20190529: 20190529
  CI_DRM_11550: 56b089ae03ef8ea8ab7f474eaa70367898891ef0 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_7027: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/index.html
  IGT_6462: ca6db54be6bf51958ad99bb319bdb514ae58eb25 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git


Testlist changes
----------------

+igt@i915_suspend@basic-s2idle-without-i915
+igt@i915_suspend@basic-s3-without-i915
-igt@i915_suspend@system-suspend-without-i915

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7027/index.html

[-- Attachment #2: Type: text/html, Size: 14298 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915
  2022-04-29  4:40   ` Gupta, Anshuman
@ 2022-04-29 10:58     ` Gupta, Anshuman
  0 siblings, 0 replies; 13+ messages in thread
From: Gupta, Anshuman @ 2022-04-29 10:58 UTC (permalink / raw)
  To: Tauro, Riana, igt-dev



> -----Original Message-----
> From: Gupta, Anshuman
> Sent: Friday, April 29, 2022 10:11 AM
> To: Tauro, Riana <riana.tauro@intel.com>; igt-dev@lists.freedesktop.org
> Subject: RE: [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3
> subtests without i915
> 
> 
> 
> > -----Original Message-----
> > From: Tauro, Riana <riana.tauro@intel.com>
> > Sent: Monday, April 25, 2022 5:52 PM
> > To: igt-dev@lists.freedesktop.org
> > Cc: Tauro, Riana <riana.tauro@intel.com>; Gupta, Anshuman
> > <anshuman.gupta@intel.com>
> > Subject: [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3
> > subtests without i915
> >
> > From: "Tauro, Riana" <riana.tauro@intel.com>
> >
> > Add new tests basic-s2idle-without-i915 and basic-s3-without-i915 to
> > exercise s2idle and s3 cycles to know the system wide health of
> > suspend/resume
> >
> > Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
> > ---
> >  tests/i915/i915_suspend.c | 14 +++++++++-----
> >  1 file changed, 9 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
> > index
> > d34b7a5d..8203b806 100644
> > --- a/tests/i915/i915_suspend.c
> > +++ b/tests/i915/i915_suspend.c
> > @@ -204,12 +204,12 @@ test_forcewake(int fd, bool hibernate)  }
> >
> >  static void
> > -test_suspend_without_i915(void)
> > +test_suspend_without_i915(int state)
> >  {
> >  	igt_kmsg(KMSG_INFO "Unloading i915\n");
> >  	igt_assert_eq(igt_i915_driver_unload(),0);
> >
> > -	igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
> > SUSPEND_TEST_NONE);
> > +	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
> >
> >  	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
> >  	igt_assert_eq(igt_i915_driver_load(NULL), 0); @@ -219,9 +219,13 @@
> > int fd;
> >
> >  igt_main
> >  {
> > -	igt_describe("Validate system suspend cycle without i915 module");
> > -	igt_subtest("system-suspend-without-i915")
> > -		test_suspend_without_i915();
> > +	igt_describe("Validate suspend-to-idle without i915 module");
> > +	igt_subtest("basic-s2idle-without-i915")
> > +		test_suspend_without_i915(SUSPEND_STATE_FREEZE);
> > +
> > +	igt_describe("Validate S3 without i915 module");
> > +	igt_subtest("basic-s3-without-i915")
> > +		test_suspend_without_i915(SUSPEND_STATE_MEM);
I think , we can optimize this test.
Please probe /sys/power/mem_sleep sys fs.
It shows the current default suspend state.
[s2idle] deep
In case [s2idle] is default , system will trigger always s2idle despite of SUSPEND_STATE_MEM input.
In case [deep] is default  then only system will trigger S3  upon requesting SUSPEND_STATE_MEM input.
Thanks,
Anshuman Gupta.
> Looks Good to me.
> Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> >
> >  	igt_fixture
> >  		fd = drm_open_driver(DRIVER_INTEL);
> > --
> > 2.25.1

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v3 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
  2022-04-29  9:54     ` [igt-dev] [PATCH i-g-t v3 " Riana Tauro
@ 2022-04-29 10:58       ` Gupta, Anshuman
  2022-04-29 11:24         ` Petri Latvala
  0 siblings, 1 reply; 13+ messages in thread
From: Gupta, Anshuman @ 2022-04-29 10:58 UTC (permalink / raw)
  To: Tauro, Riana, igt-dev, Latvala, Petri



> -----Original Message-----
> From: Tauro, Riana <riana.tauro@intel.com>
> Sent: Friday, April 29, 2022 3:25 PM
> To: igt-dev@lists.freedesktop.org
> Cc: Tauro, Riana <riana.tauro@intel.com>; Gupta, Anshuman
> <anshuman.gupta@intel.com>
> Subject: [PATCH i-g-t v3 2/2] tests/intel-ci/fast-feedback: Add suspend tests
> without i915 to fast-feedback list
> 
> From: "Tauro, Riana" <riana.tauro@intel.com>
> 
> Add basic-s2idle-without-i915 and basic-s3-without-i915 to fast-feedback list.
> 
> v2: rebase
> v3: rebase
> 
> Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
> ---
>  tests/intel-ci/fast-feedback.testlist | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-
> feedback.testlist
> index d075aa31..b579c20a 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -176,7 +176,8 @@ igt@i915_selftest@live  igt@dmabuf@all
> 
>  # System wide suspend tests
> -igt@i915_suspend@system-suspend-without-i915
> +igt@i915_suspend@basic-s2idle-without-i915
> +igt@i915_suspend@basic-s3-without-i915
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
Hi Petri,
Is it ok to add extension of this test to fast feedback list.
Thanks,
Anshuman Gupta.
>  igt@gem_exec_suspend@basic-s0
>  igt@gem_exec_suspend@basic-s3
>  igt@kms_chamelium@common-hpd-after-suspend
> --
> 2.25.1

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v3 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list
  2022-04-29 10:58       ` Gupta, Anshuman
@ 2022-04-29 11:24         ` Petri Latvala
  0 siblings, 0 replies; 13+ messages in thread
From: Petri Latvala @ 2022-04-29 11:24 UTC (permalink / raw)
  To: Gupta, Anshuman; +Cc: igt-dev

On Fri, Apr 29, 2022 at 01:58:55PM +0300, Gupta, Anshuman wrote:
> 
> 
> > -----Original Message-----
> > From: Tauro, Riana <riana.tauro@intel.com>
> > Sent: Friday, April 29, 2022 3:25 PM
> > To: igt-dev@lists.freedesktop.org
> > Cc: Tauro, Riana <riana.tauro@intel.com>; Gupta, Anshuman
> > <anshuman.gupta@intel.com>
> > Subject: [PATCH i-g-t v3 2/2] tests/intel-ci/fast-feedback: Add suspend tests
> > without i915 to fast-feedback list
> > 
> > From: "Tauro, Riana" <riana.tauro@intel.com>
> > 
> > Add basic-s2idle-without-i915 and basic-s3-without-i915 to fast-feedback list.
> > 
> > v2: rebase
> > v3: rebase
> > 
> > Signed-off-by: Tauro, Riana <riana.tauro@intel.com>
> > ---
> >  tests/intel-ci/fast-feedback.testlist | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-
> > feedback.testlist
> > index d075aa31..b579c20a 100644
> > --- a/tests/intel-ci/fast-feedback.testlist
> > +++ b/tests/intel-ci/fast-feedback.testlist
> > @@ -176,7 +176,8 @@ igt@i915_selftest@live  igt@dmabuf@all
> > 
> >  # System wide suspend tests
> > -igt@i915_suspend@system-suspend-without-i915
> > +igt@i915_suspend@basic-s2idle-without-i915
> > +igt@i915_suspend@basic-s3-without-i915
> Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> Hi Petri,
> Is it ok to add extension of this test to fast feedback list.

Acked-by: Petri Latvala <petri.latvala@intel.com>

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2022-04-29 11:25 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-25 12:21 [igt-dev] [PATCH i-g-t 0/2] Extend system-suspend-without-i915 Riana Tauro
2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/i915_suspend: Add s2idle and s3 subtests without i915 Riana Tauro
2022-04-29  4:40   ` Gupta, Anshuman
2022-04-29 10:58     ` Gupta, Anshuman
2022-04-25 12:21 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add suspend tests without i915 to fast-feedback list Riana Tauro
2022-04-25 13:28   ` [igt-dev] [PATCH i-g-t v2 " Riana Tauro
2022-04-29  4:49     ` Gupta, Anshuman
2022-04-29  9:54     ` [igt-dev] [PATCH i-g-t v3 " Riana Tauro
2022-04-29 10:58       ` Gupta, Anshuman
2022-04-29 11:24         ` Petri Latvala
2022-04-25 12:33 ` [igt-dev] ✗ Fi.CI.BUILD: failure for Extend system-suspend-without-i915 (rev2) Patchwork
2022-04-25 14:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev3) Patchwork
2022-04-29 10:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for Extend system-suspend-without-i915 (rev4) Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.