* [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe
@ 2020-02-04 0:02 Vivek Kasireddy
2020-02-04 10:50 ` Jani Nikula
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Vivek Kasireddy @ 2020-02-04 0:02 UTC (permalink / raw)
To: intel-gfx
Since the pipe->transcoder mapping is not expected to change unless
there is either eDP or DSI connectors present, check the VBT to confirm
their presence in addition to checking TRANS_DDI_FUNC_CTL(transcoder).
This additional check is needed on platforms like Elkhart Lake because
we cannot just rely on GOP/Firmware programmed values in
TRANS_DDI_FUNC_CTL(transcoder) before updating the transcoder mapping.
This patch is only relevant to EHL -- and a no-op on others --
because some of the PHYs are shared between the different DDIs and
we rely on the VBT to present the most accurate information to the
driver.
Cc: Matt Roper <matthew.d.roper@intel.com>
Cc: José Roberto de Souza <jose.souza@intel.com>
Signed-off-by: Vivek Kasireddy <vivek.kasireddy@intel.com>
---
drivers/gpu/drm/i915/display/intel_display.c | 15 ++++++++++++++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index c0e5002ce64c..4b38f293bd88 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -10805,6 +10805,18 @@ static void hsw_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
pipe_config->shared_dpll = intel_get_shared_dpll_by_id(dev_priv, id);
}
+static bool ehl_vbt_edp_dsi_present(struct drm_i915_private *dev_priv,
+ enum transcoder transcoder)
+{
+ bool edp_present = intel_bios_is_port_present(dev_priv, PORT_A);
+ bool dsi_present = intel_bios_is_dsi_present(dev_priv, NULL);
+
+ if (IS_ELKHARTLAKE(dev_priv))
+ return transcoder == TRANSCODER_EDP ? edp_present : dsi_present;
+
+ return true;
+}
+
static bool hsw_get_transcoder_state(struct intel_crtc *crtc,
struct intel_crtc_state *pipe_config,
u64 *power_domain_mask,
@@ -10844,7 +10856,8 @@ static bool hsw_get_transcoder_state(struct intel_crtc *crtc,
tmp = intel_de_read(dev_priv,
TRANS_DDI_FUNC_CTL(panel_transcoder));
- if (!(tmp & TRANS_DDI_FUNC_ENABLE))
+ if (!(tmp & TRANS_DDI_FUNC_ENABLE) ||
+ !ehl_vbt_edp_dsi_present(dev_priv, panel_transcoder))
continue;
/*
--
2.21.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe
2020-02-04 0:02 [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe Vivek Kasireddy
@ 2020-02-04 10:50 ` Jani Nikula
2020-02-04 22:44 ` Vivek Kasireddy
2020-02-04 22:13 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2020-02-06 22:50 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2 siblings, 1 reply; 6+ messages in thread
From: Jani Nikula @ 2020-02-04 10:50 UTC (permalink / raw)
To: Vivek Kasireddy, intel-gfx
On Mon, 03 Feb 2020, Vivek Kasireddy <vivek.kasireddy@intel.com> wrote:
> Since the pipe->transcoder mapping is not expected to change unless
> there is either eDP or DSI connectors present, check the VBT to confirm
> their presence in addition to checking TRANS_DDI_FUNC_CTL(transcoder).
> This additional check is needed on platforms like Elkhart Lake because
> we cannot just rely on GOP/Firmware programmed values in
> TRANS_DDI_FUNC_CTL(transcoder) before updating the transcoder mapping.
>
> This patch is only relevant to EHL -- and a no-op on others --
> because some of the PHYs are shared between the different DDIs and
> we rely on the VBT to present the most accurate information to the
> driver.
>
> Cc: Matt Roper <matthew.d.roper@intel.com>
> Cc: José Roberto de Souza <jose.souza@intel.com>
> Signed-off-by: Vivek Kasireddy <vivek.kasireddy@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_display.c | 15 ++++++++++++++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index c0e5002ce64c..4b38f293bd88 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -10805,6 +10805,18 @@ static void hsw_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
> pipe_config->shared_dpll = intel_get_shared_dpll_by_id(dev_priv, id);
> }
>
> +static bool ehl_vbt_edp_dsi_present(struct drm_i915_private *dev_priv,
> + enum transcoder transcoder)
> +{
> + bool edp_present = intel_bios_is_port_present(dev_priv, PORT_A);
> + bool dsi_present = intel_bios_is_dsi_present(dev_priv, NULL);
> +
> + if (IS_ELKHARTLAKE(dev_priv))
> + return transcoder == TRANSCODER_EDP ? edp_present : dsi_present;
> +
> + return true;
> +}
One of those things... this jumps out and immediately feels all wrong,
just like ehl_vbt_ddi_d_present() feels all wrong in
intel_combo_phy.c. But I don't know what would be the right thing to do
without spending time that I don't have on this.
BR,
Jani.
> +
> static bool hsw_get_transcoder_state(struct intel_crtc *crtc,
> struct intel_crtc_state *pipe_config,
> u64 *power_domain_mask,
> @@ -10844,7 +10856,8 @@ static bool hsw_get_transcoder_state(struct intel_crtc *crtc,
>
> tmp = intel_de_read(dev_priv,
> TRANS_DDI_FUNC_CTL(panel_transcoder));
> - if (!(tmp & TRANS_DDI_FUNC_ENABLE))
> + if (!(tmp & TRANS_DDI_FUNC_ENABLE) ||
> + !ehl_vbt_edp_dsi_present(dev_priv, panel_transcoder))
> continue;
>
> /*
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/ehl: Check VBT before updating the transcoder for pipe
2020-02-04 0:02 [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe Vivek Kasireddy
2020-02-04 10:50 ` Jani Nikula
@ 2020-02-04 22:13 ` Patchwork
2020-02-06 22:50 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2020-02-04 22:13 UTC (permalink / raw)
To: Vivek Kasireddy; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/ehl: Check VBT before updating the transcoder for pipe
URL : https://patchwork.freedesktop.org/series/72949/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_7865 -> Patchwork_16407
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/index.html
Known issues
------------
Here are the changes found in Patchwork_16407 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_close_race@basic-threads:
- fi-byt-j1900: [PASS][1] -> [INCOMPLETE][2] ([i915#45])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/fi-byt-j1900/igt@gem_close_race@basic-threads.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/fi-byt-j1900/igt@gem_close_race@basic-threads.html
- fi-byt-n2820: [PASS][3] -> [INCOMPLETE][4] ([i915#45])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/fi-byt-n2820/igt@gem_close_race@basic-threads.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/fi-byt-n2820/igt@gem_close_race@basic-threads.html
* igt@i915_selftest@live_blt:
- fi-ivb-3770: [PASS][5] -> [DMESG-FAIL][6] ([i915#553])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/fi-ivb-3770/igt@i915_selftest@live_blt.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/fi-ivb-3770/igt@i915_selftest@live_blt.html
#### Possible fixes ####
* igt@i915_selftest@live_blt:
- fi-hsw-4770r: [DMESG-FAIL][7] ([i915#553] / [i915#725]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/fi-hsw-4770r/igt@i915_selftest@live_blt.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/fi-hsw-4770r/igt@i915_selftest@live_blt.html
[i915#45]: https://gitlab.freedesktop.org/drm/intel/issues/45
[i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553
[i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725
Participating hosts (47 -> 45)
------------------------------
Additional (5): fi-bsw-n3050 fi-hsw-peppy fi-kbl-7500u fi-cfl-8109u fi-snb-2600
Missing (7): fi-hsw-4200u fi-skl-6770hq fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7865 -> Patchwork_16407
CI-20190529: 20190529
CI_DRM_7865: ac0f322e67a6a466d0d09ba722957608c20cf339 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5417: 33cc93c8ba5daa0b7498f297a4f626844d895d06 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_16407: 1043d319d289332d435146c72ea5ac519c8225fe @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
1043d319d289 drm/i915/ehl: Check VBT before updating the transcoder for pipe
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe
2020-02-04 10:50 ` Jani Nikula
@ 2020-02-04 22:44 ` Vivek Kasireddy
2020-02-04 23:49 ` Souza, Jose
0 siblings, 1 reply; 6+ messages in thread
From: Vivek Kasireddy @ 2020-02-04 22:44 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
On Tue, 4 Feb 2020 12:50:25 +0200
Jani Nikula <jani.nikula@linux.intel.com> wrote:
Hi Jani,
> On Mon, 03 Feb 2020, Vivek Kasireddy <vivek.kasireddy@intel.com>
> wrote:
> > Since the pipe->transcoder mapping is not expected to change unless
> > there is either eDP or DSI connectors present, check the VBT to
> > confirm their presence in addition to checking
> > TRANS_DDI_FUNC_CTL(transcoder). This additional check is needed on
> > platforms like Elkhart Lake because we cannot just rely on
> > GOP/Firmware programmed values in TRANS_DDI_FUNC_CTL(transcoder)
> > before updating the transcoder mapping.
> >
> > This patch is only relevant to EHL -- and a no-op on others --
> > because some of the PHYs are shared between the different DDIs and
> > we rely on the VBT to present the most accurate information to the
> > driver.
> >
> > Cc: Matt Roper <matthew.d.roper@intel.com>
> > Cc: José Roberto de Souza <jose.souza@intel.com>
> > Signed-off-by: Vivek Kasireddy <vivek.kasireddy@intel.com>
> > ---
> > drivers/gpu/drm/i915/display/intel_display.c | 15 ++++++++++++++-
> > 1 file changed, 14 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> > b/drivers/gpu/drm/i915/display/intel_display.c index
> > c0e5002ce64c..4b38f293bd88 100644 ---
> > a/drivers/gpu/drm/i915/display/intel_display.c +++
> > b/drivers/gpu/drm/i915/display/intel_display.c @@ -10805,6
> > +10805,18 @@ static void hsw_get_ddi_pll(struct drm_i915_private
> > *dev_priv, enum port port, pipe_config->shared_dpll =
> > intel_get_shared_dpll_by_id(dev_priv, id); }
> > +static bool ehl_vbt_edp_dsi_present(struct drm_i915_private
> > *dev_priv,
> > + enum transcoder transcoder)
> > +{
> > + bool edp_present = intel_bios_is_port_present(dev_priv,
> > PORT_A);
> > + bool dsi_present = intel_bios_is_dsi_present(dev_priv,
> > NULL); +
> > + if (IS_ELKHARTLAKE(dev_priv))
> > + return transcoder == TRANSCODER_EDP ? edp_present
> > : dsi_present; +
> > + return true;
> > +}
>
> One of those things... this jumps out and immediately feels all wrong,
> just like ehl_vbt_ddi_d_present() feels all wrong in
> intel_combo_phy.c. But I don't know what would be the right thing to
> do without spending time that I don't have on this.
Is there a particular approach you want me to take to address this
issue? All I am trying to do is address the plausible scenario(s) where
the GOP/firmware may program the hardware in a certain way that seems
incorrect from what i915 does based on the info in the VBT. I noticed
this issue on the EHL board I am working on; therefore, I limited the
fix to EHL only.
Thanks,
Vivek
>
> BR,
> Jani.
>
>
>
> > +
> > static bool hsw_get_transcoder_state(struct intel_crtc *crtc,
> > struct intel_crtc_state
> > *pipe_config, u64 *power_domain_mask,
> > @@ -10844,7 +10856,8 @@ static bool hsw_get_transcoder_state(struct
> > intel_crtc *crtc,
> > tmp = intel_de_read(dev_priv,
> > TRANS_DDI_FUNC_CTL(panel_transcoder));
> > - if (!(tmp & TRANS_DDI_FUNC_ENABLE))
> > + if (!(tmp & TRANS_DDI_FUNC_ENABLE) ||
> > + !ehl_vbt_edp_dsi_present(dev_priv,
> > panel_transcoder)) continue;
> >
> > /*
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe
2020-02-04 22:44 ` Vivek Kasireddy
@ 2020-02-04 23:49 ` Souza, Jose
0 siblings, 0 replies; 6+ messages in thread
From: Souza, Jose @ 2020-02-04 23:49 UTC (permalink / raw)
To: jani.nikula, Kasireddy, Vivek; +Cc: intel-gfx
I guess a better solution would be do the HW state sanitization during
the HW readout(intel_modeset_setup_hw_state())
On Tue, 2020-02-04 at 14:44 -0800, Vivek Kasireddy wrote:
> On Tue, 4 Feb 2020 12:50:25 +0200
> Jani Nikula <jani.nikula@linux.intel.com> wrote:
> Hi Jani,
>
> > On Mon, 03 Feb 2020, Vivek Kasireddy <vivek.kasireddy@intel.com>
> > wrote:
> > > Since the pipe->transcoder mapping is not expected to change
> > > unless
> > > there is either eDP or DSI connectors present, check the VBT to
> > > confirm their presence in addition to checking
> > > TRANS_DDI_FUNC_CTL(transcoder). This additional check is needed
> > > on
> > > platforms like Elkhart Lake because we cannot just rely on
> > > GOP/Firmware programmed values in TRANS_DDI_FUNC_CTL(transcoder)
> > > before updating the transcoder mapping.
> > >
> > > This patch is only relevant to EHL -- and a no-op on others --
> > > because some of the PHYs are shared between the different DDIs
> > > and
> > > we rely on the VBT to present the most accurate information to
> > > the
> > > driver.
> > >
> > > Cc: Matt Roper <matthew.d.roper@intel.com>
> > > Cc: José Roberto de Souza <jose.souza@intel.com>
> > > Signed-off-by: Vivek Kasireddy <vivek.kasireddy@intel.com>
> > > ---
> > > drivers/gpu/drm/i915/display/intel_display.c | 15
> > > ++++++++++++++-
> > > 1 file changed, 14 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> > > b/drivers/gpu/drm/i915/display/intel_display.c index
> > > c0e5002ce64c..4b38f293bd88 100644 ---
> > > a/drivers/gpu/drm/i915/display/intel_display.c +++
> > > b/drivers/gpu/drm/i915/display/intel_display.c @@ -10805,6
> > > +10805,18 @@ static void hsw_get_ddi_pll(struct drm_i915_private
> > > *dev_priv, enum port port, pipe_config->shared_dpll =
> > > intel_get_shared_dpll_by_id(dev_priv, id); }
> > > +static bool ehl_vbt_edp_dsi_present(struct drm_i915_private
> > > *dev_priv,
> > > + enum transcoder transcoder)
> > > +{
> > > + bool edp_present = intel_bios_is_port_present(dev_priv,
> > > PORT_A);
> > > + bool dsi_present = intel_bios_is_dsi_present(dev_priv,
> > > NULL); +
> > > + if (IS_ELKHARTLAKE(dev_priv))
> > > + return transcoder == TRANSCODER_EDP ? edp_present
> > > : dsi_present; +
> > > + return true;
> > > +}
> >
> > One of those things... this jumps out and immediately feels all
> > wrong,
> > just like ehl_vbt_ddi_d_present() feels all wrong in
> > intel_combo_phy.c. But I don't know what would be the right thing
> > to
> > do without spending time that I don't have on this.
>
> Is there a particular approach you want me to take to address this
> issue? All I am trying to do is address the plausible scenario(s)
> where
> the GOP/firmware may program the hardware in a certain way that seems
> incorrect from what i915 does based on the info in the VBT. I
> noticed
> this issue on the EHL board I am working on; therefore, I limited the
> fix to EHL only.
>
> Thanks,
> Vivek
>
> > BR,
> > Jani.
> >
> >
> >
> > > +
> > > static bool hsw_get_transcoder_state(struct intel_crtc *crtc,
> > > struct intel_crtc_state
> > > *pipe_config, u64 *power_domain_mask,
> > > @@ -10844,7 +10856,8 @@ static bool
> > > hsw_get_transcoder_state(struct
> > > intel_crtc *crtc,
> > > tmp = intel_de_read(dev_priv,
> > > TRANS_DDI_FUNC_CTL(panel_transcoder
> > > ));
> > > - if (!(tmp & TRANS_DDI_FUNC_ENABLE))
> > > + if (!(tmp & TRANS_DDI_FUNC_ENABLE) ||
> > > + !ehl_vbt_edp_dsi_present(dev_priv,
> > > panel_transcoder)) continue;
> > >
> > > /*
>
> _______________________________________________
> 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] 6+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/ehl: Check VBT before updating the transcoder for pipe
2020-02-04 0:02 [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe Vivek Kasireddy
2020-02-04 10:50 ` Jani Nikula
2020-02-04 22:13 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
@ 2020-02-06 22:50 ` Patchwork
2 siblings, 0 replies; 6+ messages in thread
From: Patchwork @ 2020-02-06 22:50 UTC (permalink / raw)
To: Vivek Kasireddy; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/ehl: Check VBT before updating the transcoder for pipe
URL : https://patchwork.freedesktop.org/series/72949/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_7865_full -> Patchwork_16407_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Known issues
------------
Here are the changes found in Patchwork_16407_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_close_race@basic-threads:
- shard-hsw: [PASS][1] -> [INCOMPLETE][2] ([i915#61] / [i915#694] / [i915#816])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-hsw2/igt@gem_close_race@basic-threads.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-hsw2/igt@gem_close_race@basic-threads.html
* igt@gem_ctx_shared@exec-single-timeline-bsd:
- shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#110841])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb8/igt@gem_ctx_shared@exec-single-timeline-bsd.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb1/igt@gem_ctx_shared@exec-single-timeline-bsd.html
* igt@gem_exec_balancer@hang:
- shard-tglb: [PASS][5] -> [TIMEOUT][6] ([fdo#112271])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-tglb2/igt@gem_exec_balancer@hang.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-tglb6/igt@gem_exec_balancer@hang.html
* igt@gem_exec_parallel@vcs1-fds:
- shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#112080]) +10 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb1/igt@gem_exec_parallel@vcs1-fds.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb6/igt@gem_exec_parallel@vcs1-fds.html
* igt@gem_exec_schedule@independent-bsd2:
- shard-iclb: [PASS][9] -> [SKIP][10] ([fdo#109276]) +18 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb1/igt@gem_exec_schedule@independent-bsd2.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb6/igt@gem_exec_schedule@independent-bsd2.html
* igt@gem_exec_schedule@pi-ringfull-bsd:
- shard-iclb: [PASS][11] -> [SKIP][12] ([fdo#112146]) +1 similar issue
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb3/igt@gem_exec_schedule@pi-ringfull-bsd.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb2/igt@gem_exec_schedule@pi-ringfull-bsd.html
* igt@gem_partial_pwrite_pread@writes-after-reads-display:
- shard-hsw: [PASS][13] -> [FAIL][14] ([i915#694]) +2 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-hsw7/igt@gem_partial_pwrite_pread@writes-after-reads-display.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-hsw6/igt@gem_partial_pwrite_pread@writes-after-reads-display.html
* igt@gem_ppgtt@flink-and-close-vma-leak:
- shard-kbl: [PASS][15] -> [FAIL][16] ([i915#644])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-kbl4/igt@gem_ppgtt@flink-and-close-vma-leak.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-kbl6/igt@gem_ppgtt@flink-and-close-vma-leak.html
* igt@i915_pm_dc@dc5-dpms:
- shard-iclb: [PASS][17] -> [FAIL][18] ([i915#447])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb6/igt@i915_pm_dc@dc5-dpms.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb3/igt@i915_pm_dc@dc5-dpms.html
* igt@i915_pm_rpm@fences-dpms:
- shard-iclb: [PASS][19] -> [INCOMPLETE][20] ([i915#189])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb3/igt@i915_pm_rpm@fences-dpms.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb2/igt@i915_pm_rpm@fences-dpms.html
* igt@i915_pm_rps@reset:
- shard-iclb: [PASS][21] -> [FAIL][22] ([i915#413])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb5/igt@i915_pm_rps@reset.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb8/igt@i915_pm_rps@reset.html
* igt@i915_selftest@live_gtt:
- shard-skl: [PASS][23] -> [TIMEOUT][24] ([fdo#111732] / [fdo#112271])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl9/igt@i915_selftest@live_gtt.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl9/igt@i915_selftest@live_gtt.html
* igt@kms_pipe_crc_basic@nonblocking-crc-pipe-c-frame-sequence:
- shard-skl: [PASS][25] -> [FAIL][26] ([i915#53])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl7/igt@kms_pipe_crc_basic@nonblocking-crc-pipe-c-frame-sequence.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl1/igt@kms_pipe_crc_basic@nonblocking-crc-pipe-c-frame-sequence.html
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes:
- shard-apl: [PASS][27] -> [DMESG-WARN][28] ([i915#180]) +1 similar issue
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-apl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-apl6/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html
* igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
- shard-skl: [PASS][29] -> [FAIL][30] ([fdo#108145] / [i915#265])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl7/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl6/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-kbl: [PASS][31] -> [DMESG-WARN][32] ([i915#180]) +4 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-kbl3/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-kbl2/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
#### Possible fixes ####
* igt@gem_ctx_isolation@rcs0-s3:
- shard-kbl: [DMESG-WARN][33] ([i915#180]) -> [PASS][34] +4 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-kbl2/igt@gem_ctx_isolation@rcs0-s3.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-kbl6/igt@gem_ctx_isolation@rcs0-s3.html
* igt@gem_ctx_shared@q-smoketest-all:
- shard-kbl: [INCOMPLETE][35] ([fdo#103665]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-kbl2/igt@gem_ctx_shared@q-smoketest-all.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-kbl6/igt@gem_ctx_shared@q-smoketest-all.html
* igt@gem_exec_schedule@preempt-queue-bsd2:
- shard-iclb: [SKIP][37] ([fdo#109276]) -> [PASS][38] +6 similar issues
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb8/igt@gem_exec_schedule@preempt-queue-bsd2.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb1/igt@gem_exec_schedule@preempt-queue-bsd2.html
* igt@gem_exec_schedule@reorder-wide-bsd:
- shard-iclb: [SKIP][39] ([fdo#112146]) -> [PASS][40] +6 similar issues
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb2/igt@gem_exec_schedule@reorder-wide-bsd.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb8/igt@gem_exec_schedule@reorder-wide-bsd.html
* igt@gem_ppgtt@flink-and-close-vma-leak:
- shard-glk: [FAIL][41] ([i915#644]) -> [PASS][42]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-glk9/igt@gem_ppgtt@flink-and-close-vma-leak.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-glk4/igt@gem_ppgtt@flink-and-close-vma-leak.html
* igt@gem_tiled_blits@normal:
- shard-hsw: [FAIL][43] ([i915#694]) -> [PASS][44]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-hsw7/igt@gem_tiled_blits@normal.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-hsw6/igt@gem_tiled_blits@normal.html
* igt@i915_pm_dc@dc6-psr:
- shard-iclb: [FAIL][45] ([i915#454]) -> [PASS][46]
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb2/igt@i915_pm_dc@dc6-psr.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb6/igt@i915_pm_dc@dc6-psr.html
* igt@i915_pm_rps@waitboost:
- shard-iclb: [FAIL][47] ([i915#413]) -> [PASS][48]
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb4/igt@i915_pm_rps@waitboost.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb7/igt@i915_pm_rps@waitboost.html
* igt@i915_selftest@live_gtt:
- shard-apl: [TIMEOUT][49] ([fdo#112271]) -> [PASS][50]
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-apl4/igt@i915_selftest@live_gtt.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-apl7/igt@i915_selftest@live_gtt.html
* igt@i915_suspend@fence-restore-tiled2untiled:
- shard-skl: [INCOMPLETE][51] ([i915#69]) -> [PASS][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl6/igt@i915_suspend@fence-restore-tiled2untiled.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl5/igt@i915_suspend@fence-restore-tiled2untiled.html
* igt@i915_suspend@sysfs-reader:
- shard-apl: [DMESG-WARN][53] ([i915#180]) -> [PASS][54] +4 similar issues
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-apl2/igt@i915_suspend@sysfs-reader.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-apl2/igt@i915_suspend@sysfs-reader.html
* igt@kms_cursor_crc@pipe-c-cursor-256x256-random:
- shard-apl: [FAIL][55] ([i915#54]) -> [PASS][56]
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-apl2/igt@kms_cursor_crc@pipe-c-cursor-256x256-random.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-apl3/igt@kms_cursor_crc@pipe-c-cursor-256x256-random.html
* igt@kms_flip@flip-vs-expired-vblank:
- shard-skl: [FAIL][57] ([i915#79]) -> [PASS][58]
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl7/igt@kms_flip@flip-vs-expired-vblank.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl6/igt@kms_flip@flip-vs-expired-vblank.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-hsw: [FAIL][59] ([i915#79]) -> [PASS][60]
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-hsw5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-hsw1/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@plain-flip-fb-recreate-interruptible:
- shard-skl: [FAIL][61] ([i915#34]) -> [PASS][62]
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl1/igt@kms_flip@plain-flip-fb-recreate-interruptible.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl1/igt@kms_flip@plain-flip-fb-recreate-interruptible.html
* igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min:
- shard-skl: [FAIL][63] ([fdo#108145]) -> [PASS][64] +2 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl1/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl1/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html
* igt@kms_plane_lowres@pipe-a-tiling-y:
- shard-glk: [FAIL][65] ([i915#899]) -> [PASS][66]
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-glk7/igt@kms_plane_lowres@pipe-a-tiling-y.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-glk1/igt@kms_plane_lowres@pipe-a-tiling-y.html
* igt@kms_psr@no_drrs:
- shard-iclb: [FAIL][67] ([i915#173]) -> [PASS][68]
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb1/igt@kms_psr@no_drrs.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb6/igt@kms_psr@no_drrs.html
* igt@kms_psr@psr2_cursor_render:
- shard-iclb: [SKIP][69] ([fdo#109441]) -> [PASS][70] +1 similar issue
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb3/igt@kms_psr@psr2_cursor_render.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb2/igt@kms_psr@psr2_cursor_render.html
* igt@kms_psr@suspend:
- shard-skl: [INCOMPLETE][71] ([i915#198]) -> [PASS][72]
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl1/igt@kms_psr@suspend.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl3/igt@kms_psr@suspend.html
* igt@kms_setmode@basic:
- shard-skl: [FAIL][73] ([i915#31]) -> [PASS][74]
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-skl3/igt@kms_setmode@basic.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-skl1/igt@kms_setmode@basic.html
* igt@perf_pmu@init-busy-vcs1:
- shard-iclb: [SKIP][75] ([fdo#112080]) -> [PASS][76] +4 similar issues
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb3/igt@perf_pmu@init-busy-vcs1.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb2/igt@perf_pmu@init-busy-vcs1.html
#### Warnings ####
* igt@gem_ctx_isolation@vcs1-nonpriv:
- shard-iclb: [FAIL][77] ([IGT#28]) -> [SKIP][78] ([fdo#112080])
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb1/igt@gem_ctx_isolation@vcs1-nonpriv.html
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb6/igt@gem_ctx_isolation@vcs1-nonpriv.html
* igt@gem_ctx_isolation@vcs1-nonpriv-switch:
- shard-iclb: [SKIP][79] ([fdo#112080]) -> [FAIL][80] ([IGT#28])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7865/shard-iclb8/igt@gem_ctx_isolation@vcs1-nonpriv-switch.html
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/shard-iclb1/igt@gem_ctx_isolation@vcs1-nonpriv-switch.html
[IGT#28]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/28
[fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841
[fdo#111732]: https://bugs.freedesktop.org/show_bug.cgi?id=111732
[fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
[fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146
[fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271
[i915#173]: https://gitlab.freedesktop.org/drm/intel/issues/173
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#189]: https://gitlab.freedesktop.org/drm/intel/issues/189
[i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31
[i915#34]: https://gitlab.freedesktop.org/drm/intel/issues/34
[i915#413]: https://gitlab.freedesktop.org/drm/intel/issues/413
[i915#447]: https://gitlab.freedesktop.org/drm/intel/issues/447
[i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
[i915#53]: https://gitlab.freedesktop.org/drm/intel/issues/53
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#61]: https://gitlab.freedesktop.org/drm/intel/issues/61
[i915#644]: https://gitlab.freedesktop.org/drm/intel/issues/644
[i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69
[i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816
[i915#899]: https://gitlab.freedesktop.org/drm/intel/issues/899
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7865 -> Patchwork_16407
CI-20190529: 20190529
CI_DRM_7865: ac0f322e67a6a466d0d09ba722957608c20cf339 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5417: 33cc93c8ba5daa0b7498f297a4f626844d895d06 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_16407: 1043d319d289332d435146c72ea5ac519c8225fe @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16407/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-02-06 22:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-04 0:02 [Intel-gfx] [PATCH] drm/i915/ehl: Check VBT before updating the transcoder for pipe Vivek Kasireddy
2020-02-04 10:50 ` Jani Nikula
2020-02-04 22:44 ` Vivek Kasireddy
2020-02-04 23:49 ` Souza, Jose
2020-02-04 22:13 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2020-02-06 22:50 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.