public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t 1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable
@ 2019-10-17  9:17 Petri Latvala
  2019-10-17  9:17 ` [igt-dev] [PATCH i-g-t 2/2] intel-ci: Adjust pipe-specific blacklisting Petri Latvala
                   ` (9 more replies)
  0 siblings, 10 replies; 14+ messages in thread
From: Petri Latvala @ 2019-10-17  9:17 UTC (permalink / raw)
  To: igt-dev; +Cc: Petri Latvala

The blacklist for pipes E and F are not capturing kms_busy's usage of
them, because of the differing naming convention. Make that more
uniform.

Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Martin Peres <martin.peres@linux.intel.com>
---

Martin, test renaming here. Needs your ack for the cibuglog changes
needed.

tests/kms_busy.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/tests/kms_busy.c b/tests/kms_busy.c
index 5ff2b846..1426a5f0 100644
--- a/tests/kms_busy.c
+++ b/tests/kms_busy.c
@@ -313,14 +313,14 @@ igt_main
 			igt_display_require_output_on_pipe(&display, n);
 		}
 
-		igt_subtest_f("basic-flip-%s",
+		igt_subtest_f("basic-flip-pipe-%s",
 			kmstest_pipe_name(n)) {
 			igt_require(gem_has_ring(display.drm_fd,
 						e->exec_id | e->flags));
 
 			test_flip(&display, e->exec_id | e->flags, n, false);
 		}
-		igt_subtest_f("basic-modeset-%s",
+		igt_subtest_f("basic-modeset-pipe-%s",
 			kmstest_pipe_name(n)) {
 			igt_require(gem_has_ring(display.drm_fd,
 						e->exec_id | e->flags));
@@ -335,7 +335,7 @@ igt_main
 			hang = igt_allow_hang(display.drm_fd, 0, 0);
 		}
 
-		igt_subtest_f("extended-pageflip-modeset-hang-oldfb-%s-%s",
+		igt_subtest_f("extended-pageflip-modeset-hang-oldfb-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n)) {
 			igt_require(gem_has_ring(display.drm_fd,
 						e->exec_id | e->flags));
@@ -346,23 +346,23 @@ igt_main
 		igt_fixture
 			igt_require(display.is_atomic);
 
-		igt_subtest_f("extended-pageflip-hang-oldfb-%s-%s",
+		igt_subtest_f("extended-pageflip-hang-oldfb-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n))
 			test_hang(&display, e->exec_id | e->flags, n, false, false);
 
-		igt_subtest_f("extended-pageflip-hang-newfb-%s-%s",
+		igt_subtest_f("extended-pageflip-hang-newfb-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n))
 			test_hang(&display, e->exec_id | e->flags, n, false, true);
 
-		igt_subtest_f("extended-modeset-hang-oldfb-%s-%s",
+		igt_subtest_f("extended-modeset-hang-oldfb-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n))
 			test_hang(&display, e->exec_id | e->flags, n, true, false);
 
-		igt_subtest_f("extended-modeset-hang-newfb-%s-%s",
+		igt_subtest_f("extended-modeset-hang-newfb-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n))
 			test_hang(&display, e->exec_id | e->flags, n, true, true);
 
-		igt_subtest_f("extended-modeset-hang-oldfb-with-reset-%s-%s",
+		igt_subtest_f("extended-modeset-hang-oldfb-with-reset-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n)) {
 			igt_set_module_param_int("force_reset_modeset_test", 1);
 
@@ -371,7 +371,7 @@ igt_main
 			igt_set_module_param_int("force_reset_modeset_test", 0);
 		}
 
-		igt_subtest_f("extended-modeset-hang-newfb-with-reset-%s-%s",
+		igt_subtest_f("extended-modeset-hang-newfb-with-reset-%s-pipe-%s",
 				e->name, kmstest_pipe_name(n)) {
 			igt_set_module_param_int("force_reset_modeset_test", 1);
 
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-11-11 11:31 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-17  9:17 [igt-dev] [PATCH i-g-t 1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable Petri Latvala
2019-10-17  9:17 ` [igt-dev] [PATCH i-g-t 2/2] intel-ci: Adjust pipe-specific blacklisting Petri Latvala
2019-10-17  9:21   ` [igt-dev] [PATCH i-g-t v2 " Petri Latvala
2019-11-07 14:45   ` [igt-dev] [PATCH i-g-t " Arkadiusz Hiler
2019-11-07 14:54     ` Arkadiusz Hiler
2019-10-17  9:41 ` [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t,1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable (rev2) Patchwork
2019-10-17 10:00 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
2019-10-20 10:51 ` [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t,1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable (rev3) Patchwork
2019-10-20 11:08 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
2019-10-21 11:42 ` [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t,1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable (rev4) Patchwork
2019-10-21 12:17 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2019-10-21 15:22 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2019-11-07 14:45 ` [igt-dev] [PATCH i-g-t 1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable Arkadiusz Hiler
2019-11-11 11:31 ` [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [i-g-t,1/2] kms_busy: Rename tests so pipe-specific subtests are blacklistable (rev4) Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox