* [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
@ 2019-04-29 23:08 Aditya Swarup
2019-04-30 8:21 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8) Patchwork
` (4 more replies)
0 siblings, 5 replies; 11+ messages in thread
From: Aditya Swarup @ 2019-04-29 23:08 UTC (permalink / raw)
To: intel-gfx; +Cc: Jani Nikula
From: Clinton Taylor <Clinton.A.Taylor@intel.com>
v2: Fix commit msg to reflect why issue occurs(Jani)
Set GCP_COLOR_INDICATION only when we set 10/12 bit deep color.
Changing settings from 10/12 bit deep color to 8 bit(& vice versa)
doesn't work correctly using xrandr max bpc property. When we
connect a monitor which supports deep color, the highest deep color
setting is selected; which sets GCP_COLOR_INDICATION. When we change
the setting to 8 bit color, we still set GCP_COLOR_INDICATION which
doesn't allow the switch back to 8 bit color.
v3,4: Add comments & drop changes in intel_hdmi_compute_config(Ville)
Since HSW+, GCP_COLOR_INDICATION is not required for 8bpc.
Drop the changes in intel_hdmi_compute_config as desired_bpp
is needed to change values for pipe_bpp based on bw_constrained flag.
v5: Fix missing logical && in condition for setting GCP_COLOR_INDICATION.
v6: Fix comment formatting (Ville)
v7: Add reviewed by Ville
v8: Set GCP_COLOR_INDICATION based on spec:
For Gen 7.5 or later platforms, indicate color depth only for deep
color modes. Bspec: 8135,7751,50524
Pre DDI platforms, indicate color depth if deep color is supported
by sink. Bspec: 7854
Exception: CHERRYVIEW behaves like Pre DDI platforms.
Bspec: 15975
Check pipe_bpp is less than bpp * 3 in hdmi_deep_color_possible,
to not set 12 bit deep color for every modeset. This fixes the issue
where 12 bit color was selected even when user selected 10 bit.(Ville)
v9: Maintain a consistent behavior for all platforms and support
GCP_COLOR_INDICATION only when we are in deep color mode. Remove
hdmi_sink_is_deep_color() - no longer needed as checking pipe_bpp > 24
takes care of the deep color mode scenario.
Separate patch for fixing switch from 12 bit to 10 bit deep color
mode.
Co-developed-by: Aditya Swarup <aditya.swarup@intel.com>
Signed-off-by: Clinton Taylor <Clinton.A.Taylor@intel.com>
Signed-off-by: Aditya Swarup <aditya.swarup@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Manasi Navare <manasi.d.navare@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_hdmi.c | 17 ++---------------
1 file changed, 2 insertions(+), 15 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c
index e1005d7b75fd..991eb362ef4f 100644
--- a/drivers/gpu/drm/i915/intel_hdmi.c
+++ b/drivers/gpu/drm/i915/intel_hdmi.c
@@ -856,19 +856,6 @@ static void g4x_set_infoframes(struct intel_encoder *encoder,
&crtc_state->infoframes.hdmi);
}
-static bool hdmi_sink_is_deep_color(const struct drm_connector_state *conn_state)
-{
- struct drm_connector *connector = conn_state->connector;
-
- /*
- * HDMI cloning is only supported on g4x which doesn't
- * support deep color or GCP infoframes anyway so no
- * need to worry about multiple HDMI sinks here.
- */
-
- return connector->display_info.bpc > 8;
-}
-
/*
* Determine if default_phase=1 can be indicated in the GCP infoframe.
*
@@ -973,8 +960,8 @@ static void intel_hdmi_compute_gcp_infoframe(struct intel_encoder *encoder,
crtc_state->infoframes.enable |=
intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GENERAL_CONTROL);
- /* Indicate color depth whenever the sink supports deep color */
- if (hdmi_sink_is_deep_color(conn_state))
+ /* Indicate color indication for deep color mode */
+ if (crtc_state->pipe_bpp > 24)
crtc_state->infoframes.gcp |= GCP_COLOR_INDICATION;
/* Enable default_phase whenever the display mode is suitably aligned */
--
2.17.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
2019-04-29 23:08 [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Aditya Swarup
@ 2019-04-30 8:21 ` Patchwork
2019-04-30 13:46 ` ✗ Fi.CI.IGT: failure " Patchwork
` (3 subsequent siblings)
4 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-04-30 8:21 UTC (permalink / raw)
To: Aditya Swarup; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
URL : https://patchwork.freedesktop.org/series/58912/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6012 -> Patchwork_12900
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/58912/revisions/8/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_12900:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@gem_ctx_create@basic-files:
- {fi-cml-u2}: [PASS][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-cml-u2/igt@gem_ctx_create@basic-files.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-cml-u2/igt@gem_ctx_create@basic-files.html
Known issues
------------
Here are the changes found in Patchwork_12900 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_exec@basic:
- fi-icl-y: [PASS][3] -> [INCOMPLETE][4] ([fdo#107713])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-icl-y/igt@gem_ctx_exec@basic.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-icl-y/igt@gem_ctx_exec@basic.html
* igt@gem_exec_suspend@basic-s4-devices:
- fi-blb-e6850: [PASS][5] -> [INCOMPLETE][6] ([fdo#107718])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-blb-e6850/igt@gem_exec_suspend@basic-s4-devices.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-blb-e6850/igt@gem_exec_suspend@basic-s4-devices.html
* igt@i915_selftest@live_hangcheck:
- fi-skl-iommu: [PASS][7] -> [INCOMPLETE][8] ([fdo#108602] / [fdo#108744])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-skl-iommu/igt@i915_selftest@live_hangcheck.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-skl-iommu/igt@i915_selftest@live_hangcheck.html
* igt@kms_chamelium@dp-crc-fast:
- fi-kbl-7500u: [PASS][9] -> [DMESG-WARN][10] ([fdo#103841])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size:
- fi-glk-dsi: [PASS][11] -> [INCOMPLETE][12] ([fdo#103359] / [k.org#198133])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-glk-dsi/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-glk-dsi/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.html
#### Possible fixes ####
* igt@amdgpu/amd_basic@userptr:
- fi-kbl-8809g: [DMESG-WARN][13] ([fdo#108965]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-kbl-8809g/igt@amdgpu/amd_basic@userptr.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-kbl-8809g/igt@amdgpu/amd_basic@userptr.html
* igt@gem_exec_basic@basic-blt:
- {fi-icl-u2}: [INCOMPLETE][15] ([fdo#107713] / [fdo#110246]) -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/fi-icl-u2/igt@gem_exec_basic@basic-blt.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/fi-icl-u2/igt@gem_exec_basic@basic-blt.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103841]: https://bugs.freedesktop.org/show_bug.cgi?id=103841
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
[fdo#108602]: https://bugs.freedesktop.org/show_bug.cgi?id=108602
[fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744
[fdo#108965]: https://bugs.freedesktop.org/show_bug.cgi?id=108965
[fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
[fdo#110246]: https://bugs.freedesktop.org/show_bug.cgi?id=110246
[fdo#110514]: https://bugs.freedesktop.org/show_bug.cgi?id=110514
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (52 -> 46)
------------------------------
Missing (6): fi-kbl-soraka fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-byt-clapper fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_6012 -> Patchwork_12900
CI_DRM_6012: e4882f199157e3fb73d1791352931096f6ecfcfd @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4968: caed251990f35bfe45368f803980071a73e36315 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_12900: 1707feeade4b0b06b27845ac8177c942984f1c35 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
1707feeade4b drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✗ Fi.CI.IGT: failure for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
2019-04-29 23:08 [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Aditya Swarup
2019-04-30 8:21 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8) Patchwork
@ 2019-04-30 13:46 ` Patchwork
2019-05-03 23:04 ` Aditya Swarup
2019-05-03 20:44 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10) Patchwork
` (2 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Patchwork @ 2019-04-30 13:46 UTC (permalink / raw)
To: Aditya Swarup; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
URL : https://patchwork.freedesktop.org/series/58912/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6012_full -> Patchwork_12900_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_12900_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_12900_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_12900_full:
### IGT changes ###
#### Possible regressions ####
* igt@gem_caching@read-writes:
- shard-skl: [PASS][1] -> [INCOMPLETE][2] +6 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl9/igt@gem_caching@read-writes.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/igt@gem_caching@read-writes.html
* igt@gem_mmap_gtt@forked-big-copy-odd:
- shard-skl: NOTRUN -> [INCOMPLETE][3] +1 similar issue
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl2/igt@gem_mmap_gtt@forked-big-copy-odd.html
#### Warnings ####
* igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size:
- shard-skl: [SKIP][4] ([fdo#109271]) -> [INCOMPLETE][5] +1 similar issue
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl6/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl8/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
Known issues
------------
Here are the changes found in Patchwork_12900_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_cpu_reloc@forked:
- shard-iclb: [PASS][6] -> [INCOMPLETE][7] ([fdo#107713] / [fdo#109100])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb4/igt@gem_cpu_reloc@forked.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb3/igt@gem_cpu_reloc@forked.html
* igt@i915_pm_rpm@pm-tiling:
- shard-skl: [PASS][8] -> [INCOMPLETE][9] ([fdo#107807])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl10/igt@i915_pm_rpm@pm-tiling.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/igt@i915_pm_rpm@pm-tiling.html
* igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
- shard-snb: [PASS][10] -> [SKIP][11] ([fdo#109271] / [fdo#109278]) +1 similar issue
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-snb1/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb6/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
* igt@kms_cursor_crc@cursor-64x21-offscreen:
- shard-skl: [PASS][12] -> [FAIL][13] ([fdo#103232])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl2/igt@kms_cursor_crc@cursor-64x21-offscreen.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/igt@kms_cursor_crc@cursor-64x21-offscreen.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-glk: [PASS][14] -> [INCOMPLETE][15] ([fdo#103359] / [k.org#198133]) +2 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk5/igt@kms_flip@flip-vs-suspend-interruptible.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk7/igt@kms_flip@flip-vs-suspend-interruptible.html
- shard-apl: [PASS][16] -> [DMESG-WARN][17] ([fdo#108566]) +1 similar issue
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
- shard-snb: [PASS][18] -> [INCOMPLETE][19] ([fdo#105411])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-snb5/igt@kms_flip@flip-vs-suspend-interruptible.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb1/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_flip@nonexisting-fb-interruptible:
- shard-iclb: [PASS][20] -> [INCOMPLETE][21] ([fdo#107713])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb3/igt@kms_flip@nonexisting-fb-interruptible.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb4/igt@kms_flip@nonexisting-fb-interruptible.html
* igt@kms_flip@plain-flip-ts-check-interruptible:
- shard-skl: [PASS][22] -> [FAIL][23] ([fdo#100368])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl2/igt@kms_flip@plain-flip-ts-check-interruptible.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl1/igt@kms_flip@plain-flip-ts-check-interruptible.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render:
- shard-iclb: [PASS][24] -> [FAIL][25] ([fdo#103167]) +4 similar issues
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render:
- shard-skl: [PASS][26] -> [INCOMPLETE][27] ([fdo#106978])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render.html
* igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min:
- shard-skl: [PASS][28] -> [FAIL][29] ([fdo#108145])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl5/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl7/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html
* igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
- shard-skl: [PASS][30] -> [FAIL][31] ([fdo#108145] / [fdo#110403])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl2/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
* igt@kms_plane_cursor@pipe-b-overlay-size-256:
- shard-snb: [PASS][32] -> [SKIP][33] ([fdo#109271]) +5 similar issues
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-snb1/igt@kms_plane_cursor@pipe-b-overlay-size-256.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb6/igt@kms_plane_cursor@pipe-b-overlay-size-256.html
* igt@kms_psr@psr2_cursor_mmap_cpu:
- shard-iclb: [PASS][34] -> [SKIP][35] ([fdo#109441]) +2 similar issues
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb6/igt@kms_psr@psr2_cursor_mmap_cpu.html
* igt@kms_rotation_crc@multiplane-rotation-cropping-top:
- shard-kbl: [PASS][36] -> [FAIL][37] ([fdo#109016])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-kbl7/igt@kms_rotation_crc@multiplane-rotation-cropping-top.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-kbl4/igt@kms_rotation_crc@multiplane-rotation-cropping-top.html
* igt@kms_rotation_crc@sprite-rotation-180:
- shard-skl: [PASS][38] -> [INCOMPLETE][39] ([fdo#108972]) +1 similar issue
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl1/igt@kms_rotation_crc@sprite-rotation-180.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl4/igt@kms_rotation_crc@sprite-rotation-180.html
* igt@kms_setmode@basic:
- shard-kbl: [PASS][40] -> [FAIL][41] ([fdo#99912])
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-kbl5/igt@kms_setmode@basic.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-kbl4/igt@kms_setmode@basic.html
* igt@kms_sysfs_edid_timing:
- shard-iclb: [PASS][42] -> [FAIL][43] ([fdo#100047])
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb6/igt@kms_sysfs_edid_timing.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb3/igt@kms_sysfs_edid_timing.html
* igt@perf_pmu@busy-idle-rcs0:
- shard-hsw: [PASS][44] -> [INCOMPLETE][45] ([fdo#103540]) +2 similar issues
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-hsw5/igt@perf_pmu@busy-idle-rcs0.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-hsw6/igt@perf_pmu@busy-idle-rcs0.html
#### Possible fixes ####
* igt@i915_suspend@fence-restore-untiled:
- shard-apl: [DMESG-WARN][46] ([fdo#108566]) -> [PASS][47] +3 similar issues
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-apl2/igt@i915_suspend@fence-restore-untiled.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl3/igt@i915_suspend@fence-restore-untiled.html
* igt@kms_cursor_crc@cursor-64x64-suspend:
- shard-skl: [INCOMPLETE][48] ([fdo#104108]) -> [PASS][49]
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl3/igt@kms_cursor_crc@cursor-64x64-suspend.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl10/igt@kms_cursor_crc@cursor-64x64-suspend.html
* igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic:
- shard-glk: [FAIL][50] ([fdo#106509] / [fdo#107409]) -> [PASS][51]
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk2/igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk1/igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-skl: [FAIL][52] ([fdo#105363]) -> [PASS][53]
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl3/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl6/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@modeset-vs-vblank-race:
- shard-glk: [FAIL][54] ([fdo#103060]) -> [PASS][55]
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk8/igt@kms_flip@modeset-vs-vblank-race.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk1/igt@kms_flip@modeset-vs-vblank-race.html
* igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render:
- shard-iclb: [FAIL][56] ([fdo#103167]) -> [PASS][57] +3 similar issues
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html
* igt@kms_frontbuffer_tracking@fbcpsr-suspend:
- shard-skl: [INCOMPLETE][58] ([fdo#104108] / [fdo#106978]) -> [PASS][59]
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl7/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
- shard-iclb: [INCOMPLETE][60] ([fdo#106978] / [fdo#107713]) -> [PASS][61]
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb4/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
* igt@kms_plane@pixel-format-pipe-c-planes:
- shard-glk: [SKIP][62] ([fdo#109271]) -> [PASS][63]
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk4/igt@kms_plane@pixel-format-pipe-c-planes.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk9/igt@kms_plane@pixel-format-pipe-c-planes.html
* igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format:
- shard-glk: [SKIP][64] ([fdo#109271] / [fdo#109278]) -> [PASS][65]
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk7/igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk9/igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format.html
* igt@kms_psr@psr2_cursor_plane_onoff:
- shard-iclb: [SKIP][66] ([fdo#109441]) -> [PASS][67] +3 similar issues
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb6/igt@kms_psr@psr2_cursor_plane_onoff.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
* igt@kms_setmode@basic:
- shard-apl: [FAIL][68] ([fdo#99912]) -> [PASS][69]
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-apl5/igt@kms_setmode@basic.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl4/igt@kms_setmode@basic.html
* igt@tools_test@tools_test:
- shard-iclb: [SKIP][70] ([fdo#109352]) -> [PASS][71]
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb8/igt@tools_test@tools_test.html
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb6/igt@tools_test@tools_test.html
#### Warnings ####
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt:
- shard-skl: [FAIL][72] ([fdo#108040]) -> [FAIL][73] ([fdo#103167])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl8/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
[fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
[fdo#103060]: https://bugs.freedesktop.org/show_bug.cgi?id=103060
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
[fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
[fdo#106509]: https://bugs.freedesktop.org/show_bug.cgi?id=106509
[fdo#106978]: https://bugs.freedesktop.org/show_bug.cgi?id=106978
[fdo#107409]: https://bugs.freedesktop.org/show_bug.cgi?id=107409
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
[fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108972]: https://bugs.freedesktop.org/show_bug.cgi?id=108972
[fdo#109016]: https://bugs.freedesktop.org/show_bug.cgi?id=109016
[fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109352]: https://bugs.freedesktop.org/show_bug.cgi?id=109352
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403
[fdo#110519]: https://bugs.freedesktop.org/show_bug.cgi?id=110519
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_6012 -> Patchwork_12900
CI_DRM_6012: e4882f199157e3fb73d1791352931096f6ecfcfd @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4968: caed251990f35bfe45368f803980071a73e36315 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_12900: 1707feeade4b0b06b27845ac8177c942984f1c35 @ 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_12900/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10)
2019-04-29 23:08 [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Aditya Swarup
2019-04-30 8:21 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8) Patchwork
2019-04-30 13:46 ` ✗ Fi.CI.IGT: failure " Patchwork
@ 2019-05-03 20:44 ` Patchwork
2019-05-04 2:00 ` ✓ Fi.CI.IGT: " Patchwork
2019-05-07 6:42 ` [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Jani Nikula
4 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-05-03 20:44 UTC (permalink / raw)
To: Aditya Swarup; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10)
URL : https://patchwork.freedesktop.org/series/58912/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6041 -> Patchwork_12964
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/
Known issues
------------
Here are the changes found in Patchwork_12964 that come from known issues:
### IGT changes ###
#### Possible fixes ####
* igt@i915_selftest@live_hangcheck:
- fi-skl-iommu: [INCOMPLETE][1] ([fdo#108602] / [fdo#108744] / [fdo#110581]) -> [PASS][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/fi-skl-iommu/igt@i915_selftest@live_hangcheck.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/fi-skl-iommu/igt@i915_selftest@live_hangcheck.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [FAIL][3] ([fdo#109485]) -> [PASS][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- {fi-icl-u2}: [DMESG-WARN][5] -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102505]: https://bugs.freedesktop.org/show_bug.cgi?id=102505
[fdo#108602]: https://bugs.freedesktop.org/show_bug.cgi?id=108602
[fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744
[fdo#109485]: https://bugs.freedesktop.org/show_bug.cgi?id=109485
[fdo#110581]: https://bugs.freedesktop.org/show_bug.cgi?id=110581
[fdo#110595]: https://bugs.freedesktop.org/show_bug.cgi?id=110595
Participating hosts (51 -> 44)
------------------------------
Additional (1): fi-bsw-n3050
Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-j1900 fi-byt-squawks fi-bsw-cyan fi-byt-clapper fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_6041 -> Patchwork_12964
CI_DRM_6041: 014903e8b7de5d69a17de628345ed31db1600b73 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4972: f052e49a43cc9704ea5f240df15dd9d3dfed68ab @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_12964: 53926fe20d0a3628bbcb96193aa6b8ebc3da13ab @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
53926fe20d0a drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: ✗ Fi.CI.IGT: failure for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
2019-04-30 13:46 ` ✗ Fi.CI.IGT: failure " Patchwork
@ 2019-05-03 23:04 ` Aditya Swarup
2019-05-04 5:55 ` Saarinen, Jani
0 siblings, 1 reply; 11+ messages in thread
From: Aditya Swarup @ 2019-05-03 23:04 UTC (permalink / raw)
To: intel-gfx; +Cc: Martin Peres
On Tue, Apr 30, 2019 at 01:46:15PM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
> URL : https://patchwork.freedesktop.org/series/58912/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_6012_full -> Patchwork_12900_full
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_12900_full absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_12900_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_12900_full:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@gem_caching@read-writes:
> - shard-skl: [PASS][1] -> [INCOMPLETE][2] +6 similar issues
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl9/igt@gem_caching@read-writes.html
> [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/igt@gem_caching@read-writes.html
>
> * igt@gem_mmap_gtt@forked-big-copy-odd:
> - shard-skl: NOTRUN -> [INCOMPLETE][3] +1 similar issue
> [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl2/igt@gem_mmap_gtt@forked-big-copy-odd.html
Hi Martin,
These failures shouldn't be due to my patch. Is there a separate bug
filed for gem_caching and gem_mmap_gtt failures?
Regards,
Aditya Swarup
>
>
> #### Warnings ####
>
> * igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size:
> - shard-skl: [SKIP][4] ([fdo#109271]) -> [INCOMPLETE][5] +1 similar issue
> [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl6/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl8/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_12900_full that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@gem_cpu_reloc@forked:
> - shard-iclb: [PASS][6] -> [INCOMPLETE][7] ([fdo#107713] / [fdo#109100])
> [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb4/igt@gem_cpu_reloc@forked.html
> [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb3/igt@gem_cpu_reloc@forked.html
>
> * igt@i915_pm_rpm@pm-tiling:
> - shard-skl: [PASS][8] -> [INCOMPLETE][9] ([fdo#107807])
> [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl10/igt@i915_pm_rpm@pm-tiling.html
> [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/igt@i915_pm_rpm@pm-tiling.html
>
> * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
> - shard-snb: [PASS][10] -> [SKIP][11] ([fdo#109271] / [fdo#109278]) +1 similar issue
> [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-snb1/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb6/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
>
> * igt@kms_cursor_crc@cursor-64x21-offscreen:
> - shard-skl: [PASS][12] -> [FAIL][13] ([fdo#103232])
> [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl2/igt@kms_cursor_crc@cursor-64x21-offscreen.html
> [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/igt@kms_cursor_crc@cursor-64x21-offscreen.html
>
> * igt@kms_flip@flip-vs-suspend-interruptible:
> - shard-glk: [PASS][14] -> [INCOMPLETE][15] ([fdo#103359] / [k.org#198133]) +2 similar issues
> [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk5/igt@kms_flip@flip-vs-suspend-interruptible.html
> [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk7/igt@kms_flip@flip-vs-suspend-interruptible.html
> - shard-apl: [PASS][16] -> [DMESG-WARN][17] ([fdo#108566]) +1 similar issue
> [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
> [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
> - shard-snb: [PASS][18] -> [INCOMPLETE][19] ([fdo#105411])
> [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-snb5/igt@kms_flip@flip-vs-suspend-interruptible.html
> [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb1/igt@kms_flip@flip-vs-suspend-interruptible.html
>
> * igt@kms_flip@nonexisting-fb-interruptible:
> - shard-iclb: [PASS][20] -> [INCOMPLETE][21] ([fdo#107713])
> [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb3/igt@kms_flip@nonexisting-fb-interruptible.html
> [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb4/igt@kms_flip@nonexisting-fb-interruptible.html
>
> * igt@kms_flip@plain-flip-ts-check-interruptible:
> - shard-skl: [PASS][22] -> [FAIL][23] ([fdo#100368])
> [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl2/igt@kms_flip@plain-flip-ts-check-interruptible.html
> [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl1/igt@kms_flip@plain-flip-ts-check-interruptible.html
>
> * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render:
> - shard-iclb: [PASS][24] -> [FAIL][25] ([fdo#103167]) +4 similar issues
> [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
> [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html
>
> * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render:
> - shard-skl: [PASS][26] -> [INCOMPLETE][27] ([fdo#106978])
> [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render.html
> [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render.html
>
> * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min:
> - shard-skl: [PASS][28] -> [FAIL][29] ([fdo#108145])
> [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl5/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html
> [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl7/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html
>
> * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
> - shard-skl: [PASS][30] -> [FAIL][31] ([fdo#108145] / [fdo#110403])
> [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl2/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
> [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
>
> * igt@kms_plane_cursor@pipe-b-overlay-size-256:
> - shard-snb: [PASS][32] -> [SKIP][33] ([fdo#109271]) +5 similar issues
> [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-snb1/igt@kms_plane_cursor@pipe-b-overlay-size-256.html
> [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb6/igt@kms_plane_cursor@pipe-b-overlay-size-256.html
>
> * igt@kms_psr@psr2_cursor_mmap_cpu:
> - shard-iclb: [PASS][34] -> [SKIP][35] ([fdo#109441]) +2 similar issues
> [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html
> [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb6/igt@kms_psr@psr2_cursor_mmap_cpu.html
>
> * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
> - shard-kbl: [PASS][36] -> [FAIL][37] ([fdo#109016])
> [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-kbl7/igt@kms_rotation_crc@multiplane-rotation-cropping-top.html
> [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-kbl4/igt@kms_rotation_crc@multiplane-rotation-cropping-top.html
>
> * igt@kms_rotation_crc@sprite-rotation-180:
> - shard-skl: [PASS][38] -> [INCOMPLETE][39] ([fdo#108972]) +1 similar issue
> [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl1/igt@kms_rotation_crc@sprite-rotation-180.html
> [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl4/igt@kms_rotation_crc@sprite-rotation-180.html
>
> * igt@kms_setmode@basic:
> - shard-kbl: [PASS][40] -> [FAIL][41] ([fdo#99912])
> [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-kbl5/igt@kms_setmode@basic.html
> [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-kbl4/igt@kms_setmode@basic.html
>
> * igt@kms_sysfs_edid_timing:
> - shard-iclb: [PASS][42] -> [FAIL][43] ([fdo#100047])
> [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb6/igt@kms_sysfs_edid_timing.html
> [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb3/igt@kms_sysfs_edid_timing.html
>
> * igt@perf_pmu@busy-idle-rcs0:
> - shard-hsw: [PASS][44] -> [INCOMPLETE][45] ([fdo#103540]) +2 similar issues
> [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-hsw5/igt@perf_pmu@busy-idle-rcs0.html
> [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-hsw6/igt@perf_pmu@busy-idle-rcs0.html
>
>
> #### Possible fixes ####
>
> * igt@i915_suspend@fence-restore-untiled:
> - shard-apl: [DMESG-WARN][46] ([fdo#108566]) -> [PASS][47] +3 similar issues
> [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-apl2/igt@i915_suspend@fence-restore-untiled.html
> [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl3/igt@i915_suspend@fence-restore-untiled.html
>
> * igt@kms_cursor_crc@cursor-64x64-suspend:
> - shard-skl: [INCOMPLETE][48] ([fdo#104108]) -> [PASS][49]
> [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl3/igt@kms_cursor_crc@cursor-64x64-suspend.html
> [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl10/igt@kms_cursor_crc@cursor-64x64-suspend.html
>
> * igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic:
> - shard-glk: [FAIL][50] ([fdo#106509] / [fdo#107409]) -> [PASS][51]
> [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk2/igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic.html
> [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk1/igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic.html
>
> * igt@kms_flip@flip-vs-expired-vblank-interruptible:
> - shard-skl: [FAIL][52] ([fdo#105363]) -> [PASS][53]
> [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl3/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
> [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl6/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
>
> * igt@kms_flip@modeset-vs-vblank-race:
> - shard-glk: [FAIL][54] ([fdo#103060]) -> [PASS][55]
> [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk8/igt@kms_flip@modeset-vs-vblank-race.html
> [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk1/igt@kms_flip@modeset-vs-vblank-race.html
>
> * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render:
> - shard-iclb: [FAIL][56] ([fdo#103167]) -> [PASS][57] +3 similar issues
> [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html
> [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html
>
> * igt@kms_frontbuffer_tracking@fbcpsr-suspend:
> - shard-skl: [INCOMPLETE][58] ([fdo#104108] / [fdo#106978]) -> [PASS][59]
> [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl7/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> - shard-iclb: [INCOMPLETE][60] ([fdo#106978] / [fdo#107713]) -> [PASS][61]
> [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb4/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
>
> * igt@kms_plane@pixel-format-pipe-c-planes:
> - shard-glk: [SKIP][62] ([fdo#109271]) -> [PASS][63]
> [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk4/igt@kms_plane@pixel-format-pipe-c-planes.html
> [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk9/igt@kms_plane@pixel-format-pipe-c-planes.html
>
> * igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format:
> - shard-glk: [SKIP][64] ([fdo#109271] / [fdo#109278]) -> [PASS][65]
> [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-glk7/igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format.html
> [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk9/igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format.html
>
> * igt@kms_psr@psr2_cursor_plane_onoff:
> - shard-iclb: [SKIP][66] ([fdo#109441]) -> [PASS][67] +3 similar issues
> [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb6/igt@kms_psr@psr2_cursor_plane_onoff.html
> [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
>
> * igt@kms_setmode@basic:
> - shard-apl: [FAIL][68] ([fdo#99912]) -> [PASS][69]
> [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-apl5/igt@kms_setmode@basic.html
> [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl4/igt@kms_setmode@basic.html
>
> * igt@tools_test@tools_test:
> - shard-iclb: [SKIP][70] ([fdo#109352]) -> [PASS][71]
> [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-iclb8/igt@tools_test@tools_test.html
> [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb6/igt@tools_test@tools_test.html
>
>
> #### Warnings ####
>
> * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt:
> - shard-skl: [FAIL][72] ([fdo#108040]) -> [FAIL][73] ([fdo#103167])
> [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-skl8/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt.html
> [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt.html
>
>
> {name}: This element is suppressed. This means it is ignored when computing
> the status of the difference (SUCCESS, WARNING, or FAILURE).
>
> [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
> [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
> [fdo#103060]: https://bugs.freedesktop.org/show_bug.cgi?id=103060
> [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
> [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
> [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
> [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
> [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
> [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
> [fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
> [fdo#106509]: https://bugs.freedesktop.org/show_bug.cgi?id=106509
> [fdo#106978]: https://bugs.freedesktop.org/show_bug.cgi?id=106978
> [fdo#107409]: https://bugs.freedesktop.org/show_bug.cgi?id=107409
> [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
> [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
> [fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
> [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
> [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
> [fdo#108972]: https://bugs.freedesktop.org/show_bug.cgi?id=108972
> [fdo#109016]: https://bugs.freedesktop.org/show_bug.cgi?id=109016
> [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
> [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
> [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
> [fdo#109352]: https://bugs.freedesktop.org/show_bug.cgi?id=109352
> [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
> [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403
> [fdo#110519]: https://bugs.freedesktop.org/show_bug.cgi?id=110519
> [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
> [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
>
>
> Participating hosts (10 -> 10)
> ------------------------------
>
> No changes in participating hosts
>
>
> Build changes
> -------------
>
> * Linux: CI_DRM_6012 -> Patchwork_12900
>
> CI_DRM_6012: e4882f199157e3fb73d1791352931096f6ecfcfd @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4968: caed251990f35bfe45368f803980071a73e36315 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_12900: 1707feeade4b0b06b27845ac8177c942984f1c35 @ 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_12900/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Fi.CI.IGT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10)
2019-04-29 23:08 [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Aditya Swarup
` (2 preceding siblings ...)
2019-05-03 20:44 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10) Patchwork
@ 2019-05-04 2:00 ` Patchwork
2019-05-07 6:42 ` [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Jani Nikula
4 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-05-04 2:00 UTC (permalink / raw)
To: Aditya Swarup; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10)
URL : https://patchwork.freedesktop.org/series/58912/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6041_full -> Patchwork_12964_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Known issues
------------
Here are the changes found in Patchwork_12964_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_isolation@bcs0-s3:
- shard-apl: [PASS][1] -> [DMESG-WARN][2] ([fdo#108566]) +4 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-apl2/igt@gem_ctx_isolation@bcs0-s3.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-apl5/igt@gem_ctx_isolation@bcs0-s3.html
* igt@gem_tiled_swapping@non-threaded:
- shard-hsw: [PASS][3] -> [DMESG-WARN][4] ([fdo#108686])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-hsw1/igt@gem_tiled_swapping@non-threaded.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-hsw2/igt@gem_tiled_swapping@non-threaded.html
* igt@gem_workarounds@suspend-resume-fd:
- shard-kbl: [PASS][5] -> [DMESG-WARN][6] ([fdo#108566]) +1 similar issue
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-kbl6/igt@gem_workarounds@suspend-resume-fd.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-kbl1/igt@gem_workarounds@suspend-resume-fd.html
* igt@i915_pm_rpm@dpms-lpsp:
- shard-skl: [PASS][7] -> [INCOMPLETE][8] ([fdo#107807] / [fdo#110581])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl1/igt@i915_pm_rpm@dpms-lpsp.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl9/igt@i915_pm_rpm@dpms-lpsp.html
* igt@kms_cursor_edge_walk@pipe-a-128x128-bottom-edge:
- shard-snb: [PASS][9] -> [SKIP][10] ([fdo#109271] / [fdo#109278])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-snb4/igt@kms_cursor_edge_walk@pipe-a-128x128-bottom-edge.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-snb4/igt@kms_cursor_edge_walk@pipe-a-128x128-bottom-edge.html
* igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled:
- shard-skl: [PASS][11] -> [FAIL][12] ([fdo#103184] / [fdo#103232])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl8/igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl6/igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled.html
* igt@kms_flip@2x-flip-vs-suspend-interruptible:
- shard-glk: [PASS][13] -> [INCOMPLETE][14] ([fdo#103359] / [fdo#110581] / [k.org#198133])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-glk6/igt@kms_flip@2x-flip-vs-suspend-interruptible.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-glk3/igt@kms_flip@2x-flip-vs-suspend-interruptible.html
- shard-hsw: [PASS][15] -> [INCOMPLETE][16] ([fdo#103540] / [fdo#110581])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-hsw6/igt@kms_flip@2x-flip-vs-suspend-interruptible.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-hsw7/igt@kms_flip@2x-flip-vs-suspend-interruptible.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite:
- shard-iclb: [PASS][17] -> [FAIL][18] ([fdo#103167]) +10 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
* igt@kms_plane_cursor@pipe-a-viewport-size-128:
- shard-snb: [PASS][19] -> [SKIP][20] ([fdo#109271]) +4 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-snb4/igt@kms_plane_cursor@pipe-a-viewport-size-128.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-snb4/igt@kms_plane_cursor@pipe-a-viewport-size-128.html
* igt@kms_plane_scaling@pipe-a-scaler-with-pixel-format:
- shard-glk: [PASS][21] -> [SKIP][22] ([fdo#109271] / [fdo#109278]) +1 similar issue
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-glk9/igt@kms_plane_scaling@pipe-a-scaler-with-pixel-format.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-glk8/igt@kms_plane_scaling@pipe-a-scaler-with-pixel-format.html
* igt@kms_psr@psr2_primary_page_flip:
- shard-iclb: [PASS][23] -> [SKIP][24] ([fdo#109441]) +1 similar issue
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-iclb6/igt@kms_psr@psr2_primary_page_flip.html
* igt@kms_setmode@basic:
- shard-apl: [PASS][25] -> [FAIL][26] ([fdo#99912])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-apl7/igt@kms_setmode@basic.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-apl8/igt@kms_setmode@basic.html
- shard-kbl: [PASS][27] -> [FAIL][28] ([fdo#99912])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-kbl2/igt@kms_setmode@basic.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-kbl5/igt@kms_setmode@basic.html
#### Possible fixes ####
* igt@debugfs_test@read_all_entries_display_off:
- shard-skl: [INCOMPLETE][29] ([fdo#104108] / [fdo#110581]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl5/igt@debugfs_test@read_all_entries_display_off.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl5/igt@debugfs_test@read_all_entries_display_off.html
* igt@gem_tiled_swapping@non-threaded:
- shard-snb: [DMESG-WARN][31] ([fdo#108686]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-snb6/igt@gem_tiled_swapping@non-threaded.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-snb2/igt@gem_tiled_swapping@non-threaded.html
* igt@gem_workarounds@suspend-resume-context:
- shard-apl: [DMESG-WARN][33] ([fdo#108566]) -> [PASS][34] +9 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-apl7/igt@gem_workarounds@suspend-resume-context.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-apl8/igt@gem_workarounds@suspend-resume-context.html
* igt@i915_pm_rpm@i2c:
- shard-iclb: [FAIL][35] ([fdo#104097]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-iclb6/igt@i915_pm_rpm@i2c.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-iclb7/igt@i915_pm_rpm@i2c.html
* igt@i915_pm_rpm@pm-tiling:
- shard-skl: [INCOMPLETE][37] ([fdo#107807] / [fdo#110581]) -> [PASS][38]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl6/igt@i915_pm_rpm@pm-tiling.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl9/igt@i915_pm_rpm@pm-tiling.html
* igt@kms_flip@flip-vs-suspend:
- shard-skl: [INCOMPLETE][39] ([fdo#107773] / [fdo#109507] / [fdo#110581]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl5/igt@kms_flip@flip-vs-suspend.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl9/igt@kms_flip@flip-vs-suspend.html
* igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu:
- shard-skl: [FAIL][41] ([fdo#103167] / [fdo#110379]) -> [PASS][42]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl8/igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl6/igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-indfb-draw-blt:
- shard-iclb: [FAIL][43] ([fdo#103167]) -> [PASS][44] +2 similar issues
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-iclb4/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-indfb-draw-blt.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-indfb-draw-blt.html
* igt@kms_plane@pixel-format-pipe-b-planes:
- shard-glk: [SKIP][45] ([fdo#109271]) -> [PASS][46]
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-glk4/igt@kms_plane@pixel-format-pipe-b-planes.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-glk9/igt@kms_plane@pixel-format-pipe-b-planes.html
* igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min:
- shard-skl: [FAIL][47] ([fdo#108145]) -> [PASS][48]
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-skl9/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-skl1/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html
* igt@kms_plane_lowres@pipe-a-tiling-x:
- shard-iclb: [FAIL][49] ([fdo#103166]) -> [PASS][50]
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-iclb6/igt@kms_plane_lowres@pipe-a-tiling-x.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-iclb7/igt@kms_plane_lowres@pipe-a-tiling-x.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-iclb: [SKIP][51] ([fdo#109441]) -> [PASS][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6041/shard-iclb7/igt@kms_psr@psr2_sprite_plane_move.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12964/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html
[fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
[fdo#104097]: https://bugs.freedesktop.org/show_bug.cgi?id=104097
[fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
[fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
[fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109507]: https://bugs.freedesktop.org/show_bug.cgi?id=109507
[fdo#110379]: https://bugs.freedesktop.org/show_bug.cgi?id=110379
[fdo#110581]: https://bugs.freedesktop.org/show_bug.cgi?id=110581
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_6041 -> Patchwork_12964
CI_DRM_6041: 014903e8b7de5d69a17de628345ed31db1600b73 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4972: f052e49a43cc9704ea5f240df15dd9d3dfed68ab @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_12964: 53926fe20d0a3628bbcb96193aa6b8ebc3da13ab @ 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_12964/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: ✗ Fi.CI.IGT: failure for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
2019-05-03 23:04 ` Aditya Swarup
@ 2019-05-04 5:55 ` Saarinen, Jani
2019-05-06 6:59 ` Tomi Sarvela
0 siblings, 1 reply; 11+ messages in thread
From: Saarinen, Jani @ 2019-05-04 5:55 UTC (permalink / raw)
To: Swarup, Aditya, intel-gfx, Sarvela, Tomi P; +Cc: Peres, Martin
Hi,
> -----Original Message-----
> From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of
> Aditya Swarup
> Sent: lauantai 4. toukokuuta 2019 2.04
> To: intel-gfx@lists.freedesktop.org
> Cc: Peres, Martin <martin.peres@intel.com>
> Subject: Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/icl: Set
> GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
>
> On Tue, Apr 30, 2019 at 01:46:15PM +0000, Patchwork wrote:
> > == Series Details ==
> >
> > Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
> (rev8)
> > URL : https://patchwork.freedesktop.org/series/58912/
> > State : failure
> >
> > == Summary ==
> >
> > CI Bug Log - changes from CI_DRM_6012_full -> Patchwork_12900_full
> > ====================================================
> >
> > Summary
> > -------
> >
> > **FAILURE**
> >
> > Serious unknown changes coming with Patchwork_12900_full absolutely need to
> be
> > verified manually.
> >
> > If you think the reported changes have nothing to do with the changes
> > introduced in Patchwork_12900_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_12900_full:
> >
> > ### IGT changes ###
> >
> > #### Possible regressions ####
> >
> > * igt@gem_caching@read-writes:
> > - shard-skl: [PASS][1] -> [INCOMPLETE][2] +6 similar issues
> > [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl9/igt@gem_caching@read-writes.html
> > [2]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/ig
> > t@gem_caching@read-writes.html
> >
> > * igt@gem_mmap_gtt@forked-big-copy-odd:
> > - shard-skl: NOTRUN -> [INCOMPLETE][3] +1 similar issue
> > [3]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl2/ig
> > t@gem_mmap_gtt@forked-big-copy-odd.html
> Hi Martin,
>
> These failures shouldn't be due to my patch. Is there a separate bug filed for
> gem_caching and gem_mmap_gtt failures?
Yes, this still I guess due to : https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl2/run25.log
=> Tomi, still happening: https://bugs.freedesktop.org/show_bug.cgi?id=110581
Br,
Jani
>
> Regards,
> Aditya Swarup
> >
> >
> > #### Warnings ####
> >
> > * igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size:
> > - shard-skl: [SKIP][4] ([fdo#109271]) -> [INCOMPLETE][5] +1 similar issue
> > [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl6/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
> > [5]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl8/ig
> > t@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
> >
> >
> > Known issues
> > ------------
> >
> > Here are the changes found in Patchwork_12900_full that come from known
> issues:
> >
> > ### IGT changes ###
> >
> > #### Issues hit ####
> >
> > * igt@gem_cpu_reloc@forked:
> > - shard-iclb: [PASS][6] -> [INCOMPLETE][7] ([fdo#107713] / [fdo#109100])
> > [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb4/igt@gem_cpu_reloc@forked.html
> > [7]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb3/i
> > gt@gem_cpu_reloc@forked.html
> >
> > * igt@i915_pm_rpm@pm-tiling:
> > - shard-skl: [PASS][8] -> [INCOMPLETE][9] ([fdo#107807])
> > [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl10/igt@i915_pm_rpm@pm-tiling.html
> > [9]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/ig
> > t@i915_pm_rpm@pm-tiling.html
> >
> > * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
> > - shard-snb: [PASS][10] -> [SKIP][11] ([fdo#109271] / [fdo#109278]) +1
> similar issue
> > [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> snb1/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
> > [11]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb6/ig
> > t@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
> >
> > * igt@kms_cursor_crc@cursor-64x21-offscreen:
> > - shard-skl: [PASS][12] -> [FAIL][13] ([fdo#103232])
> > [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl2/igt@kms_cursor_crc@cursor-64x21-offscreen.html
> > [13]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/ig
> > t@kms_cursor_crc@cursor-64x21-offscreen.html
> >
> > * igt@kms_flip@flip-vs-suspend-interruptible:
> > - shard-glk: [PASS][14] -> [INCOMPLETE][15] ([fdo#103359] /
> [k.org#198133]) +2 similar issues
> > [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> glk5/igt@kms_flip@flip-vs-suspend-interruptible.html
> > [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-
> glk7/igt@kms_flip@flip-vs-suspend-interruptible.html
> > - shard-apl: [PASS][16] -> [DMESG-WARN][17] ([fdo#108566]) +1 similar
> issue
> > [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
> > [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-
> apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
> > - shard-snb: [PASS][18] -> [INCOMPLETE][19] ([fdo#105411])
> > [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> snb5/igt@kms_flip@flip-vs-suspend-interruptible.html
> > [19]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb1/ig
> > t@kms_flip@flip-vs-suspend-interruptible.html
> >
> > * igt@kms_flip@nonexisting-fb-interruptible:
> > - shard-iclb: [PASS][20] -> [INCOMPLETE][21] ([fdo#107713])
> > [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb3/igt@kms_flip@nonexisting-fb-interruptible.html
> > [21]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb4/i
> > gt@kms_flip@nonexisting-fb-interruptible.html
> >
> > * igt@kms_flip@plain-flip-ts-check-interruptible:
> > - shard-skl: [PASS][22] -> [FAIL][23] ([fdo#100368])
> > [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl2/igt@kms_flip@plain-flip-ts-check-interruptible.html
> > [23]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl1/ig
> > t@kms_flip@plain-flip-ts-check-interruptible.html
> >
> > * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render:
> > - shard-iclb: [PASS][24] -> [FAIL][25] ([fdo#103167]) +4 similar issues
> > [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-
> render.html
> > [25]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/i
> > gt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.h
> > tml
> >
> > * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render:
> > - shard-skl: [PASS][26] -> [INCOMPLETE][27] ([fdo#106978])
> > [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render.html
> > [27]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl7/ig
> > t@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-render.ht
> > ml
> >
> > * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min:
> > - shard-skl: [PASS][28] -> [FAIL][29] ([fdo#108145])
> > [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl5/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html
> > [29]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl7/ig
> > t@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html
> >
> > * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
> > - shard-skl: [PASS][30] -> [FAIL][31] ([fdo#108145] / [fdo#110403])
> > [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl2/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
> > [31]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl3/ig
> > t@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
> >
> > * igt@kms_plane_cursor@pipe-b-overlay-size-256:
> > - shard-snb: [PASS][32] -> [SKIP][33] ([fdo#109271]) +5 similar issues
> > [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> snb1/igt@kms_plane_cursor@pipe-b-overlay-size-256.html
> > [33]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-snb6/ig
> > t@kms_plane_cursor@pipe-b-overlay-size-256.html
> >
> > * igt@kms_psr@psr2_cursor_mmap_cpu:
> > - shard-iclb: [PASS][34] -> [SKIP][35] ([fdo#109441]) +2 similar issues
> > [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html
> > [35]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb6/i
> > gt@kms_psr@psr2_cursor_mmap_cpu.html
> >
> > * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
> > - shard-kbl: [PASS][36] -> [FAIL][37] ([fdo#109016])
> > [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> kbl7/igt@kms_rotation_crc@multiplane-rotation-cropping-top.html
> > [37]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-kbl4/ig
> > t@kms_rotation_crc@multiplane-rotation-cropping-top.html
> >
> > * igt@kms_rotation_crc@sprite-rotation-180:
> > - shard-skl: [PASS][38] -> [INCOMPLETE][39] ([fdo#108972]) +1 similar issue
> > [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl1/igt@kms_rotation_crc@sprite-rotation-180.html
> > [39]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl4/ig
> > t@kms_rotation_crc@sprite-rotation-180.html
> >
> > * igt@kms_setmode@basic:
> > - shard-kbl: [PASS][40] -> [FAIL][41] ([fdo#99912])
> > [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> kbl5/igt@kms_setmode@basic.html
> > [41]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-kbl4/ig
> > t@kms_setmode@basic.html
> >
> > * igt@kms_sysfs_edid_timing:
> > - shard-iclb: [PASS][42] -> [FAIL][43] ([fdo#100047])
> > [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb6/igt@kms_sysfs_edid_timing.html
> > [43]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb3/i
> > gt@kms_sysfs_edid_timing.html
> >
> > * igt@perf_pmu@busy-idle-rcs0:
> > - shard-hsw: [PASS][44] -> [INCOMPLETE][45] ([fdo#103540]) +2 similar
> issues
> > [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> hsw5/igt@perf_pmu@busy-idle-rcs0.html
> > [45]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-hsw6/ig
> > t@perf_pmu@busy-idle-rcs0.html
> >
> >
> > #### Possible fixes ####
> >
> > * igt@i915_suspend@fence-restore-untiled:
> > - shard-apl: [DMESG-WARN][46] ([fdo#108566]) -> [PASS][47] +3 similar
> issues
> > [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> apl2/igt@i915_suspend@fence-restore-untiled.html
> > [47]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl3/ig
> > t@i915_suspend@fence-restore-untiled.html
> >
> > * igt@kms_cursor_crc@cursor-64x64-suspend:
> > - shard-skl: [INCOMPLETE][48] ([fdo#104108]) -> [PASS][49]
> > [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl3/igt@kms_cursor_crc@cursor-64x64-suspend.html
> > [49]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl10/i
> > gt@kms_cursor_crc@cursor-64x64-suspend.html
> >
> > * igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic:
> > - shard-glk: [FAIL][50] ([fdo#106509] / [fdo#107409]) -> [PASS][51]
> > [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> glk2/igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic.html
> > [51]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk1/ig
> > t@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic.html
> >
> > * igt@kms_flip@flip-vs-expired-vblank-interruptible:
> > - shard-skl: [FAIL][52] ([fdo#105363]) -> [PASS][53]
> > [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl3/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
> > [53]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl6/ig
> > t@kms_flip@flip-vs-expired-vblank-interruptible.html
> >
> > * igt@kms_flip@modeset-vs-vblank-race:
> > - shard-glk: [FAIL][54] ([fdo#103060]) -> [PASS][55]
> > [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> glk8/igt@kms_flip@modeset-vs-vblank-race.html
> > [55]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk1/ig
> > t@kms_flip@modeset-vs-vblank-race.html
> >
> > * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render:
> > - shard-iclb: [FAIL][56] ([fdo#103167]) -> [PASS][57] +3 similar issues
> > [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb1/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html
> > [57]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb5/i
> > gt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html
> >
> > * igt@kms_frontbuffer_tracking@fbcpsr-suspend:
> > - shard-skl: [INCOMPLETE][58] ([fdo#104108] / [fdo#106978]) -> [PASS][59]
> > [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl7/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> > [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-
> skl3/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> > - shard-iclb: [INCOMPLETE][60] ([fdo#106978] / [fdo#107713]) ->
> [PASS][61]
> > [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb4/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> > [61]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/i
> > gt@kms_frontbuffer_tracking@fbcpsr-suspend.html
> >
> > * igt@kms_plane@pixel-format-pipe-c-planes:
> > - shard-glk: [SKIP][62] ([fdo#109271]) -> [PASS][63]
> > [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> glk4/igt@kms_plane@pixel-format-pipe-c-planes.html
> > [63]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk9/ig
> > t@kms_plane@pixel-format-pipe-c-planes.html
> >
> > * igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format:
> > - shard-glk: [SKIP][64] ([fdo#109271] / [fdo#109278]) -> [PASS][65]
> > [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> glk7/igt@kms_plane_scaling@pipe-c-scaler-with-pixel-format.html
> > [65]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-glk9/ig
> > t@kms_plane_scaling@pipe-c-scaler-with-pixel-format.html
> >
> > * igt@kms_psr@psr2_cursor_plane_onoff:
> > - shard-iclb: [SKIP][66] ([fdo#109441]) -> [PASS][67] +3 similar issues
> > [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb6/igt@kms_psr@psr2_cursor_plane_onoff.html
> > [67]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb2/i
> > gt@kms_psr@psr2_cursor_plane_onoff.html
> >
> > * igt@kms_setmode@basic:
> > - shard-apl: [FAIL][68] ([fdo#99912]) -> [PASS][69]
> > [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> apl5/igt@kms_setmode@basic.html
> > [69]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-apl4/ig
> > t@kms_setmode@basic.html
> >
> > * igt@tools_test@tools_test:
> > - shard-iclb: [SKIP][70] ([fdo#109352]) -> [PASS][71]
> > [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> iclb8/igt@tools_test@tools_test.html
> > [71]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-iclb6/i
> > gt@tools_test@tools_test.html
> >
> >
> > #### Warnings ####
> >
> > * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt:
> > - shard-skl: [FAIL][72] ([fdo#108040]) -> [FAIL][73] ([fdo#103167])
> > [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
> skl8/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt.html
> > [73]:
> > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl5/ig
> > t@kms_frontbuffer_tracking@fbc-1p-primscrn-shrfb-plflip-blt.html
> >
> >
> > {name}: This element is suppressed. This means it is ignored when computing
> > the status of the difference (SUCCESS, WARNING, or FAILURE).
> >
> > [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
> > [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
> > [fdo#103060]: https://bugs.freedesktop.org/show_bug.cgi?id=103060
> > [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
> > [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
> > [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
> > [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
> > [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
> > [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
> > [fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
> > [fdo#106509]: https://bugs.freedesktop.org/show_bug.cgi?id=106509
> > [fdo#106978]: https://bugs.freedesktop.org/show_bug.cgi?id=106978
> > [fdo#107409]: https://bugs.freedesktop.org/show_bug.cgi?id=107409
> > [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
> > [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
> > [fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
> > [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
> > [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
> > [fdo#108972]: https://bugs.freedesktop.org/show_bug.cgi?id=108972
> > [fdo#109016]: https://bugs.freedesktop.org/show_bug.cgi?id=109016
> > [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
> > [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
> > [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
> > [fdo#109352]: https://bugs.freedesktop.org/show_bug.cgi?id=109352
> > [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
> > [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403
> > [fdo#110519]: https://bugs.freedesktop.org/show_bug.cgi?id=110519
> > [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
> > [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
> >
> >
> > Participating hosts (10 -> 10)
> > ------------------------------
> >
> > No changes in participating hosts
> >
> >
> > Build changes
> > -------------
> >
> > * Linux: CI_DRM_6012 -> Patchwork_12900
> >
> > CI_DRM_6012: e4882f199157e3fb73d1791352931096f6ecfcfd @
> git://anongit.freedesktop.org/gfx-ci/linux
> > IGT_4968: caed251990f35bfe45368f803980071a73e36315 @
> git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> > Patchwork_12900: 1707feeade4b0b06b27845ac8177c942984f1c35 @
> 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_12900/
> _______________________________________________
> 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] 11+ messages in thread
* Re: ✗ Fi.CI.IGT: failure for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
2019-05-04 5:55 ` Saarinen, Jani
@ 2019-05-06 6:59 ` Tomi Sarvela
0 siblings, 0 replies; 11+ messages in thread
From: Tomi Sarvela @ 2019-05-06 6:59 UTC (permalink / raw)
To: Saarinen, Jani, Swarup, Aditya, intel-gfx; +Cc: Peres, Martin
On 5/4/19 8:55 AM, Saarinen, Jani wrote:
> Hi,
>> -----Original Message-----
>> From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of
>> Aditya Swarup
>> Sent: lauantai 4. toukokuuta 2019 2.04
>> To: intel-gfx@lists.freedesktop.org
>> Cc: Peres, Martin <martin.peres@intel.com>
>> Subject: Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/icl: Set
>> GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8)
>>
>> On Tue, Apr 30, 2019 at 01:46:15PM +0000, Patchwork wrote:
>>> == Series Details ==
>>>
>>> Series: drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
>> (rev8)
>>> URL : https://patchwork.freedesktop.org/series/58912/
>>> State : failure
>>>
>>> == Summary ==
>>>
>>> CI Bug Log - changes from CI_DRM_6012_full -> Patchwork_12900_full
>>> ====================================================
>>>
>>> Summary
>>> -------
>>>
>>> **FAILURE**
>>>
>>> Serious unknown changes coming with Patchwork_12900_full absolutely need to
>> be
>>> verified manually.
>>>
>>> If you think the reported changes have nothing to do with the changes
>>> introduced in Patchwork_12900_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_12900_full:
>>>
>>> ### IGT changes ###
>>>
>>> #### Possible regressions ####
>>>
>>> * igt@gem_caching@read-writes:
>>> - shard-skl: [PASS][1] -> [INCOMPLETE][2] +6 similar issues
>>> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6012/shard-
>> skl9/igt@gem_caching@read-writes.html
>>> [2]:
>>> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl9/ig
>>> t@gem_caching@read-writes.html
>>>
>>> * igt@gem_mmap_gtt@forked-big-copy-odd:
>>> - shard-skl: NOTRUN -> [INCOMPLETE][3] +1 similar issue
>>> [3]:
>>> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl2/ig
>>> t@gem_mmap_gtt@forked-big-copy-odd.html
>> Hi Martin,
>>
>> These failures shouldn't be due to my patch. Is there a separate bug filed for
>> gem_caching and gem_mmap_gtt failures?
> Yes, this still I guess due to : https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12900/shard-skl2/run25.log
> => Tomi, still happening: https://bugs.freedesktop.org/show_bug.cgi?id=110581
>
> Br,
> Jani
This was fixed around IGT_4972 / CI_DRM_6026. We're looking at results
from around CI_DRM_6012, so yes, 10min timeouts were still happening.
Tomi
>>> Build changes
>>> -------------
>>>
>>> * Linux: CI_DRM_6012 -> Patchwork_12900
Tomi
--
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
2019-04-29 23:08 [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Aditya Swarup
` (3 preceding siblings ...)
2019-05-04 2:00 ` ✓ Fi.CI.IGT: " Patchwork
@ 2019-05-07 6:42 ` Jani Nikula
2019-05-07 10:34 ` Ville Syrjälä
4 siblings, 1 reply; 11+ messages in thread
From: Jani Nikula @ 2019-05-07 6:42 UTC (permalink / raw)
To: Aditya Swarup, intel-gfx
On Mon, 29 Apr 2019, Aditya Swarup <aditya.swarup@intel.com> wrote:
> From: Clinton Taylor <Clinton.A.Taylor@intel.com>
>
> v2: Fix commit msg to reflect why issue occurs(Jani)
> Set GCP_COLOR_INDICATION only when we set 10/12 bit deep color.
>
> Changing settings from 10/12 bit deep color to 8 bit(& vice versa)
> doesn't work correctly using xrandr max bpc property. When we
> connect a monitor which supports deep color, the highest deep color
> setting is selected; which sets GCP_COLOR_INDICATION. When we change
> the setting to 8 bit color, we still set GCP_COLOR_INDICATION which
> doesn't allow the switch back to 8 bit color.
>
> v3,4: Add comments & drop changes in intel_hdmi_compute_config(Ville)
> Since HSW+, GCP_COLOR_INDICATION is not required for 8bpc.
>
> Drop the changes in intel_hdmi_compute_config as desired_bpp
> is needed to change values for pipe_bpp based on bw_constrained flag.
>
> v5: Fix missing logical && in condition for setting GCP_COLOR_INDICATION.
>
> v6: Fix comment formatting (Ville)
>
> v7: Add reviewed by Ville
>
> v8: Set GCP_COLOR_INDICATION based on spec:
> For Gen 7.5 or later platforms, indicate color depth only for deep
> color modes. Bspec: 8135,7751,50524
>
> Pre DDI platforms, indicate color depth if deep color is supported
> by sink. Bspec: 7854
>
> Exception: CHERRYVIEW behaves like Pre DDI platforms.
> Bspec: 15975
>
> Check pipe_bpp is less than bpp * 3 in hdmi_deep_color_possible,
> to not set 12 bit deep color for every modeset. This fixes the issue
> where 12 bit color was selected even when user selected 10 bit.(Ville)
>
> v9: Maintain a consistent behavior for all platforms and support
> GCP_COLOR_INDICATION only when we are in deep color mode. Remove
> hdmi_sink_is_deep_color() - no longer needed as checking pipe_bpp > 24
> takes care of the deep color mode scenario.
>
> Separate patch for fixing switch from 12 bit to 10 bit deep color
> mode.
>
> Co-developed-by: Aditya Swarup <aditya.swarup@intel.com>
> Signed-off-by: Clinton Taylor <Clinton.A.Taylor@intel.com>
> Signed-off-by: Aditya Swarup <aditya.swarup@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: Jani Nikula <jani.nikula@intel.com>
> Cc: Manasi Navare <manasi.d.navare@intel.com>
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Ville, does this apply to this version of the patch?
BR,
Jani.
> ---
> drivers/gpu/drm/i915/intel_hdmi.c | 17 ++---------------
> 1 file changed, 2 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c
> index e1005d7b75fd..991eb362ef4f 100644
> --- a/drivers/gpu/drm/i915/intel_hdmi.c
> +++ b/drivers/gpu/drm/i915/intel_hdmi.c
> @@ -856,19 +856,6 @@ static void g4x_set_infoframes(struct intel_encoder *encoder,
> &crtc_state->infoframes.hdmi);
> }
>
> -static bool hdmi_sink_is_deep_color(const struct drm_connector_state *conn_state)
> -{
> - struct drm_connector *connector = conn_state->connector;
> -
> - /*
> - * HDMI cloning is only supported on g4x which doesn't
> - * support deep color or GCP infoframes anyway so no
> - * need to worry about multiple HDMI sinks here.
> - */
> -
> - return connector->display_info.bpc > 8;
> -}
> -
> /*
> * Determine if default_phase=1 can be indicated in the GCP infoframe.
> *
> @@ -973,8 +960,8 @@ static void intel_hdmi_compute_gcp_infoframe(struct intel_encoder *encoder,
> crtc_state->infoframes.enable |=
> intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GENERAL_CONTROL);
>
> - /* Indicate color depth whenever the sink supports deep color */
> - if (hdmi_sink_is_deep_color(conn_state))
> + /* Indicate color indication for deep color mode */
> + if (crtc_state->pipe_bpp > 24)
> crtc_state->infoframes.gcp |= GCP_COLOR_INDICATION;
>
> /* Enable default_phase whenever the display mode is suitably aligned */
--
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] 11+ messages in thread
* Re: [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
2019-05-07 6:42 ` [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Jani Nikula
@ 2019-05-07 10:34 ` Ville Syrjälä
2019-05-07 11:19 ` Jani Nikula
0 siblings, 1 reply; 11+ messages in thread
From: Ville Syrjälä @ 2019-05-07 10:34 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
On Tue, May 07, 2019 at 09:42:48AM +0300, Jani Nikula wrote:
> On Mon, 29 Apr 2019, Aditya Swarup <aditya.swarup@intel.com> wrote:
> > From: Clinton Taylor <Clinton.A.Taylor@intel.com>
> >
> > v2: Fix commit msg to reflect why issue occurs(Jani)
> > Set GCP_COLOR_INDICATION only when we set 10/12 bit deep color.
> >
> > Changing settings from 10/12 bit deep color to 8 bit(& vice versa)
> > doesn't work correctly using xrandr max bpc property. When we
> > connect a monitor which supports deep color, the highest deep color
> > setting is selected; which sets GCP_COLOR_INDICATION. When we change
> > the setting to 8 bit color, we still set GCP_COLOR_INDICATION which
> > doesn't allow the switch back to 8 bit color.
> >
> > v3,4: Add comments & drop changes in intel_hdmi_compute_config(Ville)
> > Since HSW+, GCP_COLOR_INDICATION is not required for 8bpc.
> >
> > Drop the changes in intel_hdmi_compute_config as desired_bpp
> > is needed to change values for pipe_bpp based on bw_constrained flag.
> >
> > v5: Fix missing logical && in condition for setting GCP_COLOR_INDICATION.
> >
> > v6: Fix comment formatting (Ville)
> >
> > v7: Add reviewed by Ville
> >
> > v8: Set GCP_COLOR_INDICATION based on spec:
> > For Gen 7.5 or later platforms, indicate color depth only for deep
> > color modes. Bspec: 8135,7751,50524
> >
> > Pre DDI platforms, indicate color depth if deep color is supported
> > by sink. Bspec: 7854
> >
> > Exception: CHERRYVIEW behaves like Pre DDI platforms.
> > Bspec: 15975
> >
> > Check pipe_bpp is less than bpp * 3 in hdmi_deep_color_possible,
> > to not set 12 bit deep color for every modeset. This fixes the issue
> > where 12 bit color was selected even when user selected 10 bit.(Ville)
> >
> > v9: Maintain a consistent behavior for all platforms and support
> > GCP_COLOR_INDICATION only when we are in deep color mode. Remove
> > hdmi_sink_is_deep_color() - no longer needed as checking pipe_bpp > 24
> > takes care of the deep color mode scenario.
> >
> > Separate patch for fixing switch from 12 bit to 10 bit deep color
> > mode.
> >
> > Co-developed-by: Aditya Swarup <aditya.swarup@intel.com>
> > Signed-off-by: Clinton Taylor <Clinton.A.Taylor@intel.com>
> > Signed-off-by: Aditya Swarup <aditya.swarup@intel.com>
> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > Cc: Jani Nikula <jani.nikula@intel.com>
> > Cc: Manasi Navare <manasi.d.navare@intel.com>
> > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Ville, does this apply to this version of the patch?
Aye. lgtm
>
> BR,
> Jani.
>
>
> > ---
> > drivers/gpu/drm/i915/intel_hdmi.c | 17 ++---------------
> > 1 file changed, 2 insertions(+), 15 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c
> > index e1005d7b75fd..991eb362ef4f 100644
> > --- a/drivers/gpu/drm/i915/intel_hdmi.c
> > +++ b/drivers/gpu/drm/i915/intel_hdmi.c
> > @@ -856,19 +856,6 @@ static void g4x_set_infoframes(struct intel_encoder *encoder,
> > &crtc_state->infoframes.hdmi);
> > }
> >
> > -static bool hdmi_sink_is_deep_color(const struct drm_connector_state *conn_state)
> > -{
> > - struct drm_connector *connector = conn_state->connector;
> > -
> > - /*
> > - * HDMI cloning is only supported on g4x which doesn't
> > - * support deep color or GCP infoframes anyway so no
> > - * need to worry about multiple HDMI sinks here.
> > - */
> > -
> > - return connector->display_info.bpc > 8;
> > -}
> > -
> > /*
> > * Determine if default_phase=1 can be indicated in the GCP infoframe.
> > *
> > @@ -973,8 +960,8 @@ static void intel_hdmi_compute_gcp_infoframe(struct intel_encoder *encoder,
> > crtc_state->infoframes.enable |=
> > intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GENERAL_CONTROL);
> >
> > - /* Indicate color depth whenever the sink supports deep color */
> > - if (hdmi_sink_is_deep_color(conn_state))
> > + /* Indicate color indication for deep color mode */
> > + if (crtc_state->pipe_bpp > 24)
> > crtc_state->infoframes.gcp |= GCP_COLOR_INDICATION;
> >
> > /* Enable default_phase whenever the display mode is suitably aligned */
>
> --
> Jani Nikula, Intel Open Source Graphics Center
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color
2019-05-07 10:34 ` Ville Syrjälä
@ 2019-05-07 11:19 ` Jani Nikula
0 siblings, 0 replies; 11+ messages in thread
From: Jani Nikula @ 2019-05-07 11:19 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
On Tue, 07 May 2019, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Tue, May 07, 2019 at 09:42:48AM +0300, Jani Nikula wrote:
>> On Mon, 29 Apr 2019, Aditya Swarup <aditya.swarup@intel.com> wrote:
>> > From: Clinton Taylor <Clinton.A.Taylor@intel.com>
>> >
>> > v2: Fix commit msg to reflect why issue occurs(Jani)
>> > Set GCP_COLOR_INDICATION only when we set 10/12 bit deep color.
>> >
>> > Changing settings from 10/12 bit deep color to 8 bit(& vice versa)
>> > doesn't work correctly using xrandr max bpc property. When we
>> > connect a monitor which supports deep color, the highest deep color
>> > setting is selected; which sets GCP_COLOR_INDICATION. When we change
>> > the setting to 8 bit color, we still set GCP_COLOR_INDICATION which
>> > doesn't allow the switch back to 8 bit color.
>> >
>> > v3,4: Add comments & drop changes in intel_hdmi_compute_config(Ville)
>> > Since HSW+, GCP_COLOR_INDICATION is not required for 8bpc.
>> >
>> > Drop the changes in intel_hdmi_compute_config as desired_bpp
>> > is needed to change values for pipe_bpp based on bw_constrained flag.
>> >
>> > v5: Fix missing logical && in condition for setting GCP_COLOR_INDICATION.
>> >
>> > v6: Fix comment formatting (Ville)
>> >
>> > v7: Add reviewed by Ville
>> >
>> > v8: Set GCP_COLOR_INDICATION based on spec:
>> > For Gen 7.5 or later platforms, indicate color depth only for deep
>> > color modes. Bspec: 8135,7751,50524
>> >
>> > Pre DDI platforms, indicate color depth if deep color is supported
>> > by sink. Bspec: 7854
>> >
>> > Exception: CHERRYVIEW behaves like Pre DDI platforms.
>> > Bspec: 15975
>> >
>> > Check pipe_bpp is less than bpp * 3 in hdmi_deep_color_possible,
>> > to not set 12 bit deep color for every modeset. This fixes the issue
>> > where 12 bit color was selected even when user selected 10 bit.(Ville)
>> >
>> > v9: Maintain a consistent behavior for all platforms and support
>> > GCP_COLOR_INDICATION only when we are in deep color mode. Remove
>> > hdmi_sink_is_deep_color() - no longer needed as checking pipe_bpp > 24
>> > takes care of the deep color mode scenario.
>> >
>> > Separate patch for fixing switch from 12 bit to 10 bit deep color
>> > mode.
>> >
>> > Co-developed-by: Aditya Swarup <aditya.swarup@intel.com>
>> > Signed-off-by: Clinton Taylor <Clinton.A.Taylor@intel.com>
>> > Signed-off-by: Aditya Swarup <aditya.swarup@intel.com>
>> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>> > Cc: Jani Nikula <jani.nikula@intel.com>
>> > Cc: Manasi Navare <manasi.d.navare@intel.com>
>> > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>
>> Ville, does this apply to this version of the patch?
>
> Aye. lgtm
Thanks for the patch and review, pushed to dinq.
BR,
Jani.
>
>>
>> BR,
>> Jani.
>>
>>
>> > ---
>> > drivers/gpu/drm/i915/intel_hdmi.c | 17 ++---------------
>> > 1 file changed, 2 insertions(+), 15 deletions(-)
>> >
>> > diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c
>> > index e1005d7b75fd..991eb362ef4f 100644
>> > --- a/drivers/gpu/drm/i915/intel_hdmi.c
>> > +++ b/drivers/gpu/drm/i915/intel_hdmi.c
>> > @@ -856,19 +856,6 @@ static void g4x_set_infoframes(struct intel_encoder *encoder,
>> > &crtc_state->infoframes.hdmi);
>> > }
>> >
>> > -static bool hdmi_sink_is_deep_color(const struct drm_connector_state *conn_state)
>> > -{
>> > - struct drm_connector *connector = conn_state->connector;
>> > -
>> > - /*
>> > - * HDMI cloning is only supported on g4x which doesn't
>> > - * support deep color or GCP infoframes anyway so no
>> > - * need to worry about multiple HDMI sinks here.
>> > - */
>> > -
>> > - return connector->display_info.bpc > 8;
>> > -}
>> > -
>> > /*
>> > * Determine if default_phase=1 can be indicated in the GCP infoframe.
>> > *
>> > @@ -973,8 +960,8 @@ static void intel_hdmi_compute_gcp_infoframe(struct intel_encoder *encoder,
>> > crtc_state->infoframes.enable |=
>> > intel_hdmi_infoframe_enable(HDMI_PACKET_TYPE_GENERAL_CONTROL);
>> >
>> > - /* Indicate color depth whenever the sink supports deep color */
>> > - if (hdmi_sink_is_deep_color(conn_state))
>> > + /* Indicate color indication for deep color mode */
>> > + if (crtc_state->pipe_bpp > 24)
>> > crtc_state->infoframes.gcp |= GCP_COLOR_INDICATION;
>> >
>> > /* Enable default_phase whenever the display mode is suitably aligned */
>>
>> --
>> Jani Nikula, Intel Open Source Graphics Center
--
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] 11+ messages in thread
end of thread, other threads:[~2019-05-07 11:17 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-29 23:08 [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Aditya Swarup
2019-04-30 8:21 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev8) Patchwork
2019-04-30 13:46 ` ✗ Fi.CI.IGT: failure " Patchwork
2019-05-03 23:04 ` Aditya Swarup
2019-05-04 5:55 ` Saarinen, Jani
2019-05-06 6:59 ` Tomi Sarvela
2019-05-03 20:44 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color (rev10) Patchwork
2019-05-04 2:00 ` ✓ Fi.CI.IGT: " Patchwork
2019-05-07 6:42 ` [PATCH v9] drm/i915/icl: Set GCP_COLOR_INDICATION only for 10/12 bit deep color Jani Nikula
2019-05-07 10:34 ` Ville Syrjälä
2019-05-07 11:19 ` Jani Nikula
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.