* [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
@ 2018-11-07 20:08 Imre Deak
2018-11-07 20:08 ` [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports Imre Deak
` (4 more replies)
0 siblings, 5 replies; 12+ messages in thread
From: Imre Deak @ 2018-11-07 20:08 UTC (permalink / raw)
To: intel-gfx; +Cc: Paulo Zanoni, Rodrigo Vivi
Check for reserved register field values and conflicting
transcoder->port mappings (both MST and non-MST mappings or multiple SST
mappings).
This is also needed for the next patch to determine if a port is in MST
mode during sanitization after HW readout.
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Antonio Argenziano <antonio.argenziano@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
---
drivers/gpu/drm/i915/intel_ddi.c | 76 +++++++++++++++++++++++++++-------------
1 file changed, 52 insertions(+), 24 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
index ab9a36c4ba3b..abc51693eec9 100644
--- a/drivers/gpu/drm/i915/intel_ddi.c
+++ b/drivers/gpu/drm/i915/intel_ddi.c
@@ -2005,24 +2005,24 @@ bool intel_ddi_connector_get_hw_state(struct intel_connector *intel_connector)
return ret;
}
-bool intel_ddi_get_hw_state(struct intel_encoder *encoder,
- enum pipe *pipe)
+static void intel_ddi_get_encoder_pipes(struct intel_encoder *encoder,
+ u8 *pipe_mask, bool *is_dp_mst)
{
struct drm_device *dev = encoder->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
enum port port = encoder->port;
enum pipe p;
u32 tmp;
- bool ret;
+ u8 mst_pipe_mask;
+
+ *pipe_mask = 0;
+ *is_dp_mst = false;
if (!intel_display_power_get_if_enabled(dev_priv,
encoder->power_domain))
- return false;
-
- ret = false;
+ return;
tmp = I915_READ(DDI_BUF_CTL(port));
-
if (!(tmp & DDI_BUF_CTL_ENABLE))
goto out;
@@ -2030,44 +2030,58 @@ bool intel_ddi_get_hw_state(struct intel_encoder *encoder,
tmp = I915_READ(TRANS_DDI_FUNC_CTL(TRANSCODER_EDP));
switch (tmp & TRANS_DDI_EDP_INPUT_MASK) {
+ default:
+ MISSING_CASE(tmp & TRANS_DDI_EDP_INPUT_MASK);
+ /* fallthrough */
case TRANS_DDI_EDP_INPUT_A_ON:
case TRANS_DDI_EDP_INPUT_A_ONOFF:
- *pipe = PIPE_A;
+ *pipe_mask = BIT(PIPE_A);
break;
case TRANS_DDI_EDP_INPUT_B_ONOFF:
- *pipe = PIPE_B;
+ *pipe_mask = BIT(PIPE_B);
break;
case TRANS_DDI_EDP_INPUT_C_ONOFF:
- *pipe = PIPE_C;
+ *pipe_mask = BIT(PIPE_C);
break;
}
- ret = true;
-
goto out;
}
+ mst_pipe_mask = 0;
for_each_pipe(dev_priv, p) {
- enum transcoder cpu_transcoder = (enum transcoder) p;
+ enum transcoder cpu_transcoder = (enum transcoder)p;
tmp = I915_READ(TRANS_DDI_FUNC_CTL(cpu_transcoder));
- if ((tmp & TRANS_DDI_PORT_MASK) == TRANS_DDI_SELECT_PORT(port)) {
- if ((tmp & TRANS_DDI_MODE_SELECT_MASK) ==
- TRANS_DDI_MODE_SELECT_DP_MST)
- goto out;
+ if ((tmp & TRANS_DDI_PORT_MASK) != TRANS_DDI_SELECT_PORT(port))
+ continue;
- *pipe = p;
- ret = true;
+ if ((tmp & TRANS_DDI_MODE_SELECT_MASK) ==
+ TRANS_DDI_MODE_SELECT_DP_MST)
+ mst_pipe_mask |= BIT(p);
- goto out;
- }
+ *pipe_mask |= BIT(p);
}
- DRM_DEBUG_KMS("No pipe for ddi port %c found\n", port_name(port));
+ if (!*pipe_mask)
+ DRM_DEBUG_KMS("No pipe for ddi port %c found\n",
+ port_name(port));
+
+ if (!mst_pipe_mask && hweight8(*pipe_mask) > 1) {
+ DRM_DEBUG_KMS("Multiple pipes for non DP-MST port %c (pipe_mask %02x)\n",
+ port_name(port), *pipe_mask);
+ *pipe_mask = BIT(ffs(*pipe_mask) - 1);
+ }
+
+ if (mst_pipe_mask && mst_pipe_mask != *pipe_mask)
+ DRM_DEBUG_KMS("Conflicting MST and non-MST encoders for port %c (pipe_mask %02x mst_pipe_mask %02x)\n",
+ port_name(port), *pipe_mask, mst_pipe_mask);
+ else
+ *is_dp_mst = mst_pipe_mask;
out:
- if (ret && IS_GEN9_LP(dev_priv)) {
+ if (*pipe_mask && IS_GEN9_LP(dev_priv)) {
tmp = I915_READ(BXT_PHY_CTL(port));
if ((tmp & (BXT_PHY_CMNLANE_POWERDOWN_ACK |
BXT_PHY_LANE_POWERDOWN_ACK |
@@ -2077,8 +2091,22 @@ bool intel_ddi_get_hw_state(struct intel_encoder *encoder,
}
intel_display_power_put(dev_priv, encoder->power_domain);
+}
- return ret;
+bool intel_ddi_get_hw_state(struct intel_encoder *encoder,
+ enum pipe *pipe)
+{
+ u8 pipe_mask;
+ bool is_mst;
+
+ intel_ddi_get_encoder_pipes(encoder, &pipe_mask, &is_mst);
+
+ if (is_mst || !pipe_mask)
+ return false;
+
+ *pipe = ffs(pipe_mask) - 1;
+
+ return true;
}
static inline enum intel_display_power_domain
--
2.13.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports
2018-11-07 20:08 [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Imre Deak
@ 2018-11-07 20:08 ` Imre Deak
2018-11-09 0:19 ` Clint Taylor
2018-11-07 20:39 ` ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Patchwork
` (3 subsequent siblings)
4 siblings, 1 reply; 12+ messages in thread
From: Imre Deak @ 2018-11-07 20:08 UTC (permalink / raw)
To: intel-gfx; +Cc: Rodrigo Vivi, Paulo Zanoni
We shouldn't consider an encoder inactive if it doesn't have a CRTC
linked, but has virtual MST encoders with a crtc linked. Fix this.
Also we should not sanitize the mapping for MST encoders, as it's always
their primary encoder (which could be even in SST mode) whose active
state determines if we need the clock being enabled for the
corresponding physical port. Fix this too.
This fixes at least an existing breakage where we incorrectly disabled
the clock for an active DP encoder when sanitizing its MST virtual
encoders. Not sure if there are BIOSes that enable an output in MST
mode, but our HW readout is mostly missing for it anyway, so just warn
for that case.
Fixes: 70332ac539c5 ("drm/i915/icl+: Sanitize port to PLL mapping")
Cc: Antonio Argenziano <antonio.argenziano@intel.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reported-by: Antonio Argenziano <antonio.argenziano@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
---
drivers/gpu/drm/i915/intel_ddi.c | 27 +++++++++++++++++++++++++--
1 file changed, 25 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
index abc51693eec9..4913bbdac843 100644
--- a/drivers/gpu/drm/i915/intel_ddi.c
+++ b/drivers/gpu/drm/i915/intel_ddi.c
@@ -2853,9 +2853,32 @@ void icl_unmap_plls_to_ports(struct drm_crtc *crtc,
void icl_sanitize_encoder_pll_mapping(struct intel_encoder *encoder)
{
struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
- u32 val = I915_READ(DPCLKA_CFGCR0_ICL);
+ u32 val;
enum port port = encoder->port;
- bool clk_enabled = !(val & icl_dpclka_cfgcr0_clk_off(dev_priv, port));
+ bool clk_enabled;
+
+ /*
+ * In case of DP MST, we sanitize the primary encoder only, not the
+ * virtual ones.
+ */
+ if (encoder->type == INTEL_OUTPUT_DP_MST)
+ return;
+
+ val = I915_READ(DPCLKA_CFGCR0_ICL);
+ clk_enabled = !(val & icl_dpclka_cfgcr0_clk_off(dev_priv, port));
+
+ if (!encoder->base.crtc && intel_encoder_is_dp(encoder)) {
+ u8 pipe_mask;
+ bool is_mst;
+
+ intel_ddi_get_encoder_pipes(encoder, &pipe_mask, &is_mst);
+ /*
+ * In the unlikely case that BIOS enables DP in MST mode, just
+ * warn since our MST HW readout is incomplete.
+ */
+ if (WARN_ON(is_mst))
+ return;
+ }
if (clk_enabled == !!encoder->base.crtc)
return;
--
2.13.2
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 12+ messages in thread
* ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-07 20:08 [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Imre Deak
2018-11-07 20:08 ` [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports Imre Deak
@ 2018-11-07 20:39 ` Patchwork
2018-11-07 21:23 ` Imre Deak
2018-11-09 12:40 ` Imre Deak
2018-11-09 0:09 ` ✓ Fi.CI.IGT: success " Patchwork
` (2 subsequent siblings)
4 siblings, 2 replies; 12+ messages in thread
From: Patchwork @ 2018-11-07 20:39 UTC (permalink / raw)
To: Imre Deak; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
URL : https://patchwork.freedesktop.org/series/52187/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_5100 -> Patchwork_10758 =
== Summary - FAILURE ==
Serious unknown changes coming with Patchwork_10758 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_10758, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/52187/revisions/1/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in Patchwork_10758:
=== IGT changes ===
==== Possible regressions ====
igt@gem_exec_nop@basic-parallel:
fi-icl-u: PASS -> INCOMPLETE
== Known issues ==
Here are the changes found in Patchwork_10758 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@gem_ctx_create@basic-files:
fi-bsw-kefka: PASS -> FAIL (fdo#108656)
igt@gem_mmap_gtt@basic-small-bo-tiledx:
fi-glk-dsi: PASS -> INCOMPLETE (fdo#103359, k.org#198133)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
fi-byt-clapper: PASS -> FAIL (fdo#103191, fdo#107362)
==== Possible fixes ====
igt@drv_getparams_basic@basic-subslice-total:
fi-cfl-8109u: DMESG-WARN (fdo#106107) -> PASS
igt@drv_selftest@live_coherency:
fi-gdg-551: DMESG-FAIL (fdo#107164) -> PASS
igt@gem_exec_suspend@basic-s3:
fi-cfl-8109u: INCOMPLETE (fdo#108126, fdo#107187) -> PASS
igt@kms_chamelium@common-hpd-after-suspend:
fi-skl-6700k2: WARN (fdo#108680) -> PASS
igt@kms_frontbuffer_tracking@basic:
fi-hsw-peppy: DMESG-WARN (fdo#102614) -> PASS
fi-byt-clapper: FAIL (fdo#103167) -> PASS
==== Warnings ====
igt@drv_selftest@live_contexts:
fi-icl-u2: DMESG-FAIL (fdo#108569) -> INCOMPLETE (fdo#108315)
fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#106107 https://bugs.freedesktop.org/show_bug.cgi?id=106107
fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
fdo#107187 https://bugs.freedesktop.org/show_bug.cgi?id=107187
fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
fdo#108126 https://bugs.freedesktop.org/show_bug.cgi?id=108126
fdo#108315 https://bugs.freedesktop.org/show_bug.cgi?id=108315
fdo#108569 https://bugs.freedesktop.org/show_bug.cgi?id=108569
fdo#108656 https://bugs.freedesktop.org/show_bug.cgi?id=108656
fdo#108680 https://bugs.freedesktop.org/show_bug.cgi?id=108680
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (54 -> 46) ==
Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-snb-2520m fi-ctg-p8600 fi-pnv-d510
== Build changes ==
* Linux: CI_DRM_5100 -> Patchwork_10758
CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
60ace73cb0f8 drm/i915/icl: Fix PLL mapping sanitization for DP ports
ee7b2ea5708d drm/i915/ddi: Add more sanity check to the encoder HW readout
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10758/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-07 20:39 ` ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Patchwork
@ 2018-11-07 21:23 ` Imre Deak
2018-11-09 12:40 ` Imre Deak
1 sibling, 0 replies; 12+ messages in thread
From: Imre Deak @ 2018-11-07 21:23 UTC (permalink / raw)
To: intel-gfx, Martin Peres
On Wed, Nov 07, 2018 at 08:39:24PM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
> URL : https://patchwork.freedesktop.org/series/52187/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from CI_DRM_5100 -> Patchwork_10758 =
>
> == Summary - FAILURE ==
>
> Serious unknown changes coming with Patchwork_10758 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_10758, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://patchwork.freedesktop.org/api/1.0/series/52187/revisions/1/mbox/
>
> == Possible new issues ==
>
> Here are the unknown changes that may have been introduced in Patchwork_10758:
>
> === IGT changes ===
>
> ==== Possible regressions ====
>
> igt@gem_exec_nop@basic-parallel:
> fi-icl-u: PASS -> INCOMPLETE
Can't see how this can be related, the encoders are ok during HW
readout, so the new PLL sanitize function won't do anything on them. I
found earlier instances of this issue (at least the same machine/test/error message):
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10556/fi-icl-u/igt@gem_exec_nop@basic-series.html
https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4696/fi-icl-u/igt@drv_selftest@live_requests.html
https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_3187/fi-icl-u/igt@gem_exec_nop@basic-parallel.html
https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10569/fi-icl-u/igt@gem_exec_nop@basic-series.html
https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_3185/fi-icl-u/igt@drv_selftest@live_requests.html
https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_5089/fi-icl-u/dmesg0.log
https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_3148/fi-icl-u/igt@drv_selftest@live_requests.html
https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4701/fi-icl-u/igt@drv_selftest@live_requests.html
>
>
> == Known issues ==
>
> Here are the changes found in Patchwork_10758 that come from known issues:
>
> === IGT changes ===
>
> ==== Issues hit ====
>
> igt@gem_ctx_create@basic-files:
> fi-bsw-kefka: PASS -> FAIL (fdo#108656)
>
> igt@gem_mmap_gtt@basic-small-bo-tiledx:
> fi-glk-dsi: PASS -> INCOMPLETE (fdo#103359, k.org#198133)
>
> igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
> fi-byt-clapper: PASS -> FAIL (fdo#103191, fdo#107362)
>
>
> ==== Possible fixes ====
>
> igt@drv_getparams_basic@basic-subslice-total:
> fi-cfl-8109u: DMESG-WARN (fdo#106107) -> PASS
>
> igt@drv_selftest@live_coherency:
> fi-gdg-551: DMESG-FAIL (fdo#107164) -> PASS
>
> igt@gem_exec_suspend@basic-s3:
> fi-cfl-8109u: INCOMPLETE (fdo#108126, fdo#107187) -> PASS
>
> igt@kms_chamelium@common-hpd-after-suspend:
> fi-skl-6700k2: WARN (fdo#108680) -> PASS
>
> igt@kms_frontbuffer_tracking@basic:
> fi-hsw-peppy: DMESG-WARN (fdo#102614) -> PASS
> fi-byt-clapper: FAIL (fdo#103167) -> PASS
>
>
> ==== Warnings ====
>
> igt@drv_selftest@live_contexts:
> fi-icl-u2: DMESG-FAIL (fdo#108569) -> INCOMPLETE (fdo#108315)
>
>
> fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
> fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
> fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
> fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
> fdo#106107 https://bugs.freedesktop.org/show_bug.cgi?id=106107
> fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
> fdo#107187 https://bugs.freedesktop.org/show_bug.cgi?id=107187
> fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
> fdo#108126 https://bugs.freedesktop.org/show_bug.cgi?id=108126
> fdo#108315 https://bugs.freedesktop.org/show_bug.cgi?id=108315
> fdo#108569 https://bugs.freedesktop.org/show_bug.cgi?id=108569
> fdo#108656 https://bugs.freedesktop.org/show_bug.cgi?id=108656
> fdo#108680 https://bugs.freedesktop.org/show_bug.cgi?id=108680
> k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
>
>
> == Participating hosts (54 -> 46) ==
>
> Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-snb-2520m fi-ctg-p8600 fi-pnv-d510
>
>
> == Build changes ==
>
> * Linux: CI_DRM_5100 -> Patchwork_10758
>
> CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> 60ace73cb0f8 drm/i915/icl: Fix PLL mapping sanitization for DP ports
> ee7b2ea5708d drm/i915/ddi: Add more sanity check to the encoder HW readout
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10758/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-07 20:08 [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Imre Deak
2018-11-07 20:08 ` [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports Imre Deak
2018-11-07 20:39 ` ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Patchwork
@ 2018-11-09 0:09 ` Patchwork
2018-11-09 13:44 ` Imre Deak
2018-11-09 13:25 ` ✓ Fi.CI.BAT: " Patchwork
2018-11-09 17:13 ` ✓ Fi.CI.IGT: " Patchwork
4 siblings, 1 reply; 12+ messages in thread
From: Patchwork @ 2018-11-09 0:09 UTC (permalink / raw)
To: Imre Deak; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
URL : https://patchwork.freedesktop.org/series/52187/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_5100_full -> Patchwork_10758_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork_10758_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_10758_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_10758_full:
=== IGT changes ===
==== Warnings ====
igt@pm_rc6_residency@rc6-accuracy:
shard-snb: PASS -> SKIP
igt@tools_test@tools_test:
shard-kbl: SKIP -> PASS
== Known issues ==
Here are the changes found in Patchwork_10758_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@gem_exec_suspend@basic-s3:
shard-kbl: PASS -> FAIL (fdo#103375)
igt@kms_color@pipe-b-legacy-gamma:
shard-apl: PASS -> FAIL (fdo#104782)
igt@kms_cursor_crc@cursor-128x128-random:
shard-glk: PASS -> FAIL (fdo#103232)
igt@kms_cursor_crc@cursor-128x128-suspend:
shard-apl: PASS -> FAIL (fdo#103232, fdo#103191) +1
igt@kms_cursor_crc@cursor-256x256-suspend:
shard-skl: PASS -> INCOMPLETE (fdo#104108)
igt@kms_cursor_legacy@cursora-vs-flipa-toggle:
shard-glk: PASS -> DMESG-WARN (fdo#105763, fdo#106538)
igt@kms_draw_crc@draw-method-rgb565-mmap-wc-untiled:
shard-skl: PASS -> FAIL (fdo#103184)
igt@kms_flip@flip-vs-expired-vblank:
shard-glk: PASS -> FAIL (fdo#105363, fdo#102887)
igt@kms_flip@flip-vs-expired-vblank-interruptible:
shard-glk: PASS -> FAIL (fdo#105363)
igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-wc:
shard-apl: PASS -> FAIL (fdo#103167)
igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-plflip-blt:
shard-glk: PASS -> FAIL (fdo#103167) +4
igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-shrfb-pgflip-blt:
shard-apl: SKIP -> INCOMPLETE (fdo#103927)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
shard-skl: PASS -> INCOMPLETE (fdo#104108, fdo#107773)
igt@pm_rpm@legacy-planes-dpms:
shard-skl: PASS -> INCOMPLETE (fdo#107807, fdo#105959)
igt@pm_rpm@modeset-lpsp:
shard-skl: PASS -> INCOMPLETE (fdo#107807) +1
==== Possible fixes ====
igt@kms_busy@extended-modeset-hang-newfb-render-b:
shard-kbl: DMESG-WARN (fdo#107956) -> PASS +1
igt@kms_cursor_crc@cursor-128x128-random:
shard-apl: FAIL (fdo#103232) -> PASS +2
igt@kms_cursor_crc@cursor-64x64-random:
shard-glk: FAIL (fdo#103232) -> PASS +1
igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size:
shard-glk: DMESG-WARN (fdo#105763, fdo#106538) -> PASS
igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff:
shard-glk: FAIL (fdo#103167) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
shard-glk: INCOMPLETE (k.org#198133, fdo#103359) -> PASS
igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
shard-glk: FAIL (fdo#103166) -> PASS
igt@kms_setmode@basic:
shard-kbl: FAIL (fdo#99912) -> PASS
igt@kms_universal_plane@universal-plane-pipe-b-functional:
shard-apl: FAIL (fdo#103166) -> PASS +2
igt@perf@polling:
shard-hsw: FAIL (fdo#102252) -> PASS
==== Warnings ====
igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu:
shard-snb: DMESG-WARN (fdo#107469) -> INCOMPLETE (fdo#105411)
fdo#102252 https://bugs.freedesktop.org/show_bug.cgi?id=102252
fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
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#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#103232 https://bugs.freedesktop.org/show_bug.cgi?id=103232
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
fdo#104108 https://bugs.freedesktop.org/show_bug.cgi?id=104108
fdo#104782 https://bugs.freedesktop.org/show_bug.cgi?id=104782
fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
fdo#105763 https://bugs.freedesktop.org/show_bug.cgi?id=105763
fdo#105959 https://bugs.freedesktop.org/show_bug.cgi?id=105959
fdo#106538 https://bugs.freedesktop.org/show_bug.cgi?id=106538
fdo#107469 https://bugs.freedesktop.org/show_bug.cgi?id=107469
fdo#107773 https://bugs.freedesktop.org/show_bug.cgi?id=107773
fdo#107807 https://bugs.freedesktop.org/show_bug.cgi?id=107807
fdo#107956 https://bugs.freedesktop.org/show_bug.cgi?id=107956
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 (6 -> 6) ==
No changes in participating hosts
== Build changes ==
* Linux: CI_DRM_5100 -> Patchwork_10758
CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ 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_10758/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports
2018-11-07 20:08 ` [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports Imre Deak
@ 2018-11-09 0:19 ` Clint Taylor
0 siblings, 0 replies; 12+ messages in thread
From: Clint Taylor @ 2018-11-09 0:19 UTC (permalink / raw)
To: Imre Deak, intel-gfx; +Cc: Paulo Zanoni, Rodrigo Vivi
On 11/07/2018 12:08 PM, Imre Deak wrote:
> We shouldn't consider an encoder inactive if it doesn't have a CRTC
> linked, but has virtual MST encoders with a crtc linked. Fix this.
>
> Also we should not sanitize the mapping for MST encoders, as it's always
> their primary encoder (which could be even in SST mode) whose active
> state determines if we need the clock being enabled for the
> corresponding physical port. Fix this too.
>
> This fixes at least an existing breakage where we incorrectly disabled
> the clock for an active DP encoder when sanitizing its MST virtual
> encoders. Not sure if there are BIOSes that enable an output in MST
> mode, but our HW readout is mostly missing for it anyway, so just warn
> for that case.
>
> Fixes: 70332ac539c5 ("drm/i915/icl+: Sanitize port to PLL mapping")
> Cc: Antonio Argenziano <antonio.argenziano@intel.com>
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Reported-by: Antonio Argenziano <antonio.argenziano@intel.com>
> Signed-off-by: Imre Deak <imre.deak@intel.com>
> ---
> drivers/gpu/drm/i915/intel_ddi.c | 27 +++++++++++++++++++++++++--
> 1 file changed, 25 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
> index abc51693eec9..4913bbdac843 100644
> --- a/drivers/gpu/drm/i915/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/intel_ddi.c
> @@ -2853,9 +2853,32 @@ void icl_unmap_plls_to_ports(struct drm_crtc *crtc,
> void icl_sanitize_encoder_pll_mapping(struct intel_encoder *encoder)
> {
> struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
> - u32 val = I915_READ(DPCLKA_CFGCR0_ICL);
> + u32 val;
> enum port port = encoder->port;
> - bool clk_enabled = !(val & icl_dpclka_cfgcr0_clk_off(dev_priv, port));
> + bool clk_enabled;
> +
> + /*
> + * In case of DP MST, we sanitize the primary encoder only, not the
> + * virtual ones.
> + */
> + if (encoder->type == INTEL_OUTPUT_DP_MST)
> + return;
> +
> + val = I915_READ(DPCLKA_CFGCR0_ICL);
> + clk_enabled = !(val & icl_dpclka_cfgcr0_clk_off(dev_priv, port));
> +
> + if (!encoder->base.crtc && intel_encoder_is_dp(encoder)) {
> + u8 pipe_mask;
> + bool is_mst;
> +
> + intel_ddi_get_encoder_pipes(encoder, &pipe_mask, &is_mst);
> + /*
> + * In the unlikely case that BIOS enables DP in MST mode, just
> + * warn since our MST HW readout is incomplete.
> + */
> + if (WARN_ON(is_mst))
> + return;
> + }
>
> if (clk_enabled == !!encoder->base.crtc)
> return;
Fixes the mDP lock up issue.
Reviewed-by: Clint Taylor <Clinton.A.Taylor@intel.com>
Tested-by: Clint Taylor <Clinton.A.Taylor@intel.com>
-Clint
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-07 20:39 ` ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Patchwork
2018-11-07 21:23 ` Imre Deak
@ 2018-11-09 12:40 ` Imre Deak
2018-11-09 13:20 ` Peres, Martin
1 sibling, 1 reply; 12+ messages in thread
From: Imre Deak @ 2018-11-09 12:40 UTC (permalink / raw)
To: intel-gfx; +Cc: Tomi P Sarvela, Martin Peres
On Wed, Nov 07, 2018 at 08:39:24PM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
> URL : https://patchwork.freedesktop.org/series/52187/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from CI_DRM_5100 -> Patchwork_10758 =
>
> == Summary - FAILURE ==
>
> Serious unknown changes coming with Patchwork_10758 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_10758, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://patchwork.freedesktop.org/api/1.0/series/52187/revisions/1/mbox/
>
> == Possible new issues ==
>
> Here are the unknown changes that may have been introduced in Patchwork_10758:
>
> === IGT changes ===
>
> ==== Possible regressions ====
>
> igt@gem_exec_nop@basic-parallel:
> fi-icl-u: PASS -> INCOMPLETE
https://bugs.freedesktop.org/show_bug.cgi?id=108564
>
>
> == Known issues ==
>
> Here are the changes found in Patchwork_10758 that come from known issues:
>
> === IGT changes ===
>
> ==== Issues hit ====
>
> igt@gem_ctx_create@basic-files:
> fi-bsw-kefka: PASS -> FAIL (fdo#108656)
>
> igt@gem_mmap_gtt@basic-small-bo-tiledx:
> fi-glk-dsi: PASS -> INCOMPLETE (fdo#103359, k.org#198133)
>
> igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
> fi-byt-clapper: PASS -> FAIL (fdo#103191, fdo#107362)
>
>
> ==== Possible fixes ====
>
> igt@drv_getparams_basic@basic-subslice-total:
> fi-cfl-8109u: DMESG-WARN (fdo#106107) -> PASS
>
> igt@drv_selftest@live_coherency:
> fi-gdg-551: DMESG-FAIL (fdo#107164) -> PASS
>
> igt@gem_exec_suspend@basic-s3:
> fi-cfl-8109u: INCOMPLETE (fdo#108126, fdo#107187) -> PASS
>
> igt@kms_chamelium@common-hpd-after-suspend:
> fi-skl-6700k2: WARN (fdo#108680) -> PASS
>
> igt@kms_frontbuffer_tracking@basic:
> fi-hsw-peppy: DMESG-WARN (fdo#102614) -> PASS
> fi-byt-clapper: FAIL (fdo#103167) -> PASS
>
>
> ==== Warnings ====
>
> igt@drv_selftest@live_contexts:
> fi-icl-u2: DMESG-FAIL (fdo#108569) -> INCOMPLETE (fdo#108315)
>
>
> fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
> fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
> fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
> fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
> fdo#106107 https://bugs.freedesktop.org/show_bug.cgi?id=106107
> fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
> fdo#107187 https://bugs.freedesktop.org/show_bug.cgi?id=107187
> fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
> fdo#108126 https://bugs.freedesktop.org/show_bug.cgi?id=108126
> fdo#108315 https://bugs.freedesktop.org/show_bug.cgi?id=108315
> fdo#108569 https://bugs.freedesktop.org/show_bug.cgi?id=108569
> fdo#108656 https://bugs.freedesktop.org/show_bug.cgi?id=108656
> fdo#108680 https://bugs.freedesktop.org/show_bug.cgi?id=108680
> k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
>
>
> == Participating hosts (54 -> 46) ==
>
> Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-snb-2520m fi-ctg-p8600 fi-pnv-d510
>
>
> == Build changes ==
>
> * Linux: CI_DRM_5100 -> Patchwork_10758
>
> CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> 60ace73cb0f8 drm/i915/icl: Fix PLL mapping sanitization for DP ports
> ee7b2ea5708d drm/i915/ddi: Add more sanity check to the encoder HW readout
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10758/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-09 12:40 ` Imre Deak
@ 2018-11-09 13:20 ` Peres, Martin
0 siblings, 0 replies; 12+ messages in thread
From: Peres, Martin @ 2018-11-09 13:20 UTC (permalink / raw)
To: Deak, Imre, intel-gfx; +Cc: Sarvela, Tomi P
On 09/11/2018 14:40, Deak, Imre wrote:
> On Wed, Nov 07, 2018 at 08:39:24PM +0000, Patchwork wrote:
>> == Series Details ==
>>
>> Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
>> URL : https://patchwork.freedesktop.org/series/52187/
>> State : failure
>>
>> == Summary ==
>>
>> = CI Bug Log - changes from CI_DRM_5100 -> Patchwork_10758 =
>>
>> == Summary - FAILURE ==
>>
>> Serious unknown changes coming with Patchwork_10758 absolutely need to be
>> verified manually.
>>
>> If you think the reported changes have nothing to do with the changes
>> introduced in Patchwork_10758, please notify your bug team to allow them
>> to document this new failure mode, which will reduce false positives in CI.
>>
>> External URL: https://patchwork.freedesktop.org/api/1.0/series/52187/revisions/1/mbox/
>>
>> == Possible new issues ==
>>
>> Here are the unknown changes that may have been introduced in Patchwork_10758:
>>
>> === IGT changes ===
>>
>> ==== Possible regressions ====
>>
>> igt@gem_exec_nop@basic-parallel:
>> fi-icl-u: PASS -> INCOMPLETE
>
> https://bugs.freedesktop.org/show_bug.cgi?id=108564
>
Filed and re-reporting queued. Thanks!
>>
>>
>> == Known issues ==
>>
>> Here are the changes found in Patchwork_10758 that come from known issues:
>>
>> === IGT changes ===
>>
>> ==== Issues hit ====
>>
>> igt@gem_ctx_create@basic-files:
>> fi-bsw-kefka: PASS -> FAIL (fdo#108656)
>>
>> igt@gem_mmap_gtt@basic-small-bo-tiledx:
>> fi-glk-dsi: PASS -> INCOMPLETE (fdo#103359, k.org#198133)
>>
>> igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
>> fi-byt-clapper: PASS -> FAIL (fdo#103191, fdo#107362)
>>
>>
>> ==== Possible fixes ====
>>
>> igt@drv_getparams_basic@basic-subslice-total:
>> fi-cfl-8109u: DMESG-WARN (fdo#106107) -> PASS
>>
>> igt@drv_selftest@live_coherency:
>> fi-gdg-551: DMESG-FAIL (fdo#107164) -> PASS
>>
>> igt@gem_exec_suspend@basic-s3:
>> fi-cfl-8109u: INCOMPLETE (fdo#108126, fdo#107187) -> PASS
>>
>> igt@kms_chamelium@common-hpd-after-suspend:
>> fi-skl-6700k2: WARN (fdo#108680) -> PASS
>>
>> igt@kms_frontbuffer_tracking@basic:
>> fi-hsw-peppy: DMESG-WARN (fdo#102614) -> PASS
>> fi-byt-clapper: FAIL (fdo#103167) -> PASS
>>
>>
>> ==== Warnings ====
>>
>> igt@drv_selftest@live_contexts:
>> fi-icl-u2: DMESG-FAIL (fdo#108569) -> INCOMPLETE (fdo#108315)
>>
>>
>> fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
>> fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
>> fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
>> fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
>> fdo#106107 https://bugs.freedesktop.org/show_bug.cgi?id=106107
>> fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
>> fdo#107187 https://bugs.freedesktop.org/show_bug.cgi?id=107187
>> fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
>> fdo#108126 https://bugs.freedesktop.org/show_bug.cgi?id=108126
>> fdo#108315 https://bugs.freedesktop.org/show_bug.cgi?id=108315
>> fdo#108569 https://bugs.freedesktop.org/show_bug.cgi?id=108569
>> fdo#108656 https://bugs.freedesktop.org/show_bug.cgi?id=108656
>> fdo#108680 https://bugs.freedesktop.org/show_bug.cgi?id=108680
>> k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
>>
>>
>> == Participating hosts (54 -> 46) ==
>>
>> Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-snb-2520m fi-ctg-p8600 fi-pnv-d510
>>
>>
>> == Build changes ==
>>
>> * Linux: CI_DRM_5100 -> Patchwork_10758
>>
>> CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
>> IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
>> Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ git://anongit.freedesktop.org/gfx-ci/linux
>>
>>
>> == Linux commits ==
>>
>> 60ace73cb0f8 drm/i915/icl: Fix PLL mapping sanitization for DP ports
>> ee7b2ea5708d drm/i915/ddi: Add more sanity check to the encoder HW readout
>>
>> == Logs ==
>>
>> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10758/issues.html
>
---------------------------------------------------------------------
Intel Finland Oy
Registered Address: PL 281, 00181 Helsinki
Business Identity Code: 0357606 - 4
Domiciled in Helsinki
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-07 20:08 [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Imre Deak
` (2 preceding siblings ...)
2018-11-09 0:09 ` ✓ Fi.CI.IGT: success " Patchwork
@ 2018-11-09 13:25 ` Patchwork
2018-11-09 17:13 ` ✓ Fi.CI.IGT: " Patchwork
4 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2018-11-09 13:25 UTC (permalink / raw)
To: Imre Deak; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
URL : https://patchwork.freedesktop.org/series/52187/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_5100 -> Patchwork_10758 =
== Summary - SUCCESS ==
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/52187/revisions/1/mbox/
== Known issues ==
Here are the changes found in Patchwork_10758 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@gem_ctx_create@basic-files:
fi-bsw-kefka: PASS -> FAIL (fdo#108656)
igt@gem_exec_nop@basic-parallel:
fi-icl-u: PASS -> INCOMPLETE (fdo#108315)
igt@gem_mmap_gtt@basic-small-bo-tiledx:
fi-glk-dsi: PASS -> INCOMPLETE (fdo#103359, k.org#198133)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
fi-byt-clapper: PASS -> FAIL (fdo#107362, fdo#103191)
==== Possible fixes ====
igt@drv_getparams_basic@basic-subslice-total:
fi-cfl-8109u: DMESG-WARN (fdo#106107) -> PASS
igt@drv_selftest@live_coherency:
fi-gdg-551: DMESG-FAIL (fdo#107164) -> PASS
igt@gem_exec_suspend@basic-s3:
fi-cfl-8109u: INCOMPLETE (fdo#107187, fdo#108126) -> PASS
igt@kms_chamelium@common-hpd-after-suspend:
fi-skl-6700k2: WARN (fdo#108680) -> PASS
igt@kms_frontbuffer_tracking@basic:
fi-hsw-peppy: DMESG-WARN (fdo#102614) -> PASS
fi-byt-clapper: FAIL (fdo#103167) -> PASS
==== Warnings ====
igt@drv_selftest@live_contexts:
fi-icl-u2: DMESG-FAIL (fdo#108569) -> INCOMPLETE (fdo#108315)
fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#106107 https://bugs.freedesktop.org/show_bug.cgi?id=106107
fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
fdo#107187 https://bugs.freedesktop.org/show_bug.cgi?id=107187
fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
fdo#108126 https://bugs.freedesktop.org/show_bug.cgi?id=108126
fdo#108315 https://bugs.freedesktop.org/show_bug.cgi?id=108315
fdo#108569 https://bugs.freedesktop.org/show_bug.cgi?id=108569
fdo#108656 https://bugs.freedesktop.org/show_bug.cgi?id=108656
fdo#108680 https://bugs.freedesktop.org/show_bug.cgi?id=108680
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (54 -> 46) ==
Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-snb-2520m fi-ctg-p8600 fi-pnv-d510
== Build changes ==
* Linux: CI_DRM_5100 -> Patchwork_10758
CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
60ace73cb0f8 drm/i915/icl: Fix PLL mapping sanitization for DP ports
ee7b2ea5708d drm/i915/ddi: Add more sanity check to the encoder HW readout
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10758/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-09 0:09 ` ✓ Fi.CI.IGT: success " Patchwork
@ 2018-11-09 13:44 ` Imre Deak
2018-11-09 14:00 ` Imre Deak
0 siblings, 1 reply; 12+ messages in thread
From: Imre Deak @ 2018-11-09 13:44 UTC (permalink / raw)
To: intel-gfx; +Cc: Tomi P Sarvela, Martin Peres
On Fri, Nov 09, 2018 at 12:09:40AM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
> URL : https://patchwork.freedesktop.org/series/52187/
> State : success
>
> == Summary ==
>
> = CI Bug Log - changes from CI_DRM_5100_full -> Patchwork_10758_full =
>
> == Summary - WARNING ==
>
> Minor unknown changes coming with Patchwork_10758_full need to be verified
> manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_10758_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_10758_full:
>
> === IGT changes ===
>
> ==== Warnings ====
>
> igt@pm_rc6_residency@rc6-accuracy:
> shard-snb: PASS -> SKIP
Independent platform, looks like
https://bugs.freedesktop.org/show_bug.cgi?id=108664
will add there a note about this skip.
>
> igt@tools_test@tools_test:
> shard-kbl: SKIP -> PASS
>
>
> == Known issues ==
>
> Here are the changes found in Patchwork_10758_full that come from known issues:
>
> === IGT changes ===
>
> ==== Issues hit ====
>
> igt@gem_exec_suspend@basic-s3:
> shard-kbl: PASS -> FAIL (fdo#103375)
>
> igt@kms_color@pipe-b-legacy-gamma:
> shard-apl: PASS -> FAIL (fdo#104782)
>
> igt@kms_cursor_crc@cursor-128x128-random:
> shard-glk: PASS -> FAIL (fdo#103232)
>
> igt@kms_cursor_crc@cursor-128x128-suspend:
> shard-apl: PASS -> FAIL (fdo#103232, fdo#103191) +1
>
> igt@kms_cursor_crc@cursor-256x256-suspend:
> shard-skl: PASS -> INCOMPLETE (fdo#104108)
>
> igt@kms_cursor_legacy@cursora-vs-flipa-toggle:
> shard-glk: PASS -> DMESG-WARN (fdo#105763, fdo#106538)
>
> igt@kms_draw_crc@draw-method-rgb565-mmap-wc-untiled:
> shard-skl: PASS -> FAIL (fdo#103184)
>
> igt@kms_flip@flip-vs-expired-vblank:
> shard-glk: PASS -> FAIL (fdo#105363, fdo#102887)
>
> igt@kms_flip@flip-vs-expired-vblank-interruptible:
> shard-glk: PASS -> FAIL (fdo#105363)
>
> igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-wc:
> shard-apl: PASS -> FAIL (fdo#103167)
>
> igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-plflip-blt:
> shard-glk: PASS -> FAIL (fdo#103167) +4
>
> igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-shrfb-pgflip-blt:
> shard-apl: SKIP -> INCOMPLETE (fdo#103927)
>
> igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> shard-skl: PASS -> INCOMPLETE (fdo#104108, fdo#107773)
>
> igt@pm_rpm@legacy-planes-dpms:
> shard-skl: PASS -> INCOMPLETE (fdo#107807, fdo#105959)
>
> igt@pm_rpm@modeset-lpsp:
> shard-skl: PASS -> INCOMPLETE (fdo#107807) +1
>
>
> ==== Possible fixes ====
>
> igt@kms_busy@extended-modeset-hang-newfb-render-b:
> shard-kbl: DMESG-WARN (fdo#107956) -> PASS +1
>
> igt@kms_cursor_crc@cursor-128x128-random:
> shard-apl: FAIL (fdo#103232) -> PASS +2
>
> igt@kms_cursor_crc@cursor-64x64-random:
> shard-glk: FAIL (fdo#103232) -> PASS +1
>
> igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size:
> shard-glk: DMESG-WARN (fdo#105763, fdo#106538) -> PASS
>
> igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff:
> shard-glk: FAIL (fdo#103167) -> PASS
>
> igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> shard-glk: INCOMPLETE (k.org#198133, fdo#103359) -> PASS
>
> igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
> shard-glk: FAIL (fdo#103166) -> PASS
>
> igt@kms_setmode@basic:
> shard-kbl: FAIL (fdo#99912) -> PASS
>
> igt@kms_universal_plane@universal-plane-pipe-b-functional:
> shard-apl: FAIL (fdo#103166) -> PASS +2
>
> igt@perf@polling:
> shard-hsw: FAIL (fdo#102252) -> PASS
>
>
> ==== Warnings ====
>
> igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu:
> shard-snb: DMESG-WARN (fdo#107469) -> INCOMPLETE (fdo#105411)
>
>
> fdo#102252 https://bugs.freedesktop.org/show_bug.cgi?id=102252
> fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
> 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#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
> fdo#103232 https://bugs.freedesktop.org/show_bug.cgi?id=103232
> fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
> fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
> fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
> fdo#104108 https://bugs.freedesktop.org/show_bug.cgi?id=104108
> fdo#104782 https://bugs.freedesktop.org/show_bug.cgi?id=104782
> fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
> fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
> fdo#105763 https://bugs.freedesktop.org/show_bug.cgi?id=105763
> fdo#105959 https://bugs.freedesktop.org/show_bug.cgi?id=105959
> fdo#106538 https://bugs.freedesktop.org/show_bug.cgi?id=106538
> fdo#107469 https://bugs.freedesktop.org/show_bug.cgi?id=107469
> fdo#107773 https://bugs.freedesktop.org/show_bug.cgi?id=107773
> fdo#107807 https://bugs.freedesktop.org/show_bug.cgi?id=107807
> fdo#107956 https://bugs.freedesktop.org/show_bug.cgi?id=107956
> 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 (6 -> 6) ==
>
> No changes in participating hosts
>
>
> == Build changes ==
>
> * Linux: CI_DRM_5100 -> Patchwork_10758
>
> CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ 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_10758/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-09 13:44 ` Imre Deak
@ 2018-11-09 14:00 ` Imre Deak
0 siblings, 0 replies; 12+ messages in thread
From: Imre Deak @ 2018-11-09 14:00 UTC (permalink / raw)
To: intel-gfx, Antonio Argenziano, Clint A Taylor,
Ville Syrjälä,
Martin Peres
On Fri, Nov 09, 2018 at 03:44:47PM +0200, Imre Deak wrote:
> On Fri, Nov 09, 2018 at 12:09:40AM +0000, Patchwork wrote:
> > == Series Details ==
> >
> > Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
> > URL : https://patchwork.freedesktop.org/series/52187/
> > State : success
Pushed to -dinq, thanks for the report, review and testing.
> >
> > == Summary ==
> >
> > = CI Bug Log - changes from CI_DRM_5100_full -> Patchwork_10758_full =
> >
> > == Summary - WARNING ==
> >
> > Minor unknown changes coming with Patchwork_10758_full need to be verified
> > manually.
> >
> > If you think the reported changes have nothing to do with the changes
> > introduced in Patchwork_10758_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_10758_full:
> >
> > === IGT changes ===
> >
> > ==== Warnings ====
> >
> > igt@pm_rc6_residency@rc6-accuracy:
> > shard-snb: PASS -> SKIP
>
> Independent platform, looks like
> https://bugs.freedesktop.org/show_bug.cgi?id=108664
>
> will add there a note about this skip.
>
>
> >
> > igt@tools_test@tools_test:
> > shard-kbl: SKIP -> PASS
> >
> >
> > == Known issues ==
> >
> > Here are the changes found in Patchwork_10758_full that come from known issues:
> >
> > === IGT changes ===
> >
> > ==== Issues hit ====
> >
> > igt@gem_exec_suspend@basic-s3:
> > shard-kbl: PASS -> FAIL (fdo#103375)
> >
> > igt@kms_color@pipe-b-legacy-gamma:
> > shard-apl: PASS -> FAIL (fdo#104782)
> >
> > igt@kms_cursor_crc@cursor-128x128-random:
> > shard-glk: PASS -> FAIL (fdo#103232)
> >
> > igt@kms_cursor_crc@cursor-128x128-suspend:
> > shard-apl: PASS -> FAIL (fdo#103232, fdo#103191) +1
> >
> > igt@kms_cursor_crc@cursor-256x256-suspend:
> > shard-skl: PASS -> INCOMPLETE (fdo#104108)
> >
> > igt@kms_cursor_legacy@cursora-vs-flipa-toggle:
> > shard-glk: PASS -> DMESG-WARN (fdo#105763, fdo#106538)
> >
> > igt@kms_draw_crc@draw-method-rgb565-mmap-wc-untiled:
> > shard-skl: PASS -> FAIL (fdo#103184)
> >
> > igt@kms_flip@flip-vs-expired-vblank:
> > shard-glk: PASS -> FAIL (fdo#105363, fdo#102887)
> >
> > igt@kms_flip@flip-vs-expired-vblank-interruptible:
> > shard-glk: PASS -> FAIL (fdo#105363)
> >
> > igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-wc:
> > shard-apl: PASS -> FAIL (fdo#103167)
> >
> > igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-plflip-blt:
> > shard-glk: PASS -> FAIL (fdo#103167) +4
> >
> > igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-shrfb-pgflip-blt:
> > shard-apl: SKIP -> INCOMPLETE (fdo#103927)
> >
> > igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> > shard-skl: PASS -> INCOMPLETE (fdo#104108, fdo#107773)
> >
> > igt@pm_rpm@legacy-planes-dpms:
> > shard-skl: PASS -> INCOMPLETE (fdo#107807, fdo#105959)
> >
> > igt@pm_rpm@modeset-lpsp:
> > shard-skl: PASS -> INCOMPLETE (fdo#107807) +1
> >
> >
> > ==== Possible fixes ====
> >
> > igt@kms_busy@extended-modeset-hang-newfb-render-b:
> > shard-kbl: DMESG-WARN (fdo#107956) -> PASS +1
> >
> > igt@kms_cursor_crc@cursor-128x128-random:
> > shard-apl: FAIL (fdo#103232) -> PASS +2
> >
> > igt@kms_cursor_crc@cursor-64x64-random:
> > shard-glk: FAIL (fdo#103232) -> PASS +1
> >
> > igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size:
> > shard-glk: DMESG-WARN (fdo#105763, fdo#106538) -> PASS
> >
> > igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff:
> > shard-glk: FAIL (fdo#103167) -> PASS
> >
> > igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> > shard-glk: INCOMPLETE (k.org#198133, fdo#103359) -> PASS
> >
> > igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
> > shard-glk: FAIL (fdo#103166) -> PASS
> >
> > igt@kms_setmode@basic:
> > shard-kbl: FAIL (fdo#99912) -> PASS
> >
> > igt@kms_universal_plane@universal-plane-pipe-b-functional:
> > shard-apl: FAIL (fdo#103166) -> PASS +2
> >
> > igt@perf@polling:
> > shard-hsw: FAIL (fdo#102252) -> PASS
> >
> >
> > ==== Warnings ====
> >
> > igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu:
> > shard-snb: DMESG-WARN (fdo#107469) -> INCOMPLETE (fdo#105411)
> >
> >
> > fdo#102252 https://bugs.freedesktop.org/show_bug.cgi?id=102252
> > fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
> > 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#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
> > fdo#103232 https://bugs.freedesktop.org/show_bug.cgi?id=103232
> > fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
> > fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
> > fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
> > fdo#104108 https://bugs.freedesktop.org/show_bug.cgi?id=104108
> > fdo#104782 https://bugs.freedesktop.org/show_bug.cgi?id=104782
> > fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
> > fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
> > fdo#105763 https://bugs.freedesktop.org/show_bug.cgi?id=105763
> > fdo#105959 https://bugs.freedesktop.org/show_bug.cgi?id=105959
> > fdo#106538 https://bugs.freedesktop.org/show_bug.cgi?id=106538
> > fdo#107469 https://bugs.freedesktop.org/show_bug.cgi?id=107469
> > fdo#107773 https://bugs.freedesktop.org/show_bug.cgi?id=107773
> > fdo#107807 https://bugs.freedesktop.org/show_bug.cgi?id=107807
> > fdo#107956 https://bugs.freedesktop.org/show_bug.cgi?id=107956
> > 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 (6 -> 6) ==
> >
> > No changes in participating hosts
> >
> >
> > == Build changes ==
> >
> > * Linux: CI_DRM_5100 -> Patchwork_10758
> >
> > CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
> > IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> > Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ 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_10758/shards.html
> _______________________________________________
> 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] 12+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
2018-11-07 20:08 [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Imre Deak
` (3 preceding siblings ...)
2018-11-09 13:25 ` ✓ Fi.CI.BAT: " Patchwork
@ 2018-11-09 17:13 ` Patchwork
4 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2018-11-09 17:13 UTC (permalink / raw)
To: Imre Deak; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout
URL : https://patchwork.freedesktop.org/series/52187/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_5100_full -> Patchwork_10758_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork_10758_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_10758_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_10758_full:
=== IGT changes ===
==== Warnings ====
igt@gem_exec_capture@capture-bsd2:
shard-snb: SKIP -> ( 2 SKIP ) +1156
igt@gem_exec_reloc@basic-cpu-gtt-noreloc:
shard-skl: PASS -> ( 2 PASS ) +792
igt@gem_pwrite@small-cpu-fbr:
shard-kbl: PASS -> ( 2 PASS ) +1783
igt@gem_userptr_blits@map-fixed-invalidate-overlap:
shard-glk: PASS -> ( 2 PASS ) +1601
igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-f:
shard-hsw: SKIP -> ( 2 SKIP ) +983
igt@kms_flip@bo-too-big-interruptible:
shard-snb: PASS -> ( 2 PASS ) +1183
igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-render:
shard-glk: SKIP -> ( 2 SKIP ) +639
igt@kms_frontbuffer_tracking@psr-rgb101010-draw-mmap-wc:
shard-kbl: SKIP -> ( 2 SKIP ) +683
igt@kms_rotation_crc@sprite-rotation-90-pos-100-0:
shard-apl: PASS -> ( 2 PASS ) +1732
igt@perf_pmu@rc6:
shard-kbl: SKIP -> ( 1 PASS, 1 SKIP )
igt@perf_pmu@semaphore-wait-vcs1:
shard-skl: SKIP -> ( 2 SKIP ) +302
igt@pm_rc6_residency@rc6-accuracy:
shard-snb: PASS -> ( 2 SKIP )
igt@prime_vgem@sync-bsd1:
shard-apl: SKIP -> ( 2 SKIP ) +816
igt@syncobj_basic@bad-create-flags:
shard-hsw: PASS -> ( 2 PASS ) +1797
igt@tools_test@tools_test:
shard-kbl: SKIP -> PASS
== Known issues ==
Here are the changes found in Patchwork_10758_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@gem_exec_suspend@basic-s3:
shard-kbl: PASS -> FAIL (fdo#103375)
igt@kms_atomic@plane_cursor_legacy:
shard-glk: PASS -> ( 1 INCOMPLETE, 1 PASS ) (fdo#103359, k.org#198133)
igt@kms_color@pipe-a-degamma:
shard-apl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#108145, fdo#104782)
igt@kms_color@pipe-b-legacy-gamma:
shard-apl: PASS -> ( 2 FAIL ) (fdo#104782)
igt@kms_cursor_crc@cursor-128x128-offscreen:
shard-skl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103232) +1
igt@kms_cursor_crc@cursor-128x128-random:
shard-glk: PASS -> ( 2 FAIL ) (fdo#103232)
igt@kms_cursor_crc@cursor-128x128-suspend:
shard-apl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103191, fdo#103232)
igt@kms_cursor_crc@cursor-128x42-onscreen:
shard-apl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103232) +1
igt@kms_cursor_crc@cursor-256x256-suspend:
shard-skl: PASS -> ( 1 INCOMPLETE, 1 PASS ) (fdo#104108)
shard-apl: PASS -> ( 2 FAIL ) (fdo#103191, fdo#103232)
igt@kms_cursor_legacy@cursora-vs-flipa-toggle:
shard-glk: PASS -> ( 2 DMESG-WARN ) (fdo#105763, fdo#106538)
igt@kms_draw_crc@draw-method-rgb565-mmap-wc-untiled:
shard-skl: PASS -> ( 2 FAIL ) (fdo#103184)
igt@kms_flip@flip-vs-expired-vblank:
shard-glk: PASS -> ( 1 FAIL, 1 PASS ) (fdo#105363, fdo#102887)
igt@kms_flip@flip-vs-expired-vblank-interruptible:
shard-glk: PASS -> ( 1 FAIL, 1 PASS ) (fdo#105363)
igt@kms_flip_tiling@flip-to-y-tiled:
shard-skl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#107931) +1
igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-wc:
shard-apl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103167) +4
igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-mmap-wc:
shard-glk: PASS -> ( 2 FAIL ) (fdo#103167) +2
igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-plflip-blt:
shard-glk: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103167) +3
igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-shrfb-pgflip-blt:
shard-apl: SKIP -> ( 1 INCOMPLETE, 1 SKIP ) (fdo#103927)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
shard-skl: PASS -> ( 1 INCOMPLETE, 1 PASS ) (fdo#104108, fdo#107773)
igt@kms_plane@pixel-format-pipe-a-planes:
shard-apl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103166) +2
igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
shard-skl: PASS -> ( 1 FAIL, 1 PASS ) (fdo#107815)
igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
shard-skl: PASS -> FAIL (fdo#107815)
igt@kms_plane_multiple@atomic-pipe-b-tiling-x:
shard-glk: PASS -> ( 1 FAIL, 1 PASS ) (fdo#103166) +1
igt@perf_pmu@frequency:
shard-snb: NOTRUN -> INCOMPLETE (fdo#105411)
igt@pm_rpm@legacy-planes-dpms:
shard-skl: PASS -> INCOMPLETE (fdo#107807, fdo#105959)
igt@pm_rpm@modeset-lpsp:
shard-skl: PASS -> ( 1 INCOMPLETE, 1 PASS ) (fdo#107807) +2
==== Possible fixes ====
igt@drm_import_export@import-close-race-flink:
shard-skl: TIMEOUT (fdo#108667) -> PASS
igt@gem_pwrite_pread@snooped-copy-correctness:
shard-snb: INCOMPLETE (fdo#105411) -> SKIP
igt@kms_cursor_crc@cursor-256x256-suspend:
shard-glk: FAIL (fdo#103232) -> PASS
igt@kms_cursor_crc@cursor-64x64-random:
shard-glk: FAIL (fdo#103232) -> ( 2 PASS )
shard-apl: FAIL (fdo#103232) -> ( 2 PASS )
igt@kms_cursor_crc@cursor-64x64-suspend:
shard-skl: INCOMPLETE (fdo#104108) -> PASS
igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size:
shard-glk: DMESG-WARN (fdo#105763, fdo#106538) -> ( 2 PASS )
igt@kms_flip@flip-vs-expired-vblank-interruptible:
shard-skl: FAIL (fdo#105363) -> PASS
igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff:
shard-glk: FAIL (fdo#103167) -> ( 2 PASS )
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
shard-glk: INCOMPLETE (fdo#103359, k.org#198133) -> ( 2 PASS )
igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
shard-glk: FAIL (fdo#103166) -> ( 2 PASS )
igt@kms_setmode@basic:
shard-kbl: FAIL (fdo#99912) -> ( 2 PASS )
igt@kms_universal_plane@universal-plane-pipe-b-functional:
shard-apl: FAIL (fdo#103166) -> ( 2 PASS ) +1
igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend:
shard-skl: INCOMPLETE (fdo#104108, fdo#107773) -> PASS +1
==== Warnings ====
igt@gem_exec_reuse@baggage:
shard-apl: DMESG-WARN (fdo#108691) -> ( 1 DMESG-WARN, 1 PASS ) (fdo#108690, fdo#108691)
igt@gem_exec_schedule@pi-ringfull-bsd:
shard-glk: FAIL (fdo#103158) -> ( 2 FAIL ) (fdo#103158) +1
igt@gem_exec_schedule@pi-ringfull-vebox:
shard-kbl: FAIL (fdo#103158) -> ( 2 FAIL ) (fdo#103158) +4
shard-apl: FAIL (fdo#103158) -> ( 2 FAIL ) (fdo#103158) +3
igt@gem_ppgtt@blt-vs-render-ctxn:
shard-skl: TIMEOUT (fdo#108039) -> ( 2 TIMEOUT ) (fdo#108039)
igt@gem_userptr_blits@readonly-unsync:
shard-skl: INCOMPLETE (fdo#108074) -> ( 2 INCOMPLETE ) (fdo#108074)
igt@kms_available_modes_crc@available_mode_test_crc:
shard-hsw: FAIL (fdo#106641) -> ( 2 FAIL ) (fdo#106641)
shard-kbl: FAIL (fdo#106641) -> ( 2 FAIL ) (fdo#106641)
shard-snb: FAIL (fdo#106641) -> ( 2 FAIL ) (fdo#106641)
igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-a:
shard-hsw: DMESG-WARN (fdo#107956) -> ( 2 DMESG-WARN ) (fdo#107956) +8
shard-kbl: DMESG-WARN (fdo#107956) -> ( 2 DMESG-WARN ) (fdo#107956) +5
igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
shard-skl: DMESG-WARN (fdo#107956) -> ( 2 DMESG-WARN ) (fdo#107956) +2
igt@kms_busy@extended-pageflip-hang-newfb-render-b:
shard-apl: DMESG-WARN (fdo#107956) -> ( 2 DMESG-WARN ) (fdo#107956) +10
igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
shard-snb: DMESG-WARN (fdo#107956) -> ( 2 DMESG-WARN ) (fdo#107956) +5
igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-c:
shard-glk: DMESG-WARN (fdo#107956) -> ( 2 DMESG-WARN ) (fdo#107956) +9
shard-kbl: DMESG-WARN (fdo#107956) -> ( 1 DMESG-WARN, 1 PASS ) (fdo#107956) +1
igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
shard-apl: FAIL (fdo#106510, fdo#105458) -> ( 2 FAIL ) (fdo#106510, fdo#105458) +1
igt@kms_chv_cursor_fail@pipe-a-64x64-right-edge:
shard-skl: FAIL (fdo#104671) -> ( 2 FAIL ) (fdo#104671)
igt@kms_content_protection@atomic:
shard-kbl: DMESG-FAIL (fdo#108550) -> ( 2 DMESG-FAIL ) (fdo#108550) +1
shard-apl: FAIL (fdo#108597) -> ( 2 FAIL ) (fdo#108597) +1
igt@kms_cursor_crc@cursor-128x128-sliding:
shard-glk: FAIL (fdo#103232) -> ( 2 FAIL ) (fdo#103232) +1
igt@kms_cursor_crc@cursor-128x42-sliding:
shard-apl: FAIL (fdo#103232) -> ( 1 FAIL, 1 PASS ) (fdo#103232) +1
igt@kms_cursor_crc@cursor-256x85-onscreen:
shard-apl: FAIL (fdo#103232) -> ( 2 FAIL ) (fdo#103232) +5
igt@kms_cursor_crc@cursor-64x64-dpms:
shard-kbl: FAIL (fdo#103232) -> ( 2 FAIL ) (fdo#103232) +2
shard-skl: FAIL (fdo#103232) -> ( 2 FAIL ) (fdo#103232) +4
igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic:
shard-glk: FAIL (fdo#106509, fdo#105454) -> ( 2 FAIL ) (fdo#106509, fdo#105454) +1
igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move:
shard-apl: FAIL (fdo#103167) -> ( 1 FAIL, 1 PASS ) (fdo#103167)
shard-glk: FAIL (fdo#103167) -> ( 1 FAIL, 1 PASS ) (fdo#103167) +3
igt@kms_frontbuffer_tracking@fbc-farfromfence:
shard-skl: FAIL (fdo#103167) -> ( 2 FAIL ) (fdo#105682, fdo#103167) +1
igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-cpu:
shard-snb: DMESG-WARN (fdo#107469) -> ( 1 DMESG-WARN, 1 INCOMPLETE ) (fdo#107469, fdo#105411)
igt@kms_frontbuffer_tracking@fbc-stridechange:
shard-hsw: FAIL (fdo#105682) -> ( 2 FAIL ) (fdo#105682)
igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-wc:
shard-skl: FAIL (fdo#103167) -> ( 2 FAIL ) (fdo#103167) +1
igt@kms_panel_fitting@legacy:
shard-skl: FAIL (fdo#105456) -> ( 2 FAIL ) (fdo#105456)
igt@kms_plane@plane-position-covered-pipe-b-planes:
shard-glk: FAIL (fdo#103166) -> ( 1 FAIL, 1 PASS ) (fdo#103166) +1
shard-apl: FAIL (fdo#103166) -> ( 1 FAIL, 1 PASS ) (fdo#103166)
igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
shard-skl: FAIL (fdo#108145, fdo#107815) -> ( 2 FAIL ) (fdo#108145, fdo#107815) +3
igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
shard-apl: FAIL (fdo#108145) -> ( 2 FAIL ) (fdo#108145) +14
igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
shard-skl: FAIL (fdo#108145) -> ( 2 FAIL ) (fdo#108145) +5
igt@kms_plane_alpha_blend@pipe-b-alpha-transparant-fb:
shard-kbl: FAIL (fdo#108145) -> ( 2 FAIL ) (fdo#108145) +10
igt@kms_plane_alpha_blend@pipe-c-alpha-7efc:
shard-kbl: FAIL (fdo#108145, fdo#108590) -> ( 2 FAIL ) (fdo#108145, fdo#108590) +1
igt@kms_plane_alpha_blend@pipe-c-alpha-transparant-fb:
shard-glk: FAIL (fdo#108145) -> ( 2 FAIL ) (fdo#108145) +10
igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
shard-apl: FAIL (fdo#103166) -> ( 2 FAIL ) (fdo#103166)
igt@kms_plane_multiple@atomic-pipe-b-tiling-y:
shard-kbl: FAIL (fdo#103166) -> ( 2 FAIL ) (fdo#103166)
shard-skl: FAIL (fdo#103166, fdo#107815) -> ( 2 FAIL ) (fdo#103166, fdo#107815)
igt@kms_properties@connector-properties-atomic:
shard-apl: FAIL (fdo#108642) -> ( 2 FAIL ) (fdo#108642)
shard-glk: FAIL (fdo#108642) -> ( 2 FAIL ) (fdo#108642)
shard-hsw: FAIL (fdo#108642) -> ( 2 FAIL ) (fdo#108642)
shard-kbl: FAIL (fdo#108642) -> ( 2 FAIL ) (fdo#108642)
igt@kms_rotation_crc@primary-rotation-180:
shard-snb: FAIL (fdo#103925) -> ( 2 FAIL ) (fdo#103925)
igt@kms_setmode@basic:
shard-glk: FAIL (fdo#99912) -> ( 2 FAIL ) (fdo#99912)
shard-hsw: FAIL (fdo#99912) -> ( 1 FAIL, 1 PASS ) (fdo#99912)
shard-snb: FAIL (fdo#99912) -> ( 2 FAIL ) (fdo#99912)
igt@kms_sysfs_edid_timing:
shard-apl: FAIL (fdo#100047) -> ( 2 FAIL ) (fdo#100047)
shard-kbl: FAIL (fdo#100047) -> ( 2 FAIL ) (fdo#100047)
igt@perf@polling:
shard-hsw: FAIL (fdo#102252) -> ( 1 FAIL, 1 PASS ) (fdo#102252)
fdo#100047 https://bugs.freedesktop.org/show_bug.cgi?id=100047
fdo#102252 https://bugs.freedesktop.org/show_bug.cgi?id=102252
fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
fdo#103158 https://bugs.freedesktop.org/show_bug.cgi?id=103158
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#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#103232 https://bugs.freedesktop.org/show_bug.cgi?id=103232
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
fdo#104108 https://bugs.freedesktop.org/show_bug.cgi?id=104108
fdo#104671 https://bugs.freedesktop.org/show_bug.cgi?id=104671
fdo#104782 https://bugs.freedesktop.org/show_bug.cgi?id=104782
fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
fdo#105454 https://bugs.freedesktop.org/show_bug.cgi?id=105454
fdo#105456 https://bugs.freedesktop.org/show_bug.cgi?id=105456
fdo#105458 https://bugs.freedesktop.org/show_bug.cgi?id=105458
fdo#105682 https://bugs.freedesktop.org/show_bug.cgi?id=105682
fdo#105763 https://bugs.freedesktop.org/show_bug.cgi?id=105763
fdo#105959 https://bugs.freedesktop.org/show_bug.cgi?id=105959
fdo#106509 https://bugs.freedesktop.org/show_bug.cgi?id=106509
fdo#106510 https://bugs.freedesktop.org/show_bug.cgi?id=106510
fdo#106538 https://bugs.freedesktop.org/show_bug.cgi?id=106538
fdo#106641 https://bugs.freedesktop.org/show_bug.cgi?id=106641
fdo#107469 https://bugs.freedesktop.org/show_bug.cgi?id=107469
fdo#107773 https://bugs.freedesktop.org/show_bug.cgi?id=107773
fdo#107807 https://bugs.freedesktop.org/show_bug.cgi?id=107807
fdo#107815 https://bugs.freedesktop.org/show_bug.cgi?id=107815
fdo#107931 https://bugs.freedesktop.org/show_bug.cgi?id=107931
fdo#107956 https://bugs.freedesktop.org/show_bug.cgi?id=107956
fdo#108039 https://bugs.freedesktop.org/show_bug.cgi?id=108039
fdo#108074 https://bugs.freedesktop.org/show_bug.cgi?id=108074
fdo#108145 https://bugs.freedesktop.org/show_bug.cgi?id=108145
fdo#108550 https://bugs.freedesktop.org/show_bug.cgi?id=108550
fdo#108590 https://bugs.freedesktop.org/show_bug.cgi?id=108590
fdo#108597 https://bugs.freedesktop.org/show_bug.cgi?id=108597
fdo#108642 https://bugs.freedesktop.org/show_bug.cgi?id=108642
fdo#108667 https://bugs.freedesktop.org/show_bug.cgi?id=108667
fdo#108690 https://bugs.freedesktop.org/show_bug.cgi?id=108690
fdo#108691 https://bugs.freedesktop.org/show_bug.cgi?id=108691
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 (6 -> 6) ==
No changes in participating hosts
== Build changes ==
* Linux: CI_DRM_5100 -> Patchwork_10758
CI_DRM_5100: c5f9a3064d90c86dab99d5aef011b87f6f8921dc @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4712: a3ede1b535ac8137f6949c468edd7054453d5dae @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_10758: 60ace73cb0f8e814e6a6e34d6c9775d5fcf077a8 @ 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_10758/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2018-11-09 17:13 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-07 20:08 [PATCH 1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Imre Deak
2018-11-07 20:08 ` [PATCH 2/2] drm/i915/icl: Fix PLL mapping sanitization for DP ports Imre Deak
2018-11-09 0:19 ` Clint Taylor
2018-11-07 20:39 ` ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/ddi: Add more sanity check to the encoder HW readout Patchwork
2018-11-07 21:23 ` Imre Deak
2018-11-09 12:40 ` Imre Deak
2018-11-09 13:20 ` Peres, Martin
2018-11-09 0:09 ` ✓ Fi.CI.IGT: success " Patchwork
2018-11-09 13:44 ` Imre Deak
2018-11-09 14:00 ` Imre Deak
2018-11-09 13:25 ` ✓ Fi.CI.BAT: " Patchwork
2018-11-09 17:13 ` ✓ Fi.CI.IGT: " Patchwork
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.