* [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing
@ 2019-08-23 18:33 Ville Syrjala
2019-08-23 18:33 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors Ville Syrjala
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Ville Syrjala @ 2019-08-23 18:33 UTC (permalink / raw)
To: igt-dev
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
We have no problems forcing HDMI connector status on HSW/BDW these days.
Remove the bogus restriction.
Also DP forcing doesn't really work on any platform because the kernel
can't get the DPCD if there's nothing actually connected. So keep
refusing DP forcing, and in fact extend it to all platforms.
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
lib/igt_kms.c | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 17a7d2b67f82..d8cdae76c913 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -980,23 +980,15 @@ bool kmstest_force_connector(int drm_fd, drmModeConnector *connector,
char *path, **tmp;
const char *value;
drmModeConnector *temp;
- uint32_t devid;
int len, dir, idx;
- if (is_i915_device(drm_fd)) {
- devid = intel_get_drm_devid(drm_fd);
-
- /*
- * forcing hdmi or dp connectors on HSW and BDW doesn't
- * currently work, so fail early to allow the test to skip if
- * required
- */
- if ((connector->connector_type == DRM_MODE_CONNECTOR_HDMIA ||
- connector->connector_type == DRM_MODE_CONNECTOR_HDMIB ||
- connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort)
- && (IS_HASWELL(devid) || IS_BROADWELL(devid)))
- return false;
- }
+ /*
+ * Forcing DP connectors doesn't currently work, so
+ * fail early to allow the test to skip if required.
+ */
+ if (is_i915_device(drm_fd) &&
+ connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort)
+ return false;
switch (state) {
case FORCE_CONNECTOR_ON:
--
2.21.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors
2019-08-23 18:33 [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Ville Syrjala
@ 2019-08-23 18:33 ` Ville Syrjala
2019-08-26 13:36 ` Ser, Simon
2019-08-23 20:07 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Patchwork
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: Ville Syrjala @ 2019-08-23 18:33 UTC (permalink / raw)
To: igt-dev
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
If we can't find a VGA connector, let's look for a HDMI connector
instead. We can run all but the load detect subtests with HDMI.
v2: Rebase
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
tests/kms_force_connector_basic.c | 95 +++++++++++++++++++------------
1 file changed, 59 insertions(+), 36 deletions(-)
diff --git a/tests/kms_force_connector_basic.c b/tests/kms_force_connector_basic.c
index f1533e5415c0..ddbd97efd29e 100644
--- a/tests/kms_force_connector_basic.c
+++ b/tests/kms_force_connector_basic.c
@@ -80,43 +80,63 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
/* force the VGA output and test that it worked */
int drm_fd = 0;
drmModeRes *res;
- drmModeConnector *vga_connector = NULL, *temp;
+ drmModeConnector *connector = NULL, *temp;
int start_n_modes, start_connection;
igt_fixture {
- unsigned vga_connector_id = 0;
+ unsigned connector_id = 0;
drm_fd = drm_open_driver_master(DRIVER_INTEL);
res = drmModeGetResources(drm_fd);
igt_require(res);
- /* find the vga connector */
+ /* find a vga connector */
for (int i = 0; i < res->count_connectors; i++) {
- vga_connector = drmModeGetConnectorCurrent(drm_fd,
- res->connectors[i]);
+ connector = drmModeGetConnectorCurrent(drm_fd,
+ res->connectors[i]);
- if (vga_connector->connector_type == DRM_MODE_CONNECTOR_VGA) {
+ if (connector->connector_type == DRM_MODE_CONNECTOR_VGA) {
/* Ensure that no override was left in place. */
kmstest_force_connector(drm_fd,
- vga_connector,
+ connector,
FORCE_CONNECTOR_UNSPECIFIED);
/* Only use the first VGA connector. */
- if (!vga_connector_id)
- vga_connector_id = res->connectors[i];
+ if (!connector_id)
+ connector_id = res->connectors[i];
}
- drmModeFreeConnector(vga_connector);
+ drmModeFreeConnector(connector);
}
- igt_require(vga_connector_id);
+ /* find a hdmi connector if we didn't find vga */
+ for (int i = 0; i < res->count_connectors; i++) {
+ connector = drmModeGetConnectorCurrent(drm_fd,
+ res->connectors[i]);
+
+ if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA ||
+ connector->connector_type == DRM_MODE_CONNECTOR_HDMIB) {
+ /* Ensure that no override was left in place. */
+ kmstest_force_connector(drm_fd,
+ connector,
+ FORCE_CONNECTOR_UNSPECIFIED);
+
+ /* Use the the first HDMI connector. */
+ if (!connector_id)
+ connector_id = res->connectors[i];
+ }
+
+ drmModeFreeConnector(connector);
+ }
+
+ igt_require(connector_id);
/* Reacquire status after clearing any previous overrides */
- vga_connector = drmModeGetConnector(drm_fd, vga_connector_id);
+ connector = drmModeGetConnector(drm_fd, connector_id);
- start_n_modes = vga_connector->count_modes;
- start_connection = vga_connector->connection;
+ start_n_modes = connector->count_modes;
+ start_connection = connector->connection;
}
igt_subtest("force-load-detect") {
@@ -124,6 +144,9 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
drmModePlaneRes *plane_resources;
struct igt_fb xrgb_fb, argb_fb;
+ /* no load detect on HDMI */
+ igt_require(connector->connector_type == DRM_MODE_CONNECTOR_VGA);
+
igt_create_fb(drm_fd, w, h, DRM_FORMAT_XRGB8888, 0, &xrgb_fb);
igt_create_fb(drm_fd, w, h, DRM_FORMAT_ARGB8888, 0, &argb_fb);
igt_assert(drmSetClientCap(drm_fd, DRM_CLIENT_CAP_UNIVERSAL_PLANES, 1) == 0);
@@ -176,7 +199,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
/* This can't use drmModeGetConnectorCurrent
* because connector probing is the point of this test.
*/
- temp = drmModeGetConnector(drm_fd, vga_connector->connector_id);
+ temp = drmModeGetConnector(drm_fd, connector->connector_id);
igt_set_module_param_int("load_detect_test", 0);
@@ -206,9 +229,9 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
igt_display_t display;
/* force the connector on and check the reported values */
- kmstest_force_connector(drm_fd, vga_connector, FORCE_CONNECTOR_ON);
+ kmstest_force_connector(drm_fd, connector, FORCE_CONNECTOR_ON);
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
igt_assert_eq(temp->connection, DRM_MODE_CONNECTED);
igt_assert_lt(0, temp->count_modes);
drmModeFreeConnector(temp);
@@ -221,35 +244,35 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
/* force the connector off */
- kmstest_force_connector(drm_fd, vga_connector,
+ kmstest_force_connector(drm_fd, connector,
FORCE_CONNECTOR_OFF);
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
igt_assert_eq(temp->connection, DRM_MODE_DISCONNECTED);
igt_assert_eq(0, temp->count_modes);
drmModeFreeConnector(temp);
/* check that the previous state is restored */
- kmstest_force_connector(drm_fd, vga_connector,
+ kmstest_force_connector(drm_fd, connector,
FORCE_CONNECTOR_UNSPECIFIED);
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
igt_assert_eq(temp->connection, start_connection);
drmModeFreeConnector(temp);
}
igt_subtest("force-edid") {
- kmstest_force_connector(drm_fd, vga_connector,
+ kmstest_force_connector(drm_fd, connector,
FORCE_CONNECTOR_ON);
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
drmModeFreeConnector(temp);
/* test edid forcing */
- kmstest_force_edid(drm_fd, vga_connector,
+ kmstest_force_edid(drm_fd, connector,
igt_kms_get_base_edid());
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
igt_debug("num_conn %i\n", temp->count_modes);
@@ -260,11 +283,11 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
drmModeFreeConnector(temp);
/* remove edid */
- kmstest_force_edid(drm_fd, vga_connector, NULL);
- kmstest_force_connector(drm_fd, vga_connector,
+ kmstest_force_edid(drm_fd, connector, NULL);
+ kmstest_force_connector(drm_fd, connector,
FORCE_CONNECTOR_UNSPECIFIED);
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
/* the connector should now have the same number of modes that
* it started with */
igt_assert_eq(temp->count_modes, start_n_modes);
@@ -275,14 +298,14 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
igt_subtest("prune-stale-modes") {
int i;
- kmstest_force_connector(drm_fd, vga_connector,
+ kmstest_force_connector(drm_fd, connector,
FORCE_CONNECTOR_ON);
/* test pruning of stale modes */
- kmstest_force_edid(drm_fd, vga_connector,
+ kmstest_force_edid(drm_fd, connector,
igt_kms_get_alt_edid());
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
for (i = 0; i < temp->count_modes; i++) {
if (temp->modes[i].hdisplay == 1400 &&
@@ -293,10 +316,10 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
drmModeFreeConnector(temp);
- kmstest_force_edid(drm_fd, vga_connector,
+ kmstest_force_edid(drm_fd, connector,
igt_kms_get_base_edid());
temp = drmModeGetConnectorCurrent(drm_fd,
- vga_connector->connector_id);
+ connector->connector_id);
for (i = 0; i < temp->count_modes; i++) {
if (temp->modes[i].hdisplay == 1400 &&
@@ -307,13 +330,13 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
drmModeFreeConnector(temp);
- kmstest_force_edid(drm_fd, vga_connector, NULL);
- kmstest_force_connector(drm_fd, vga_connector,
+ kmstest_force_edid(drm_fd, connector, NULL);
+ kmstest_force_connector(drm_fd, connector,
FORCE_CONNECTOR_UNSPECIFIED);
}
igt_fixture {
- drmModeFreeConnector(vga_connector);
+ drmModeFreeConnector(connector);
close(drm_fd);
reset_connectors();
--
2.21.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing
2019-08-23 18:33 [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Ville Syrjala
2019-08-23 18:33 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors Ville Syrjala
@ 2019-08-23 20:07 ` Patchwork
2019-08-24 23:16 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-08-26 13:05 ` [igt-dev] [PATCH i-g-t 1/2] " Ser, Simon
3 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2019-08-23 20:07 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing
URL : https://patchwork.freedesktop.org/series/65713/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6779 -> IGTPW_3377
====================================================
Summary
-------
**WARNING**
Minor unknown changes coming with IGTPW_3377 need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_3377, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/65713/revisions/1/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_3377:
### IGT changes ###
#### Warnings ####
* igt@kms_force_connector_basic@force-load-detect:
- fi-cml-u2: [SKIP][1] ([fdo#109285]) -> [SKIP][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-cml-u2/igt@kms_force_connector_basic@force-load-detect.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-cml-u2/igt@kms_force_connector_basic@force-load-detect.html
- fi-icl-u2: [SKIP][3] ([fdo#109285]) -> [SKIP][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html
- fi-icl-u3: [SKIP][5] ([fdo#109285]) -> [SKIP][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u3/igt@kms_force_connector_basic@force-load-detect.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u3/igt@kms_force_connector_basic@force-load-detect.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@kms_force_connector_basic@force-connector-state:
- {fi-icl-dsi}: [SKIP][7] ([fdo#109285]) -> [SKIP][8] +3 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-dsi/igt@kms_force_connector_basic@force-connector-state.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-dsi/igt@kms_force_connector_basic@force-connector-state.html
* igt@kms_force_connector_basic@force-load-detect:
- {fi-icl-guc}: [SKIP][9] ([fdo#109285]) -> [SKIP][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-guc/igt@kms_force_connector_basic@force-load-detect.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-guc/igt@kms_force_connector_basic@force-load-detect.html
Known issues
------------
Here are the changes found in IGTPW_3377 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live_reset:
- fi-icl-u2: [PASS][11] -> [INCOMPLETE][12] ([fdo#107713])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u2/igt@i915_selftest@live_reset.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u2/igt@i915_selftest@live_reset.html
#### Possible fixes ####
* igt@i915_module_load@reload-no-display:
- {fi-icl-u4}: [DMESG-WARN][13] ([fdo#105602]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u4/igt@i915_module_load@reload-no-display.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u4/igt@i915_module_load@reload-no-display.html
* igt@kms_chamelium@dp-edid-read:
- fi-cml-u2: [FAIL][15] ([fdo#109483]) -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-cml-u2/igt@kms_chamelium@dp-edid-read.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-cml-u2/igt@kms_chamelium@dp-edid-read.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-bsw-kefka: [SKIP][17] ([fdo#109271]) -> [PASS][18] +2 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-bsw-kefka/igt@kms_force_connector_basic@force-connector-state.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-bsw-kefka/igt@kms_force_connector_basic@force-connector-state.html
- fi-cfl-8700k: [SKIP][19] ([fdo#109271]) -> [PASS][20] +2 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-cfl-8700k/igt@kms_force_connector_basic@force-connector-state.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-cfl-8700k/igt@kms_force_connector_basic@force-connector-state.html
- fi-skl-6600u: [SKIP][21] ([fdo#109271]) -> [PASS][22] +2 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-6600u/igt@kms_force_connector_basic@force-connector-state.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-6600u/igt@kms_force_connector_basic@force-connector-state.html
- fi-skl-lmem: [SKIP][23] ([fdo#109271]) -> [PASS][24] +2 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-lmem/igt@kms_force_connector_basic@force-connector-state.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-lmem/igt@kms_force_connector_basic@force-connector-state.html
- fi-kbl-r: [SKIP][25] ([fdo#109271]) -> [PASS][26] +2 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-kbl-r/igt@kms_force_connector_basic@force-connector-state.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-kbl-r/igt@kms_force_connector_basic@force-connector-state.html
- fi-byt-n2820: [SKIP][27] ([fdo#109271]) -> [PASS][28] +2 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-byt-n2820/igt@kms_force_connector_basic@force-connector-state.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-byt-n2820/igt@kms_force_connector_basic@force-connector-state.html
- fi-skl-6770hq: [SKIP][29] ([fdo#109271]) -> [PASS][30] +2 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-6770hq/igt@kms_force_connector_basic@force-connector-state.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-6770hq/igt@kms_force_connector_basic@force-connector-state.html
- fi-kbl-guc: [SKIP][31] ([fdo#109271]) -> [PASS][32] +2 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-kbl-guc/igt@kms_force_connector_basic@force-connector-state.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-kbl-guc/igt@kms_force_connector_basic@force-connector-state.html
* igt@kms_force_connector_basic@force-edid:
- fi-glk-dsi: [SKIP][33] ([fdo#109271]) -> [PASS][34] +2 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-glk-dsi/igt@kms_force_connector_basic@force-edid.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-glk-dsi/igt@kms_force_connector_basic@force-edid.html
- fi-bdw-gvtdvm: [SKIP][35] ([fdo#109271]) -> [PASS][36] +2 similar issues
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-bdw-gvtdvm/igt@kms_force_connector_basic@force-edid.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-bdw-gvtdvm/igt@kms_force_connector_basic@force-edid.html
- fi-skl-6260u: [SKIP][37] ([fdo#109271]) -> [PASS][38] +2 similar issues
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-6260u/igt@kms_force_connector_basic@force-edid.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-6260u/igt@kms_force_connector_basic@force-edid.html
- fi-skl-gvtdvm: [SKIP][39] ([fdo#109271]) -> [PASS][40] +2 similar issues
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-gvtdvm/igt@kms_force_connector_basic@force-edid.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-gvtdvm/igt@kms_force_connector_basic@force-edid.html
- fi-bdw-5557u: [SKIP][41] ([fdo#109271]) -> [PASS][42] +2 similar issues
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-bdw-5557u/igt@kms_force_connector_basic@force-edid.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-bdw-5557u/igt@kms_force_connector_basic@force-edid.html
- fi-hsw-4770r: [SKIP][43] ([fdo#109271]) -> [PASS][44] +2 similar issues
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-hsw-4770r/igt@kms_force_connector_basic@force-edid.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-hsw-4770r/igt@kms_force_connector_basic@force-edid.html
- fi-kbl-x1275: [SKIP][45] ([fdo#109271]) -> [PASS][46] +2 similar issues
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-kbl-x1275/igt@kms_force_connector_basic@force-edid.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-kbl-x1275/igt@kms_force_connector_basic@force-edid.html
* igt@kms_force_connector_basic@prune-stale-modes:
- fi-hsw-peppy: [SKIP][47] ([fdo#109271]) -> [PASS][48] +2 similar issues
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-hsw-peppy/igt@kms_force_connector_basic@prune-stale-modes.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-hsw-peppy/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-icl-u2: [SKIP][49] ([fdo#109285]) -> [PASS][50] +2 similar issues
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u2/igt@kms_force_connector_basic@prune-stale-modes.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u2/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-cfl-8109u: [SKIP][51] ([fdo#109271]) -> [PASS][52] +2 similar issues
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-cfl-8109u/igt@kms_force_connector_basic@prune-stale-modes.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-cfl-8109u/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-kbl-7500u: [SKIP][53] ([fdo#109271]) -> [PASS][54] +2 similar issues
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-kbl-7500u/igt@kms_force_connector_basic@prune-stale-modes.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-kbl-7500u/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-cml-u2: [SKIP][55] ([fdo#109285]) -> [PASS][56] +2 similar issues
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-cml-u2/igt@kms_force_connector_basic@prune-stale-modes.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-cml-u2/igt@kms_force_connector_basic@prune-stale-modes.html
- {fi-icl-guc}: [SKIP][57] ([fdo#109285]) -> [PASS][58] +2 similar issues
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-guc/igt@kms_force_connector_basic@prune-stale-modes.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-guc/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-cfl-guc: [SKIP][59] ([fdo#109271]) -> [PASS][60] +2 similar issues
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-cfl-guc/igt@kms_force_connector_basic@prune-stale-modes.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-cfl-guc/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-whl-u: [SKIP][61] ([fdo#109271]) -> [PASS][62] +2 similar issues
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-whl-u/igt@kms_force_connector_basic@prune-stale-modes.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-whl-u/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-icl-u3: [SKIP][63] ([fdo#109285]) -> [PASS][64] +2 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u3/igt@kms_force_connector_basic@prune-stale-modes.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u3/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-kbl-7567u: [SKIP][65] ([fdo#109271]) -> [PASS][66] +2 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-kbl-7567u/igt@kms_force_connector_basic@prune-stale-modes.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-kbl-7567u/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-skl-guc: [SKIP][67] ([fdo#109271]) -> [PASS][68] +2 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-guc/igt@kms_force_connector_basic@prune-stale-modes.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-guc/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-bsw-n3050: [SKIP][69] ([fdo#109271]) -> [PASS][70] +2 similar issues
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-bsw-n3050/igt@kms_force_connector_basic@prune-stale-modes.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-bsw-n3050/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-skl-iommu: [SKIP][71] ([fdo#109271]) -> [PASS][72] +2 similar issues
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-iommu/igt@kms_force_connector_basic@prune-stale-modes.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-iommu/igt@kms_force_connector_basic@prune-stale-modes.html
- fi-skl-6700k2: [SKIP][73] ([fdo#109271]) -> [PASS][74] +2 similar issues
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-skl-6700k2/igt@kms_force_connector_basic@prune-stale-modes.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-skl-6700k2/igt@kms_force_connector_basic@prune-stale-modes.html
* igt@kms_frontbuffer_tracking@basic:
- fi-icl-u2: [FAIL][75] ([fdo#103167]) -> [PASS][76]
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.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#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109483]: https://bugs.freedesktop.org/show_bug.cgi?id=109483
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
Participating hosts (54 -> 47)
------------------------------
Additional (1): fi-bxt-dsi
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_5149 -> IGTPW_3377
CI-20190529: 20190529
CI_DRM_6779: 96b21ba1b7912952fef64efcaa6ee1e4c4182b92 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3377: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/
IGT_5149: 6756ede680ee12745393360d7cc87cc0eb733ff6 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing
2019-08-23 18:33 [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Ville Syrjala
2019-08-23 18:33 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors Ville Syrjala
2019-08-23 20:07 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Patchwork
@ 2019-08-24 23:16 ` Patchwork
2019-08-26 13:05 ` [igt-dev] [PATCH i-g-t 1/2] " Ser, Simon
3 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2019-08-24 23:16 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing
URL : https://patchwork.freedesktop.org/series/65713/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6779_full -> IGTPW_3377_full
====================================================
Summary
-------
**WARNING**
Minor unknown changes coming with IGTPW_3377_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_3377_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/65713/revisions/1/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_3377_full:
### IGT changes ###
#### Warnings ####
* igt@kms_force_connector_basic@force-load-detect:
- shard-iclb: [SKIP][1] ([fdo#109285]) -> [SKIP][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb5/igt@kms_force_connector_basic@force-load-detect.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb2/igt@kms_force_connector_basic@force-load-detect.html
Known issues
------------
Here are the changes found in IGTPW_3377_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_switch@legacy-bsd2-heavy:
- shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#109276]) +18 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb2/igt@gem_ctx_switch@legacy-bsd2-heavy.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb8/igt@gem_ctx_switch@legacy-bsd2-heavy.html
* igt@gem_exec_balancer@smoke:
- shard-iclb: [PASS][5] -> [SKIP][6] ([fdo#110854])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb2/igt@gem_exec_balancer@smoke.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb6/igt@gem_exec_balancer@smoke.html
* igt@gem_exec_schedule@preempt-hang-bsd:
- shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#111325]) +1 similar issue
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb7/igt@gem_exec_schedule@preempt-hang-bsd.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb2/igt@gem_exec_schedule@preempt-hang-bsd.html
* igt@gem_tiled_swapping@non-threaded:
- shard-glk: [PASS][9] -> [DMESG-WARN][10] ([fdo#108686])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-glk4/igt@gem_tiled_swapping@non-threaded.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-glk3/igt@gem_tiled_swapping@non-threaded.html
- shard-hsw: [PASS][11] -> [FAIL][12] ([fdo#108686])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-hsw6/igt@gem_tiled_swapping@non-threaded.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-hsw2/igt@gem_tiled_swapping@non-threaded.html
* igt@i915_suspend@debugfs-reader:
- shard-apl: [PASS][13] -> [DMESG-WARN][14] ([fdo#108566]) +5 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-apl5/igt@i915_suspend@debugfs-reader.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-apl3/igt@i915_suspend@debugfs-reader.html
* igt@kms_cursor_crc@pipe-c-cursor-64x21-random:
- shard-apl: [PASS][15] -> [FAIL][16] ([fdo#103232])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-apl5/igt@kms_cursor_crc@pipe-c-cursor-64x21-random.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-apl7/igt@kms_cursor_crc@pipe-c-cursor-64x21-random.html
- shard-kbl: [PASS][17] -> [FAIL][18] ([fdo#103232])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-kbl1/igt@kms_cursor_crc@pipe-c-cursor-64x21-random.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-kbl7/igt@kms_cursor_crc@pipe-c-cursor-64x21-random.html
* igt@kms_flip@flip-vs-expired-vblank:
- shard-kbl: [PASS][19] -> [FAIL][20] ([fdo#105363])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-kbl2/igt@kms_flip@flip-vs-expired-vblank.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-kbl4/igt@kms_flip@flip-vs-expired-vblank.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-cpu:
- shard-glk: [PASS][21] -> [FAIL][22] ([fdo#103167])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-glk7/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-cpu.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-glk2/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-cpu.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render:
- shard-iclb: [PASS][23] -> [FAIL][24] ([fdo#103167]) +3 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
* igt@kms_plane_lowres@pipe-a-tiling-x:
- shard-iclb: [PASS][25] -> [FAIL][26] ([fdo#103166])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb1/igt@kms_plane_lowres@pipe-a-tiling-x.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb7/igt@kms_plane_lowres@pipe-a-tiling-x.html
* igt@kms_psr@psr2_primary_page_flip:
- shard-iclb: [PASS][27] -> [SKIP][28] ([fdo#109441]) +2 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb3/igt@kms_psr@psr2_primary_page_flip.html
#### Possible fixes ####
* igt@gem_eio@in-flight-suspend:
- shard-apl: [DMESG-WARN][29] ([fdo#108566]) -> [PASS][30] +3 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-apl1/igt@gem_eio@in-flight-suspend.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-apl1/igt@gem_eio@in-flight-suspend.html
* igt@gem_exec_schedule@preempt-contexts-bsd:
- shard-iclb: [SKIP][31] ([fdo#111325]) -> [PASS][32] +4 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb2/igt@gem_exec_schedule@preempt-contexts-bsd.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb8/igt@gem_exec_schedule@preempt-contexts-bsd.html
* igt@gem_exec_schedule@reorder-wide-bsd1:
- shard-iclb: [SKIP][33] ([fdo#109276]) -> [PASS][34] +15 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb5/igt@gem_exec_schedule@reorder-wide-bsd1.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb2/igt@gem_exec_schedule@reorder-wide-bsd1.html
* {igt@gem_userptr_blits@stress-purge}:
- shard-apl: [INCOMPLETE][35] ([fdo#103927]) -> [PASS][36] +1 similar issue
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-apl6/igt@gem_userptr_blits@stress-purge.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-apl8/igt@gem_userptr_blits@stress-purge.html
* igt@kms_cursor_legacy@flip-vs-cursor-crc-atomic:
- shard-snb: [SKIP][37] ([fdo#109271]) -> [PASS][38]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-snb2/igt@kms_cursor_legacy@flip-vs-cursor-crc-atomic.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-snb4/igt@kms_cursor_legacy@flip-vs-cursor-crc-atomic.html
* igt@kms_flip@flip-vs-expired-vblank:
- shard-snb: [FAIL][39] ([fdo#105363]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-snb2/igt@kms_flip@flip-vs-expired-vblank.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-snb7/igt@kms_flip@flip-vs-expired-vblank.html
* igt@kms_force_connector_basic@force-edid:
- shard-kbl: [SKIP][41] ([fdo#109271]) -> [PASS][42] +2 similar issues
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-kbl7/igt@kms_force_connector_basic@force-edid.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-kbl7/igt@kms_force_connector_basic@force-edid.html
- shard-iclb: [SKIP][43] ([fdo#109285]) -> [PASS][44] +2 similar issues
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb1/igt@kms_force_connector_basic@force-edid.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb5/igt@kms_force_connector_basic@force-edid.html
* igt@kms_force_connector_basic@prune-stale-modes:
- shard-glk: [SKIP][45] ([fdo#109271]) -> [PASS][46] +2 similar issues
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-glk1/igt@kms_force_connector_basic@prune-stale-modes.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-glk2/igt@kms_force_connector_basic@prune-stale-modes.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite:
- shard-iclb: [FAIL][47] ([fdo#103167]) -> [PASS][48] +4 similar issues
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
* igt@kms_hdmi_inject@inject-4k:
- shard-hsw: [SKIP][49] ([fdo#109271]) -> [PASS][50] +1 similar issue
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-hsw1/igt@kms_hdmi_inject@inject-4k.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-hsw5/igt@kms_hdmi_inject@inject-4k.html
* igt@kms_psr@psr2_cursor_plane_onoff:
- shard-iclb: [SKIP][51] ([fdo#109441]) -> [PASS][52] +3 similar issues
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6779/shard-iclb8/igt@kms_psr@psr2_cursor_plane_onoff.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854
[fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325
Participating hosts (9 -> 6)
------------------------------
Missing (3): pig-skl-6260u shard-skl pig-hsw-4770r
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5149 -> IGTPW_3377
* Piglit: piglit_4509 -> None
CI-20190529: 20190529
CI_DRM_6779: 96b21ba1b7912952fef64efcaa6ee1e4c4182b92 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3377: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3377/
IGT_5149: 6756ede680ee12745393360d7cc87cc0eb733ff6 @ 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_3377/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing
2019-08-23 18:33 [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Ville Syrjala
` (2 preceding siblings ...)
2019-08-24 23:16 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
@ 2019-08-26 13:05 ` Ser, Simon
3 siblings, 0 replies; 6+ messages in thread
From: Ser, Simon @ 2019-08-26 13:05 UTC (permalink / raw)
To: ville.syrjala, igt-dev
On Fri, 2019-08-23 at 21:33 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> We have no problems forcing HDMI connector status on HSW/BDW these days.
> Remove the bogus restriction.
>
> Also DP forcing doesn't really work on any platform because the kernel
> can't get the DPCD if there's nothing actually connected. So keep
> refusing DP forcing, and in fact extend it to all platforms.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
LGTM and CI is happy.
Reviewed-by: Simon Ser <simon.ser@intel.com>
> ---
> lib/igt_kms.c | 22 +++++++---------------
> 1 file changed, 7 insertions(+), 15 deletions(-)
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 17a7d2b67f82..d8cdae76c913 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -980,23 +980,15 @@ bool kmstest_force_connector(int drm_fd, drmModeConnector *connector,
> char *path, **tmp;
> const char *value;
> drmModeConnector *temp;
> - uint32_t devid;
> int len, dir, idx;
>
> - if (is_i915_device(drm_fd)) {
> - devid = intel_get_drm_devid(drm_fd);
> -
> - /*
> - * forcing hdmi or dp connectors on HSW and BDW doesn't
> - * currently work, so fail early to allow the test to skip if
> - * required
> - */
> - if ((connector->connector_type == DRM_MODE_CONNECTOR_HDMIA ||
> - connector->connector_type == DRM_MODE_CONNECTOR_HDMIB ||
> - connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort)
> - && (IS_HASWELL(devid) || IS_BROADWELL(devid)))
> - return false;
> - }
> + /*
> + * Forcing DP connectors doesn't currently work, so
> + * fail early to allow the test to skip if required.
> + */
> + if (is_i915_device(drm_fd) &&
> + connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort)
> + return false;
>
> switch (state) {
> case FORCE_CONNECTOR_ON:
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors
2019-08-23 18:33 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors Ville Syrjala
@ 2019-08-26 13:36 ` Ser, Simon
0 siblings, 0 replies; 6+ messages in thread
From: Ser, Simon @ 2019-08-26 13:36 UTC (permalink / raw)
To: ville.syrjala, igt-dev
On Fri, 2019-08-23 at 21:33 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> If we can't find a VGA connector, let's look for a HDMI connector
> instead. We can run all but the load detect subtests with HDMI.
>
> v2: Rebase
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
I suppose it's not worth it to try to avoid the few lines of duplicated
code.
Reviewed-by: Simon Ser <simon.ser@intel.com>
> ---
> tests/kms_force_connector_basic.c | 95 +++++++++++++++++++------------
> 1 file changed, 59 insertions(+), 36 deletions(-)
>
> diff --git a/tests/kms_force_connector_basic.c b/tests/kms_force_connector_basic.c
> index f1533e5415c0..ddbd97efd29e 100644
> --- a/tests/kms_force_connector_basic.c
> +++ b/tests/kms_force_connector_basic.c
> @@ -80,43 +80,63 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> /* force the VGA output and test that it worked */
> int drm_fd = 0;
> drmModeRes *res;
> - drmModeConnector *vga_connector = NULL, *temp;
> + drmModeConnector *connector = NULL, *temp;
> int start_n_modes, start_connection;
>
> igt_fixture {
> - unsigned vga_connector_id = 0;
> + unsigned connector_id = 0;
>
> drm_fd = drm_open_driver_master(DRIVER_INTEL);
>
> res = drmModeGetResources(drm_fd);
> igt_require(res);
>
> - /* find the vga connector */
> + /* find a vga connector */
> for (int i = 0; i < res->count_connectors; i++) {
> - vga_connector = drmModeGetConnectorCurrent(drm_fd,
> - res->connectors[i]);
> + connector = drmModeGetConnectorCurrent(drm_fd,
> + res->connectors[i]);
>
> - if (vga_connector->connector_type == DRM_MODE_CONNECTOR_VGA) {
> + if (connector->connector_type == DRM_MODE_CONNECTOR_VGA) {
> /* Ensure that no override was left in place. */
> kmstest_force_connector(drm_fd,
> - vga_connector,
> + connector,
> FORCE_CONNECTOR_UNSPECIFIED);
>
> /* Only use the first VGA connector. */
> - if (!vga_connector_id)
> - vga_connector_id = res->connectors[i];
> + if (!connector_id)
> + connector_id = res->connectors[i];
> }
>
> - drmModeFreeConnector(vga_connector);
> + drmModeFreeConnector(connector);
> }
>
> - igt_require(vga_connector_id);
> + /* find a hdmi connector if we didn't find vga */
> + for (int i = 0; i < res->count_connectors; i++) {
> + connector = drmModeGetConnectorCurrent(drm_fd,
> + res->connectors[i]);
> +
> + if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA ||
> + connector->connector_type == DRM_MODE_CONNECTOR_HDMIB) {
> + /* Ensure that no override was left in place. */
> + kmstest_force_connector(drm_fd,
> + connector,
> + FORCE_CONNECTOR_UNSPECIFIED);
> +
> + /* Use the the first HDMI connector. */
> + if (!connector_id)
> + connector_id = res->connectors[i];
> + }
> +
> + drmModeFreeConnector(connector);
> + }
> +
> + igt_require(connector_id);
>
> /* Reacquire status after clearing any previous overrides */
> - vga_connector = drmModeGetConnector(drm_fd, vga_connector_id);
> + connector = drmModeGetConnector(drm_fd, connector_id);
>
> - start_n_modes = vga_connector->count_modes;
> - start_connection = vga_connector->connection;
> + start_n_modes = connector->count_modes;
> + start_connection = connector->connection;
> }
>
> igt_subtest("force-load-detect") {
> @@ -124,6 +144,9 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> drmModePlaneRes *plane_resources;
> struct igt_fb xrgb_fb, argb_fb;
>
> + /* no load detect on HDMI */
> + igt_require(connector->connector_type == DRM_MODE_CONNECTOR_VGA);
> +
> igt_create_fb(drm_fd, w, h, DRM_FORMAT_XRGB8888, 0, &xrgb_fb);
> igt_create_fb(drm_fd, w, h, DRM_FORMAT_ARGB8888, 0, &argb_fb);
> igt_assert(drmSetClientCap(drm_fd, DRM_CLIENT_CAP_UNIVERSAL_PLANES, 1) == 0);
> @@ -176,7 +199,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> /* This can't use drmModeGetConnectorCurrent
> * because connector probing is the point of this test.
> */
> - temp = drmModeGetConnector(drm_fd, vga_connector->connector_id);
> + temp = drmModeGetConnector(drm_fd, connector->connector_id);
>
> igt_set_module_param_int("load_detect_test", 0);
>
> @@ -206,9 +229,9 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> igt_display_t display;
>
> /* force the connector on and check the reported values */
> - kmstest_force_connector(drm_fd, vga_connector, FORCE_CONNECTOR_ON);
> + kmstest_force_connector(drm_fd, connector, FORCE_CONNECTOR_ON);
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
> igt_assert_eq(temp->connection, DRM_MODE_CONNECTED);
> igt_assert_lt(0, temp->count_modes);
> drmModeFreeConnector(temp);
> @@ -221,35 +244,35 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
>
>
> /* force the connector off */
> - kmstest_force_connector(drm_fd, vga_connector,
> + kmstest_force_connector(drm_fd, connector,
> FORCE_CONNECTOR_OFF);
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
> igt_assert_eq(temp->connection, DRM_MODE_DISCONNECTED);
> igt_assert_eq(0, temp->count_modes);
> drmModeFreeConnector(temp);
>
> /* check that the previous state is restored */
> - kmstest_force_connector(drm_fd, vga_connector,
> + kmstest_force_connector(drm_fd, connector,
> FORCE_CONNECTOR_UNSPECIFIED);
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
> igt_assert_eq(temp->connection, start_connection);
> drmModeFreeConnector(temp);
> }
>
> igt_subtest("force-edid") {
> - kmstest_force_connector(drm_fd, vga_connector,
> + kmstest_force_connector(drm_fd, connector,
> FORCE_CONNECTOR_ON);
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
> drmModeFreeConnector(temp);
>
> /* test edid forcing */
> - kmstest_force_edid(drm_fd, vga_connector,
> + kmstest_force_edid(drm_fd, connector,
> igt_kms_get_base_edid());
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
>
> igt_debug("num_conn %i\n", temp->count_modes);
>
> @@ -260,11 +283,11 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> drmModeFreeConnector(temp);
>
> /* remove edid */
> - kmstest_force_edid(drm_fd, vga_connector, NULL);
> - kmstest_force_connector(drm_fd, vga_connector,
> + kmstest_force_edid(drm_fd, connector, NULL);
> + kmstest_force_connector(drm_fd, connector,
> FORCE_CONNECTOR_UNSPECIFIED);
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
> /* the connector should now have the same number of modes that
> * it started with */
> igt_assert_eq(temp->count_modes, start_n_modes);
> @@ -275,14 +298,14 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> igt_subtest("prune-stale-modes") {
> int i;
>
> - kmstest_force_connector(drm_fd, vga_connector,
> + kmstest_force_connector(drm_fd, connector,
> FORCE_CONNECTOR_ON);
>
> /* test pruning of stale modes */
> - kmstest_force_edid(drm_fd, vga_connector,
> + kmstest_force_edid(drm_fd, connector,
> igt_kms_get_alt_edid());
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
>
> for (i = 0; i < temp->count_modes; i++) {
> if (temp->modes[i].hdisplay == 1400 &&
> @@ -293,10 +316,10 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
>
> drmModeFreeConnector(temp);
>
> - kmstest_force_edid(drm_fd, vga_connector,
> + kmstest_force_edid(drm_fd, connector,
> igt_kms_get_base_edid());
> temp = drmModeGetConnectorCurrent(drm_fd,
> - vga_connector->connector_id);
> + connector->connector_id);
>
> for (i = 0; i < temp->count_modes; i++) {
> if (temp->modes[i].hdisplay == 1400 &&
> @@ -307,13 +330,13 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
>
> drmModeFreeConnector(temp);
>
> - kmstest_force_edid(drm_fd, vga_connector, NULL);
> - kmstest_force_connector(drm_fd, vga_connector,
> + kmstest_force_edid(drm_fd, connector, NULL);
> + kmstest_force_connector(drm_fd, connector,
> FORCE_CONNECTOR_UNSPECIFIED);
> }
>
> igt_fixture {
> - drmModeFreeConnector(vga_connector);
> + drmModeFreeConnector(connector);
> close(drm_fd);
>
> reset_connectors();
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-08-26 13:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-23 18:33 [igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Ville Syrjala
2019-08-23 18:33 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_force_connector_basic: Execute with HDMI connectors Ville Syrjala
2019-08-26 13:36 ` Ser, Simon
2019-08-23 20:07 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] lib/igt_kms: Remove stale restrictions for HSW/BDW HDMI connector forcing Patchwork
2019-08-24 23:16 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-08-26 13:05 ` [igt-dev] [PATCH i-g-t 1/2] " Ser, Simon
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.