* [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
@ 2021-01-06 22:38 Sean Paul
2021-01-06 23:41 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
` (7 more replies)
0 siblings, 8 replies; 23+ messages in thread
From: Sean Paul @ 2021-01-06 22:38 UTC (permalink / raw)
To: dri-devel, intel-gfx; +Cc: David Airlie, Sean Paul
From: Sean Paul <seanpaul@chromium.org>
The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
check, it was always a nice-to-have. After deploying this across various
devices, we've determined that some MST bridge chips do not properly
support this call for HDCP 1.4 (namely Synaptics and Realtek).
I had considered creating a quirk for this, but I think it's more
prudent to just disable the check entirely since I don't have an idea
how widespread support is.
Signed-off-by: Sean Paul <seanpaul@chromium.org>
---
drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
1 file changed, 1 insertion(+), 25 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
index 03424d20e9f7..b6a9606bf09a 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
@@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
return ret;
}
-static
-bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
- struct intel_connector *connector)
-{
- struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
- struct intel_dp *intel_dp = &dig_port->dp;
- struct drm_dp_query_stream_enc_status_ack_reply reply;
- int ret;
-
- if (!intel_dp_hdcp_check_link(dig_port, connector))
- return false;
-
- ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
- connector->port, &reply);
- if (ret) {
- drm_dbg_kms(&i915->drm,
- "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
- connector->base.base.id, connector->base.name, ret);
- return false;
- }
-
- return reply.auth_completed && reply.encryption_enabled;
-}
-
static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
.write_an_aksv = intel_dp_hdcp_write_an_aksv,
.read_bksv = intel_dp_hdcp_read_bksv,
@@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
.read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
.read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
.toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
- .check_link = intel_dp_mst_hdcp_check_link,
+ .check_link = intel_dp_hdcp_check_link,
.hdcp_capable = intel_dp_hdcp_capable,
.protocol = HDCP_PROTOCOL_DP,
--
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 23+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
@ 2021-01-06 23:41 ` Patchwork
2021-01-07 10:13 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
` (6 subsequent siblings)
7 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2021-01-06 23:41 UTC (permalink / raw)
To: Sean Paul; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 2422 bytes --]
== Series Details ==
Series: drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
URL : https://patchwork.freedesktop.org/series/85555/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9554 -> Patchwork_19275
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/index.html
Known issues
------------
Here are the changes found in Patchwork_19275 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@fbdev@read:
- fi-tgl-y: [PASS][1] -> [DMESG-WARN][2] ([i915#402]) +2 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/fi-tgl-y/igt@fbdev@read.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/fi-tgl-y/igt@fbdev@read.html
#### Possible fixes ####
* igt@i915_pm_rpm@module-reload:
- fi-kbl-guc: [FAIL][3] ([i915#579]) -> [PASS][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/fi-kbl-guc/igt@i915_pm_rpm@module-reload.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/fi-kbl-guc/igt@i915_pm_rpm@module-reload.html
* igt@vgem_basic@dmabuf-fence:
- fi-tgl-y: [DMESG-WARN][5] ([i915#402]) -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/fi-tgl-y/igt@vgem_basic@dmabuf-fence.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/fi-tgl-y/igt@vgem_basic@dmabuf-fence.html
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
[i915#579]: https://gitlab.freedesktop.org/drm/intel/issues/579
Participating hosts (42 -> 37)
------------------------------
Missing (5): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_9554 -> Patchwork_19275
CI-20190529: 20190529
CI_DRM_9554: 4786f2a4c7e280974d0b46ca9e30a156eb528d59 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5945: f0ad1a564956d6796d9ff09182c48d78fb00eefe @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19275: bd6eed3a74e7aef1dde724eacb490413d8b3ee79 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
bd6eed3a74e7 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/index.html
[-- Attachment #1.2: Type: text/html, Size: 3147 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
2021-01-06 23:41 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
@ 2021-01-07 10:13 ` Patchwork
2021-01-12 17:51 ` [Intel-gfx] [PATCH] " Jani Nikula
` (5 subsequent siblings)
7 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2021-01-07 10:13 UTC (permalink / raw)
To: Sean Paul; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 20875 bytes --]
== Series Details ==
Series: drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
URL : https://patchwork.freedesktop.org/series/85555/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9554_full -> Patchwork_19275_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_19275_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_19275_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_19275_full:
### IGT changes ###
#### Possible regressions ####
* igt@kms_flip_tiling@flip-to-y-tiled@edp-1-pipe-c:
- shard-skl: [PASS][1] -> [FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl6/igt@kms_flip_tiling@flip-to-y-tiled@edp-1-pipe-c.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl10/igt@kms_flip_tiling@flip-to-y-tiled@edp-1-pipe-c.html
Known issues
------------
Here are the changes found in Patchwork_19275_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@cursor-dpms:
- shard-iclb: [PASS][3] -> [DMESG-WARN][4] ([i915#1226])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb6/igt@i915_pm_rpm@cursor-dpms.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb5/igt@i915_pm_rpm@cursor-dpms.html
* igt@i915_pm_rpm@system-suspend-execbuf:
- shard-skl: [PASS][5] -> [INCOMPLETE][6] ([i915#146] / [i915#151])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl7/igt@i915_pm_rpm@system-suspend-execbuf.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl3/igt@i915_pm_rpm@system-suspend-execbuf.html
* igt@kms_async_flips@test-time-stamp:
- shard-skl: [PASS][7] -> [DMESG-FAIL][8] ([i915#142])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl8/igt@kms_async_flips@test-time-stamp.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl1/igt@kms_async_flips@test-time-stamp.html
* igt@kms_chamelium@dp-frame-dump:
- shard-skl: NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +1 similar issue
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl8/igt@kms_chamelium@dp-frame-dump.html
* igt@kms_cursor_crc@pipe-a-cursor-256x85-offscreen:
- shard-skl: [PASS][10] -> [FAIL][11] ([i915#54]) +5 similar issues
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl5/igt@kms_cursor_crc@pipe-a-cursor-256x85-offscreen.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl1/igt@kms_cursor_crc@pipe-a-cursor-256x85-offscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-128x42-offscreen:
- shard-skl: NOTRUN -> [FAIL][12] ([i915#54])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl8/igt@kms_cursor_crc@pipe-c-cursor-128x42-offscreen.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions:
- shard-skl: NOTRUN -> [FAIL][13] ([i915#2346])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl5/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
* igt@kms_flip@flip-vs-expired-vblank@c-edp1:
- shard-skl: [PASS][14] -> [FAIL][15] ([i915#79])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl5/igt@kms_flip@flip-vs-expired-vblank@c-edp1.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl1/igt@kms_flip@flip-vs-expired-vblank@c-edp1.html
* igt@kms_flip@flip-vs-expired-vblank@c-hdmi-a2:
- shard-glk: [PASS][16] -> [FAIL][17] ([i915#79])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-glk9/igt@kms_flip@flip-vs-expired-vblank@c-hdmi-a2.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-glk9/igt@kms_flip@flip-vs-expired-vblank@c-hdmi-a2.html
* igt@kms_flip@flip-vs-suspend-interruptible@c-edp1:
- shard-skl: [PASS][18] -> [INCOMPLETE][19] ([i915#2295])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl6/igt@kms_flip@flip-vs-suspend-interruptible@c-edp1.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl10/igt@kms_flip@flip-vs-suspend-interruptible@c-edp1.html
* igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1:
- shard-skl: [PASS][20] -> [FAIL][21] ([i915#2122])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl9/igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl9/igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-blt:
- shard-skl: NOTRUN -> [SKIP][22] ([fdo#109271]) +31 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl8/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render:
- shard-kbl: NOTRUN -> [SKIP][23] ([fdo#109271]) +12 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl1/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb:
- shard-skl: NOTRUN -> [FAIL][24] ([i915#265])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl8/igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb.html
* igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
- shard-skl: [PASS][25] -> [FAIL][26] ([fdo#108145] / [i915#265])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl6/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl10/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
* igt@kms_psr2_su@frontbuffer:
- shard-iclb: [PASS][27] -> [SKIP][28] ([fdo#109642] / [fdo#111068])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb2/igt@kms_psr2_su@frontbuffer.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb8/igt@kms_psr2_su@frontbuffer.html
* igt@kms_psr@psr2_primary_mmap_cpu:
- shard-iclb: [PASS][29] -> [SKIP][30] ([fdo#109441]) +2 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb2/igt@kms_psr@psr2_primary_mmap_cpu.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb1/igt@kms_psr@psr2_primary_mmap_cpu.html
* igt@kms_vblank@pipe-d-query-idle:
- shard-iclb: NOTRUN -> [SKIP][31] ([fdo#109278])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb8/igt@kms_vblank@pipe-d-query-idle.html
#### Possible fixes ####
* {igt@gem_exec_fair@basic-none@rcs0}:
- shard-kbl: [SKIP][32] ([fdo#109271]) -> [PASS][33]
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-kbl6/igt@gem_exec_fair@basic-none@rcs0.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl7/igt@gem_exec_fair@basic-none@rcs0.html
* {igt@gem_exec_fair@basic-throttle@rcs0}:
- shard-glk: [FAIL][34] ([i915#2842]) -> [PASS][35]
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-glk5/igt@gem_exec_fair@basic-throttle@rcs0.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-glk5/igt@gem_exec_fair@basic-throttle@rcs0.html
* {igt@gem_exec_schedule@u-fairslice@rcs0}:
- shard-skl: [DMESG-WARN][36] ([i915#1610] / [i915#2803]) -> [PASS][37]
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl10/igt@gem_exec_schedule@u-fairslice@rcs0.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl8/igt@gem_exec_schedule@u-fairslice@rcs0.html
* igt@i915_pm_rpm@cursor-dpms:
- shard-glk: [DMESG-WARN][38] ([i915#118] / [i915#95]) -> [PASS][39] +1 similar issue
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-glk2/igt@i915_pm_rpm@cursor-dpms.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-glk3/igt@i915_pm_rpm@cursor-dpms.html
* igt@kms_async_flips@test-time-stamp:
- shard-tglb: [FAIL][40] ([i915#2574]) -> [PASS][41]
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-tglb5/igt@kms_async_flips@test-time-stamp.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-tglb8/igt@kms_async_flips@test-time-stamp.html
* igt@kms_color@pipe-b-ctm-0-5:
- shard-skl: [DMESG-WARN][42] ([i915#1982]) -> [PASS][43]
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl7/igt@kms_color@pipe-b-ctm-0-5.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl3/igt@kms_color@pipe-b-ctm-0-5.html
* igt@kms_cursor_crc@pipe-c-cursor-128x128-offscreen:
- shard-skl: [FAIL][44] ([i915#54]) -> [PASS][45] +8 similar issues
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl2/igt@kms_cursor_crc@pipe-c-cursor-128x128-offscreen.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl3/igt@kms_cursor_crc@pipe-c-cursor-128x128-offscreen.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:
- shard-skl: [FAIL][46] ([i915#2346] / [i915#533]) -> [PASS][47]
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl5/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl5/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@flip-vs-cursor-busy-crc-legacy:
- shard-skl: [FAIL][48] ([i915#2346]) -> [PASS][49]
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl8/igt@kms_cursor_legacy@flip-vs-cursor-busy-crc-legacy.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl4/igt@kms_cursor_legacy@flip-vs-cursor-busy-crc-legacy.html
* igt@kms_cursor_legacy@pipe-c-single-bo:
- shard-kbl: [INCOMPLETE][50] -> [PASS][51]
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-kbl1/igt@kms_cursor_legacy@pipe-c-single-bo.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl1/igt@kms_cursor_legacy@pipe-c-single-bo.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@a-edp1:
- shard-tglb: [FAIL][52] ([i915#2598]) -> [PASS][53]
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-tglb2/igt@kms_flip@flip-vs-expired-vblank-interruptible@a-edp1.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-tglb6/igt@kms_flip@flip-vs-expired-vblank-interruptible@a-edp1.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
- shard-skl: [FAIL][54] ([i915#79]) -> [PASS][55]
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl10/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl2/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@c-dp1:
- shard-kbl: [DMESG-WARN][56] ([i915#180] / [i915#78]) -> [PASS][57] +4 similar issues
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-kbl2/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-dp1.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl6/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-dp1.html
* igt@kms_flip@modeset-vs-vblank-race@c-dp1:
- shard-kbl: [DMESG-WARN][58] ([i915#165]) -> [PASS][59]
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-kbl2/igt@kms_flip@modeset-vs-vblank-race@c-dp1.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl3/igt@kms_flip@modeset-vs-vblank-race@c-dp1.html
* igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1:
- shard-skl: [FAIL][60] ([i915#2122]) -> [PASS][61]
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl5/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl4/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-onoff:
- shard-iclb: [FAIL][62] ([i915#49]) -> [PASS][63]
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-onoff.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb5/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-onoff.html
* igt@kms_hdr@bpc-switch-dpms:
- shard-skl: [FAIL][64] ([i915#1188]) -> [PASS][65] +1 similar issue
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl7/igt@kms_hdr@bpc-switch-dpms.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl8/igt@kms_hdr@bpc-switch-dpms.html
* igt@kms_psr@psr2_cursor_plane_onoff:
- shard-iclb: [SKIP][66] ([fdo#109441]) -> [PASS][67]
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb3/igt@kms_psr@psr2_cursor_plane_onoff.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
* igt@perf@polling:
- shard-skl: [FAIL][68] ([i915#1542]) -> [PASS][69] +1 similar issue
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl8/igt@perf@polling.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl4/igt@perf@polling.html
* igt@testdisplay:
- shard-kbl: [DMESG-WARN][70] ([i915#165] / [i915#180] / [i915#78]) -> [PASS][71] +1 similar issue
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-kbl2/igt@testdisplay.html
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl3/igt@testdisplay.html
#### Warnings ####
* igt@i915_pm_rc6_residency@rc6-fence:
- shard-iclb: [WARN][72] ([i915#2681] / [i915#2684]) -> [WARN][73] ([i915#1804] / [i915#2684])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb1/igt@i915_pm_rc6_residency@rc6-fence.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb3/igt@i915_pm_rc6_residency@rc6-fence.html
* igt@i915_pm_rc6_residency@rc6-idle:
- shard-iclb: [WARN][74] ([i915#1804] / [i915#2684]) -> [FAIL][75] ([i915#2680])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb4/igt@i915_pm_rc6_residency@rc6-idle.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb3/igt@i915_pm_rc6_residency@rc6-idle.html
* igt@runner@aborted:
- shard-kbl: [FAIL][76] ([i915#2295] / [i915#2505] / [i915#483]) -> [FAIL][77] ([i915#2295] / [i915#483])
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-kbl6/igt@runner@aborted.html
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-kbl2/igt@runner@aborted.html
- shard-iclb: [FAIL][78] ([i915#2295] / [i915#2724] / [i915#483]) -> [FAIL][79] ([i915#2295] / [i915#2724])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-iclb6/igt@runner@aborted.html
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-iclb2/igt@runner@aborted.html
- shard-glk: ([FAIL][80], [FAIL][81]) ([i915#2295] / [i915#2426] / [i915#483] / [k.org#202321]) -> ([FAIL][82], [FAIL][83]) ([i915#2295] / [i915#2426] / [k.org#202321])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-glk7/igt@runner@aborted.html
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-glk8/igt@runner@aborted.html
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-glk6/igt@runner@aborted.html
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-glk2/igt@runner@aborted.html
- shard-skl: ([FAIL][84], [FAIL][85]) ([i915#2295] / [i915#2426]) -> ([FAIL][86], [FAIL][87]) ([i915#2295])
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl2/igt@runner@aborted.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9554/shard-skl10/igt@runner@aborted.html
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl2/igt@runner@aborted.html
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/shard-skl1/igt@runner@aborted.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
[i915#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188
[i915#1226]: https://gitlab.freedesktop.org/drm/intel/issues/1226
[i915#142]: https://gitlab.freedesktop.org/drm/intel/issues/142
[i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146
[i915#151]: https://gitlab.freedesktop.org/drm/intel/issues/151
[i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
[i915#1610]: https://gitlab.freedesktop.org/drm/intel/issues/1610
[i915#165]: https://gitlab.freedesktop.org/drm/intel/issues/165
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2295]: https://gitlab.freedesktop.org/drm/intel/issues/2295
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#2505]: https://gitlab.freedesktop.org/drm/intel/issues/2505
[i915#2574]: https://gitlab.freedesktop.org/drm/intel/issues/2574
[i915#2598]: https://gitlab.freedesktop.org/drm/intel/issues/2598
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#2680]: https://gitlab.freedesktop.org/drm/intel/issues/2680
[i915#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681
[i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684
[i915#2724]: https://gitlab.freedesktop.org/drm/intel/issues/2724
[i915#2802]: https://gitlab.freedesktop.org/drm/intel/issues/2802
[i915#2803]: https://gitlab.freedesktop.org/drm/intel/issues/2803
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846
[i915#483]: https://gitlab.freedesktop.org/drm/intel/issues/483
[i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#78]: https://gitlab.freedesktop.org/drm/intel/issues/78
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
[k.org#202321]: https://bugzilla.kernel.org/show_bug.cgi?id=202321
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_9554 -> Patchwork_19275
CI-20190529: 20190529
CI_DRM_9554: 4786f2a4c7e280974d0b46ca9e30a156eb528d59 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5945: f0ad1a564956d6796d9ff09182c48d78fb00eefe @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19275: bd6eed3a74e7aef1dde724eacb490413d8b3ee79 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19275/index.html
[-- Attachment #1.2: Type: text/html, Size: 24910 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
2021-01-06 23:41 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2021-01-07 10:13 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
@ 2021-01-12 17:51 ` Jani Nikula
2021-01-13 10:19 ` Anshuman Gupta
` (4 subsequent siblings)
7 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2021-01-12 17:51 UTC (permalink / raw)
To: Sean Paul, dri-devel, intel-gfx, Anshuman Gupta; +Cc: , David Airlie, Sean Paul
Anshuman, please review.
BR,
Jani.
On Wed, 06 Jan 2021, Sean Paul <sean@poorly.run> wrote:
> From: Sean Paul <seanpaul@chromium.org>
>
> The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
> check, it was always a nice-to-have. After deploying this across various
> devices, we've determined that some MST bridge chips do not properly
> support this call for HDCP 1.4 (namely Synaptics and Realtek).
>
> I had considered creating a quirk for this, but I think it's more
> prudent to just disable the check entirely since I don't have an idea
> how widespread support is.
>
> Signed-off-by: Sean Paul <seanpaul@chromium.org>
> ---
> drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
> 1 file changed, 1 insertion(+), 25 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> index 03424d20e9f7..b6a9606bf09a 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> @@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> return ret;
> }
>
> -static
> -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
> - struct intel_connector *connector)
> -{
> - struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
> - struct intel_dp *intel_dp = &dig_port->dp;
> - struct drm_dp_query_stream_enc_status_ack_reply reply;
> - int ret;
> -
> - if (!intel_dp_hdcp_check_link(dig_port, connector))
> - return false;
> -
> - ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
> - connector->port, &reply);
> - if (ret) {
> - drm_dbg_kms(&i915->drm,
> - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> - connector->base.base.id, connector->base.name, ret);
> - return false;
> - }
> -
> - return reply.auth_completed && reply.encryption_enabled;
> -}
> -
> static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> .read_bksv = intel_dp_hdcp_read_bksv,
> @@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> - .check_link = intel_dp_mst_hdcp_check_link,
> + .check_link = intel_dp_hdcp_check_link,
> .hdcp_capable = intel_dp_hdcp_capable,
>
> .protocol = HDCP_PROTOCOL_DP,
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
` (2 preceding siblings ...)
2021-01-12 17:51 ` [Intel-gfx] [PATCH] " Jani Nikula
@ 2021-01-13 10:19 ` Anshuman Gupta
2021-01-13 13:39 ` Sean Paul
2021-01-21 17:26 ` [Intel-gfx] [PATCH v2] " Sean Paul
` (3 subsequent siblings)
7 siblings, 1 reply; 23+ messages in thread
From: Anshuman Gupta @ 2021-01-13 10:19 UTC (permalink / raw)
To: Sean Paul; +Cc: David Airlie, intel-gfx, Sean Paul, dri-devel
On 2021-01-07 at 04:08:58 +0530, Sean Paul wrote:
> From: Sean Paul <seanpaul@chromium.org>
>
> The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
IMHO DP 1.4 vesa specs I.3.5 mark QSES as desirale for both HDCP 1.4 and HDCP 2.2.
"The MST Source device may use a QUERY_STREAM_ENCRYPTION_STATUS message
transaction to query the downstream status for a particular stream."
I feel it useful for scenario in which a non hdcp supported monitor
is hot plugged to MST branch. Source really doesn't know about the hdcp
capable device on MST branch, it just know the capability of immediate
downstream device. QSES can fetch the HDCP capability from MST topology.
We don't require to enable stream encryption for such streams.
> check, it was always a nice-to-have. After deploying this across various
> devices, we've determined that some MST bridge chips do not properly
> support this call for HDCP 1.4 (namely Synaptics and Realtek).
>
> I had considered creating a quirk for this, but I think it's more
> prudent to just disable the check entirely since I don't have an idea
> how widespread support is.
May be we can remove it from the link check and can retain as utility ?
Thanks,
Anshuman Gupta.
>
> Signed-off-by: Sean Paul <seanpaul@chromium.org>
> ---
> drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
> 1 file changed, 1 insertion(+), 25 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> index 03424d20e9f7..b6a9606bf09a 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> @@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> return ret;
> }
>
> -static
> -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
> - struct intel_connector *connector)
> -{
> - struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
> - struct intel_dp *intel_dp = &dig_port->dp;
> - struct drm_dp_query_stream_enc_status_ack_reply reply;
> - int ret;
> -
> - if (!intel_dp_hdcp_check_link(dig_port, connector))
> - return false;
> -
> - ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
> - connector->port, &reply);
> - if (ret) {
> - drm_dbg_kms(&i915->drm,
> - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> - connector->base.base.id, connector->base.name, ret);
> - return false;
> - }
> -
> - return reply.auth_completed && reply.encryption_enabled;
> -}
> -
> static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> .read_bksv = intel_dp_hdcp_read_bksv,
> @@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> - .check_link = intel_dp_mst_hdcp_check_link,
> + .check_link = intel_dp_hdcp_check_link,
> .hdcp_capable = intel_dp_hdcp_capable,
>
> .protocol = HDCP_PROTOCOL_DP,
> --
> Sean Paul, Software Engineer, Google / Chromium OS
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-13 10:19 ` Anshuman Gupta
@ 2021-01-13 13:39 ` Sean Paul
2021-01-13 14:31 ` Daniel Vetter
0 siblings, 1 reply; 23+ messages in thread
From: Sean Paul @ 2021-01-13 13:39 UTC (permalink / raw)
To: Anshuman Gupta; +Cc: David Airlie, intel-gfx, Sean Paul, dri-devel
On Wed, Jan 13, 2021 at 5:34 AM Anshuman Gupta <anshuman.gupta@intel.com> wrote:
>
> On 2021-01-07 at 04:08:58 +0530, Sean Paul wrote:
> > From: Sean Paul <seanpaul@chromium.org>
> >
> > The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
> IMHO DP 1.4 vesa specs I.3.5 mark QSES as desirale for both HDCP 1.4 and HDCP 2.2.
> "The MST Source device may use a QUERY_STREAM_ENCRYPTION_STATUS message
> transaction to query the downstream status for a particular stream."
>
> I feel it useful for scenario in which a non hdcp supported monitor
> is hot plugged to MST branch. Source really doesn't know about the hdcp
> capable device on MST branch, it just know the capability of immediate
> downstream device. QSES can fetch the HDCP capability from MST topology.
> We don't require to enable stream encryption for such streams.
I agree it's useful when it works, but unfortunately it's broken on at
least 2 MST bridge chips I've encountered :/
Until we can figure out a) how to fix them (ie: firmware updates), or
b) how to enumerate all of the broken chips to create quirks, we
probably just want to disable QSES for HDCP 1.4.
Sean
> > check, it was always a nice-to-have. After deploying this across various
> > devices, we've determined that some MST bridge chips do not properly
> > support this call for HDCP 1.4 (namely Synaptics and Realtek).
> >
> > I had considered creating a quirk for this, but I think it's more
> > prudent to just disable the check entirely since I don't have an idea
> > how widespread support is.
> May be we can remove it from the link check and can retain as utility ?
> Thanks,
> Anshuman Gupta.
> >
> > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > ---
> > drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
> > 1 file changed, 1 insertion(+), 25 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > index 03424d20e9f7..b6a9606bf09a 100644
> > --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > @@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> > return ret;
> > }
> >
> > -static
> > -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
> > - struct intel_connector *connector)
> > -{
> > - struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
> > - struct intel_dp *intel_dp = &dig_port->dp;
> > - struct drm_dp_query_stream_enc_status_ack_reply reply;
> > - int ret;
> > -
> > - if (!intel_dp_hdcp_check_link(dig_port, connector))
> > - return false;
> > -
> > - ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
> > - connector->port, &reply);
> > - if (ret) {
> > - drm_dbg_kms(&i915->drm,
> > - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> > - connector->base.base.id, connector->base.name, ret);
> > - return false;
> > - }
> > -
> > - return reply.auth_completed && reply.encryption_enabled;
> > -}
> > -
> > static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> > .read_bksv = intel_dp_hdcp_read_bksv,
> > @@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> > .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> > .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> > - .check_link = intel_dp_mst_hdcp_check_link,
> > + .check_link = intel_dp_hdcp_check_link,
> > .hdcp_capable = intel_dp_hdcp_capable,
> >
> > .protocol = HDCP_PROTOCOL_DP,
> > --
> > Sean Paul, Software Engineer, Google / Chromium OS
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-13 13:39 ` Sean Paul
@ 2021-01-13 14:31 ` Daniel Vetter
2021-01-13 18:33 ` Sean Paul
0 siblings, 1 reply; 23+ messages in thread
From: Daniel Vetter @ 2021-01-13 14:31 UTC (permalink / raw)
To: Sean Paul; +Cc: David Airlie, intel-gfx, Sean Paul, dri-devel
On Wed, Jan 13, 2021 at 2:39 PM Sean Paul <sean@poorly.run> wrote:
>
> On Wed, Jan 13, 2021 at 5:34 AM Anshuman Gupta <anshuman.gupta@intel.com> wrote:
> >
> > On 2021-01-07 at 04:08:58 +0530, Sean Paul wrote:
> > > From: Sean Paul <seanpaul@chromium.org>
> > >
> > > The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
> > IMHO DP 1.4 vesa specs I.3.5 mark QSES as desirale for both HDCP 1.4 and HDCP 2.2.
> > "The MST Source device may use a QUERY_STREAM_ENCRYPTION_STATUS message
> > transaction to query the downstream status for a particular stream."
> >
> > I feel it useful for scenario in which a non hdcp supported monitor
> > is hot plugged to MST branch. Source really doesn't know about the hdcp
> > capable device on MST branch, it just know the capability of immediate
> > downstream device. QSES can fetch the HDCP capability from MST topology.
> > We don't require to enable stream encryption for such streams.
>
> I agree it's useful when it works, but unfortunately it's broken on at
> least 2 MST bridge chips I've encountered :/
>
> Until we can figure out a) how to fix them (ie: firmware updates), or
> b) how to enumerate all of the broken chips to create quirks, we
> probably just want to disable QSES for HDCP 1.4.
What happens when the user plugs in a non-hdcp screen into a hub which
doesn't do QSES? Just black screen?
That would suck a bit, otoh with broken hw I don't see how we could do
better :-/
-Daniel
> Sean
>
> > > check, it was always a nice-to-have. After deploying this across various
> > > devices, we've determined that some MST bridge chips do not properly
> > > support this call for HDCP 1.4 (namely Synaptics and Realtek).
> > >
> > > I had considered creating a quirk for this, but I think it's more
> > > prudent to just disable the check entirely since I don't have an idea
> > > how widespread support is.
> > May be we can remove it from the link check and can retain as utility ?
> > Thanks,
> > Anshuman Gupta.
> > >
> > > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > > ---
> > > drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
> > > 1 file changed, 1 insertion(+), 25 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > index 03424d20e9f7..b6a9606bf09a 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > @@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> > > return ret;
> > > }
> > >
> > > -static
> > > -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
> > > - struct intel_connector *connector)
> > > -{
> > > - struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
> > > - struct intel_dp *intel_dp = &dig_port->dp;
> > > - struct drm_dp_query_stream_enc_status_ack_reply reply;
> > > - int ret;
> > > -
> > > - if (!intel_dp_hdcp_check_link(dig_port, connector))
> > > - return false;
> > > -
> > > - ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
> > > - connector->port, &reply);
> > > - if (ret) {
> > > - drm_dbg_kms(&i915->drm,
> > > - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> > > - connector->base.base.id, connector->base.name, ret);
> > > - return false;
> > > - }
> > > -
> > > - return reply.auth_completed && reply.encryption_enabled;
> > > -}
> > > -
> > > static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> > > .read_bksv = intel_dp_hdcp_read_bksv,
> > > @@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> > > .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> > > .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> > > - .check_link = intel_dp_mst_hdcp_check_link,
> > > + .check_link = intel_dp_hdcp_check_link,
> > > .hdcp_capable = intel_dp_hdcp_capable,
> > >
> > > .protocol = HDCP_PROTOCOL_DP,
> > > --
> > > Sean Paul, Software Engineer, Google / Chromium OS
> > >
> > > _______________________________________________
> > > Intel-gfx mailing list
> > > Intel-gfx@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-13 14:31 ` Daniel Vetter
@ 2021-01-13 18:33 ` Sean Paul
2021-01-13 20:45 ` Daniel Vetter
2021-01-15 10:22 ` Gupta, Anshuman
0 siblings, 2 replies; 23+ messages in thread
From: Sean Paul @ 2021-01-13 18:33 UTC (permalink / raw)
To: Daniel Vetter; +Cc: David Airlie, intel-gfx, Sean Paul, dri-devel
On Wed, Jan 13, 2021 at 9:31 AM Daniel Vetter <daniel@ffwll.ch> wrote:
>
> On Wed, Jan 13, 2021 at 2:39 PM Sean Paul <sean@poorly.run> wrote:
> >
> > On Wed, Jan 13, 2021 at 5:34 AM Anshuman Gupta <anshuman.gupta@intel.com> wrote:
> > >
> > > On 2021-01-07 at 04:08:58 +0530, Sean Paul wrote:
> > > > From: Sean Paul <seanpaul@chromium.org>
> > > >
> > > > The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
> > > IMHO DP 1.4 vesa specs I.3.5 mark QSES as desirale for both HDCP 1.4 and HDCP 2.2.
> > > "The MST Source device may use a QUERY_STREAM_ENCRYPTION_STATUS message
> > > transaction to query the downstream status for a particular stream."
> > >
> > > I feel it useful for scenario in which a non hdcp supported monitor
> > > is hot plugged to MST branch. Source really doesn't know about the hdcp
> > > capable device on MST branch, it just know the capability of immediate
> > > downstream device. QSES can fetch the HDCP capability from MST topology.
> > > We don't require to enable stream encryption for such streams.
> >
> > I agree it's useful when it works, but unfortunately it's broken on at
> > least 2 MST bridge chips I've encountered :/
> >
> > Until we can figure out a) how to fix them (ie: firmware updates), or
> > b) how to enumerate all of the broken chips to create quirks, we
> > probably just want to disable QSES for HDCP 1.4.
>
> What happens when the user plugs in a non-hdcp screen into a hub which
> doesn't do QSES? Just black screen?
>
Good question, thanks for forcing me to explain myself more thoroughly :)
This patch doesn't change that behavior, QSES is currently only used
as a means for verifying the stream continues to be encrypted in
steady-state (ie: after auth has already completed and the pixels are
flowing).
If one wanted to check HDCP 1.4 capability upfront, QSES wouldn't be
the way to do it. Instead you would tunnel a remote DPCD to the sink
to read the BCAPS register (ie: the same way we check non-MST
connectors).
So QSES is currently only around in HDCP 1.4 as an extra precaution
against a bug in the code preventing the MST stream from being
encrypted. IMO broken HW overrules suspenders when we already have a
belt :)
Sean
> That would suck a bit, otoh with broken hw I don't see how we could do
> better :-/
> -Daniel
>
> > Sean
> >
> > > > check, it was always a nice-to-have. After deploying this across various
> > > > devices, we've determined that some MST bridge chips do not properly
> > > > support this call for HDCP 1.4 (namely Synaptics and Realtek).
> > > >
> > > > I had considered creating a quirk for this, but I think it's more
> > > > prudent to just disable the check entirely since I don't have an idea
> > > > how widespread support is.
> > > May be we can remove it from the link check and can retain as utility ?
> > > Thanks,
> > > Anshuman Gupta.
> > > >
> > > > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > > > ---
> > > > drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
> > > > 1 file changed, 1 insertion(+), 25 deletions(-)
> > > >
> > > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > index 03424d20e9f7..b6a9606bf09a 100644
> > > > --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > @@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> > > > return ret;
> > > > }
> > > >
> > > > -static
> > > > -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
> > > > - struct intel_connector *connector)
> > > > -{
> > > > - struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
> > > > - struct intel_dp *intel_dp = &dig_port->dp;
> > > > - struct drm_dp_query_stream_enc_status_ack_reply reply;
> > > > - int ret;
> > > > -
> > > > - if (!intel_dp_hdcp_check_link(dig_port, connector))
> > > > - return false;
> > > > -
> > > > - ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
> > > > - connector->port, &reply);
> > > > - if (ret) {
> > > > - drm_dbg_kms(&i915->drm,
> > > > - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> > > > - connector->base.base.id, connector->base.name, ret);
> > > > - return false;
> > > > - }
> > > > -
> > > > - return reply.auth_completed && reply.encryption_enabled;
> > > > -}
> > > > -
> > > > static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > > .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> > > > .read_bksv = intel_dp_hdcp_read_bksv,
> > > > @@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > > .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> > > > .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> > > > .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> > > > - .check_link = intel_dp_mst_hdcp_check_link,
> > > > + .check_link = intel_dp_hdcp_check_link,
> > > > .hdcp_capable = intel_dp_hdcp_capable,
> > > >
> > > > .protocol = HDCP_PROTOCOL_DP,
> > > > --
> > > > Sean Paul, Software Engineer, Google / Chromium OS
> > > >
> > > > _______________________________________________
> > > > Intel-gfx mailing list
> > > > Intel-gfx@lists.freedesktop.org
> > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-13 18:33 ` Sean Paul
@ 2021-01-13 20:45 ` Daniel Vetter
2021-01-15 10:22 ` Gupta, Anshuman
1 sibling, 0 replies; 23+ messages in thread
From: Daniel Vetter @ 2021-01-13 20:45 UTC (permalink / raw)
To: Sean Paul; +Cc: David Airlie, intel-gfx, Sean Paul, dri-devel
On Wed, Jan 13, 2021 at 7:34 PM Sean Paul <sean@poorly.run> wrote:
>
> On Wed, Jan 13, 2021 at 9:31 AM Daniel Vetter <daniel@ffwll.ch> wrote:
> >
> > On Wed, Jan 13, 2021 at 2:39 PM Sean Paul <sean@poorly.run> wrote:
> > >
> > > On Wed, Jan 13, 2021 at 5:34 AM Anshuman Gupta <anshuman.gupta@intel.com> wrote:
> > > >
> > > > On 2021-01-07 at 04:08:58 +0530, Sean Paul wrote:
> > > > > From: Sean Paul <seanpaul@chromium.org>
> > > > >
> > > > > The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
> > > > IMHO DP 1.4 vesa specs I.3.5 mark QSES as desirale for both HDCP 1.4 and HDCP 2.2.
> > > > "The MST Source device may use a QUERY_STREAM_ENCRYPTION_STATUS message
> > > > transaction to query the downstream status for a particular stream."
> > > >
> > > > I feel it useful for scenario in which a non hdcp supported monitor
> > > > is hot plugged to MST branch. Source really doesn't know about the hdcp
> > > > capable device on MST branch, it just know the capability of immediate
> > > > downstream device. QSES can fetch the HDCP capability from MST topology.
> > > > We don't require to enable stream encryption for such streams.
> > >
> > > I agree it's useful when it works, but unfortunately it's broken on at
> > > least 2 MST bridge chips I've encountered :/
> > >
> > > Until we can figure out a) how to fix them (ie: firmware updates), or
> > > b) how to enumerate all of the broken chips to create quirks, we
> > > probably just want to disable QSES for HDCP 1.4.
> >
> > What happens when the user plugs in a non-hdcp screen into a hub which
> > doesn't do QSES? Just black screen?
> >
>
> Good question, thanks for forcing me to explain myself more thoroughly :)
>
> This patch doesn't change that behavior, QSES is currently only used
> as a means for verifying the stream continues to be encrypted in
> steady-state (ie: after auth has already completed and the pixels are
> flowing).
>
> If one wanted to check HDCP 1.4 capability upfront, QSES wouldn't be
> the way to do it. Instead you would tunnel a remote DPCD to the sink
> to read the BCAPS register (ie: the same way we check non-MST
> connectors).
>
> So QSES is currently only around in HDCP 1.4 as an extra precaution
> against a bug in the code preventing the MST stream from being
> encrypted. IMO broken HW overrules suspenders when we already have a
> belt :)
I think with the above explanation added to the commit message this
makes sense to merge. Fwiw, i.e. not much: Acked-by: Daniel Vetter
<daniel.vetter@ffwll.ch>
Cheers, Daniel
>
>
> Sean
>
> > That would suck a bit, otoh with broken hw I don't see how we could do
> > better :-/
> > -Daniel
> >
> > > Sean
> > >
> > > > > check, it was always a nice-to-have. After deploying this across various
> > > > > devices, we've determined that some MST bridge chips do not properly
> > > > > support this call for HDCP 1.4 (namely Synaptics and Realtek).
> > > > >
> > > > > I had considered creating a quirk for this, but I think it's more
> > > > > prudent to just disable the check entirely since I don't have an idea
> > > > > how widespread support is.
> > > > May be we can remove it from the link check and can retain as utility ?
> > > > Thanks,
> > > > Anshuman Gupta.
> > > > >
> > > > > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > > > > ---
> > > > > drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26 +-------------------
> > > > > 1 file changed, 1 insertion(+), 25 deletions(-)
> > > > >
> > > > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > index 03424d20e9f7..b6a9606bf09a 100644
> > > > > --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > @@ -640,30 +640,6 @@ intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> > > > > return ret;
> > > > > }
> > > > >
> > > > > -static
> > > > > -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
> > > > > - struct intel_connector *connector)
> > > > > -{
> > > > > - struct drm_i915_private *i915 = to_i915(dig_port->base.base.dev);
> > > > > - struct intel_dp *intel_dp = &dig_port->dp;
> > > > > - struct drm_dp_query_stream_enc_status_ack_reply reply;
> > > > > - int ret;
> > > > > -
> > > > > - if (!intel_dp_hdcp_check_link(dig_port, connector))
> > > > > - return false;
> > > > > -
> > > > > - ret = drm_dp_send_query_stream_enc_status(&intel_dp->mst_mgr,
> > > > > - connector->port, &reply);
> > > > > - if (ret) {
> > > > > - drm_dbg_kms(&i915->drm,
> > > > > - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> > > > > - connector->base.base.id, connector->base.name, ret);
> > > > > - return false;
> > > > > - }
> > > > > -
> > > > > - return reply.auth_completed && reply.encryption_enabled;
> > > > > -}
> > > > > -
> > > > > static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > > > .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> > > > > .read_bksv = intel_dp_hdcp_read_bksv,
> > > > > @@ -674,7 +650,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > > > .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> > > > > .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> > > > > .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> > > > > - .check_link = intel_dp_mst_hdcp_check_link,
> > > > > + .check_link = intel_dp_hdcp_check_link,
> > > > > .hdcp_capable = intel_dp_hdcp_capable,
> > > > >
> > > > > .protocol = HDCP_PROTOCOL_DP,
> > > > > --
> > > > > Sean Paul, Software Engineer, Google / Chromium OS
> > > > >
> > > > > _______________________________________________
> > > > > Intel-gfx mailing list
> > > > > Intel-gfx@lists.freedesktop.org
> > > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
> >
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-13 18:33 ` Sean Paul
2021-01-13 20:45 ` Daniel Vetter
@ 2021-01-15 10:22 ` Gupta, Anshuman
1 sibling, 0 replies; 23+ messages in thread
From: Gupta, Anshuman @ 2021-01-15 10:22 UTC (permalink / raw)
To: Sean Paul, Daniel Vetter; +Cc: David Airlie, intel-gfx, Sean Paul, dri-devel
> -----Original Message-----
> From: Sean Paul <sean@poorly.run>
> Sent: Thursday, January 14, 2021 12:04 AM
> To: Daniel Vetter <daniel@ffwll.ch>
> Cc: Gupta, Anshuman <anshuman.gupta@intel.com>; David Airlie
> <airlied@linux.ie>; intel-gfx@lists.freedesktop.org; Sean Paul
> <seanpaul@chromium.org>; dri-devel@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for
> HDCP 1.4 over MST
>
> On Wed, Jan 13, 2021 at 9:31 AM Daniel Vetter <daniel@ffwll.ch> wrote:
> >
> > On Wed, Jan 13, 2021 at 2:39 PM Sean Paul <sean@poorly.run> wrote:
> > >
> > > On Wed, Jan 13, 2021 at 5:34 AM Anshuman Gupta
> <anshuman.gupta@intel.com> wrote:
> > > >
> > > > On 2021-01-07 at 04:08:58 +0530, Sean Paul wrote:
> > > > > From: Sean Paul <seanpaul@chromium.org>
> > > > >
> > > > > The HDCP 1.4 spec does not require the
> > > > > QUERY_STREAM_ENCRYPTION_STATUS
> > > > IMHO DP 1.4 vesa specs I.3.5 mark QSES as desirale for both HDCP 1.4
> and HDCP 2.2.
> > > > "The MST Source device may use a
> QUERY_STREAM_ENCRYPTION_STATUS
> > > > message transaction to query the downstream status for a particular
> stream."
> > > >
> > > > I feel it useful for scenario in which a non hdcp supported
> > > > monitor is hot plugged to MST branch. Source really doesn't know
> > > > about the hdcp capable device on MST branch, it just know the
> > > > capability of immediate downstream device. QSES can fetch the HDCP
> capability from MST topology.
> > > > We don't require to enable stream encryption for such streams.
> > >
> > > I agree it's useful when it works, but unfortunately it's broken on
> > > at least 2 MST bridge chips I've encountered :/
> > >
> > > Until we can figure out a) how to fix them (ie: firmware updates),
> > > or
> > > b) how to enumerate all of the broken chips to create quirks, we
> > > probably just want to disable QSES for HDCP 1.4.
> >
> > What happens when the user plugs in a non-hdcp screen into a hub which
> > doesn't do QSES? Just black screen?
> >
>
> Good question, thanks for forcing me to explain myself more thoroughly :)
>
> This patch doesn't change that behavior, QSES is currently only used as a
> means for verifying the stream continues to be encrypted in steady-state
> (ie: after auth has already completed and the pixels are flowing).
>
> If one wanted to check HDCP 1.4 capability upfront, QSES wouldn't be the
> way to do it. Instead you would tunnel a remote DPCD to the sink to read
> the BCAPS register (ie: the same way we check non-MST connectors).
AFAIK in case of MST topology source can only get BCAPS/RXCAPS of the immediate downstream sink
(in this particular case it will be BCAPS/RXCAPS of MST hub).
Ex. I have a HDCP 2.2 (DP-6 immediate downstream) and HDCP 1.4 (DP-7) connector setup in daisy chain setup, DP-7 just reports the RXCAPS of DP-6.
cat /sys/kernel/debug/dri/0/DP-7/i915_hdcp_sink_capability
DP-7:369 HDCP version: HDCP1.4 HDCP2.2
root@linux-Tiger-Lake-Client-Platform:/home/linux# dmesg
[180919.808209] [drm:drm_dp_dpcd_read] AUX B/DDI B/PHY B: 0x68028 AUX -> (ret= 1) 03
[180919.809089] [drm:drm_dp_dpcd_read] AUX B/DDI B/PHY B: 0x6921d AUX -> (ret= 3) 02 00 03
I was planning to use QSES to query HDCP in MST topology but now not sure what to do with not supported MST hubs.
However agree with you for this patch, it seems to disable the QSES link check the right thing to do.
It would require to rebase.
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
>
> So QSES is currently only around in HDCP 1.4 as an extra precaution against
> a bug in the code preventing the MST stream from being encrypted. IMO
> broken HW overrules suspenders when we already have a belt :)
>
>
> Sean
>
> > That would suck a bit, otoh with broken hw I don't see how we could do
> > better :-/ -Daniel
> >
> > > Sean
> > >
> > > > > check, it was always a nice-to-have. After deploying this across
> > > > > various devices, we've determined that some MST bridge chips do
> > > > > not properly support this call for HDCP 1.4 (namely Synaptics and
> Realtek).
> > > > >
> > > > > I had considered creating a quirk for this, but I think it's
> > > > > more prudent to just disable the check entirely since I don't
> > > > > have an idea how widespread support is.
> > > > May be we can remove it from the link check and can retain as utility
> ?
> > > > Thanks,
> > > > Anshuman Gupta.
> > > > >
> > > > > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > > > > ---
> > > > > drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 26
> > > > > +-------------------
> > > > > 1 file changed, 1 insertion(+), 25 deletions(-)
> > > > >
> > > > > diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > index 03424d20e9f7..b6a9606bf09a 100644
> > > > > --- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > +++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
> > > > > @@ -640,30 +640,6 @@
> intel_dp_mst_hdcp_toggle_signalling(struct intel_digital_port *dig_port,
> > > > > return ret;
> > > > > }
> > > > >
> > > > > -static
> > > > > -bool intel_dp_mst_hdcp_check_link(struct intel_digital_port
> *dig_port,
> > > > > - struct intel_connector *connector)
> > > > > -{
> > > > > - struct drm_i915_private *i915 = to_i915(dig_port-
> >base.base.dev);
> > > > > - struct intel_dp *intel_dp = &dig_port->dp;
> > > > > - struct drm_dp_query_stream_enc_status_ack_reply reply;
> > > > > - int ret;
> > > > > -
> > > > > - if (!intel_dp_hdcp_check_link(dig_port, connector))
> > > > > - return false;
> > > > > -
> > > > > - ret = drm_dp_send_query_stream_enc_status(&intel_dp-
> >mst_mgr,
> > > > > - connector->port, &reply);
> > > > > - if (ret) {
> > > > > - drm_dbg_kms(&i915->drm,
> > > > > - "[CONNECTOR:%d:%s] failed QSES ret=%d\n",
> > > > > - connector->base.base.id, connector->base.name,
> ret);
> > > > > - return false;
> > > > > - }
> > > > > -
> > > > > - return reply.auth_completed && reply.encryption_enabled;
> > > > > -}
> > > > > -
> > > > > static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > > > .write_an_aksv = intel_dp_hdcp_write_an_aksv,
> > > > > .read_bksv = intel_dp_hdcp_read_bksv, @@ -674,7 +650,7 @@
> > > > > static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
> > > > > .read_ksv_fifo = intel_dp_hdcp_read_ksv_fifo,
> > > > > .read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
> > > > > .toggle_signalling = intel_dp_mst_hdcp_toggle_signalling,
> > > > > - .check_link = intel_dp_mst_hdcp_check_link,
> > > > > + .check_link = intel_dp_hdcp_check_link,
> > > > > .hdcp_capable = intel_dp_hdcp_capable,
> > > > >
> > > > > .protocol = HDCP_PROTOCOL_DP,
> > > > > --
> > > > > Sean Paul, Software Engineer, Google / Chromium OS
> > > > >
> > > > > _______________________________________________
> > > > > Intel-gfx mailing list
> > > > > Intel-gfx@lists.freedesktop.org
> > > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
> >
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Intel-gfx] [PATCH v2] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
` (3 preceding siblings ...)
2021-01-13 10:19 ` Anshuman Gupta
@ 2021-01-21 17:26 ` Sean Paul
2021-01-22 2:42 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2) Patchwork
` (2 subsequent siblings)
7 siblings, 0 replies; 23+ messages in thread
From: Sean Paul @ 2021-01-21 17:26 UTC (permalink / raw)
To: dri-devel, intel-gfx; +Cc: Daniel Vetter, Sean Paul
From: Sean Paul <seanpaul@chromium.org>
The HDCP 1.4 spec does not require the QUERY_STREAM_ENCRYPTION_STATUS
check, it was always a nice-to-have. After deploying this across various
devices, we've determined that some MST bridge chips do not properly
support this call for HDCP 1.4 (namely Synaptics and Realtek).
I had considered creating a quirk for this, but I think it's more
prudent to just disable the check entirely since I don't have an idea
how widespread support is.
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210106223909.34476-1-sean@poorly.run #v1
Changes in v2:
-Rebased on -tip
---
drivers/gpu/drm/i915/display/intel_dp_hdcp.c | 12 +-----------
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
index f372e25edab4..4dba5bb15af5 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_hdcp.c
@@ -722,16 +722,6 @@ static bool intel_dp_mst_get_qses_status(struct intel_digital_port *dig_port,
return reply.auth_completed && reply.encryption_enabled;
}
-static
-bool intel_dp_mst_hdcp_check_link(struct intel_digital_port *dig_port,
- struct intel_connector *connector)
-{
- if (!intel_dp_hdcp_check_link(dig_port, connector))
- return false;
-
- return intel_dp_mst_get_qses_status(dig_port, connector);
-}
-
static int
intel_dp_mst_hdcp2_stream_encryption(struct intel_connector *connector,
bool enable)
@@ -805,7 +795,7 @@ static const struct intel_hdcp_shim intel_dp_mst_hdcp_shim = {
.read_v_prime_part = intel_dp_hdcp_read_v_prime_part,
.toggle_signalling = intel_dp_hdcp_toggle_signalling,
.stream_encryption = intel_dp_mst_hdcp_stream_encryption,
- .check_link = intel_dp_mst_hdcp_check_link,
+ .check_link = intel_dp_hdcp_check_link,
.hdcp_capable = intel_dp_hdcp_capable,
.write_2_2_msg = intel_dp_hdcp2_write_msg,
.read_2_2_msg = intel_dp_hdcp2_read_msg,
--
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 23+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
` (4 preceding siblings ...)
2021-01-21 17:26 ` [Intel-gfx] [PATCH v2] " Sean Paul
@ 2021-01-22 2:42 ` Patchwork
2021-01-25 15:26 ` Gupta, Anshuman
2021-01-25 17:02 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-01-28 7:05 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
7 siblings, 1 reply; 23+ messages in thread
From: Patchwork @ 2021-01-22 2:42 UTC (permalink / raw)
To: Sean Paul; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 4444 bytes --]
== Series Details ==
Series: drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL : https://patchwork.freedesktop.org/series/85555/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_19447 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_19447, 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/Patchwork_19447/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_19447:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live@gt_lrc:
- fi-bsw-n3050: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
Known issues
------------
Here are the changes found in Patchwork_19447 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_mmap_gtt@basic:
- fi-tgl-y: [PASS][3] -> [DMESG-WARN][4] ([i915#402]) +2 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_mmap_gtt@basic.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html
* igt@i915_selftest@live@hugepages:
- fi-bsw-n3050: [PASS][5] -> [SKIP][6] ([fdo#109271]) +20 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html
* igt@i915_selftest@live@ring_submission:
- fi-bsw-n3050: NOTRUN -> [SKIP][7] ([fdo#109271]) +2 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@ring_submission.html
* igt@kms_chamelium@dp-crc-fast:
- fi-kbl-7500u: [PASS][8] -> [FAIL][9] ([i915#1161] / [i915#262])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
#### Possible fixes ####
* igt@gem_tiled_blits@basic:
- fi-tgl-y: [DMESG-WARN][10] ([i915#402]) -> [PASS][11] +1 similar issue
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_tiled_blits@basic.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_tiled_blits@basic.html
#### Warnings ####
* igt@i915_selftest@live@execlists:
- fi-bsw-n3050: [INCOMPLETE][12] ([i915#2940]) -> [SKIP][13] ([fdo#109271])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1161]: https://gitlab.freedesktop.org/drm/intel/issues/1161
[i915#262]: https://gitlab.freedesktop.org/drm/intel/issues/262
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
Participating hosts (41 -> 37)
------------------------------
Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u
Build changes
-------------
* Linux: CI_DRM_9666 -> Patchwork_19447
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
[-- Attachment #1.2: Type: text/html, Size: 5412 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-22 2:42 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2) Patchwork
@ 2021-01-25 15:26 ` Gupta, Anshuman
2021-01-25 16:41 ` Vudum, Lakshminarayana
2021-01-25 17:10 ` Vudum, Lakshminarayana
0 siblings, 2 replies; 23+ messages in thread
From: Gupta, Anshuman @ 2021-01-25 15:26 UTC (permalink / raw)
To: intel-gfx, Vudum, Lakshminarayana
Hi Lakshmi ,
We need to merge the patch https://patchwork.freedesktop.org/series/85555/
Below are the gem failures causing the BAT to fail , those were not seen on Rev1 of this patch.
Could you please rereport-the results to trigger CI-IGT run.
Thanks,
Anshuman Gupta.
From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Patchwork
Sent: Friday, January 22, 2021 8:13 AM
To: Sean Paul <sean@poorly.run>
Cc: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Patch Details
Series:
drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL:
https://patchwork.freedesktop.org/series/85555/
State:
failure
Details:
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447
Summary
FAILURE
Serious unknown changes coming with Patchwork_19447 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_19447, 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/Patchwork_19447/index.html
Possible new issues
Here are the unknown changes that may have been introduced in Patchwork_19447:
IGT changes
Possible regressions
• igt@i915_selftest@live@gt_lrc:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
Known issues
Here are the changes found in Patchwork_19447 that come from known issues:
IGT changes
Issues hit
• igt@gem_mmap_gtt@basic:
o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar issues
• igt@i915_selftest@live@hugepages:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar issues
• igt@i915_selftest@live@ring_submission:
o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@ring_submission.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar issues
• igt@kms_chamelium@dp-crc-fast:
o fi-kbl-7500u: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html (https://gitlab.freedesktop.org/drm/intel/issues/1161 / https://gitlab.freedesktop.org/drm/intel/issues/262)
Possible fixes
• igt@gem_tiled_blits@basic:
o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_tiled_blits@basic.html (https://gitlab.freedesktop.org/drm/intel/issues/402) -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_tiled_blits@basic.html +1 similar issue
Warnings
• igt@i915_selftest@live@execlists:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@execlists.html (https://gitlab.freedesktop.org/drm/intel/issues/2940) -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@execlists.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
Participating hosts (41 -> 37)
Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u
Build changes
• Linux: CI_DRM_9666 -> Patchwork_19447
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-25 15:26 ` Gupta, Anshuman
@ 2021-01-25 16:41 ` Vudum, Lakshminarayana
2021-01-25 17:10 ` Vudum, Lakshminarayana
1 sibling, 0 replies; 23+ messages in thread
From: Vudum, Lakshminarayana @ 2021-01-25 16:41 UTC (permalink / raw)
To: Gupta, Anshuman, intel-gfx
Hi,
CI bug log crashes while creating an issue but I have created bug in gitlab. I will try to create issue in CI bug log again later today.
https://gitlab.freedesktop.org/drm/intel/-/issues/2998
Thanks,
Lakshmi.
-----Original Message-----
From: Gupta, Anshuman <anshuman.gupta@intel.com>
Sent: Monday, January 25, 2021 7:26 AM
To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Cc: Sean Paul <sean@poorly.run>
Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Hi Lakshmi ,
We need to merge the patch https://patchwork.freedesktop.org/series/85555/
Below are the gem failures causing the BAT to fail , those were not seen on Rev1 of this patch.
Could you please rereport-the results to trigger CI-IGT run.
Thanks,
Anshuman Gupta.
From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Patchwork
Sent: Friday, January 22, 2021 8:13 AM
To: Sean Paul <sean@poorly.run>
Cc: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Patch Details
Series:
drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL:
https://patchwork.freedesktop.org/series/85555/
State:
failure
Details:
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447 Summary FAILURE Serious unknown changes coming with Patchwork_19447 absolutely need to be verified manually.
If you think the reported changes have nothing to do with the changes introduced in Patchwork_19447, 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/Patchwork_19447/index.html
Possible new issues
Here are the unknown changes that may have been introduced in Patchwork_19447:
IGT changes
Possible regressions
• igt@i915_selftest@live@gt_lrc:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
Known issues
Here are the changes found in Patchwork_19447 that come from known issues:
IGT changes
Issues hit
• igt@gem_mmap_gtt@basic:
o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar issues • igt@i915_selftest@live@hugepages:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar issues • igt@i915_selftest@live@ring_submission:
o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@ring_submission.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar issues • igt@kms_chamelium@dp-crc-fast:
o fi-kbl-7500u: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html (https://gitlab.freedesktop.org/drm/intel/issues/1161 / https://gitlab.freedesktop.org/drm/intel/issues/262)
Possible fixes
• igt@gem_tiled_blits@basic:
o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_tiled_blits@basic.html (https://gitlab.freedesktop.org/drm/intel/issues/402) -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_tiled_blits@basic.html +1 similar issue Warnings • igt@i915_selftest@live@execlists:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@execlists.html (https://gitlab.freedesktop.org/drm/intel/issues/2940) -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@execlists.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
Participating hosts (41 -> 37)
Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u Build changes • Linux: CI_DRM_9666 -> Patchwork_19447
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
` (5 preceding siblings ...)
2021-01-22 2:42 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2) Patchwork
@ 2021-01-25 17:02 ` Patchwork
2021-01-28 7:05 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
7 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2021-01-25 17:02 UTC (permalink / raw)
To: Sean Paul; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 4066 bytes --]
== Series Details ==
Series: drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL : https://patchwork.freedesktop.org/series/85555/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
Known issues
------------
Here are the changes found in Patchwork_19447 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_mmap_gtt@basic:
- fi-tgl-y: [PASS][1] -> [DMESG-WARN][2] ([i915#402]) +2 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_mmap_gtt@basic.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html
* igt@i915_selftest@live@gt_lrc:
- fi-bsw-n3050: [PASS][3] -> [DMESG-FAIL][4] ([i915#2998])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
* igt@i915_selftest@live@hugepages:
- fi-bsw-n3050: [PASS][5] -> [SKIP][6] ([fdo#109271]) +20 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html
* igt@i915_selftest@live@ring_submission:
- fi-bsw-n3050: NOTRUN -> [SKIP][7] ([fdo#109271]) +2 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@ring_submission.html
* igt@kms_chamelium@dp-crc-fast:
- fi-kbl-7500u: [PASS][8] -> [FAIL][9] ([i915#1161] / [i915#262])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
#### Possible fixes ####
* igt@gem_tiled_blits@basic:
- fi-tgl-y: [DMESG-WARN][10] ([i915#402]) -> [PASS][11] +1 similar issue
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_tiled_blits@basic.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_tiled_blits@basic.html
#### Warnings ####
* igt@i915_selftest@live@execlists:
- fi-bsw-n3050: [INCOMPLETE][12] ([i915#2940]) -> [SKIP][13] ([fdo#109271])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1161]: https://gitlab.freedesktop.org/drm/intel/issues/1161
[i915#262]: https://gitlab.freedesktop.org/drm/intel/issues/262
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#2998]: https://gitlab.freedesktop.org/drm/intel/issues/2998
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
Participating hosts (43 -> 37)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-bdw-samus bat-jsl-2 bat-jsl-1
Build changes
-------------
* Linux: CI_DRM_9666 -> Patchwork_19447
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
[-- Attachment #1.2: Type: text/html, Size: 5022 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-25 15:26 ` Gupta, Anshuman
2021-01-25 16:41 ` Vudum, Lakshminarayana
@ 2021-01-25 17:10 ` Vudum, Lakshminarayana
2021-01-27 5:20 ` Gupta, Anshuman
1 sibling, 1 reply; 23+ messages in thread
From: Vudum, Lakshminarayana @ 2021-01-25 17:10 UTC (permalink / raw)
To: Gupta, Anshuman, intel-gfx
Re-reported.
-----Original Message-----
From: Gupta, Anshuman <anshuman.gupta@intel.com>
Sent: Monday, January 25, 2021 7:26 AM
To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Cc: Sean Paul <sean@poorly.run>
Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Hi Lakshmi ,
We need to merge the patch https://patchwork.freedesktop.org/series/85555/
Below are the gem failures causing the BAT to fail , those were not seen on Rev1 of this patch.
Could you please rereport-the results to trigger CI-IGT run.
Thanks,
Anshuman Gupta.
From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Patchwork
Sent: Friday, January 22, 2021 8:13 AM
To: Sean Paul <sean@poorly.run>
Cc: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Patch Details
Series:
drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL:
https://patchwork.freedesktop.org/series/85555/
State:
failure
Details:
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447 Summary FAILURE Serious unknown changes coming with Patchwork_19447 absolutely need to be verified manually.
If you think the reported changes have nothing to do with the changes introduced in Patchwork_19447, 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/Patchwork_19447/index.html
Possible new issues
Here are the unknown changes that may have been introduced in Patchwork_19447:
IGT changes
Possible regressions
• igt@i915_selftest@live@gt_lrc:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
Known issues
Here are the changes found in Patchwork_19447 that come from known issues:
IGT changes
Issues hit
• igt@gem_mmap_gtt@basic:
o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar issues • igt@i915_selftest@live@hugepages:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@hugepages.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar issues • igt@i915_selftest@live@ring_submission:
o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@ring_submission.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar issues • igt@kms_chamelium@dp-crc-fast:
o fi-kbl-7500u: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html (https://gitlab.freedesktop.org/drm/intel/issues/1161 / https://gitlab.freedesktop.org/drm/intel/issues/262)
Possible fixes
• igt@gem_tiled_blits@basic:
o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-y/igt@gem_tiled_blits@basic.html (https://gitlab.freedesktop.org/drm/intel/issues/402) -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-y/igt@gem_tiled_blits@basic.html +1 similar issue Warnings • igt@i915_selftest@live@execlists:
o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-bsw-n3050/igt@i915_selftest@live@execlists.html (https://gitlab.freedesktop.org/drm/intel/issues/2940) -> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-n3050/igt@i915_selftest@live@execlists.html (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
Participating hosts (41 -> 37)
Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u Build changes • Linux: CI_DRM_9666 -> Patchwork_19447
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-25 17:10 ` Vudum, Lakshminarayana
@ 2021-01-27 5:20 ` Gupta, Anshuman
2021-01-27 5:29 ` Vudum, Lakshminarayana
0 siblings, 1 reply; 23+ messages in thread
From: Gupta, Anshuman @ 2021-01-27 5:20 UTC (permalink / raw)
To: Vudum, Lakshminarayana, intel-gfx
Thanks Lakshmi for re-reporting it.
May I know that re-reporting will also trigger a CI-IGT run.
I am not able to see this series in CI Queue https://intel-gfx-ci.01.org/queue/index.html
Currently this patch Rev2 is not showing any CI-IGT results.
Br,
Anshuman
> -----Original Message-----
> From: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> Sent: Monday, January 25, 2021 10:40 PM
> To: Gupta, Anshuman <anshuman.gupta@intel.com>; intel-
> gfx@lists.freedesktop.org
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the
> QSES check for HDCP 1.4 over MST (rev2)
>
> Re-reported.
>
> -----Original Message-----
> From: Gupta, Anshuman <anshuman.gupta@intel.com>
> Sent: Monday, January 25, 2021 7:26 AM
> To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana
> <lakshminarayana.vudum@intel.com>
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the
> QSES check for HDCP 1.4 over MST (rev2)
>
> Hi Lakshmi ,
> We need to merge the patch
> https://patchwork.freedesktop.org/series/85555/
> Below are the gem failures causing the BAT to fail , those were not seen on
> Rev1 of this patch.
> Could you please rereport-the results to trigger CI-IGT run.
> Thanks,
> Anshuman Gupta.
>
> From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of
> Patchwork
> Sent: Friday, January 22, 2021 8:13 AM
> To: Sean Paul <sean@poorly.run>
> Cc: intel-gfx@lists.freedesktop.org
> Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES
> check for HDCP 1.4 over MST (rev2)
>
> Patch Details
> Series:
> drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
> URL:
> https://patchwork.freedesktop.org/series/85555/
> State:
> failure
> Details:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
> CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447 Summary
> FAILURE Serious unknown changes coming with Patchwork_19447
> absolutely need to be verified manually.
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_19447, 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/Patchwork_19447/index.html
> Possible new issues
> Here are the unknown changes that may have been introduced in
> Patchwork_19447:
> IGT changes
> Possible regressions
> • igt@i915_selftest@live@gt_lrc:
> o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@gt_lrc.html
> Known issues
> Here are the changes found in Patchwork_19447 that come from known
> issues:
> IGT changes
> Issues hit
> • igt@gem_mmap_gtt@basic:
> o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-ci.01.org/tree/drm-
> tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html
> (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar issues •
> igt@i915_selftest@live@hugepages:
> o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> bsw-n3050/igt@i915_selftest@live@hugepages.html -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@hugepages.html
> (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar issues
> • igt@i915_selftest@live@ring_submission:
> o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-
> tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@ring_submission.html
> (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar issues
> • igt@kms_chamelium@dp-crc-fast:
> o fi-kbl-7500u: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-
> 7500u/igt@kms_chamelium@dp-crc-fast.html
> (https://gitlab.freedesktop.org/drm/intel/issues/1161 /
> https://gitlab.freedesktop.org/drm/intel/issues/262)
> Possible fixes
> • igt@gem_tiled_blits@basic:
> o fi-tgl-y: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> y/igt@gem_tiled_blits@basic.html
> (https://gitlab.freedesktop.org/drm/intel/issues/402) -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-
> y/igt@gem_tiled_blits@basic.html +1 similar issue Warnings •
> igt@i915_selftest@live@execlists:
> o fi-bsw-n3050: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> bsw-n3050/igt@i915_selftest@live@execlists.html
> (https://gitlab.freedesktop.org/drm/intel/issues/2940) -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@execlists.html
> (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
> Participating hosts (41 -> 37)
> Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u Build
> changes • Linux: CI_DRM_9666 -> Patchwork_19447
> CI-20190529: 20190529
> CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @
> git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @
> git://anongit.freedesktop.org/gfx-ci/linux
> == Linux commits ==
> 69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over
> MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-27 5:20 ` Gupta, Anshuman
@ 2021-01-27 5:29 ` Vudum, Lakshminarayana
2021-01-27 7:37 ` Gupta, Anshuman
0 siblings, 1 reply; 23+ messages in thread
From: Vudum, Lakshminarayana @ 2021-01-27 5:29 UTC (permalink / raw)
To: Gupta, Anshuman, intel-gfx
Re-reporting will just re-report the results once more time. Re-reporting will re-check if there is an existing bug for the regression failures or not. If a bug is found, series shows as success (as the failure is not a regression). Regression failures will find a bug when a re-report as I either file a new bug or I will associate the regression failure to an existing bug before I re-report.
But re-reporting will not trigger re-run to see version 2 results. Let me know if you have any further questions.
Thanks,
Lakshmi.
-----Original Message-----
From: Gupta, Anshuman <anshuman.gupta@intel.com>
Sent: Tuesday, January 26, 2021 9:20 PM
To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>; intel-gfx@lists.freedesktop.org
Cc: Sean Paul <sean@poorly.run>
Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Thanks Lakshmi for re-reporting it.
May I know that re-reporting will also trigger a CI-IGT run.
I am not able to see this series in CI Queue https://intel-gfx-ci.01.org/queue/index.html
Currently this patch Rev2 is not showing any CI-IGT results.
Br,
Anshuman
> -----Original Message-----
> From: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> Sent: Monday, January 25, 2021 10:40 PM
> To: Gupta, Anshuman <anshuman.gupta@intel.com>; intel-
> gfx@lists.freedesktop.org
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> Disable the QSES check for HDCP 1.4 over MST (rev2)
>
> Re-reported.
>
> -----Original Message-----
> From: Gupta, Anshuman <anshuman.gupta@intel.com>
> Sent: Monday, January 25, 2021 7:26 AM
> To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana
> <lakshminarayana.vudum@intel.com>
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> Disable the QSES check for HDCP 1.4 over MST (rev2)
>
> Hi Lakshmi ,
> We need to merge the patch
> https://patchwork.freedesktop.org/series/85555/
> Below are the gem failures causing the BAT to fail , those were not
> seen on
> Rev1 of this patch.
> Could you please rereport-the results to trigger CI-IGT run.
> Thanks,
> Anshuman Gupta.
>
> From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of
> Patchwork
> Sent: Friday, January 22, 2021 8:13 AM
> To: Sean Paul <sean@poorly.run>
> Cc: intel-gfx@lists.freedesktop.org
> Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable
> the QSES check for HDCP 1.4 over MST (rev2)
>
> Patch Details
> Series:
> drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
> URL:
> https://patchwork.freedesktop.org/series/85555/
> State:
> failure
> Details:
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
> CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447 Summary
> FAILURE Serious unknown changes coming with Patchwork_19447 absolutely
> need to be verified manually.
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_19447, 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/Patchwork_19447/index.html
> Possible new issues
> Here are the unknown changes that may have been introduced in
> Patchwork_19447:
> IGT changes
> Possible regressions
> • igt@i915_selftest@live@gt_lrc:
> o fi-bsw-n3050:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@gt_lrc.html
> Known issues
> Here are the changes found in Patchwork_19447 that come from known
> issues:
> IGT changes
> Issues hit
> • igt@gem_mmap_gtt@basic:
> o fi-tgl-y:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-ci.01.org/tree/drm-
> tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html
> (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar
> issues •
> igt@i915_selftest@live@hugepages:
> o fi-bsw-n3050:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> bsw-n3050/igt@i915_selftest@live@hugepages.html -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@hugepages.html
> (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar
> issues • igt@i915_selftest@live@ring_submission:
> o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-
> tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@ring_submission.html
> (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar
> issues • igt@kms_chamelium@dp-crc-fast:
> o fi-kbl-7500u:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-
> 7500u/igt@kms_chamelium@dp-crc-fast.html
> (https://gitlab.freedesktop.org/drm/intel/issues/1161 /
> https://gitlab.freedesktop.org/drm/intel/issues/262)
> Possible fixes
> • igt@gem_tiled_blits@basic:
> o fi-tgl-y:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> y/igt@gem_tiled_blits@basic.html
> (https://gitlab.freedesktop.org/drm/intel/issues/402) ->
> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-
> y/igt@gem_tiled_blits@basic.html +1 similar issue Warnings •
> igt@i915_selftest@live@execlists:
> o fi-bsw-n3050:
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> bsw-n3050/igt@i915_selftest@live@execlists.html
> (https://gitlab.freedesktop.org/drm/intel/issues/2940) ->
> https://intel-gfx-
> ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> n3050/igt@i915_selftest@live@execlists.html
> (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
> Participating hosts (41 -> 37)
> Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u Build
> changes • Linux: CI_DRM_9666 -> Patchwork_19447
> CI-20190529: 20190529
> CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @
> git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @
> git://anongit.freedesktop.org/gfx-ci/linux
> == Linux commits ==
> 69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over
> MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-27 5:29 ` Vudum, Lakshminarayana
@ 2021-01-27 7:37 ` Gupta, Anshuman
2021-01-27 19:13 ` Vudum, Lakshminarayana
0 siblings, 1 reply; 23+ messages in thread
From: Gupta, Anshuman @ 2021-01-27 7:37 UTC (permalink / raw)
To: Vudum, Lakshminarayana, intel-gfx
> -----Original Message-----
> From: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> Sent: Wednesday, January 27, 2021 11:00 AM
> To: Gupta, Anshuman <anshuman.gupta@intel.com>; intel-
> gfx@lists.freedesktop.org
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the
> QSES check for HDCP 1.4 over MST (rev2)
>
> Re-reporting will just re-report the results once more time. Re-reporting will
> re-check if there is an existing bug for the regression failures or not. If a bug
> is found, series shows as success (as the failure is not a regression).
> Regression failures will find a bug when a re-report as I either file a new bug
> or I will associate the regression failure to an existing bug before I re-report.
>
> But re-reporting will not trigger re-run to see version 2 results. Let me know
> if you have any further questions.
We don't have CI-IGT (full) results for the series https://patchwork.freedesktop.org/series/85555/#rev2 yet.
In order to merge this patch we would require CI pre merge results.
Thanks,
Anshuman Gupta.
>
> Thanks,
> Lakshmi.
> -----Original Message-----
> From: Gupta, Anshuman <anshuman.gupta@intel.com>
> Sent: Tuesday, January 26, 2021 9:20 PM
> To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>; intel-
> gfx@lists.freedesktop.org
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the
> QSES check for HDCP 1.4 over MST (rev2)
>
> Thanks Lakshmi for re-reporting it.
> May I know that re-reporting will also trigger a CI-IGT run.
> I am not able to see this series in CI Queue https://intel-gfx-
> ci.01.org/queue/index.html
> Currently this patch Rev2 is not showing any CI-IGT results.
>
> Br,
> Anshuman
>
> > -----Original Message-----
> > From: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> > Sent: Monday, January 25, 2021 10:40 PM
> > To: Gupta, Anshuman <anshuman.gupta@intel.com>; intel-
> > gfx@lists.freedesktop.org
> > Cc: Sean Paul <sean@poorly.run>
> > Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> > Disable the QSES check for HDCP 1.4 over MST (rev2)
> >
> > Re-reported.
> >
> > -----Original Message-----
> > From: Gupta, Anshuman <anshuman.gupta@intel.com>
> > Sent: Monday, January 25, 2021 7:26 AM
> > To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana
> > <lakshminarayana.vudum@intel.com>
> > Cc: Sean Paul <sean@poorly.run>
> > Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> > Disable the QSES check for HDCP 1.4 over MST (rev2)
> >
> > Hi Lakshmi ,
> > We need to merge the patch
> > https://patchwork.freedesktop.org/series/85555/
> > Below are the gem failures causing the BAT to fail , those were not
> > seen on
> > Rev1 of this patch.
> > Could you please rereport-the results to trigger CI-IGT run.
> > Thanks,
> > Anshuman Gupta.
> >
> > From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of
> > Patchwork
> > Sent: Friday, January 22, 2021 8:13 AM
> > To: Sean Paul <sean@poorly.run>
> > Cc: intel-gfx@lists.freedesktop.org
> > Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable
> > the QSES check for HDCP 1.4 over MST (rev2)
> >
> > Patch Details
> > Series:
> > drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
> > URL:
> > https://patchwork.freedesktop.org/series/85555/
> > State:
> > failure
> > Details:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
> > CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447 Summary
> > FAILURE Serious unknown changes coming with Patchwork_19447
> absolutely
> > need to be verified manually.
> > If you think the reported changes have nothing to do with the changes
> > introduced in Patchwork_19447, 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/Patchwork_19447/index.html
> > Possible new issues
> > Here are the unknown changes that may have been introduced in
> > Patchwork_19447:
> > IGT changes
> > Possible regressions
> > • igt@i915_selftest@live@gt_lrc:
> > o fi-bsw-n3050:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@gt_lrc.html
> > Known issues
> > Here are the changes found in Patchwork_19447 that come from known
> > issues:
> > IGT changes
> > Issues hit
> > • igt@gem_mmap_gtt@basic:
> > o fi-tgl-y:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> > y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-
> ci.01.org/tree/drm-
> > tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar
> > issues •
> > igt@i915_selftest@live@hugepages:
> > o fi-bsw-n3050:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > bsw-n3050/igt@i915_selftest@live@hugepages.html -> https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@hugepages.html
> > (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar
> > issues • igt@i915_selftest@live@ring_submission:
> > o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-
> > tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@ring_submission.html
> > (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar
> > issues • igt@kms_chamelium@dp-crc-fast:
> > o fi-kbl-7500u:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-
> > 7500u/igt@kms_chamelium@dp-crc-fast.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/1161 /
> > https://gitlab.freedesktop.org/drm/intel/issues/262)
> > Possible fixes
> > • igt@gem_tiled_blits@basic:
> > o fi-tgl-y:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> > y/igt@gem_tiled_blits@basic.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/402) ->
> > https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-
> > y/igt@gem_tiled_blits@basic.html +1 similar issue Warnings •
> > igt@i915_selftest@live@execlists:
> > o fi-bsw-n3050:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > bsw-n3050/igt@i915_selftest@live@execlists.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/2940) ->
> > https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@execlists.html
> > (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
> > Participating hosts (41 -> 37)
> > Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u Build
> > changes • Linux: CI_DRM_9666 -> Patchwork_19447
> > CI-20190529: 20190529
> > CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @
> > git://anongit.freedesktop.org/gfx-ci/linux
> > IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @
> > git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> > Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @
> > git://anongit.freedesktop.org/gfx-ci/linux
> > == Linux commits ==
> > 69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over
> > MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-27 7:37 ` Gupta, Anshuman
@ 2021-01-27 19:13 ` Vudum, Lakshminarayana
2021-01-28 7:09 ` Petri Latvala
0 siblings, 1 reply; 23+ messages in thread
From: Vudum, Lakshminarayana @ 2021-01-27 19:13 UTC (permalink / raw)
To: Gupta, Anshuman, intel-gfx, Latvala, Petri
I am not totally sure why shard run is not triggered here https://patchwork.freedesktop.org/series/85555/#rev2
@Latvala, Petri any help here?
Thanks,
Lakshmi.
-----Original Message-----
From: Gupta, Anshuman <anshuman.gupta@intel.com>
Sent: Tuesday, January 26, 2021 11:38 PM
To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>; intel-gfx@lists.freedesktop.org
Cc: Sean Paul <sean@poorly.run>
Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
> -----Original Message-----
> From: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> Sent: Wednesday, January 27, 2021 11:00 AM
> To: Gupta, Anshuman <anshuman.gupta@intel.com>; intel-
> gfx@lists.freedesktop.org
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> Disable the QSES check for HDCP 1.4 over MST (rev2)
>
> Re-reporting will just re-report the results once more time.
> Re-reporting will re-check if there is an existing bug for the
> regression failures or not. If a bug is found, series shows as success (as the failure is not a regression).
> Regression failures will find a bug when a re-report as I either file
> a new bug or I will associate the regression failure to an existing bug before I re-report.
>
> But re-reporting will not trigger re-run to see version 2 results. Let
> me know if you have any further questions.
We don't have CI-IGT (full) results for the series https://patchwork.freedesktop.org/series/85555/#rev2 yet.
In order to merge this patch we would require CI pre merge results.
Thanks,
Anshuman Gupta.
>
> Thanks,
> Lakshmi.
> -----Original Message-----
> From: Gupta, Anshuman <anshuman.gupta@intel.com>
> Sent: Tuesday, January 26, 2021 9:20 PM
> To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>; intel-
> gfx@lists.freedesktop.org
> Cc: Sean Paul <sean@poorly.run>
> Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> Disable the QSES check for HDCP 1.4 over MST (rev2)
>
> Thanks Lakshmi for re-reporting it.
> May I know that re-reporting will also trigger a CI-IGT run.
> I am not able to see this series in CI Queue https://intel-gfx-
> ci.01.org/queue/index.html Currently this patch Rev2 is not showing
> any CI-IGT results.
>
> Br,
> Anshuman
>
> > -----Original Message-----
> > From: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> > Sent: Monday, January 25, 2021 10:40 PM
> > To: Gupta, Anshuman <anshuman.gupta@intel.com>; intel-
> > gfx@lists.freedesktop.org
> > Cc: Sean Paul <sean@poorly.run>
> > Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> > Disable the QSES check for HDCP 1.4 over MST (rev2)
> >
> > Re-reported.
> >
> > -----Original Message-----
> > From: Gupta, Anshuman <anshuman.gupta@intel.com>
> > Sent: Monday, January 25, 2021 7:26 AM
> > To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana
> > <lakshminarayana.vudum@intel.com>
> > Cc: Sean Paul <sean@poorly.run>
> > Subject: RE: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp:
> > Disable the QSES check for HDCP 1.4 over MST (rev2)
> >
> > Hi Lakshmi ,
> > We need to merge the patch
> > https://patchwork.freedesktop.org/series/85555/
> > Below are the gem failures causing the BAT to fail , those were not
> > seen on
> > Rev1 of this patch.
> > Could you please rereport-the results to trigger CI-IGT run.
> > Thanks,
> > Anshuman Gupta.
> >
> > From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf
> > Of Patchwork
> > Sent: Friday, January 22, 2021 8:13 AM
> > To: Sean Paul <sean@poorly.run>
> > Cc: intel-gfx@lists.freedesktop.org
> > Subject: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable
> > the QSES check for HDCP 1.4 over MST (rev2)
> >
> > Patch Details
> > Series:
> > drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
> > URL:
> > https://patchwork.freedesktop.org/series/85555/
> > State:
> > failure
> > Details:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
> > CI Bug Log - changes from CI_DRM_9666 -> Patchwork_19447 Summary
> > FAILURE Serious unknown changes coming with Patchwork_19447
> absolutely
> > need to be verified manually.
> > If you think the reported changes have nothing to do with the
> > changes introduced in Patchwork_19447, 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/Patchwork_19447/index.html
> > Possible new issues
> > Here are the unknown changes that may have been introduced in
> > Patchwork_19447:
> > IGT changes
> > Possible regressions
> > • igt@i915_selftest@live@gt_lrc:
> > o fi-bsw-n3050:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > bsw-n3050/igt@i915_selftest@live@gt_lrc.html -> https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@gt_lrc.html
> > Known issues
> > Here are the changes found in Patchwork_19447 that come from known
> > issues:
> > IGT changes
> > Issues hit
> > • igt@gem_mmap_gtt@basic:
> > o fi-tgl-y:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> > y/igt@gem_mmap_gtt@basic.html -> https://intel-gfx-
> ci.01.org/tree/drm-
> > tip/Patchwork_19447/fi-tgl-y/igt@gem_mmap_gtt@basic.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/402) +2 similar
> > issues •
> > igt@i915_selftest@live@hugepages:
> > o fi-bsw-n3050:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > bsw-n3050/igt@i915_selftest@live@hugepages.html ->
> > https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@hugepages.html
> > (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +20 similar
> > issues • igt@i915_selftest@live@ring_submission:
> > o fi-bsw-n3050: NOTRUN -> https://intel-gfx-ci.01.org/tree/drm-
> > tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@ring_submission.html
> > (https://bugs.freedesktop.org/show_bug.cgi?id=109271) +2 similar
> > issues • igt@kms_chamelium@dp-crc-fast:
> > o fi-kbl-7500u:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > kbl-7500u/igt@kms_chamelium@dp-crc-fast.html -> https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-kbl-
> > 7500u/igt@kms_chamelium@dp-crc-fast.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/1161 /
> > https://gitlab.freedesktop.org/drm/intel/issues/262)
> > Possible fixes
> > • igt@gem_tiled_blits@basic:
> > o fi-tgl-y:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-tgl-
> > y/igt@gem_tiled_blits@basic.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/402) ->
> > https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-tgl-
> > y/igt@gem_tiled_blits@basic.html +1 similar issue Warnings •
> > igt@i915_selftest@live@execlists:
> > o fi-bsw-n3050:
> > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9666/fi-
> > bsw-n3050/igt@i915_selftest@live@execlists.html
> > (https://gitlab.freedesktop.org/drm/intel/issues/2940) ->
> > https://intel-gfx-
> > ci.01.org/tree/drm-tip/Patchwork_19447/fi-bsw-
> > n3050/igt@i915_selftest@live@execlists.html
> > (https://bugs.freedesktop.org/show_bug.cgi?id=109271)
> > Participating hosts (41 -> 37)
> > Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u Build
> > changes • Linux: CI_DRM_9666 -> Patchwork_19447
> > CI-20190529: 20190529
> > CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @
> > git://anongit.freedesktop.org/gfx-ci/linux
> > IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @
> > git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> > Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @
> > git://anongit.freedesktop.org/gfx-ci/linux
> > == Linux commits ==
> > 69d7db003fa4 drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over
> > MST
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
` (6 preceding siblings ...)
2021-01-25 17:02 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
@ 2021-01-28 7:05 ` Patchwork
2021-01-28 7:22 ` Gupta, Anshuman
7 siblings, 1 reply; 23+ messages in thread
From: Patchwork @ 2021-01-28 7:05 UTC (permalink / raw)
To: Sean Paul; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 1268 bytes --]
== Series Details ==
Series: drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL : https://patchwork.freedesktop.org/series/85555/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9666_full -> Patchwork_19447_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
Changes
-------
No changes found
Participating hosts (10 -> 8)
------------------------------
Missing (2): pig-skl-6260u pig-glk-j5005
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_9666 -> Patchwork_19447
* Piglit: piglit_4509 -> None
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
[-- Attachment #1.2: Type: text/html, Size: 1850 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-27 19:13 ` Vudum, Lakshminarayana
@ 2021-01-28 7:09 ` Petri Latvala
0 siblings, 0 replies; 23+ messages in thread
From: Petri Latvala @ 2021-01-28 7:09 UTC (permalink / raw)
To: Vudum, Lakshminarayana; +Cc: intel-gfx
On Wed, Jan 27, 2021 at 09:13:36PM +0200, Vudum, Lakshminarayana wrote:
> I am not totally sure why shard run is not triggered here https://patchwork.freedesktop.org/series/85555/#rev2
> @Latvala, Petri any help here?
The results were there but reporting it failed. Re-reported it and
it's now on patchwork.
--
Petri Latvala
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
2021-01-28 7:05 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
@ 2021-01-28 7:22 ` Gupta, Anshuman
0 siblings, 0 replies; 23+ messages in thread
From: Gupta, Anshuman @ 2021-01-28 7:22 UTC (permalink / raw)
To: intel-gfx, Sean Paul
[-- Attachment #1.1: Type: text/plain, Size: 1469 bytes --]
Pushed to drm-intel-next.
From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Patchwork
Sent: Thursday, January 28, 2021 12:36 PM
To: Sean Paul <sean@poorly.run>
Cc: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
Patch Details
Series:
drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2)
URL:
https://patchwork.freedesktop.org/series/85555/
State:
success
Details:
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
CI Bug Log - changes from CI_DRM_9666_full -> Patchwork_19447_full
Summary
SUCCESS
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_19447/index.html
Changes
No changes found
Participating hosts (10 -> 8)
Missing (2): pig-skl-6260u pig-glk-j5005
Build changes
* CI: CI-20190529 -> None
* Linux: CI_DRM_9666 -> Patchwork_19447
* Piglit: piglit_4509 -> None
CI-20190529: 20190529
CI_DRM_9666: 9ccbc653bf2948d1f7e9ff300dd7679b888ffa25 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5962: 22e3daaed82ab7890018a2f2aabf5082cd536023 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_19447: 69d7db003fa406df7851ab08239dd409a78f4421 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
[-- Attachment #1.2: Type: text/html, Size: 8272 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 23+ messages in thread
end of thread, other threads:[~2021-01-28 7:22 UTC | newest]
Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-06 22:38 [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST Sean Paul
2021-01-06 23:41 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2021-01-07 10:13 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-01-12 17:51 ` [Intel-gfx] [PATCH] " Jani Nikula
2021-01-13 10:19 ` Anshuman Gupta
2021-01-13 13:39 ` Sean Paul
2021-01-13 14:31 ` Daniel Vetter
2021-01-13 18:33 ` Sean Paul
2021-01-13 20:45 ` Daniel Vetter
2021-01-15 10:22 ` Gupta, Anshuman
2021-01-21 17:26 ` [Intel-gfx] [PATCH v2] " Sean Paul
2021-01-22 2:42 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST (rev2) Patchwork
2021-01-25 15:26 ` Gupta, Anshuman
2021-01-25 16:41 ` Vudum, Lakshminarayana
2021-01-25 17:10 ` Vudum, Lakshminarayana
2021-01-27 5:20 ` Gupta, Anshuman
2021-01-27 5:29 ` Vudum, Lakshminarayana
2021-01-27 7:37 ` Gupta, Anshuman
2021-01-27 19:13 ` Vudum, Lakshminarayana
2021-01-28 7:09 ` Petri Latvala
2021-01-25 17:02 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-01-28 7:05 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2021-01-28 7:22 ` Gupta, Anshuman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).