* [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 21:07 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:07 UTC (permalink / raw)
To: intel-gfx; +Cc: Lucas De Marchi
The South Display is part of the PCH so we should technically be basing
our port detection logic off the PCH in use rather than the platform
generation.
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 3123958e2081..ddf5bad1b969 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -5487,7 +5487,7 @@ static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
}
-static bool icl_digital_port_connected(struct intel_encoder *encoder)
+static bool icp_digital_port_connected(struct intel_encoder *encoder)
{
struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
struct intel_digital_port *dig_port = enc_to_dig_port(&encoder->base);
@@ -5525,9 +5525,9 @@ static bool __intel_digital_port_connected(struct intel_encoder *encoder)
return g4x_digital_port_connected(encoder);
}
- if (INTEL_GEN(dev_priv) >= 11)
- return icl_digital_port_connected(encoder);
- else if (IS_GEN(dev_priv, 10) || IS_GEN9_BC(dev_priv))
+ if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+ return icp_digital_port_connected(encoder);
+ else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
return spt_digital_port_connected(encoder);
else if (IS_GEN9_LP(dev_priv))
return bxt_digital_port_connected(encoder);
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [Intel-gfx] [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 21:07 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:07 UTC (permalink / raw)
To: intel-gfx; +Cc: Lucas De Marchi
The South Display is part of the PCH so we should technically be basing
our port detection logic off the PCH in use rather than the platform
generation.
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 3123958e2081..ddf5bad1b969 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -5487,7 +5487,7 @@ static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
}
-static bool icl_digital_port_connected(struct intel_encoder *encoder)
+static bool icp_digital_port_connected(struct intel_encoder *encoder)
{
struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
struct intel_digital_port *dig_port = enc_to_dig_port(&encoder->base);
@@ -5525,9 +5525,9 @@ static bool __intel_digital_port_connected(struct intel_encoder *encoder)
return g4x_digital_port_connected(encoder);
}
- if (INTEL_GEN(dev_priv) >= 11)
- return icl_digital_port_connected(encoder);
- else if (IS_GEN(dev_priv, 10) || IS_GEN9_BC(dev_priv))
+ if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+ return icp_digital_port_connected(encoder);
+ else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
return spt_digital_port_connected(encoder);
else if (IS_GEN9_LP(dev_priv))
return bxt_digital_port_connected(encoder);
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
@ 2019-11-26 21:07 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:07 UTC (permalink / raw)
To: intel-gfx; +Cc: Lucas De Marchi
When looking at SDEISR to determine the connection status of combo
outputs, we should use the phy index rather than the port index.
Although they're usually the same thing, EHL's DDI-D (port D) is
attached to PHY-A and SDEISR doesn't even have bits for a "D" output.
It's also possible that future platforms may map DDIs (the internal
display engine programming units) to PHYs (the output handling on the IO
side) in ways where port!=phy, so let's look at the PHY index by
default.
Fixes: 719d24002602 ("drm/i915/ehl: Enable DDI-D")
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index ddf5bad1b969..59c5fd7bf27d 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -5477,14 +5477,12 @@ static bool bxt_digital_port_connected(struct intel_encoder *encoder)
}
static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
- struct intel_digital_port *intel_dig_port)
+ enum phy phy)
{
- enum port port = intel_dig_port->base.port;
-
- if (HAS_PCH_MCC(dev_priv) && port == PORT_C)
+ if (HAS_PCH_MCC(dev_priv) && phy == PHY_C)
return I915_READ(SDEISR) & SDE_TC_HOTPLUG_ICP(PORT_TC1);
- return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
+ return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(phy);
}
static bool icp_digital_port_connected(struct intel_encoder *encoder)
@@ -5494,7 +5492,7 @@ static bool icp_digital_port_connected(struct intel_encoder *encoder)
enum phy phy = intel_port_to_phy(dev_priv, encoder->port);
if (intel_phy_is_combo(dev_priv, phy))
- return icl_combo_port_connected(dev_priv, dig_port);
+ return icl_combo_port_connected(dev_priv, phy);
else if (intel_phy_is_tc(dev_priv, phy))
return intel_tc_port_connected(dig_port);
else
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [Intel-gfx] [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
@ 2019-11-26 21:07 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:07 UTC (permalink / raw)
To: intel-gfx; +Cc: Lucas De Marchi
When looking at SDEISR to determine the connection status of combo
outputs, we should use the phy index rather than the port index.
Although they're usually the same thing, EHL's DDI-D (port D) is
attached to PHY-A and SDEISR doesn't even have bits for a "D" output.
It's also possible that future platforms may map DDIs (the internal
display engine programming units) to PHYs (the output handling on the IO
side) in ways where port!=phy, so let's look at the PHY index by
default.
Fixes: 719d24002602 ("drm/i915/ehl: Enable DDI-D")
Cc: José Roberto de Souza <jose.souza@intel.com>
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index ddf5bad1b969..59c5fd7bf27d 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -5477,14 +5477,12 @@ static bool bxt_digital_port_connected(struct intel_encoder *encoder)
}
static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
- struct intel_digital_port *intel_dig_port)
+ enum phy phy)
{
- enum port port = intel_dig_port->base.port;
-
- if (HAS_PCH_MCC(dev_priv) && port == PORT_C)
+ if (HAS_PCH_MCC(dev_priv) && phy == PHY_C)
return I915_READ(SDEISR) & SDE_TC_HOTPLUG_ICP(PORT_TC1);
- return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
+ return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(phy);
}
static bool icp_digital_port_connected(struct intel_encoder *encoder)
@@ -5494,7 +5492,7 @@ static bool icp_digital_port_connected(struct intel_encoder *encoder)
enum phy phy = intel_port_to_phy(dev_priv, encoder->port);
if (intel_phy_is_combo(dev_priv, phy))
- return icl_combo_port_connected(dev_priv, dig_port);
+ return icl_combo_port_connected(dev_priv, phy);
else if (intel_phy_is_tc(dev_priv, phy))
return intel_tc_port_connected(dig_port);
else
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:07 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:07 UTC (permalink / raw)
To: intel-gfx
The bspec tells us 'Program SHPD_FILTER_CNT with the "500 microseconds
adjusted" value before enabling hotplug detection' on CNP+. We haven't
been touching this register at all thus far, but we should probably
follow the bspec's guidance.
The register also exists on LPT and SPT, but there isn't any specific
guidance I can find on how we should be programming it there so let's
leave it be for now.
Bspec: 4342
Bspec: 31297
Bspec: 8407
Bspec: 49305
Bspec: 50473
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
drivers/gpu/drm/i915/i915_irq.c | 5 +++++
drivers/gpu/drm/i915/i915_reg.h | 4 ++++
2 files changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index dae00f7dd7df..028cb6239c12 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct drm_i915_private *dev_priv,
hotplug_irqs = sde_ddi_mask | sde_tc_mask;
enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
+ I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
+
ibx_display_interrupt_update(dev_priv, hotplug_irqs, enabled_irqs);
icp_hpd_detection_setup(dev_priv, ddi_enable_mask, tc_enable_mask);
@@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct drm_i915_private *dev_priv)
{
u32 hotplug_irqs, enabled_irqs;
+ if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
+ I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
+
hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 94d0f593eeb7..74cf45de162e 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -8110,6 +8110,10 @@ enum {
#define SHOTPLUG_CTL_TC _MMIO(0xc4034)
#define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port) * 4)
+
+#define SHPD_FILTER_CNT _MMIO(0xc4038)
+#define SHPD_FILTER_CNT_500_ADJ 0x001D9
+
/* Icelake DSC Rate Control Range Parameter Registers */
#define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
#define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 + 4)
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [Intel-gfx] [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:07 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:07 UTC (permalink / raw)
To: intel-gfx
The bspec tells us 'Program SHPD_FILTER_CNT with the "500 microseconds
adjusted" value before enabling hotplug detection' on CNP+. We haven't
been touching this register at all thus far, but we should probably
follow the bspec's guidance.
The register also exists on LPT and SPT, but there isn't any specific
guidance I can find on how we should be programming it there so let's
leave it be for now.
Bspec: 4342
Bspec: 31297
Bspec: 8407
Bspec: 49305
Bspec: 50473
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
drivers/gpu/drm/i915/i915_irq.c | 5 +++++
drivers/gpu/drm/i915/i915_reg.h | 4 ++++
2 files changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index dae00f7dd7df..028cb6239c12 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct drm_i915_private *dev_priv,
hotplug_irqs = sde_ddi_mask | sde_tc_mask;
enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
+ I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
+
ibx_display_interrupt_update(dev_priv, hotplug_irqs, enabled_irqs);
icp_hpd_detection_setup(dev_priv, ddi_enable_mask, tc_enable_mask);
@@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct drm_i915_private *dev_priv)
{
u32 hotplug_irqs, enabled_irqs;
+ if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
+ I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
+
hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 94d0f593eeb7..74cf45de162e 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -8110,6 +8110,10 @@ enum {
#define SHOTPLUG_CTL_TC _MMIO(0xc4034)
#define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port) * 4)
+
+#define SHPD_FILTER_CNT _MMIO(0xc4038)
+#define SHPD_FILTER_CNT_500_ADJ 0x001D9
+
/* Icelake DSC Rate Control Range Parameter Registers */
#define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
#define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 + 4)
--
2.23.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 32+ messages in thread
* Re: [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 21:29 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:29 UTC (permalink / raw)
To: Roper, Matthew D, intel-gfx; +Cc: De Marchi, Lucas
On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> The South Display is part of the PCH so we should technically be
> basing
> our port detection logic off the PCH in use rather than the platform
> generation.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 3123958e2081..ddf5bad1b969 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -5487,7 +5487,7 @@ static bool icl_combo_port_connected(struct
> drm_i915_private *dev_priv,
> return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
> }
>
> -static bool icl_digital_port_connected(struct intel_encoder
> *encoder)
> +static bool icp_digital_port_connected(struct intel_encoder
> *encoder)
> {
> struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
> struct intel_digital_port *dig_port = enc_to_dig_port(&encoder-
> >base);
> @@ -5525,9 +5525,9 @@ static bool
> __intel_digital_port_connected(struct intel_encoder *encoder)
> return g4x_digital_port_connected(encoder);
> }
>
> - if (INTEL_GEN(dev_priv) >= 11)
> - return icl_digital_port_connected(encoder);
> - else if (IS_GEN(dev_priv, 10) || IS_GEN9_BC(dev_priv))
> + if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
> + return icp_digital_port_connected(encoder);
> + else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
> return spt_digital_port_connected(encoder);
> else if (IS_GEN9_LP(dev_priv))
> return bxt_digital_port_connected(encoder);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 21:29 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:29 UTC (permalink / raw)
To: Roper, Matthew D, intel-gfx; +Cc: De Marchi, Lucas
On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> The South Display is part of the PCH so we should technically be
> basing
> our port detection logic off the PCH in use rather than the platform
> generation.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index 3123958e2081..ddf5bad1b969 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -5487,7 +5487,7 @@ static bool icl_combo_port_connected(struct
> drm_i915_private *dev_priv,
> return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
> }
>
> -static bool icl_digital_port_connected(struct intel_encoder
> *encoder)
> +static bool icp_digital_port_connected(struct intel_encoder
> *encoder)
> {
> struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
> struct intel_digital_port *dig_port = enc_to_dig_port(&encoder-
> >base);
> @@ -5525,9 +5525,9 @@ static bool
> __intel_digital_port_connected(struct intel_encoder *encoder)
> return g4x_digital_port_connected(encoder);
> }
>
> - if (INTEL_GEN(dev_priv) >= 11)
> - return icl_digital_port_connected(encoder);
> - else if (IS_GEN(dev_priv, 10) || IS_GEN9_BC(dev_priv))
> + if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
> + return icp_digital_port_connected(encoder);
> + else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
> return spt_digital_port_connected(encoder);
> else if (IS_GEN9_LP(dev_priv))
> return bxt_digital_port_connected(encoder);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
@ 2019-11-26 21:34 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:34 UTC (permalink / raw)
To: Roper, Matthew D, intel-gfx; +Cc: De Marchi, Lucas
On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> When looking at SDEISR to determine the connection status of combo
> outputs, we should use the phy index rather than the port index.
> Although they're usually the same thing, EHL's DDI-D (port D) is
> attached to PHY-A and SDEISR doesn't even have bits for a "D" output.
> It's also possible that future platforms may map DDIs (the internal
> display engine programming units) to PHYs (the output handling on the
> IO
> side) in ways where port!=phy, so let's look at the PHY index by
> default.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Fixes: 719d24002602 ("drm/i915/ehl: Enable DDI-D")
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index ddf5bad1b969..59c5fd7bf27d 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -5477,14 +5477,12 @@ static bool bxt_digital_port_connected(struct
> intel_encoder *encoder)
> }
>
> static bool icl_combo_port_connected(struct drm_i915_private
> *dev_priv,
> - struct intel_digital_port
> *intel_dig_port)
> + enum phy phy)
> {
> - enum port port = intel_dig_port->base.port;
> -
> - if (HAS_PCH_MCC(dev_priv) && port == PORT_C)
> + if (HAS_PCH_MCC(dev_priv) && phy == PHY_C)
> return I915_READ(SDEISR) &
> SDE_TC_HOTPLUG_ICP(PORT_TC1);
>
> - return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
> + return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(phy);
> }
>
> static bool icp_digital_port_connected(struct intel_encoder
> *encoder)
> @@ -5494,7 +5492,7 @@ static bool icp_digital_port_connected(struct
> intel_encoder *encoder)
> enum phy phy = intel_port_to_phy(dev_priv, encoder->port);
>
> if (intel_phy_is_combo(dev_priv, phy))
> - return icl_combo_port_connected(dev_priv, dig_port);
> + return icl_combo_port_connected(dev_priv, phy);
> else if (intel_phy_is_tc(dev_priv, phy))
> return intel_tc_port_connected(dig_port);
> else
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
@ 2019-11-26 21:34 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:34 UTC (permalink / raw)
To: Roper, Matthew D, intel-gfx; +Cc: De Marchi, Lucas
On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> When looking at SDEISR to determine the connection status of combo
> outputs, we should use the phy index rather than the port index.
> Although they're usually the same thing, EHL's DDI-D (port D) is
> attached to PHY-A and SDEISR doesn't even have bits for a "D" output.
> It's also possible that future platforms may map DDIs (the internal
> display engine programming units) to PHYs (the output handling on the
> IO
> side) in ways where port!=phy, so let's look at the PHY index by
> default.
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Fixes: 719d24002602 ("drm/i915/ehl: Enable DDI-D")
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index ddf5bad1b969..59c5fd7bf27d 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -5477,14 +5477,12 @@ static bool bxt_digital_port_connected(struct
> intel_encoder *encoder)
> }
>
> static bool icl_combo_port_connected(struct drm_i915_private
> *dev_priv,
> - struct intel_digital_port
> *intel_dig_port)
> + enum phy phy)
> {
> - enum port port = intel_dig_port->base.port;
> -
> - if (HAS_PCH_MCC(dev_priv) && port == PORT_C)
> + if (HAS_PCH_MCC(dev_priv) && phy == PHY_C)
> return I915_READ(SDEISR) &
> SDE_TC_HOTPLUG_ICP(PORT_TC1);
>
> - return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
> + return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(phy);
> }
>
> static bool icp_digital_port_connected(struct intel_encoder
> *encoder)
> @@ -5494,7 +5492,7 @@ static bool icp_digital_port_connected(struct
> intel_encoder *encoder)
> enum phy phy = intel_port_to_phy(dev_priv, encoder->port);
>
> if (intel_phy_is_combo(dev_priv, phy))
> - return icl_combo_port_connected(dev_priv, dig_port);
> + return icl_combo_port_connected(dev_priv, phy);
> else if (intel_phy_is_tc(dev_priv, phy))
> return intel_tc_port_connected(dig_port);
> else
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:41 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:41 UTC (permalink / raw)
To: Roper, Matthew D, intel-gfx
On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> The bspec tells us 'Program SHPD_FILTER_CNT with the "500
> microseconds
> adjusted" value before enabling hotplug detection' on CNP+. We
> haven't
> been touching this register at all thus far, but we should probably
> follow the bspec's guidance.
>
> The register also exists on LPT and SPT, but there isn't any specific
> guidance I can find on how we should be programming it there so let's
> leave it be for now.
>
> Bspec: 4342
> Bspec: 31297
> Bspec: 8407
> Bspec: 49305
> Bspec: 50473
>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
> drivers/gpu/drm/i915/i915_irq.c | 5 +++++
> drivers/gpu/drm/i915/i915_reg.h | 4 ++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c
> b/drivers/gpu/drm/i915/i915_irq.c
> index dae00f7dd7df..028cb6239c12 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct
> drm_i915_private *dev_priv,
> hotplug_irqs = sde_ddi_mask | sde_tc_mask;
> enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
>
> + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> +
> ibx_display_interrupt_update(dev_priv, hotplug_irqs,
> enabled_irqs);
>
> icp_hpd_detection_setup(dev_priv, ddi_enable_mask,
> tc_enable_mask);
> @@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct
> drm_i915_private *dev_priv)
> {
> u32 hotplug_irqs, enabled_irqs;
>
> + if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
> + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> +
> hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
> enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
>
> diff --git a/drivers/gpu/drm/i915/i915_reg.h
> b/drivers/gpu/drm/i915/i915_reg.h
> index 94d0f593eeb7..74cf45de162e 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -8110,6 +8110,10 @@ enum {
>
> #define SHOTPLUG_CTL_TC _MMIO(0xc4034)
> #define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port) * 4)
> +
> +#define SHPD_FILTER_CNT _MMIO(0xc4038)
> +#define SHPD_FILTER_CNT_500_ADJ 0x001D9
Shouldn't it be 0x1F2? Or I'm missing something?
Also this is the default value.
> +
> /* Icelake DSC Rate Control Range Parameter Registers */
> #define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
> #define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 +
> 4)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:41 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:41 UTC (permalink / raw)
To: Roper, Matthew D, intel-gfx
On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> The bspec tells us 'Program SHPD_FILTER_CNT with the "500
> microseconds
> adjusted" value before enabling hotplug detection' on CNP+. We
> haven't
> been touching this register at all thus far, but we should probably
> follow the bspec's guidance.
>
> The register also exists on LPT and SPT, but there isn't any specific
> guidance I can find on how we should be programming it there so let's
> leave it be for now.
>
> Bspec: 4342
> Bspec: 31297
> Bspec: 8407
> Bspec: 49305
> Bspec: 50473
>
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
> drivers/gpu/drm/i915/i915_irq.c | 5 +++++
> drivers/gpu/drm/i915/i915_reg.h | 4 ++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c
> b/drivers/gpu/drm/i915/i915_irq.c
> index dae00f7dd7df..028cb6239c12 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct
> drm_i915_private *dev_priv,
> hotplug_irqs = sde_ddi_mask | sde_tc_mask;
> enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
>
> + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> +
> ibx_display_interrupt_update(dev_priv, hotplug_irqs,
> enabled_irqs);
>
> icp_hpd_detection_setup(dev_priv, ddi_enable_mask,
> tc_enable_mask);
> @@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct
> drm_i915_private *dev_priv)
> {
> u32 hotplug_irqs, enabled_irqs;
>
> + if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
> + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> +
> hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
> enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
>
> diff --git a/drivers/gpu/drm/i915/i915_reg.h
> b/drivers/gpu/drm/i915/i915_reg.h
> index 94d0f593eeb7..74cf45de162e 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -8110,6 +8110,10 @@ enum {
>
> #define SHOTPLUG_CTL_TC _MMIO(0xc4034)
> #define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port) * 4)
> +
> +#define SHPD_FILTER_CNT _MMIO(0xc4038)
> +#define SHPD_FILTER_CNT_500_ADJ 0x001D9
Shouldn't it be 0x1F2? Or I'm missing something?
Also this is the default value.
> +
> /* Icelake DSC Rate Control Range Parameter Registers */
> #define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
> #define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 +
> 4)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:50 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:50 UTC (permalink / raw)
To: Souza, Jose; +Cc: intel-gfx
On Tue, Nov 26, 2019 at 01:41:15PM -0800, Souza, Jose wrote:
> On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> > The bspec tells us 'Program SHPD_FILTER_CNT with the "500
> > microseconds
> > adjusted" value before enabling hotplug detection' on CNP+. We
> > haven't
> > been touching this register at all thus far, but we should probably
> > follow the bspec's guidance.
> >
> > The register also exists on LPT and SPT, but there isn't any specific
> > guidance I can find on how we should be programming it there so let's
> > leave it be for now.
> >
> > Bspec: 4342
> > Bspec: 31297
> > Bspec: 8407
> > Bspec: 49305
> > Bspec: 50473
> >
> > Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> > ---
> > drivers/gpu/drm/i915/i915_irq.c | 5 +++++
> > drivers/gpu/drm/i915/i915_reg.h | 4 ++++
> > 2 files changed, 9 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_irq.c
> > b/drivers/gpu/drm/i915/i915_irq.c
> > index dae00f7dd7df..028cb6239c12 100644
> > --- a/drivers/gpu/drm/i915/i915_irq.c
> > +++ b/drivers/gpu/drm/i915/i915_irq.c
> > @@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct
> > drm_i915_private *dev_priv,
> > hotplug_irqs = sde_ddi_mask | sde_tc_mask;
> > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
> >
> > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > +
> > ibx_display_interrupt_update(dev_priv, hotplug_irqs,
> > enabled_irqs);
> >
> > icp_hpd_detection_setup(dev_priv, ddi_enable_mask,
> > tc_enable_mask);
> > @@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct
> > drm_i915_private *dev_priv)
> > {
> > u32 hotplug_irqs, enabled_irqs;
> >
> > + if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
> > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > +
> > hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
> > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
> >
> > diff --git a/drivers/gpu/drm/i915/i915_reg.h
> > b/drivers/gpu/drm/i915/i915_reg.h
> > index 94d0f593eeb7..74cf45de162e 100644
> > --- a/drivers/gpu/drm/i915/i915_reg.h
> > +++ b/drivers/gpu/drm/i915/i915_reg.h
> > @@ -8110,6 +8110,10 @@ enum {
> >
> > #define SHOTPLUG_CTL_TC _MMIO(0xc4034)
> > #define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port) * 4)
> > +
> > +#define SHPD_FILTER_CNT _MMIO(0xc4038)
> > +#define SHPD_FILTER_CNT_500_ADJ 0x001D9
>
> Shouldn't it be 0x1F2? Or I'm missing something?
> Also this is the default value.
>
0x1F2 is the "500 microseconds" option, whereas 0x1D9 is the "500
microseconds adjusted" option according to bspec 8407/50473. The
programming instructions tell us to use the non-default "adjusted"
variant.
I'm not sure why they don't just call it "475 microseconds" since that's
what this value really works out to...
Matt
>
> > +
> > /* Icelake DSC Rate Control Range Parameter Registers */
> > #define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
> > #define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 +
> > 4)
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:50 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 21:50 UTC (permalink / raw)
To: Souza, Jose; +Cc: intel-gfx
On Tue, Nov 26, 2019 at 01:41:15PM -0800, Souza, Jose wrote:
> On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> > The bspec tells us 'Program SHPD_FILTER_CNT with the "500
> > microseconds
> > adjusted" value before enabling hotplug detection' on CNP+. We
> > haven't
> > been touching this register at all thus far, but we should probably
> > follow the bspec's guidance.
> >
> > The register also exists on LPT and SPT, but there isn't any specific
> > guidance I can find on how we should be programming it there so let's
> > leave it be for now.
> >
> > Bspec: 4342
> > Bspec: 31297
> > Bspec: 8407
> > Bspec: 49305
> > Bspec: 50473
> >
> > Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> > ---
> > drivers/gpu/drm/i915/i915_irq.c | 5 +++++
> > drivers/gpu/drm/i915/i915_reg.h | 4 ++++
> > 2 files changed, 9 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_irq.c
> > b/drivers/gpu/drm/i915/i915_irq.c
> > index dae00f7dd7df..028cb6239c12 100644
> > --- a/drivers/gpu/drm/i915/i915_irq.c
> > +++ b/drivers/gpu/drm/i915/i915_irq.c
> > @@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct
> > drm_i915_private *dev_priv,
> > hotplug_irqs = sde_ddi_mask | sde_tc_mask;
> > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
> >
> > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > +
> > ibx_display_interrupt_update(dev_priv, hotplug_irqs,
> > enabled_irqs);
> >
> > icp_hpd_detection_setup(dev_priv, ddi_enable_mask,
> > tc_enable_mask);
> > @@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct
> > drm_i915_private *dev_priv)
> > {
> > u32 hotplug_irqs, enabled_irqs;
> >
> > + if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
> > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > +
> > hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
> > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
> >
> > diff --git a/drivers/gpu/drm/i915/i915_reg.h
> > b/drivers/gpu/drm/i915/i915_reg.h
> > index 94d0f593eeb7..74cf45de162e 100644
> > --- a/drivers/gpu/drm/i915/i915_reg.h
> > +++ b/drivers/gpu/drm/i915/i915_reg.h
> > @@ -8110,6 +8110,10 @@ enum {
> >
> > #define SHOTPLUG_CTL_TC _MMIO(0xc4034)
> > #define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port) * 4)
> > +
> > +#define SHPD_FILTER_CNT _MMIO(0xc4038)
> > +#define SHPD_FILTER_CNT_500_ADJ 0x001D9
>
> Shouldn't it be 0x1F2? Or I'm missing something?
> Also this is the default value.
>
0x1F2 is the "500 microseconds" option, whereas 0x1D9 is the "500
microseconds adjusted" option according to bspec 8407/50473. The
programming instructions tell us to use the non-default "adjusted"
variant.
I'm not sure why they don't just call it "475 microseconds" since that's
what this value really works out to...
Matt
>
> > +
> > /* Icelake DSC Rate Control Range Parameter Registers */
> > #define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
> > #define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 +
> > 4)
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:57 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:57 UTC (permalink / raw)
To: Roper, Matthew D; +Cc: intel-gfx
On Tue, 2019-11-26 at 13:50 -0800, Matt Roper wrote:
> On Tue, Nov 26, 2019 at 01:41:15PM -0800, Souza, Jose wrote:
> > On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> > > The bspec tells us 'Program SHPD_FILTER_CNT with the "500
> > > microseconds
> > > adjusted" value before enabling hotplug detection' on CNP+. We
> > > haven't
> > > been touching this register at all thus far, but we should
> > > probably
> > > follow the bspec's guidance.
> > >
> > > The register also exists on LPT and SPT, but there isn't any
> > > specific
> > > guidance I can find on how we should be programming it there so
> > > let's
> > > leave it be for now.
> > >
> > > Bspec: 4342
> > > Bspec: 31297
> > > Bspec: 8407
> > > Bspec: 49305
> > > Bspec: 50473
> > >
> > > Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> > > ---
> > > drivers/gpu/drm/i915/i915_irq.c | 5 +++++
> > > drivers/gpu/drm/i915/i915_reg.h | 4 ++++
> > > 2 files changed, 9 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/i915_irq.c
> > > b/drivers/gpu/drm/i915/i915_irq.c
> > > index dae00f7dd7df..028cb6239c12 100644
> > > --- a/drivers/gpu/drm/i915/i915_irq.c
> > > +++ b/drivers/gpu/drm/i915/i915_irq.c
> > > @@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct
> > > drm_i915_private *dev_priv,
> > > hotplug_irqs = sde_ddi_mask | sde_tc_mask;
> > > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
> > >
> > > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > > +
> > > ibx_display_interrupt_update(dev_priv, hotplug_irqs,
> > > enabled_irqs);
> > >
> > > icp_hpd_detection_setup(dev_priv, ddi_enable_mask,
> > > tc_enable_mask);
> > > @@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct
> > > drm_i915_private *dev_priv)
> > > {
> > > u32 hotplug_irqs, enabled_irqs;
> > >
> > > + if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
> > > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > > +
> > > hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
> > > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
> > >
> > > diff --git a/drivers/gpu/drm/i915/i915_reg.h
> > > b/drivers/gpu/drm/i915/i915_reg.h
> > > index 94d0f593eeb7..74cf45de162e 100644
> > > --- a/drivers/gpu/drm/i915/i915_reg.h
> > > +++ b/drivers/gpu/drm/i915/i915_reg.h
> > > @@ -8110,6 +8110,10 @@ enum {
> > >
> > > #define SHOTPLUG_CTL_TC _MMIO(0xc4034)
> > > #define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port)
> > > * 4)
> > > +
> > > +#define SHPD_FILTER_CNT _MMIO(0xc4038)
> > > +#define SHPD_FILTER_CNT_500_ADJ 0x001D9
> >
> > Shouldn't it be 0x1F2? Or I'm missing something?
> > Also this is the default value.
> >
>
> 0x1F2 is the "500 microseconds" option, whereas 0x1D9 is the "500
> microseconds adjusted" option according to bspec 8407/50473. The
> programming instructions tell us to use the non-default "adjusted"
> variant.
>
> I'm not sure why they don't just call it "475 microseconds" since
> that's
> what this value really works out to...
Oooh
"Adjusted" smells like a workaround value
Anyways:
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
>
> Matt
>
> > > +
> > > /* Icelake DSC Rate Control Range Parameter Registers */
> > > #define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
> > > #define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 +
> > > 4)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+
@ 2019-11-26 21:57 ` Souza, Jose
0 siblings, 0 replies; 32+ messages in thread
From: Souza, Jose @ 2019-11-26 21:57 UTC (permalink / raw)
To: Roper, Matthew D; +Cc: intel-gfx
On Tue, 2019-11-26 at 13:50 -0800, Matt Roper wrote:
> On Tue, Nov 26, 2019 at 01:41:15PM -0800, Souza, Jose wrote:
> > On Tue, 2019-11-26 at 13:07 -0800, Matt Roper wrote:
> > > The bspec tells us 'Program SHPD_FILTER_CNT with the "500
> > > microseconds
> > > adjusted" value before enabling hotplug detection' on CNP+. We
> > > haven't
> > > been touching this register at all thus far, but we should
> > > probably
> > > follow the bspec's guidance.
> > >
> > > The register also exists on LPT and SPT, but there isn't any
> > > specific
> > > guidance I can find on how we should be programming it there so
> > > let's
> > > leave it be for now.
> > >
> > > Bspec: 4342
> > > Bspec: 31297
> > > Bspec: 8407
> > > Bspec: 49305
> > > Bspec: 50473
> > >
> > > Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> > > ---
> > > drivers/gpu/drm/i915/i915_irq.c | 5 +++++
> > > drivers/gpu/drm/i915/i915_reg.h | 4 ++++
> > > 2 files changed, 9 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/i915_irq.c
> > > b/drivers/gpu/drm/i915/i915_irq.c
> > > index dae00f7dd7df..028cb6239c12 100644
> > > --- a/drivers/gpu/drm/i915/i915_irq.c
> > > +++ b/drivers/gpu/drm/i915/i915_irq.c
> > > @@ -2976,6 +2976,8 @@ static void icp_hpd_irq_setup(struct
> > > drm_i915_private *dev_priv,
> > > hotplug_irqs = sde_ddi_mask | sde_tc_mask;
> > > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, pins);
> > >
> > > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > > +
> > > ibx_display_interrupt_update(dev_priv, hotplug_irqs,
> > > enabled_irqs);
> > >
> > > icp_hpd_detection_setup(dev_priv, ddi_enable_mask,
> > > tc_enable_mask);
> > > @@ -3081,6 +3083,9 @@ static void spt_hpd_irq_setup(struct
> > > drm_i915_private *dev_priv)
> > > {
> > > u32 hotplug_irqs, enabled_irqs;
> > >
> > > + if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
> > > + I915_WRITE(SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
> > > +
> > > hotplug_irqs = SDE_HOTPLUG_MASK_SPT;
> > > enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_spt);
> > >
> > > diff --git a/drivers/gpu/drm/i915/i915_reg.h
> > > b/drivers/gpu/drm/i915/i915_reg.h
> > > index 94d0f593eeb7..74cf45de162e 100644
> > > --- a/drivers/gpu/drm/i915/i915_reg.h
> > > +++ b/drivers/gpu/drm/i915/i915_reg.h
> > > @@ -8110,6 +8110,10 @@ enum {
> > >
> > > #define SHOTPLUG_CTL_TC _MMIO(0xc4034)
> > > #define ICP_TC_HPD_ENABLE(tc_port) (8 << (tc_port)
> > > * 4)
> > > +
> > > +#define SHPD_FILTER_CNT _MMIO(0xc4038)
> > > +#define SHPD_FILTER_CNT_500_ADJ 0x001D9
> >
> > Shouldn't it be 0x1F2? Or I'm missing something?
> > Also this is the default value.
> >
>
> 0x1F2 is the "500 microseconds" option, whereas 0x1D9 is the "500
> microseconds adjusted" option according to bspec 8407/50473. The
> programming instructions tell us to use the non-default "adjusted"
> variant.
>
> I'm not sure why they don't just call it "475 microseconds" since
> that's
> what this value really works out to...
Oooh
"Adjusted" smells like a workaround value
Anyways:
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
>
> Matt
>
> > > +
> > > /* Icelake DSC Rate Control Range Parameter Registers */
> > > #define DSCA_RC_RANGE_PARAMETERS_0 _MMIO(0x6B240)
> > > #define DSCA_RC_RANGE_PARAMETERS_0_UDW _MMIO(0x6B240 +
> > > 4)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 23:20 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-26 23:20 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7426 -> Patchwork_15447
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15447 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15447, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15447:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_gem_contexts:
- fi-skl-6770hq: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
Known issues
------------
Here are the changes found in Patchwork_15447 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [PASS][3] -> [FAIL][4] ([fdo#111045] / [fdo#111096])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
#### Possible fixes ####
* igt@i915_selftest@live_gem_contexts:
- fi-skl-lmem: [DMESG-FAIL][5] -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
#### Warnings ####
* igt@gem_exec_suspend@basic-s4-devices:
- fi-kbl-x1275: [DMESG-WARN][7] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][8] ([fdo#103558] / [fdo#105602] / [fdo#107139])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][9] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][10] ([fdo#103558] / [fdo#105602]) +5 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-atomic:
- fi-kbl-x1275: [DMESG-WARN][11] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][12] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +4 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
Participating hosts (49 -> 45)
------------------------------
Additional (2): fi-hsw-4770r fi-whl-u
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7426 -> Patchwork_15447
CI-20190529: 20190529
CI_DRM_7426: b204d72d3485a148456e2077683974739b675b21 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15447: 86c94d0000b657e7adda0a615e7264c25e158e3b @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
86c94d0000b6 drm/i915: Program SHPD_FILTER_CNT on CNP+
4fe5a19ba44a drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
2e5ac97c5e02 drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 23:20 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-26 23:20 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7426 -> Patchwork_15447
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15447 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15447, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15447:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_gem_contexts:
- fi-skl-6770hq: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
Known issues
------------
Here are the changes found in Patchwork_15447 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [PASS][3] -> [FAIL][4] ([fdo#111045] / [fdo#111096])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
#### Possible fixes ####
* igt@i915_selftest@live_gem_contexts:
- fi-skl-lmem: [DMESG-FAIL][5] -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
#### Warnings ####
* igt@gem_exec_suspend@basic-s4-devices:
- fi-kbl-x1275: [DMESG-WARN][7] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][8] ([fdo#103558] / [fdo#105602] / [fdo#107139])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][9] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][10] ([fdo#103558] / [fdo#105602]) +5 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-atomic:
- fi-kbl-x1275: [DMESG-WARN][11] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][12] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +4 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
Participating hosts (49 -> 45)
------------------------------
Additional (2): fi-hsw-4770r fi-whl-u
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7426 -> Patchwork_15447
CI-20190529: 20190529
CI_DRM_7426: b204d72d3485a148456e2077683974739b675b21 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15447: 86c94d0000b657e7adda0a615e7264c25e158e3b @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
86c94d0000b6 drm/i915: Program SHPD_FILTER_CNT on CNP+
4fe5a19ba44a drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
2e5ac97c5e02 drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 23:38 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 23:38 UTC (permalink / raw)
To: intel-gfx
On Tue, Nov 26, 2019 at 11:20:18PM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
> URL : https://patchwork.freedesktop.org/series/70073/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_7426 -> Patchwork_15447
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_15447 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_15447, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_15447:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live_gem_contexts:
> - fi-skl-6770hq: [PASS][1] -> [DMESG-FAIL][2]
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
> [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
GEM self-test failed:
<3>[ 480.076214] idle: Failed with -11!
<3>[ 480.090075] i915/i915_gem_context_live_selftests: igt_ctx_sseu failed with error -11
Not related to this series; hitting the re-test button.
Matt
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_15447 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@kms_chamelium@hdmi-hpd-fast:
> - fi-kbl-7500u: [PASS][3] -> [FAIL][4] ([fdo#111045] / [fdo#111096])
> [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
> [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
>
>
> #### Possible fixes ####
>
> * igt@i915_selftest@live_gem_contexts:
> - fi-skl-lmem: [DMESG-FAIL][5] -> [PASS][6]
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
> [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
>
>
> #### Warnings ####
>
> * igt@gem_exec_suspend@basic-s4-devices:
> - fi-kbl-x1275: [DMESG-WARN][7] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][8] ([fdo#103558] / [fdo#105602] / [fdo#107139])
> [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
> [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
>
> * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
> - fi-kbl-x1275: [DMESG-WARN][9] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][10] ([fdo#103558] / [fdo#105602]) +5 similar issues
> [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
> [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
>
> * igt@kms_cursor_legacy@basic-flip-after-cursor-atomic:
> - fi-kbl-x1275: [DMESG-WARN][11] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][12] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +4 similar issues
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
> [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
>
>
> [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
> [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
> [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
> [fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
> [fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
> [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
>
>
> Participating hosts (49 -> 45)
> ------------------------------
>
> Additional (2): fi-hsw-4770r fi-whl-u
> Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
>
>
> Build changes
> -------------
>
> * CI: CI-20190529 -> None
> * Linux: CI_DRM_7426 -> Patchwork_15447
>
> CI-20190529: 20190529
> CI_DRM_7426: b204d72d3485a148456e2077683974739b675b21 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_15447: 86c94d0000b657e7adda0a615e7264c25e158e3b @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> 86c94d0000b6 drm/i915: Program SHPD_FILTER_CNT on CNP+
> 4fe5a19ba44a drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
> 2e5ac97c5e02 drm/i915: Handle SDEISR according to PCH rather than platform
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-26 23:38 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-26 23:38 UTC (permalink / raw)
To: intel-gfx
On Tue, Nov 26, 2019 at 11:20:18PM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform
> URL : https://patchwork.freedesktop.org/series/70073/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_7426 -> Patchwork_15447
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_15447 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_15447, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_15447:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live_gem_contexts:
> - fi-skl-6770hq: [PASS][1] -> [DMESG-FAIL][2]
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
> [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
GEM self-test failed:
<3>[ 480.076214] idle: Failed with -11!
<3>[ 480.090075] i915/i915_gem_context_live_selftests: igt_ctx_sseu failed with error -11
Not related to this series; hitting the re-test button.
Matt
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_15447 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@kms_chamelium@hdmi-hpd-fast:
> - fi-kbl-7500u: [PASS][3] -> [FAIL][4] ([fdo#111045] / [fdo#111096])
> [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
> [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
>
>
> #### Possible fixes ####
>
> * igt@i915_selftest@live_gem_contexts:
> - fi-skl-lmem: [DMESG-FAIL][5] -> [PASS][6]
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
> [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
>
>
> #### Warnings ####
>
> * igt@gem_exec_suspend@basic-s4-devices:
> - fi-kbl-x1275: [DMESG-WARN][7] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][8] ([fdo#103558] / [fdo#105602] / [fdo#107139])
> [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
> [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
>
> * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
> - fi-kbl-x1275: [DMESG-WARN][9] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][10] ([fdo#103558] / [fdo#105602]) +5 similar issues
> [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
> [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
>
> * igt@kms_cursor_legacy@basic-flip-after-cursor-atomic:
> - fi-kbl-x1275: [DMESG-WARN][11] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][12] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +4 similar issues
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7426/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
> [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
>
>
> [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
> [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
> [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
> [fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
> [fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
> [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
>
>
> Participating hosts (49 -> 45)
> ------------------------------
>
> Additional (2): fi-hsw-4770r fi-whl-u
> Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
>
>
> Build changes
> -------------
>
> * CI: CI-20190529 -> None
> * Linux: CI_DRM_7426 -> Patchwork_15447
>
> CI-20190529: 20190529
> CI_DRM_7426: b204d72d3485a148456e2077683974739b675b21 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_15447: 86c94d0000b657e7adda0a615e7264c25e158e3b @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> 86c94d0000b6 drm/i915: Program SHPD_FILTER_CNT on CNP+
> 4fe5a19ba44a drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
> 2e5ac97c5e02 drm/i915: Handle SDEISR according to PCH rather than platform
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15447/index.html
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev2)
@ 2019-11-27 2:35 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-27 2:35 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev2)
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7428 -> Patchwork_15452
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15452 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15452, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15452:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_gem_contexts:
- fi-skl-6770hq: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
Known issues
------------
Here are the changes found in Patchwork_15452 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s3:
- fi-icl-y: [PASS][3] -> [INCOMPLETE][4] ([fdo#107713])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-y/igt@gem_exec_suspend@basic-s3.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-icl-y/igt@gem_exec_suspend@basic-s3.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-skl-6770hq: [PASS][5] -> [DMESG-WARN][6] ([fdo#105541])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_gem_contexts:
- fi-skl-lmem: [PASS][7] -> [INCOMPLETE][8] ([fdo#111700])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
#### Possible fixes ####
* igt@i915_module_load@reload-with-fault-injection:
- {fi-kbl-7560u}: [INCOMPLETE][9] ([fdo#109964] / [fdo#112298]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_blt:
- fi-bsw-n3050: [DMESG-FAIL][11] ([fdo#112176]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-bsw-n3050/igt@i915_selftest@live_blt.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-bsw-n3050/igt@i915_selftest@live_blt.html
- fi-hsw-peppy: [DMESG-FAIL][13] ([fdo#112147]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-hsw-peppy/igt@i915_selftest@live_blt.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-hsw-peppy/igt@i915_selftest@live_blt.html
* igt@i915_selftest@live_execlists:
- fi-glk-dsi: [DMESG-FAIL][15] -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-glk-dsi/igt@i915_selftest@live_execlists.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-glk-dsi/igt@i915_selftest@live_execlists.html
* igt@kms_busy@basic-flip-pipe-a:
- fi-icl-u2: [TIMEOUT][17] ([fdo#111800]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
#### Warnings ####
* igt@kms_busy@basic-flip-pipe-b:
- fi-kbl-x1275: [DMESG-WARN][19] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][20] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +6 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_busy@basic-flip-pipe-b.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-x1275/igt@kms_busy@basic-flip-pipe-b.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [FAIL][21] ([fdo#111045] / [fdo#111096]) -> [FAIL][22] ([fdo#111407])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_pipe_crc_basic@read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][23] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][24] ([fdo#103558] / [fdo#105602]) +3 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105541]: https://bugs.freedesktop.org/show_bug.cgi?id=105541
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
[fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
[fdo#111700]: https://bugs.freedesktop.org/show_bug.cgi?id=111700
[fdo#111800]: https://bugs.freedesktop.org/show_bug.cgi?id=111800
[fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
[fdo#112176]: https://bugs.freedesktop.org/show_bug.cgi?id=112176
[fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
Participating hosts (51 -> 45)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7428 -> Patchwork_15452
CI-20190529: 20190529
CI_DRM_7428: d90242799e46210c5c09e1f0c64db2bb7ee870c6 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15452: 3937a13811ec4c713926fd4ed302292e53e4a86e @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
3937a13811ec drm/i915: Program SHPD_FILTER_CNT on CNP+
09130f2d60e6 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
061f4770dcee drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev2)
@ 2019-11-27 2:35 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-27 2:35 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev2)
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7428 -> Patchwork_15452
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15452 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15452, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15452:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_gem_contexts:
- fi-skl-6770hq: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-skl-6770hq/igt@i915_selftest@live_gem_contexts.html
Known issues
------------
Here are the changes found in Patchwork_15452 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s3:
- fi-icl-y: [PASS][3] -> [INCOMPLETE][4] ([fdo#107713])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-y/igt@gem_exec_suspend@basic-s3.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-icl-y/igt@gem_exec_suspend@basic-s3.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-skl-6770hq: [PASS][5] -> [DMESG-WARN][6] ([fdo#105541])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_gem_contexts:
- fi-skl-lmem: [PASS][7] -> [INCOMPLETE][8] ([fdo#111700])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
#### Possible fixes ####
* igt@i915_module_load@reload-with-fault-injection:
- {fi-kbl-7560u}: [INCOMPLETE][9] ([fdo#109964] / [fdo#112298]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_blt:
- fi-bsw-n3050: [DMESG-FAIL][11] ([fdo#112176]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-bsw-n3050/igt@i915_selftest@live_blt.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-bsw-n3050/igt@i915_selftest@live_blt.html
- fi-hsw-peppy: [DMESG-FAIL][13] ([fdo#112147]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-hsw-peppy/igt@i915_selftest@live_blt.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-hsw-peppy/igt@i915_selftest@live_blt.html
* igt@i915_selftest@live_execlists:
- fi-glk-dsi: [DMESG-FAIL][15] -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-glk-dsi/igt@i915_selftest@live_execlists.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-glk-dsi/igt@i915_selftest@live_execlists.html
* igt@kms_busy@basic-flip-pipe-a:
- fi-icl-u2: [TIMEOUT][17] ([fdo#111800]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
#### Warnings ####
* igt@kms_busy@basic-flip-pipe-b:
- fi-kbl-x1275: [DMESG-WARN][19] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][20] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +6 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_busy@basic-flip-pipe-b.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-x1275/igt@kms_busy@basic-flip-pipe-b.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [FAIL][21] ([fdo#111045] / [fdo#111096]) -> [FAIL][22] ([fdo#111407])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_pipe_crc_basic@read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][23] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][24] ([fdo#103558] / [fdo#105602]) +3 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105541]: https://bugs.freedesktop.org/show_bug.cgi?id=105541
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
[fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
[fdo#111700]: https://bugs.freedesktop.org/show_bug.cgi?id=111700
[fdo#111800]: https://bugs.freedesktop.org/show_bug.cgi?id=111800
[fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
[fdo#112176]: https://bugs.freedesktop.org/show_bug.cgi?id=112176
[fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
Participating hosts (51 -> 45)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7428 -> Patchwork_15452
CI-20190529: 20190529
CI_DRM_7428: d90242799e46210c5c09e1f0c64db2bb7ee870c6 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15452: 3937a13811ec4c713926fd4ed302292e53e4a86e @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
3937a13811ec drm/i915: Program SHPD_FILTER_CNT on CNP+
09130f2d60e6 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
061f4770dcee drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15452/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
@ 2019-11-27 7:19 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-27 7:19 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7428 -> Patchwork_15455
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15455 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15455, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15455:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_execlists:
- fi-cfl-8700k: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@i915_selftest@live_active:
- {fi-kbl-7560u}: NOTRUN -> [DMESG-FAIL][3]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_selftest@live_active.html
Known issues
------------
Here are the changes found in Patchwork_15455 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@module-reload:
- fi-skl-lmem: [PASS][4] -> [DMESG-WARN][5] ([fdo#112261])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
#### Possible fixes ####
* igt@i915_module_load@reload-with-fault-injection:
- {fi-kbl-7560u}: [INCOMPLETE][6] ([fdo#109964] / [fdo#112298]) -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_blt:
- fi-bsw-n3050: [DMESG-FAIL][8] ([fdo#112176]) -> [PASS][9]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-bsw-n3050/igt@i915_selftest@live_blt.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-bsw-n3050/igt@i915_selftest@live_blt.html
- fi-hsw-peppy: [DMESG-FAIL][10] ([fdo#112147]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-hsw-peppy/igt@i915_selftest@live_blt.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-hsw-peppy/igt@i915_selftest@live_blt.html
* igt@i915_selftest@live_execlists:
- fi-glk-dsi: [DMESG-FAIL][12] -> [PASS][13]
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-glk-dsi/igt@i915_selftest@live_execlists.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-glk-dsi/igt@i915_selftest@live_execlists.html
* igt@kms_busy@basic-flip-pipe-a:
- fi-icl-u2: [TIMEOUT][14] ([fdo#111800]) -> [PASS][15]
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [FAIL][16] ([fdo#111045] / [fdo#111096]) -> [PASS][17]
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
#### Warnings ####
* igt@gem_exec_suspend@basic-s4-devices:
- fi-kbl-x1275: [DMESG-WARN][18] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][19] ([fdo#103558] / [fdo#105602] / [fdo#107139])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
* igt@kms_flip@basic-flip-vs-modeset:
- fi-kbl-x1275: [DMESG-WARN][20] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][21] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +8 similar issues
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][22] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][23] ([fdo#103558] / [fdo#105602]) +6 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
[fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
[fdo#111800]: https://bugs.freedesktop.org/show_bug.cgi?id=111800
[fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
[fdo#112176]: https://bugs.freedesktop.org/show_bug.cgi?id=112176
[fdo#112261]: https://bugs.freedesktop.org/show_bug.cgi?id=112261
[fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
Participating hosts (51 -> 45)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7428 -> Patchwork_15455
CI-20190529: 20190529
CI_DRM_7428: d90242799e46210c5c09e1f0c64db2bb7ee870c6 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15455: d383fec4b9be3418b6ec33147c212276c2340407 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
d383fec4b9be drm/i915: Program SHPD_FILTER_CNT on CNP+
51dee6a357d0 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
d2f45eb90da9 drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
@ 2019-11-27 7:19 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-27 7:19 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7428 -> Patchwork_15455
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15455 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15455, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15455:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_execlists:
- fi-cfl-8700k: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@i915_selftest@live_active:
- {fi-kbl-7560u}: NOTRUN -> [DMESG-FAIL][3]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_selftest@live_active.html
Known issues
------------
Here are the changes found in Patchwork_15455 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@module-reload:
- fi-skl-lmem: [PASS][4] -> [DMESG-WARN][5] ([fdo#112261])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
#### Possible fixes ####
* igt@i915_module_load@reload-with-fault-injection:
- {fi-kbl-7560u}: [INCOMPLETE][6] ([fdo#109964] / [fdo#112298]) -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_blt:
- fi-bsw-n3050: [DMESG-FAIL][8] ([fdo#112176]) -> [PASS][9]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-bsw-n3050/igt@i915_selftest@live_blt.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-bsw-n3050/igt@i915_selftest@live_blt.html
- fi-hsw-peppy: [DMESG-FAIL][10] ([fdo#112147]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-hsw-peppy/igt@i915_selftest@live_blt.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-hsw-peppy/igt@i915_selftest@live_blt.html
* igt@i915_selftest@live_execlists:
- fi-glk-dsi: [DMESG-FAIL][12] -> [PASS][13]
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-glk-dsi/igt@i915_selftest@live_execlists.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-glk-dsi/igt@i915_selftest@live_execlists.html
* igt@kms_busy@basic-flip-pipe-a:
- fi-icl-u2: [TIMEOUT][14] ([fdo#111800]) -> [PASS][15]
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [FAIL][16] ([fdo#111045] / [fdo#111096]) -> [PASS][17]
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
#### Warnings ####
* igt@gem_exec_suspend@basic-s4-devices:
- fi-kbl-x1275: [DMESG-WARN][18] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][19] ([fdo#103558] / [fdo#105602] / [fdo#107139])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
* igt@kms_flip@basic-flip-vs-modeset:
- fi-kbl-x1275: [DMESG-WARN][20] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][21] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +8 similar issues
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][22] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][23] ([fdo#103558] / [fdo#105602]) +6 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
[fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
[fdo#111800]: https://bugs.freedesktop.org/show_bug.cgi?id=111800
[fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
[fdo#112176]: https://bugs.freedesktop.org/show_bug.cgi?id=112176
[fdo#112261]: https://bugs.freedesktop.org/show_bug.cgi?id=112261
[fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
Participating hosts (51 -> 45)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7428 -> Patchwork_15455
CI-20190529: 20190529
CI_DRM_7428: d90242799e46210c5c09e1f0c64db2bb7ee870c6 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15455: d383fec4b9be3418b6ec33147c212276c2340407 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
d383fec4b9be drm/i915: Program SHPD_FILTER_CNT on CNP+
51dee6a357d0 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
d2f45eb90da9 drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
@ 2019-11-27 16:38 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-27 16:38 UTC (permalink / raw)
To: intel-gfx
On Wed, Nov 27, 2019 at 07:19:08AM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
> URL : https://patchwork.freedesktop.org/series/70073/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_7428 -> Patchwork_15455
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_15455 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_15455, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_15455:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live_execlists:
> - fi-cfl-8700k: [PASS][1] -> [DMESG-FAIL][2]
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
> [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
[ 456.989562] i915/intel_execlists_live_selftests: live_preempt_hang failed with error -5
Another GEM failure that isn't related to the test. Hitting re-test yet
again.
Matt
>
>
> #### Suppressed ####
>
> The following results come from untrusted machines, tests, or statuses.
> They do not affect the overall result.
>
> * igt@i915_selftest@live_active:
> - {fi-kbl-7560u}: NOTRUN -> [DMESG-FAIL][3]
> [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_selftest@live_active.html
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_15455 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@i915_pm_rpm@module-reload:
> - fi-skl-lmem: [PASS][4] -> [DMESG-WARN][5] ([fdo#112261])
> [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
>
>
> #### Possible fixes ####
>
> * igt@i915_module_load@reload-with-fault-injection:
> - {fi-kbl-7560u}: [INCOMPLETE][6] ([fdo#109964] / [fdo#112298]) -> [PASS][7]
> [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
> [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
>
> * igt@i915_selftest@live_blt:
> - fi-bsw-n3050: [DMESG-FAIL][8] ([fdo#112176]) -> [PASS][9]
> [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-bsw-n3050/igt@i915_selftest@live_blt.html
> [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-bsw-n3050/igt@i915_selftest@live_blt.html
> - fi-hsw-peppy: [DMESG-FAIL][10] ([fdo#112147]) -> [PASS][11]
> [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-hsw-peppy/igt@i915_selftest@live_blt.html
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-hsw-peppy/igt@i915_selftest@live_blt.html
>
> * igt@i915_selftest@live_execlists:
> - fi-glk-dsi: [DMESG-FAIL][12] -> [PASS][13]
> [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-glk-dsi/igt@i915_selftest@live_execlists.html
> [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-glk-dsi/igt@i915_selftest@live_execlists.html
>
> * igt@kms_busy@basic-flip-pipe-a:
> - fi-icl-u2: [TIMEOUT][14] ([fdo#111800]) -> [PASS][15]
> [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
> [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
>
> * igt@kms_chamelium@hdmi-hpd-fast:
> - fi-kbl-7500u: [FAIL][16] ([fdo#111045] / [fdo#111096]) -> [PASS][17]
> [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
> [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
>
>
> #### Warnings ####
>
> * igt@gem_exec_suspend@basic-s4-devices:
> - fi-kbl-x1275: [DMESG-WARN][18] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][19] ([fdo#103558] / [fdo#105602] / [fdo#107139])
> [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
> [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
>
> * igt@kms_flip@basic-flip-vs-modeset:
> - fi-kbl-x1275: [DMESG-WARN][20] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][21] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +8 similar issues
> [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
> [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
>
> * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
> - fi-kbl-x1275: [DMESG-WARN][22] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][23] ([fdo#103558] / [fdo#105602]) +6 similar issues
> [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
> [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
>
>
> {name}: This element is suppressed. This means it is ignored when computing
> the status of the difference (SUCCESS, WARNING, or FAILURE).
>
> [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
> [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
> [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
> [fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
> [fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
> [fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
> [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
> [fdo#111800]: https://bugs.freedesktop.org/show_bug.cgi?id=111800
> [fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
> [fdo#112176]: https://bugs.freedesktop.org/show_bug.cgi?id=112176
> [fdo#112261]: https://bugs.freedesktop.org/show_bug.cgi?id=112261
> [fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
>
>
> Participating hosts (51 -> 45)
> ------------------------------
>
> Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
>
>
> Build changes
> -------------
>
> * CI: CI-20190529 -> None
> * Linux: CI_DRM_7428 -> Patchwork_15455
>
> CI-20190529: 20190529
> CI_DRM_7428: d90242799e46210c5c09e1f0c64db2bb7ee870c6 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_15455: d383fec4b9be3418b6ec33147c212276c2340407 @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> d383fec4b9be drm/i915: Program SHPD_FILTER_CNT on CNP+
> 51dee6a357d0 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
> d2f45eb90da9 drm/i915: Handle SDEISR according to PCH rather than platform
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
@ 2019-11-27 16:38 ` Matt Roper
0 siblings, 0 replies; 32+ messages in thread
From: Matt Roper @ 2019-11-27 16:38 UTC (permalink / raw)
To: intel-gfx
On Wed, Nov 27, 2019 at 07:19:08AM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3)
> URL : https://patchwork.freedesktop.org/series/70073/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_7428 -> Patchwork_15455
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_15455 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_15455, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_15455:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live_execlists:
> - fi-cfl-8700k: [PASS][1] -> [DMESG-FAIL][2]
> [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
> [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-cfl-8700k/igt@i915_selftest@live_execlists.html
[ 456.989562] i915/intel_execlists_live_selftests: live_preempt_hang failed with error -5
Another GEM failure that isn't related to the test. Hitting re-test yet
again.
Matt
>
>
> #### Suppressed ####
>
> The following results come from untrusted machines, tests, or statuses.
> They do not affect the overall result.
>
> * igt@i915_selftest@live_active:
> - {fi-kbl-7560u}: NOTRUN -> [DMESG-FAIL][3]
> [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_selftest@live_active.html
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_15455 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@i915_pm_rpm@module-reload:
> - fi-skl-lmem: [PASS][4] -> [DMESG-WARN][5] ([fdo#112261])
> [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
> [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-skl-lmem/igt@i915_pm_rpm@module-reload.html
>
>
> #### Possible fixes ####
>
> * igt@i915_module_load@reload-with-fault-injection:
> - {fi-kbl-7560u}: [INCOMPLETE][6] ([fdo#109964] / [fdo#112298]) -> [PASS][7]
> [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
> [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7560u/igt@i915_module_load@reload-with-fault-injection.html
>
> * igt@i915_selftest@live_blt:
> - fi-bsw-n3050: [DMESG-FAIL][8] ([fdo#112176]) -> [PASS][9]
> [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-bsw-n3050/igt@i915_selftest@live_blt.html
> [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-bsw-n3050/igt@i915_selftest@live_blt.html
> - fi-hsw-peppy: [DMESG-FAIL][10] ([fdo#112147]) -> [PASS][11]
> [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-hsw-peppy/igt@i915_selftest@live_blt.html
> [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-hsw-peppy/igt@i915_selftest@live_blt.html
>
> * igt@i915_selftest@live_execlists:
> - fi-glk-dsi: [DMESG-FAIL][12] -> [PASS][13]
> [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-glk-dsi/igt@i915_selftest@live_execlists.html
> [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-glk-dsi/igt@i915_selftest@live_execlists.html
>
> * igt@kms_busy@basic-flip-pipe-a:
> - fi-icl-u2: [TIMEOUT][14] ([fdo#111800]) -> [PASS][15]
> [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
> [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-icl-u2/igt@kms_busy@basic-flip-pipe-a.html
>
> * igt@kms_chamelium@hdmi-hpd-fast:
> - fi-kbl-7500u: [FAIL][16] ([fdo#111045] / [fdo#111096]) -> [PASS][17]
> [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
> [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
>
>
> #### Warnings ####
>
> * igt@gem_exec_suspend@basic-s4-devices:
> - fi-kbl-x1275: [DMESG-WARN][18] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139]) -> [DMESG-WARN][19] ([fdo#103558] / [fdo#105602] / [fdo#107139])
> [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
> [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
>
> * igt@kms_flip@basic-flip-vs-modeset:
> - fi-kbl-x1275: [DMESG-WARN][20] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][21] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +8 similar issues
> [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
> [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
>
> * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
> - fi-kbl-x1275: [DMESG-WARN][22] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][23] ([fdo#103558] / [fdo#105602]) +6 similar issues
> [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7428/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
> [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/fi-kbl-x1275/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
>
>
> {name}: This element is suppressed. This means it is ignored when computing
> the status of the difference (SUCCESS, WARNING, or FAILURE).
>
> [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
> [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
> [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
> [fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
> [fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
> [fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
> [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
> [fdo#111800]: https://bugs.freedesktop.org/show_bug.cgi?id=111800
> [fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
> [fdo#112176]: https://bugs.freedesktop.org/show_bug.cgi?id=112176
> [fdo#112261]: https://bugs.freedesktop.org/show_bug.cgi?id=112261
> [fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
>
>
> Participating hosts (51 -> 45)
> ------------------------------
>
> Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
>
>
> Build changes
> -------------
>
> * CI: CI-20190529 -> None
> * Linux: CI_DRM_7428 -> Patchwork_15455
>
> CI-20190529: 20190529
> CI_DRM_7428: d90242799e46210c5c09e1f0c64db2bb7ee870c6 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_5310: d1ea62b3f759f10ff6860561ba82e5c4902511d3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_15455: d383fec4b9be3418b6ec33147c212276c2340407 @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> d383fec4b9be drm/i915: Program SHPD_FILTER_CNT on CNP+
> 51dee6a357d0 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
> d2f45eb90da9 drm/i915: Handle SDEISR according to PCH rather than platform
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15455/index.html
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
@ 2019-11-27 17:59 ` Lucas De Marchi
0 siblings, 0 replies; 32+ messages in thread
From: Lucas De Marchi @ 2019-11-27 17:59 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
On Tue, Nov 26, 2019 at 01:07:31PM -0800, Matt Roper wrote:
>When looking at SDEISR to determine the connection status of combo
>outputs, we should use the phy index rather than the port index.
>Although they're usually the same thing, EHL's DDI-D (port D) is
>attached to PHY-A and SDEISR doesn't even have bits for a "D" output.
>It's also possible that future platforms may map DDIs (the internal
>display engine programming units) to PHYs (the output handling on the IO
>side) in ways where port!=phy, so let's look at the PHY index by
>default.
>
>Fixes: 719d24002602 ("drm/i915/ehl: Enable DDI-D")
>Cc: José Roberto de Souza <jose.souza@intel.com>
>Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
>---
> drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
>index ddf5bad1b969..59c5fd7bf27d 100644
>--- a/drivers/gpu/drm/i915/display/intel_dp.c
>+++ b/drivers/gpu/drm/i915/display/intel_dp.c
>@@ -5477,14 +5477,12 @@ static bool bxt_digital_port_connected(struct intel_encoder *encoder)
> }
>
> static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
while at it I think it would make more sense to call this
icl_combo_phy_connected() and maybe even intel_combo_phy_connected()
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi
>- struct intel_digital_port *intel_dig_port)
>+ enum phy phy)
> {
>- enum port port = intel_dig_port->base.port;
>-
>- if (HAS_PCH_MCC(dev_priv) && port == PORT_C)
>+ if (HAS_PCH_MCC(dev_priv) && phy == PHY_C)
> return I915_READ(SDEISR) & SDE_TC_HOTPLUG_ICP(PORT_TC1);
>
>- return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
>+ return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(phy);
> }
>
> static bool icp_digital_port_connected(struct intel_encoder *encoder)
>@@ -5494,7 +5492,7 @@ static bool icp_digital_port_connected(struct intel_encoder *encoder)
> enum phy phy = intel_port_to_phy(dev_priv, encoder->port);
>
> if (intel_phy_is_combo(dev_priv, phy))
>- return icl_combo_port_connected(dev_priv, dig_port);
>+ return icl_combo_port_connected(dev_priv, phy);
> else if (intel_phy_is_tc(dev_priv, phy))
> return intel_tc_port_connected(dig_port);
> else
>--
>2.23.0
>
>_______________________________________________
>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] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
@ 2019-11-27 17:59 ` Lucas De Marchi
0 siblings, 0 replies; 32+ messages in thread
From: Lucas De Marchi @ 2019-11-27 17:59 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
On Tue, Nov 26, 2019 at 01:07:31PM -0800, Matt Roper wrote:
>When looking at SDEISR to determine the connection status of combo
>outputs, we should use the phy index rather than the port index.
>Although they're usually the same thing, EHL's DDI-D (port D) is
>attached to PHY-A and SDEISR doesn't even have bits for a "D" output.
>It's also possible that future platforms may map DDIs (the internal
>display engine programming units) to PHYs (the output handling on the IO
>side) in ways where port!=phy, so let's look at the PHY index by
>default.
>
>Fixes: 719d24002602 ("drm/i915/ehl: Enable DDI-D")
>Cc: José Roberto de Souza <jose.souza@intel.com>
>Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
>---
> drivers/gpu/drm/i915/display/intel_dp.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
>index ddf5bad1b969..59c5fd7bf27d 100644
>--- a/drivers/gpu/drm/i915/display/intel_dp.c
>+++ b/drivers/gpu/drm/i915/display/intel_dp.c
>@@ -5477,14 +5477,12 @@ static bool bxt_digital_port_connected(struct intel_encoder *encoder)
> }
>
> static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
while at it I think it would make more sense to call this
icl_combo_phy_connected() and maybe even intel_combo_phy_connected()
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi
>- struct intel_digital_port *intel_dig_port)
>+ enum phy phy)
> {
>- enum port port = intel_dig_port->base.port;
>-
>- if (HAS_PCH_MCC(dev_priv) && port == PORT_C)
>+ if (HAS_PCH_MCC(dev_priv) && phy == PHY_C)
> return I915_READ(SDEISR) & SDE_TC_HOTPLUG_ICP(PORT_TC1);
>
>- return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
>+ return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(phy);
> }
>
> static bool icp_digital_port_connected(struct intel_encoder *encoder)
>@@ -5494,7 +5492,7 @@ static bool icp_digital_port_connected(struct intel_encoder *encoder)
> enum phy phy = intel_port_to_phy(dev_priv, encoder->port);
>
> if (intel_phy_is_combo(dev_priv, phy))
>- return icl_combo_port_connected(dev_priv, dig_port);
>+ return icl_combo_port_connected(dev_priv, phy);
> else if (intel_phy_is_tc(dev_priv, phy))
> return intel_tc_port_connected(dig_port);
> else
>--
>2.23.0
>
>_______________________________________________
>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] 32+ messages in thread
* Re: [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-27 18:00 ` Lucas De Marchi
0 siblings, 0 replies; 32+ messages in thread
From: Lucas De Marchi @ 2019-11-27 18:00 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
On Tue, Nov 26, 2019 at 01:07:30PM -0800, Matt Roper wrote:
>The South Display is part of the PCH so we should technically be basing
>our port detection logic off the PCH in use rather than the platform
>generation.
>
>Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi
>---
> drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
>index 3123958e2081..ddf5bad1b969 100644
>--- a/drivers/gpu/drm/i915/display/intel_dp.c
>+++ b/drivers/gpu/drm/i915/display/intel_dp.c
>@@ -5487,7 +5487,7 @@ static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
> return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
> }
>
>-static bool icl_digital_port_connected(struct intel_encoder *encoder)
>+static bool icp_digital_port_connected(struct intel_encoder *encoder)
> {
> struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
> struct intel_digital_port *dig_port = enc_to_dig_port(&encoder->base);
>@@ -5525,9 +5525,9 @@ static bool __intel_digital_port_connected(struct intel_encoder *encoder)
> return g4x_digital_port_connected(encoder);
> }
>
>- if (INTEL_GEN(dev_priv) >= 11)
>- return icl_digital_port_connected(encoder);
>- else if (IS_GEN(dev_priv, 10) || IS_GEN9_BC(dev_priv))
>+ if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
>+ return icp_digital_port_connected(encoder);
>+ else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
> return spt_digital_port_connected(encoder);
> else if (IS_GEN9_LP(dev_priv))
> return bxt_digital_port_connected(encoder);
>--
>2.23.0
>
>_______________________________________________
>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] 32+ messages in thread
* Re: [Intel-gfx] [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform
@ 2019-11-27 18:00 ` Lucas De Marchi
0 siblings, 0 replies; 32+ messages in thread
From: Lucas De Marchi @ 2019-11-27 18:00 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
On Tue, Nov 26, 2019 at 01:07:30PM -0800, Matt Roper wrote:
>The South Display is part of the PCH so we should technically be basing
>our port detection logic off the PCH in use rather than the platform
>generation.
>
>Cc: Lucas De Marchi <lucas.demarchi@intel.com>
>Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi
>---
> drivers/gpu/drm/i915/display/intel_dp.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
>index 3123958e2081..ddf5bad1b969 100644
>--- a/drivers/gpu/drm/i915/display/intel_dp.c
>+++ b/drivers/gpu/drm/i915/display/intel_dp.c
>@@ -5487,7 +5487,7 @@ static bool icl_combo_port_connected(struct drm_i915_private *dev_priv,
> return I915_READ(SDEISR) & SDE_DDI_HOTPLUG_ICP(port);
> }
>
>-static bool icl_digital_port_connected(struct intel_encoder *encoder)
>+static bool icp_digital_port_connected(struct intel_encoder *encoder)
> {
> struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
> struct intel_digital_port *dig_port = enc_to_dig_port(&encoder->base);
>@@ -5525,9 +5525,9 @@ static bool __intel_digital_port_connected(struct intel_encoder *encoder)
> return g4x_digital_port_connected(encoder);
> }
>
>- if (INTEL_GEN(dev_priv) >= 11)
>- return icl_digital_port_connected(encoder);
>- else if (IS_GEN(dev_priv, 10) || IS_GEN9_BC(dev_priv))
>+ if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
>+ return icp_digital_port_connected(encoder);
>+ else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
> return spt_digital_port_connected(encoder);
> else if (IS_GEN9_LP(dev_priv))
> return bxt_digital_port_connected(encoder);
>--
>2.23.0
>
>_______________________________________________
>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] 32+ messages in thread
* ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev4)
@ 2019-11-27 21:17 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-27 21:17 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev4)
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7434 -> Patchwork_15471
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15471 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15471, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15471:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_gt_heartbeat:
- fi-icl-u3: NOTRUN -> [DMESG-FAIL][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-icl-u3/igt@i915_selftest@live_gt_heartbeat.html
Known issues
------------
Here are the changes found in Patchwork_15471 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_frontbuffer_tracking@basic:
- fi-icl-u3: [PASS][2] -> [FAIL][3] ([fdo#103167])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-icl-u3/igt@kms_frontbuffer_tracking@basic.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-icl-u3/igt@kms_frontbuffer_tracking@basic.html
- fi-hsw-peppy: [PASS][4] -> [DMESG-WARN][5] ([fdo#102614])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
#### Possible fixes ####
* igt@i915_module_load@reload-no-display:
- fi-skl-lmem: [DMESG-WARN][6] ([fdo#112261]) -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-skl-lmem/igt@i915_module_load@reload-no-display.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-skl-lmem/igt@i915_module_load@reload-no-display.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-icl-u3: [INCOMPLETE][8] ([fdo#107713]) -> [PASS][9]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-icl-u3/igt@i915_module_load@reload-with-fault-injection.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-icl-u3/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_blt:
- fi-hsw-peppy: [DMESG-FAIL][10] ([fdo#112147]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-hsw-peppy/igt@i915_selftest@live_blt.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-hsw-peppy/igt@i915_selftest@live_blt.html
#### Warnings ####
* igt@gem_exec_suspend@basic-s4-devices:
- fi-kbl-x1275: [DMESG-WARN][12] ([fdo#103558] / [fdo#105602] / [fdo#107139]) -> [DMESG-WARN][13] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
* igt@i915_pm_rpm@module-reload:
- fi-skl-6770hq: [DMESG-WARN][14] ([fdo#112261]) -> [FAIL][15] ([fdo#108511])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][16] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][17] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +6 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_pipe_crc_basic@read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][18] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][19] ([fdo#103558] / [fdo#105602]) +1 similar issue
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#108511]: https://bugs.freedesktop.org/show_bug.cgi?id=108511
[fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
[fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
[fdo#112261]: https://bugs.freedesktop.org/show_bug.cgi?id=112261
[fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
Participating hosts (52 -> 45)
------------------------------
Missing (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7434 -> Patchwork_15471
CI-20190529: 20190529
CI_DRM_7434: 1bbc4d30ca9fd950cbcb73f324e00d0bc357758e @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5312: 851c75531043cd906e028632b64b02b9312e9945 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15471: 9638e9227864de0dbb086a7c2225a398e589c499 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
9638e9227864 drm/i915: Program SHPD_FILTER_CNT on CNP+
97f9c9ff7ab5 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
a9b96f566cc1 drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev4)
@ 2019-11-27 21:17 ` Patchwork
0 siblings, 0 replies; 32+ messages in thread
From: Patchwork @ 2019-11-27 21:17 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev4)
URL : https://patchwork.freedesktop.org/series/70073/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7434 -> Patchwork_15471
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15471 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15471, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15471:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_gt_heartbeat:
- fi-icl-u3: NOTRUN -> [DMESG-FAIL][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-icl-u3/igt@i915_selftest@live_gt_heartbeat.html
Known issues
------------
Here are the changes found in Patchwork_15471 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_frontbuffer_tracking@basic:
- fi-icl-u3: [PASS][2] -> [FAIL][3] ([fdo#103167])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-icl-u3/igt@kms_frontbuffer_tracking@basic.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-icl-u3/igt@kms_frontbuffer_tracking@basic.html
- fi-hsw-peppy: [PASS][4] -> [DMESG-WARN][5] ([fdo#102614])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
#### Possible fixes ####
* igt@i915_module_load@reload-no-display:
- fi-skl-lmem: [DMESG-WARN][6] ([fdo#112261]) -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-skl-lmem/igt@i915_module_load@reload-no-display.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-skl-lmem/igt@i915_module_load@reload-no-display.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-icl-u3: [INCOMPLETE][8] ([fdo#107713]) -> [PASS][9]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-icl-u3/igt@i915_module_load@reload-with-fault-injection.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-icl-u3/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_blt:
- fi-hsw-peppy: [DMESG-FAIL][10] ([fdo#112147]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-hsw-peppy/igt@i915_selftest@live_blt.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-hsw-peppy/igt@i915_selftest@live_blt.html
#### Warnings ####
* igt@gem_exec_suspend@basic-s4-devices:
- fi-kbl-x1275: [DMESG-WARN][12] ([fdo#103558] / [fdo#105602] / [fdo#107139]) -> [DMESG-WARN][13] ([fdo#103558] / [fdo#105602] / [fdo#105763] / [fdo#107139])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-kbl-x1275/igt@gem_exec_suspend@basic-s4-devices.html
* igt@i915_pm_rpm@module-reload:
- fi-skl-6770hq: [DMESG-WARN][14] ([fdo#112261]) -> [FAIL][15] ([fdo#108511])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][16] ([fdo#103558] / [fdo#105602]) -> [DMESG-WARN][17] ([fdo#103558] / [fdo#105602] / [fdo#105763]) +6 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_pipe_crc_basic@read-crc-pipe-a:
- fi-kbl-x1275: [DMESG-WARN][18] ([fdo#103558] / [fdo#105602] / [fdo#105763]) -> [DMESG-WARN][19] ([fdo#103558] / [fdo#105602]) +1 similar issue
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7434/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/fi-kbl-x1275/igt@kms_pipe_crc_basic@read-crc-pipe-a.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#107139]: https://bugs.freedesktop.org/show_bug.cgi?id=107139
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#108511]: https://bugs.freedesktop.org/show_bug.cgi?id=108511
[fdo#109964]: https://bugs.freedesktop.org/show_bug.cgi?id=109964
[fdo#112147]: https://bugs.freedesktop.org/show_bug.cgi?id=112147
[fdo#112261]: https://bugs.freedesktop.org/show_bug.cgi?id=112261
[fdo#112298]: https://bugs.freedesktop.org/show_bug.cgi?id=112298
Participating hosts (52 -> 45)
------------------------------
Missing (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7434 -> Patchwork_15471
CI-20190529: 20190529
CI_DRM_7434: 1bbc4d30ca9fd950cbcb73f324e00d0bc357758e @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5312: 851c75531043cd906e028632b64b02b9312e9945 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15471: 9638e9227864de0dbb086a7c2225a398e589c499 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
9638e9227864 drm/i915: Program SHPD_FILTER_CNT on CNP+
97f9c9ff7ab5 drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port
a9b96f566cc1 drm/i915: Handle SDEISR according to PCH rather than platform
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15471/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 32+ messages in thread
end of thread, other threads:[~2019-11-27 21:17 UTC | newest]
Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-26 21:07 [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform Matt Roper
2019-11-26 21:07 ` [Intel-gfx] " Matt Roper
2019-11-26 21:07 ` [PATCH 2/3] drm/i915/ehl: Make icp_digital_port_connected() use phy instead of port Matt Roper
2019-11-26 21:07 ` [Intel-gfx] " Matt Roper
2019-11-26 21:34 ` Souza, Jose
2019-11-26 21:34 ` [Intel-gfx] " Souza, Jose
2019-11-27 17:59 ` Lucas De Marchi
2019-11-27 17:59 ` [Intel-gfx] " Lucas De Marchi
2019-11-26 21:07 ` [PATCH 3/3] drm/i915: Program SHPD_FILTER_CNT on CNP+ Matt Roper
2019-11-26 21:07 ` [Intel-gfx] " Matt Roper
2019-11-26 21:41 ` Souza, Jose
2019-11-26 21:41 ` [Intel-gfx] " Souza, Jose
2019-11-26 21:50 ` Matt Roper
2019-11-26 21:50 ` [Intel-gfx] " Matt Roper
2019-11-26 21:57 ` Souza, Jose
2019-11-26 21:57 ` [Intel-gfx] " Souza, Jose
2019-11-26 21:29 ` [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform Souza, Jose
2019-11-26 21:29 ` [Intel-gfx] " Souza, Jose
2019-11-26 23:20 ` ✗ Fi.CI.BAT: failure for series starting with [1/3] " Patchwork
2019-11-26 23:20 ` [Intel-gfx] " Patchwork
2019-11-26 23:38 ` Matt Roper
2019-11-26 23:38 ` [Intel-gfx] " Matt Roper
2019-11-27 2:35 ` ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev2) Patchwork
2019-11-27 2:35 ` [Intel-gfx] " Patchwork
2019-11-27 7:19 ` ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev3) Patchwork
2019-11-27 7:19 ` [Intel-gfx] " Patchwork
2019-11-27 16:38 ` Matt Roper
2019-11-27 16:38 ` [Intel-gfx] " Matt Roper
2019-11-27 18:00 ` [PATCH 1/3] drm/i915: Handle SDEISR according to PCH rather than platform Lucas De Marchi
2019-11-27 18:00 ` [Intel-gfx] " Lucas De Marchi
2019-11-27 21:17 ` ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Handle SDEISR according to PCH rather than platform (rev4) Patchwork
2019-11-27 21:17 ` [Intel-gfx] " 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.