* [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector
@ 2019-08-14 15:18 Simon Ser
2019-08-14 16:09 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Simon Ser @ 2019-08-14 15:18 UTC (permalink / raw)
To: igt-dev; +Cc: Daniel Vetter
This test enables a connector, unplugs it, re-plugs it, and makes sure the
Chamelium board still receives a video signal.
Legacy userspace doesn't necessarily handle hotplug uevents. This test makes
sure we don't break it.
See [1] and [2] for more context.
[1]: https://lists.freedesktop.org/archives/igt-dev/2019-July/015071.html
[2]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
Signed-off-by: Simon Ser <simon.ser@intel.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Imre Deak <imre.deak@intel.com>
---
We should probably skip this test on DP-MST and USB-C?
tests/kms_chamelium.c | 84 +++++++++++++++++++++++++++++++++++++++++++
1 file changed, 84 insertions(+)
diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
index f25d13c6e7c3..5cb9a3b9b363 100644
--- a/tests/kms_chamelium.c
+++ b/tests/kms_chamelium.c
@@ -862,6 +862,80 @@ static void test_mode_timings(data_t *data, struct chamelium_port *port)
drmModeFreeConnector(connector);
}
+#define WAIT_VIDEO_INPUT_STABLE_TIMEOUT 5 /* seconds */
+
+/** test_hotplug_while_active:
+ *
+ * Plug a connector, enable it, unplug it, re-plug it and check we still have a
+ * stable input signal.
+ *
+ * Legacy userspace that doesn't consume hotplug events rely on this behaviour.
+ */
+static void
+test_hotplug_while_active(data_t *data, struct chamelium_port *port)
+{
+ struct udev_monitor *mon = igt_watch_hotplug();
+ igt_output_t *output;
+ drmModeConnector *connector;
+ igt_plane_t *primary;
+ struct igt_fb fb;
+ drmModeModeInfo *mode;
+ unsigned int fb_id;
+ bool ok;
+
+ reset_state(data, port);
+
+ igt_flush_hotplugs(mon);
+
+ chamelium_plug(data->chamelium, port);
+ igt_assert(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT));
+ igt_assert_eq(reprobe_connector(data, port), DRM_MODE_CONNECTED);
+
+ output = prepare_output(data, port, TEST_EDID_BASE);
+ connector = chamelium_port_get_connector(data->chamelium, port, false);
+ primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
+ igt_assert(primary);
+
+ igt_assert(connector->count_modes > 0);
+ mode = &connector->modes[0];
+
+ fb_id = igt_create_color_pattern_fb(data->drm_fd,
+ mode->hdisplay, mode->vdisplay,
+ DRM_FORMAT_XRGB8888,
+ LOCAL_DRM_FORMAT_MOD_NONE,
+ 0, 0, 0, &fb);
+ igt_assert(fb_id > 0);
+
+ enable_output(data, port, output, mode, &fb);
+
+ ok = chamelium_port_wait_video_input_stable(data->chamelium, port,
+ WAIT_VIDEO_INPUT_STABLE_TIMEOUT);
+ igt_assert_f(ok, "Failed to wait for video input to become stable "
+ "after hotplug\n");
+
+ igt_flush_hotplugs(mon);
+
+ /* Unplug and re-plug the connector without disabling the output. */
+ chamelium_unplug(data->chamelium, port);
+ igt_assert_f(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT),
+ "Expected hotplug event after unplug\n");
+ igt_assert_eq(reprobe_connector(data, port), DRM_MODE_DISCONNECTED);
+
+ igt_flush_hotplugs(mon);
+
+ chamelium_plug(data->chamelium, port);
+ igt_assert_f(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT),
+ "Expected hotplug event after unplug-replug sequence\n");
+ igt_assert_eq(reprobe_connector(data, port), DRM_MODE_CONNECTED);
+
+ ok = chamelium_port_wait_video_input_stable(data->chamelium, port,
+ WAIT_VIDEO_INPUT_STABLE_TIMEOUT);
+ igt_assert_f(ok, "Failed to wait for video input to become stable "
+ "after unplug and re-plug\n");
+
+ igt_cleanup_hotplug(mon);
+}
+
/* Playback parameters control the audio signal we synthesize and send */
#define PLAYBACK_CHANNELS 2
@@ -2267,6 +2341,9 @@ igt_main
test_basic_hotplug(&data, port,
HPD_TOGGLE_COUNT_FAST);
+ connector_subtest("dp-hpd-active", DisplayPort)
+ test_hotplug_while_active(&data, port);
+
connector_subtest("dp-edid-read", DisplayPort) {
test_edid_read(&data, port, TEST_EDID_BASE);
test_edid_read(&data, port, TEST_EDID_ALT);
@@ -2345,6 +2422,9 @@ igt_main
test_basic_hotplug(&data, port,
HPD_TOGGLE_COUNT_FAST);
+ connector_subtest("hdmi-hpd-active", HDMIA)
+ test_hotplug_while_active(&data, port);
+
connector_subtest("hdmi-edid-read", HDMIA) {
test_edid_read(&data, port, TEST_EDID_BASE);
test_edid_read(&data, port, TEST_EDID_ALT);
@@ -2501,6 +2581,10 @@ igt_main
connector_subtest("vga-hpd-fast", VGA)
test_basic_hotplug(&data, port, HPD_TOGGLE_COUNT_FAST);
+ connector_subtest("vga-hpd-active", VGA)
+ test_hotplug_while_active(&data, port);
+
+
connector_subtest("vga-edid-read", VGA) {
test_edid_read(&data, port, TEST_EDID_BASE);
test_edid_read(&data, port, TEST_EDID_ALT);
--
2.22.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for tests/kms_chamelium: add a hotplug test with active connector
2019-08-14 15:18 [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector Simon Ser
@ 2019-08-14 16:09 ` Patchwork
2019-08-14 16:14 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2019-08-14 16:09 UTC (permalink / raw)
To: Simon Ser; +Cc: igt-dev
== Series Details ==
Series: tests/kms_chamelium: add a hotplug test with active connector
URL : https://patchwork.freedesktop.org/series/65182/
State : warning
== Summary ==
Pipeline status: FAILED.
See https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/56367 for more details.
== Logs ==
For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/56367
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_chamelium: add a hotplug test with active connector
2019-08-14 15:18 [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector Simon Ser
2019-08-14 16:09 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
@ 2019-08-14 16:14 ` Patchwork
2019-08-15 7:46 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-08-16 9:23 ` [igt-dev] [PATCH i-g-t] " Imre Deak
3 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2019-08-14 16:14 UTC (permalink / raw)
To: Simon Ser; +Cc: igt-dev
== Series Details ==
Series: tests/kms_chamelium: add a hotplug test with active connector
URL : https://patchwork.freedesktop.org/series/65182/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6708 -> IGTPW_3349
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/65182/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_3349 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s3:
- fi-snb-2600: [PASS][1] -> [DMESG-WARN][2] ([fdo#102365])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/fi-snb-2600/igt@gem_exec_suspend@basic-s3.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/fi-snb-2600/igt@gem_exec_suspend@basic-s3.html
* igt@kms_busy@basic-flip-a:
- fi-kbl-7567u: [PASS][3] -> [SKIP][4] ([fdo#109271] / [fdo#109278]) +2 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/fi-kbl-7567u/igt@kms_busy@basic-flip-a.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/fi-kbl-7567u/igt@kms_busy@basic-flip-a.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-icl-u2: [PASS][5] -> [FAIL][6] ([fdo#109483])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html
#### Possible fixes ####
* igt@gem_ctx_param@basic-default:
- fi-apl-guc: [INCOMPLETE][7] ([fdo#103927]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/fi-apl-guc/igt@gem_ctx_param@basic-default.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/fi-apl-guc/igt@gem_ctx_param@basic-default.html
* igt@gem_ctx_switch@rcs0:
- fi-bxt-j4205: [INCOMPLETE][9] ([fdo#103927]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/fi-bxt-j4205/igt@gem_ctx_switch@rcs0.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/fi-bxt-j4205/igt@gem_ctx_switch@rcs0.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102365]: https://bugs.freedesktop.org/show_bug.cgi?id=102365
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109483]: https://bugs.freedesktop.org/show_bug.cgi?id=109483
Participating hosts (53 -> 46)
------------------------------
Additional (1): fi-icl-u3
Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5134 -> IGTPW_3349
CI-20190529: 20190529
CI_DRM_6708: 51b7fc7bcf508fb1b95fe6874847c945fa153316 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3349: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/
IGT_5134: 81df2f22385bc275975cf199d962eed9bc10f916 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Testlist changes ==
+igt@kms_chamelium@dp-hpd-active
+igt@kms_chamelium@hdmi-hpd-active
+igt@kms_chamelium@vga-hpd-active
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for tests/kms_chamelium: add a hotplug test with active connector
2019-08-14 15:18 [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector Simon Ser
2019-08-14 16:09 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2019-08-14 16:14 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
@ 2019-08-15 7:46 ` Patchwork
2019-08-16 9:23 ` [igt-dev] [PATCH i-g-t] " Imre Deak
3 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2019-08-15 7:46 UTC (permalink / raw)
To: Simon Ser; +Cc: igt-dev
== Series Details ==
Series: tests/kms_chamelium: add a hotplug test with active connector
URL : https://patchwork.freedesktop.org/series/65182/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6708_full -> IGTPW_3349_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/65182/revisions/1/mbox/
New tests
---------
New tests have been introduced between CI_DRM_6708_full and IGTPW_3349_full:
### New IGT tests (3) ###
* igt@kms_chamelium@dp-hpd-active:
- Statuses : 6 skip(s)
- Exec time: [0.0] s
* igt@kms_chamelium@hdmi-hpd-active:
- Statuses : 6 skip(s)
- Exec time: [0.0] s
* igt@kms_chamelium@vga-hpd-active:
- Statuses : 6 skip(s)
- Exec time: [0.0] s
Known issues
------------
Here are the changes found in IGTPW_3349_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_balancer@smoke:
- shard-iclb: [PASS][1] -> [SKIP][2] ([fdo#110854])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb2/igt@gem_exec_balancer@smoke.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb8/igt@gem_exec_balancer@smoke.html
* igt@gem_exec_schedule@deep-bsd:
- shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#111325]) +1 similar issue
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb5/igt@gem_exec_schedule@deep-bsd.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb2/igt@gem_exec_schedule@deep-bsd.html
* igt@i915_pm_rpm@i2c:
- shard-hsw: [PASS][5] -> [FAIL][6] ([fdo#104097])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-hsw4/igt@i915_pm_rpm@i2c.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-hsw7/igt@i915_pm_rpm@i2c.html
* igt@i915_pm_rpm@modeset-stress-extra-wait:
- shard-hsw: [PASS][7] -> [INCOMPLETE][8] ([fdo#103540] / [fdo#107807])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-hsw1/igt@i915_pm_rpm@modeset-stress-extra-wait.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-hsw5/igt@i915_pm_rpm@modeset-stress-extra-wait.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-render:
- shard-glk: [PASS][9] -> [FAIL][10] ([fdo#103167])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-glk8/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-render.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-glk5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-blt:
- shard-iclb: [PASS][11] -> [FAIL][12] ([fdo#103167])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-blt.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-blt.html
* igt@kms_psr@no_drrs:
- shard-iclb: [PASS][13] -> [FAIL][14] ([fdo#108341])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb2/igt@kms_psr@no_drrs.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb1/igt@kms_psr@no_drrs.html
* igt@kms_psr@psr2_cursor_plane_onoff:
- shard-iclb: [PASS][15] -> [SKIP][16] ([fdo#109441]) +1 similar issue
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb5/igt@kms_psr@psr2_cursor_plane_onoff.html
* igt@kms_vblank@pipe-c-ts-continuation-suspend:
- shard-apl: [PASS][17] -> [DMESG-WARN][18] ([fdo#108566]) +4 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-apl4/igt@kms_vblank@pipe-c-ts-continuation-suspend.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-apl6/igt@kms_vblank@pipe-c-ts-continuation-suspend.html
* igt@prime_vgem@fence-wait-bsd2:
- shard-iclb: [PASS][19] -> [SKIP][20] ([fdo#109276]) +18 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb2/igt@prime_vgem@fence-wait-bsd2.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb8/igt@prime_vgem@fence-wait-bsd2.html
#### Possible fixes ####
* igt@gem_exec_schedule@independent-bsd:
- shard-iclb: [SKIP][21] ([fdo#111325]) -> [PASS][22] +1 similar issue
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb4/igt@gem_exec_schedule@independent-bsd.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb5/igt@gem_exec_schedule@independent-bsd.html
* igt@gem_exec_schedule@preempt-bsd1:
- shard-iclb: [SKIP][23] ([fdo#109276]) -> [PASS][24] +15 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb8/igt@gem_exec_schedule@preempt-bsd1.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb1/igt@gem_exec_schedule@preempt-bsd1.html
* igt@i915_suspend@debugfs-reader:
- shard-apl: [DMESG-WARN][25] ([fdo#108566]) -> [PASS][26] +6 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-apl3/igt@i915_suspend@debugfs-reader.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-apl6/igt@i915_suspend@debugfs-reader.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw:
- shard-iclb: [FAIL][27] ([fdo#103167]) -> [PASS][28] +2 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb4/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
* igt@kms_plane_lowres@pipe-a-tiling-y:
- shard-iclb: [FAIL][29] ([fdo#103166]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb6/igt@kms_plane_lowres@pipe-a-tiling-y.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb7/igt@kms_plane_lowres@pipe-a-tiling-y.html
* igt@kms_psr2_su@page_flip:
- shard-iclb: [SKIP][31] ([fdo#109642] / [fdo#111068]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb5/igt@kms_psr2_su@page_flip.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb2/igt@kms_psr2_su@page_flip.html
#### Warnings ####
* igt@gem_mocs_settings@mocs-isolation-bsd2:
- shard-iclb: [SKIP][33] ([fdo#109276]) -> [FAIL][34] ([fdo#111330])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb7/igt@gem_mocs_settings@mocs-isolation-bsd2.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb1/igt@gem_mocs_settings@mocs-isolation-bsd2.html
* igt@gem_mocs_settings@mocs-reset-bsd2:
- shard-iclb: [FAIL][35] ([fdo#111330]) -> [SKIP][36] ([fdo#109276]) +1 similar issue
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb2/igt@gem_mocs_settings@mocs-reset-bsd2.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb8/igt@gem_mocs_settings@mocs-reset-bsd2.html
* igt@kms_atomic_transition@4x-modeset-transitions-nonblocking-fencing:
- shard-snb: [SKIP][37] ([fdo#109271]) -> [SKIP][38] ([fdo#109271] / [fdo#109278])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-snb6/igt@kms_atomic_transition@4x-modeset-transitions-nonblocking-fencing.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-snb6/igt@kms_atomic_transition@4x-modeset-transitions-nonblocking-fencing.html
* igt@kms_dp_dsc@basic-dsc-enable-edp:
- shard-iclb: [SKIP][39] ([fdo#109349]) -> [DMESG-WARN][40] ([fdo#107724])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6708/shard-iclb3/igt@kms_dp_dsc@basic-dsc-enable-edp.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/shard-iclb2/igt@kms_dp_dsc@basic-dsc-enable-edp.html
[fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
[fdo#104097]: https://bugs.freedesktop.org/show_bug.cgi?id=104097
[fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
[fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
[fdo#108341]: https://bugs.freedesktop.org/show_bug.cgi?id=108341
[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#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[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#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325
[fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330
Participating hosts (10 -> 6)
------------------------------
Missing (4): pig-skl-6260u shard-skl pig-hsw-4770r pig-glk-j5005
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5134 -> IGTPW_3349
* Piglit: piglit_4509 -> None
CI-20190529: 20190529
CI_DRM_6708: 51b7fc7bcf508fb1b95fe6874847c945fa153316 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3349: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3349/
IGT_5134: 81df2f22385bc275975cf199d962eed9bc10f916 @ 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_3349/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector
2019-08-14 15:18 [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector Simon Ser
` (2 preceding siblings ...)
2019-08-15 7:46 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
@ 2019-08-16 9:23 ` Imre Deak
3 siblings, 0 replies; 5+ messages in thread
From: Imre Deak @ 2019-08-16 9:23 UTC (permalink / raw)
To: Simon Ser; +Cc: igt-dev, Daniel Vetter
On Wed, Aug 14, 2019 at 06:18:49PM +0300, Simon Ser wrote:
> This test enables a connector, unplugs it, re-plugs it, and makes sure the
> Chamelium board still receives a video signal.
>
> Legacy userspace doesn't necessarily handle hotplug uevents. This test makes
> sure we don't break it.
>
> See [1] and [2] for more context.
>
> [1]: https://lists.freedesktop.org/archives/igt-dev/2019-July/015071.html
> [2]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
>
> Signed-off-by: Simon Ser <simon.ser@intel.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Imre Deak <imre.deak@intel.com>
> ---
>
> We should probably skip this test on DP-MST and USB-C?
As I understand the plan is to add this test and change the existing
ones to handle hotplug events properly, that is enable/disable the
output on connect/disconnect events.
This new test case should be skipped on USB-C at least, since it would
fail expectedly (I assume also on MST). For that we would need a way to
identify these connectors, I think that could be based on the platform
and port from debugfs/i915_display_info.
Btw, note that storm detection on USB-C connectors will also work only
if the output is disabled on the connector.
>
> tests/kms_chamelium.c | 84 +++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 84 insertions(+)
>
> diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> index f25d13c6e7c3..5cb9a3b9b363 100644
> --- a/tests/kms_chamelium.c
> +++ b/tests/kms_chamelium.c
> @@ -862,6 +862,80 @@ static void test_mode_timings(data_t *data, struct chamelium_port *port)
> drmModeFreeConnector(connector);
> }
>
> +#define WAIT_VIDEO_INPUT_STABLE_TIMEOUT 5 /* seconds */
> +
> +/** test_hotplug_while_active:
> + *
> + * Plug a connector, enable it, unplug it, re-plug it and check we still have a
> + * stable input signal.
> + *
> + * Legacy userspace that doesn't consume hotplug events rely on this behaviour.
> + */
> +static void
> +test_hotplug_while_active(data_t *data, struct chamelium_port *port)
> +{
> + struct udev_monitor *mon = igt_watch_hotplug();
> + igt_output_t *output;
> + drmModeConnector *connector;
> + igt_plane_t *primary;
> + struct igt_fb fb;
> + drmModeModeInfo *mode;
> + unsigned int fb_id;
> + bool ok;
> +
> + reset_state(data, port);
> +
> + igt_flush_hotplugs(mon);
> +
> + chamelium_plug(data->chamelium, port);
> + igt_assert(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT));
> + igt_assert_eq(reprobe_connector(data, port), DRM_MODE_CONNECTED);
> +
> + output = prepare_output(data, port, TEST_EDID_BASE);
> + connector = chamelium_port_get_connector(data->chamelium, port, false);
> + primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> + igt_assert(primary);
> +
> + igt_assert(connector->count_modes > 0);
> + mode = &connector->modes[0];
> +
> + fb_id = igt_create_color_pattern_fb(data->drm_fd,
> + mode->hdisplay, mode->vdisplay,
> + DRM_FORMAT_XRGB8888,
> + LOCAL_DRM_FORMAT_MOD_NONE,
> + 0, 0, 0, &fb);
> + igt_assert(fb_id > 0);
> +
> + enable_output(data, port, output, mode, &fb);
> +
> + ok = chamelium_port_wait_video_input_stable(data->chamelium, port,
> + WAIT_VIDEO_INPUT_STABLE_TIMEOUT);
> + igt_assert_f(ok, "Failed to wait for video input to become stable "
> + "after hotplug\n");
> +
> + igt_flush_hotplugs(mon);
> +
> + /* Unplug and re-plug the connector without disabling the output. */
> + chamelium_unplug(data->chamelium, port);
> + igt_assert_f(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT),
> + "Expected hotplug event after unplug\n");
> + igt_assert_eq(reprobe_connector(data, port), DRM_MODE_DISCONNECTED);
> +
> + igt_flush_hotplugs(mon);
> +
> + chamelium_plug(data->chamelium, port);
> + igt_assert_f(igt_hotplug_detected(mon, HOTPLUG_TIMEOUT),
> + "Expected hotplug event after unplug-replug sequence\n");
> + igt_assert_eq(reprobe_connector(data, port), DRM_MODE_CONNECTED);
> +
> + ok = chamelium_port_wait_video_input_stable(data->chamelium, port,
> + WAIT_VIDEO_INPUT_STABLE_TIMEOUT);
> + igt_assert_f(ok, "Failed to wait for video input to become stable "
> + "after unplug and re-plug\n");
> +
> + igt_cleanup_hotplug(mon);
> +}
> +
>
> /* Playback parameters control the audio signal we synthesize and send */
> #define PLAYBACK_CHANNELS 2
> @@ -2267,6 +2341,9 @@ igt_main
> test_basic_hotplug(&data, port,
> HPD_TOGGLE_COUNT_FAST);
>
> + connector_subtest("dp-hpd-active", DisplayPort)
> + test_hotplug_while_active(&data, port);
> +
> connector_subtest("dp-edid-read", DisplayPort) {
> test_edid_read(&data, port, TEST_EDID_BASE);
> test_edid_read(&data, port, TEST_EDID_ALT);
> @@ -2345,6 +2422,9 @@ igt_main
> test_basic_hotplug(&data, port,
> HPD_TOGGLE_COUNT_FAST);
>
> + connector_subtest("hdmi-hpd-active", HDMIA)
> + test_hotplug_while_active(&data, port);
> +
> connector_subtest("hdmi-edid-read", HDMIA) {
> test_edid_read(&data, port, TEST_EDID_BASE);
> test_edid_read(&data, port, TEST_EDID_ALT);
> @@ -2501,6 +2581,10 @@ igt_main
> connector_subtest("vga-hpd-fast", VGA)
> test_basic_hotplug(&data, port, HPD_TOGGLE_COUNT_FAST);
>
> + connector_subtest("vga-hpd-active", VGA)
> + test_hotplug_while_active(&data, port);
> +
> +
> connector_subtest("vga-edid-read", VGA) {
> test_edid_read(&data, port, TEST_EDID_BASE);
> test_edid_read(&data, port, TEST_EDID_ALT);
> --
> 2.22.0
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-08-16 9:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-14 15:18 [igt-dev] [PATCH i-g-t] tests/kms_chamelium: add a hotplug test with active connector Simon Ser
2019-08-14 16:09 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2019-08-14 16:14 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2019-08-15 7:46 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-08-16 9:23 ` [igt-dev] [PATCH i-g-t] " Imre Deak
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.