* [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
@ 2024-02-02 13:12 Gustavo Sousa
2024-02-02 18:47 ` ✓ Fi.CI.BAT: success for " Patchwork
` (2 more replies)
0 siblings, 3 replies; 13+ messages in thread
From: Gustavo Sousa @ 2024-02-02 13:12 UTC (permalink / raw)
To: intel-gfx; +Cc: gustavo.sousa
Looks like the name and description of intel_cdclk_needs_modeset()
became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
cdclk changes if possible"), when it became possible to update the cdclk
without requiring disabling the pipes when only changing the cd2x
divider was enough.
Later on we also added the same type of support with squash and crawling
with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
squasher to be reconfigured live") and commit d62686ba3b54
("drm/i915/adl_p: CDCLK crawl support for ADL").
As such, update that function's name and documentation to something more
appropriate, since the real checks for requiring modeset are done
elsewhere.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
One thing worth noting here is that, with this change, we are left with an
awkward situation where two function names related to checking changes in cdclk:
intel_cdclk_params_changed() and intel_cdclk_changed()
,
and I find it weird that we have intel_cdclk_changed(), which checks for the
voltage level as well. Shouldn't the voltage level be a function of cdclk and
ddi clock? Why do we need that?
drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
.../drm/i915/display/intel_display_power_well.c | 4 ++--
3 files changed, 11 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
index 26200ee3e23f..caadd880865f 100644
--- a/drivers/gpu/drm/i915/display/intel_cdclk.c
+++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
@@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
}
/**
- * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
- * configurations requires a modeset on all pipes
+ * intel_cdclk_params_changed - Check whether CDCLK parameters changed
* @a: first CDCLK configuration
* @b: second CDCLK configuration
*
* Returns:
- * True if changing between the two CDCLK configurations
- * requires all pipes to be off, false if not.
+ * True if parameters changed in a way that requires programming the CDCLK
+ * and False otherwise.
*/
-bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
- const struct intel_cdclk_config *b)
+bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
+ const struct intel_cdclk_config *b)
{
return a->cdclk != b->cdclk ||
a->vco != b->vco ||
@@ -2295,7 +2294,7 @@ static bool intel_cdclk_can_cd2x_update(struct drm_i915_private *dev_priv,
static bool intel_cdclk_changed(const struct intel_cdclk_config *a,
const struct intel_cdclk_config *b)
{
- return intel_cdclk_needs_modeset(a, b) ||
+ return intel_cdclk_params_changed(a, b) ||
a->voltage_level != b->voltage_level;
}
@@ -3202,7 +3201,7 @@ int intel_modeset_calc_cdclk(struct intel_atomic_state *state)
drm_dbg_kms(&dev_priv->drm,
"Can change cdclk cd2x divider with pipe %c active\n",
pipe_name(pipe));
- } else if (intel_cdclk_needs_modeset(&old_cdclk_state->actual,
+ } else if (intel_cdclk_params_changed(&old_cdclk_state->actual,
&new_cdclk_state->actual)) {
/* All pipes must be switched off while we change the cdclk. */
ret = intel_modeset_all_pipes_late(state, "CDCLK change");
diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.h b/drivers/gpu/drm/i915/display/intel_cdclk.h
index 48fd7d39e0cd..f8d3b2b3be6a 100644
--- a/drivers/gpu/drm/i915/display/intel_cdclk.h
+++ b/drivers/gpu/drm/i915/display/intel_cdclk.h
@@ -60,8 +60,8 @@ void intel_init_cdclk_hooks(struct drm_i915_private *dev_priv);
void intel_update_max_cdclk(struct drm_i915_private *dev_priv);
void intel_update_cdclk(struct drm_i915_private *dev_priv);
u32 intel_read_rawclk(struct drm_i915_private *dev_priv);
-bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
- const struct intel_cdclk_config *b);
+bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
+ const struct intel_cdclk_config *b);
void intel_set_cdclk_pre_plane_update(struct intel_atomic_state *state);
void intel_set_cdclk_post_plane_update(struct intel_atomic_state *state);
void intel_cdclk_dump_config(struct drm_i915_private *i915,
diff --git a/drivers/gpu/drm/i915/display/intel_display_power_well.c b/drivers/gpu/drm/i915/display/intel_display_power_well.c
index 47cd6bb04366..7364ca520764 100644
--- a/drivers/gpu/drm/i915/display/intel_display_power_well.c
+++ b/drivers/gpu/drm/i915/display/intel_display_power_well.c
@@ -968,8 +968,8 @@ void gen9_disable_dc_states(struct drm_i915_private *dev_priv)
intel_cdclk_get_cdclk(dev_priv, &cdclk_config);
/* Can't read out voltage_level so can't use intel_cdclk_changed() */
drm_WARN_ON(&dev_priv->drm,
- intel_cdclk_needs_modeset(&dev_priv->display.cdclk.hw,
- &cdclk_config));
+ intel_cdclk_params_changed(&dev_priv->display.cdclk.hw,
+ &cdclk_config));
gen9_assert_dbuf_enabled(dev_priv);
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-02 13:12 [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed Gustavo Sousa
@ 2024-02-02 18:47 ` Patchwork
2024-02-02 19:58 ` [PATCH] " Ville Syrjälä
2024-02-02 20:17 ` ✓ Fi.CI.IGT: success for " Patchwork
2 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2024-02-02 18:47 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 2279 bytes --]
== Series Details ==
Series: drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
URL : https://patchwork.freedesktop.org/series/129467/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14212 -> Patchwork_129467v1
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/index.html
Participating hosts (38 -> 37)
------------------------------
Additional (1): fi-bsw-n3050
Missing (2): bat-jsl-1 fi-snb-2520m
Known issues
------------
Here are the changes found in Patchwork_129467v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_lmem_swapping@random-engines:
- fi-bsw-n3050: NOTRUN -> [SKIP][1] ([fdo#109271]) +15 other tests skip
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/fi-bsw-n3050/igt@gem_lmem_swapping@random-engines.html
#### Possible fixes ####
* igt@i915_selftest@live@gt_pm:
- bat-adln-1: [DMESG-FAIL][2] ([i915#10010]) -> [PASS][3]
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/bat-adln-1/igt@i915_selftest@live@gt_pm.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/bat-adln-1/igt@i915_selftest@live@gt_pm.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#10010]: https://gitlab.freedesktop.org/drm/intel/issues/10010
Build changes
-------------
* Linux: CI_DRM_14212 -> Patchwork_129467v1
CI-20190529: 20190529
CI_DRM_14212: 1dd92467500a5ead3e44bbdfe15e064dd79b65ef @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7702: bf075a74ece1956fc0e554291591b9da3eab54cf @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_129467v1: 1dd92467500a5ead3e44bbdfe15e064dd79b65ef @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
a15a92ce56ce drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/index.html
[-- Attachment #2: Type: text/html, Size: 2913 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-02 13:12 [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed Gustavo Sousa
2024-02-02 18:47 ` ✓ Fi.CI.BAT: success for " Patchwork
@ 2024-02-02 19:58 ` Ville Syrjälä
2024-02-02 20:06 ` Ville Syrjälä
2024-02-03 13:25 ` Gustavo Sousa
2024-02-02 20:17 ` ✓ Fi.CI.IGT: success for " Patchwork
2 siblings, 2 replies; 13+ messages in thread
From: Ville Syrjälä @ 2024-02-02 19:58 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
> Looks like the name and description of intel_cdclk_needs_modeset()
> became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
> cdclk changes if possible"), when it became possible to update the cdclk
> without requiring disabling the pipes when only changing the cd2x
> divider was enough.
>
> Later on we also added the same type of support with squash and crawling
> with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
> when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
> squasher to be reconfigured live") and commit d62686ba3b54
> ("drm/i915/adl_p: CDCLK crawl support for ADL").
>
> As such, update that function's name and documentation to something more
> appropriate, since the real checks for requiring modeset are done
> elsewhere.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
>
> One thing worth noting here is that, with this change, we are left with an
> awkward situation where two function names related to checking changes in cdclk:
>
> intel_cdclk_params_changed() and intel_cdclk_changed()
>
> ,
>
> and I find it weird that we have intel_cdclk_changed(), which checks for the
> voltage level as well. Shouldn't the voltage level be a function of cdclk and
> ddi clock? Why do we need that?
>
> drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
> drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
> .../drm/i915/display/intel_display_power_well.c | 4 ++--
> 3 files changed, 11 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> index 26200ee3e23f..caadd880865f 100644
> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
> }
>
> /**
> - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
> - * configurations requires a modeset on all pipes
> + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
> * @a: first CDCLK configuration
> * @b: second CDCLK configuration
> *
> * Returns:
> - * True if changing between the two CDCLK configurations
> - * requires all pipes to be off, false if not.
> + * True if parameters changed in a way that requires programming the CDCLK
> + * and False otherwise.
> */
> -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
> - const struct intel_cdclk_config *b)
> +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
> + const struct intel_cdclk_config *b)
The new name isn't very descriptive either.
Outside the cd2x/crawl/squash cases we stil have to consider
two cases:
1. cdclk frequency/pll changes (voltage level can change or not)
2. cdclk frequency/pll doesn't change, but voltage level needs to change
And that difference is what intel_cdclk_needs_modeset() is trying
convey. And intel_cdclk_changed() tells us whether anything at all
is changing.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-02 19:58 ` [PATCH] " Ville Syrjälä
@ 2024-02-02 20:06 ` Ville Syrjälä
2024-02-03 13:32 ` Gustavo Sousa
2024-02-03 13:25 ` Gustavo Sousa
1 sibling, 1 reply; 13+ messages in thread
From: Ville Syrjälä @ 2024-02-02 20:06 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
On Fri, Feb 02, 2024 at 09:58:37PM +0200, Ville Syrjälä wrote:
> On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
> > Looks like the name and description of intel_cdclk_needs_modeset()
> > became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
> > cdclk changes if possible"), when it became possible to update the cdclk
> > without requiring disabling the pipes when only changing the cd2x
> > divider was enough.
> >
> > Later on we also added the same type of support with squash and crawling
> > with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
> > when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
> > squasher to be reconfigured live") and commit d62686ba3b54
> > ("drm/i915/adl_p: CDCLK crawl support for ADL").
> >
> > As such, update that function's name and documentation to something more
> > appropriate, since the real checks for requiring modeset are done
> > elsewhere.
> >
> > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> > ---
> >
> > One thing worth noting here is that, with this change, we are left with an
> > awkward situation where two function names related to checking changes in cdclk:
> >
> > intel_cdclk_params_changed() and intel_cdclk_changed()
> >
> > ,
> >
> > and I find it weird that we have intel_cdclk_changed(), which checks for the
> > voltage level as well. Shouldn't the voltage level be a function of cdclk and
> > ddi clock? Why do we need that?
> >
> > drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
> > drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
> > .../drm/i915/display/intel_display_power_well.c | 4 ++--
> > 3 files changed, 11 insertions(+), 12 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> > index 26200ee3e23f..caadd880865f 100644
> > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> > @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
> > }
> >
> > /**
> > - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
> > - * configurations requires a modeset on all pipes
> > + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
> > * @a: first CDCLK configuration
> > * @b: second CDCLK configuration
> > *
> > * Returns:
> > - * True if changing between the two CDCLK configurations
> > - * requires all pipes to be off, false if not.
> > + * True if parameters changed in a way that requires programming the CDCLK
> > + * and False otherwise.
> > */
> > -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
> > - const struct intel_cdclk_config *b)
> > +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
> > + const struct intel_cdclk_config *b)
>
> The new name isn't very descriptive either.
>
> Outside the cd2x/crawl/squash cases we stil have to consider
> two cases:
> 1. cdclk frequency/pll changes (voltage level can change or not)
> 2. cdclk frequency/pll doesn't change, but voltage level needs to change
>
> And that difference is what intel_cdclk_needs_modeset() is trying
> convey. And intel_cdclk_changed() tells us whether anything at all
> is changing.
intel_cdclk_clock_changed() is perhaps the best name I can
think of, off the top of my head. But I'm notoriously bad at
naming things so take that with a pinch of salt.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 13+ messages in thread
* ✓ Fi.CI.IGT: success for drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-02 13:12 [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed Gustavo Sousa
2024-02-02 18:47 ` ✓ Fi.CI.BAT: success for " Patchwork
2024-02-02 19:58 ` [PATCH] " Ville Syrjälä
@ 2024-02-02 20:17 ` Patchwork
2 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2024-02-02 20:17 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 84691 bytes --]
== Series Details ==
Series: drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
URL : https://patchwork.freedesktop.org/series/129467/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_14212_full -> Patchwork_129467v1_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (8 -> 8)
------------------------------
No changes in participating hosts
Known issues
------------
Here are the changes found in Patchwork_129467v1_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@api_intel_bb@object-reloc-purge-cache:
- shard-mtlp: NOTRUN -> [SKIP][1] ([i915#8411])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@api_intel_bb@object-reloc-purge-cache.html
- shard-dg2: NOTRUN -> [SKIP][2] ([i915#8411]) +2 other tests skip
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@api_intel_bb@object-reloc-purge-cache.html
* igt@device_reset@cold-reset-bound:
- shard-dg2: NOTRUN -> [SKIP][3] ([i915#7701])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@device_reset@cold-reset-bound.html
* igt@device_reset@unbind-cold-reset-rebind:
- shard-rkl: NOTRUN -> [SKIP][4] ([i915#7701])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@device_reset@unbind-cold-reset-rebind.html
* igt@drm_buddy@drm_buddy@drm_test_buddy_alloc_limit:
- shard-glk: NOTRUN -> [DMESG-WARN][5] ([i915#10140])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk9/igt@drm_buddy@drm_buddy@drm_test_buddy_alloc_limit.html
* igt@drm_fdinfo@busy@rcs0:
- shard-dg2: NOTRUN -> [SKIP][6] ([i915#8414]) +22 other tests skip
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@drm_fdinfo@busy@rcs0.html
* igt@drm_fdinfo@most-busy-idle-check-all@rcs0:
- shard-rkl: [PASS][7] -> [FAIL][8] ([i915#7742])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-3/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-4/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html
* igt@gem_ccs@suspend-resume@xmajor-compressed-compfmt0-lmem0-lmem0:
- shard-dg2: NOTRUN -> [INCOMPLETE][9] ([i915#10137] / [i915#7297])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@gem_ccs@suspend-resume@xmajor-compressed-compfmt0-lmem0-lmem0.html
* igt@gem_close_race@multigpu-basic-threads:
- shard-dg2: NOTRUN -> [SKIP][10] ([i915#7697]) +1 other test skip
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_close_race@multigpu-basic-threads.html
- shard-mtlp: NOTRUN -> [SKIP][11] ([i915#7697])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_close_race@multigpu-basic-threads.html
* igt@gem_create@create-ext-set-pat:
- shard-dg2: NOTRUN -> [SKIP][12] ([i915#8562])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@gem_create@create-ext-set-pat.html
* igt@gem_ctx_exec@basic-nohangcheck:
- shard-tglu: [PASS][13] -> [FAIL][14] ([i915#6268])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-2/igt@gem_ctx_exec@basic-nohangcheck.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-10/igt@gem_ctx_exec@basic-nohangcheck.html
* igt@gem_ctx_persistence@heartbeat-hostile:
- shard-dg2: NOTRUN -> [SKIP][15] ([i915#8555]) +2 other tests skip
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@gem_ctx_persistence@heartbeat-hostile.html
* igt@gem_ctx_sseu@invalid-args:
- shard-dg2: NOTRUN -> [SKIP][16] ([i915#280])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_ctx_sseu@invalid-args.html
* igt@gem_exec_balancer@bonded-sync:
- shard-dg2: NOTRUN -> [SKIP][17] ([i915#4771])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@gem_exec_balancer@bonded-sync.html
* igt@gem_exec_balancer@bonded-true-hang:
- shard-dg2: NOTRUN -> [SKIP][18] ([i915#4812]) +2 other tests skip
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@gem_exec_balancer@bonded-true-hang.html
* igt@gem_exec_balancer@noheartbeat:
- shard-dg1: NOTRUN -> [SKIP][19] ([i915#8555])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@gem_exec_balancer@noheartbeat.html
* igt@gem_exec_balancer@parallel-bb-first:
- shard-rkl: NOTRUN -> [SKIP][20] ([i915#4525])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@gem_exec_balancer@parallel-bb-first.html
* igt@gem_exec_capture@many-4k-incremental:
- shard-dg2: NOTRUN -> [FAIL][21] ([i915#9606])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@gem_exec_capture@many-4k-incremental.html
- shard-rkl: NOTRUN -> [FAIL][22] ([i915#9606])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@gem_exec_capture@many-4k-incremental.html
* igt@gem_exec_capture@pi@vcs1:
- shard-mtlp: [PASS][23] -> [FAIL][24] ([i915#4475] / [i915#7765])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-7/igt@gem_exec_capture@pi@vcs1.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-4/igt@gem_exec_capture@pi@vcs1.html
* igt@gem_exec_capture@pi@vecs0:
- shard-mtlp: [PASS][25] -> [DMESG-WARN][26] ([i915#5591])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-7/igt@gem_exec_capture@pi@vecs0.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-4/igt@gem_exec_capture@pi@vecs0.html
* igt@gem_exec_endless@dispatch@rcs0:
- shard-tglu: [PASS][27] -> [TIMEOUT][28] ([i915#3778] / [i915#7392])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-3/igt@gem_exec_endless@dispatch@rcs0.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-2/igt@gem_exec_endless@dispatch@rcs0.html
* igt@gem_exec_fair@basic-none-rrul@rcs0:
- shard-glk: NOTRUN -> [FAIL][29] ([i915#2842])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk8/igt@gem_exec_fair@basic-none-rrul@rcs0.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-rkl: [PASS][30] -> [FAIL][31] ([i915#2842])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-3/igt@gem_exec_fair@basic-none-share@rcs0.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-pace-solo:
- shard-dg2: NOTRUN -> [SKIP][32] ([i915#3539]) +1 other test skip
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_exec_fair@basic-pace-solo.html
* igt@gem_exec_flush@basic-uc-pro-default:
- shard-dg2: NOTRUN -> [SKIP][33] ([i915#3539] / [i915#4852]) +7 other tests skip
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_exec_flush@basic-uc-pro-default.html
* igt@gem_exec_gttfill@multigpu-basic:
- shard-dg1: NOTRUN -> [SKIP][34] ([i915#7697])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@gem_exec_gttfill@multigpu-basic.html
* igt@gem_exec_params@rsvd2-dirt:
- shard-dg2: NOTRUN -> [SKIP][35] ([fdo#109283] / [i915#5107])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_exec_params@rsvd2-dirt.html
* igt@gem_exec_reloc@basic-cpu-gtt-noreloc:
- shard-dg2: NOTRUN -> [SKIP][36] ([i915#3281]) +18 other tests skip
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_exec_reloc@basic-cpu-gtt-noreloc.html
* igt@gem_exec_reloc@basic-range-active:
- shard-mtlp: NOTRUN -> [SKIP][37] ([i915#3281]) +2 other tests skip
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_exec_reloc@basic-range-active.html
* igt@gem_exec_reloc@basic-write-cpu:
- shard-rkl: NOTRUN -> [SKIP][38] ([i915#3281]) +2 other tests skip
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@gem_exec_reloc@basic-write-cpu.html
* igt@gem_exec_schedule@reorder-wide:
- shard-dg2: NOTRUN -> [SKIP][39] ([i915#4537] / [i915#4812]) +2 other tests skip
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@gem_exec_schedule@reorder-wide.html
* igt@gem_exec_schedule@u-fairslice:
- shard-snb: NOTRUN -> [SKIP][40] ([fdo#109271]) +53 other tests skip
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb4/igt@gem_exec_schedule@u-fairslice.html
* igt@gem_exec_suspend@basic-s4-devices@lmem0:
- shard-dg2: NOTRUN -> [ABORT][41] ([i915#7975] / [i915#8213])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@gem_exec_suspend@basic-s4-devices@lmem0.html
* igt@gem_exec_suspend@basic-s4-devices@smem:
- shard-rkl: NOTRUN -> [ABORT][42] ([i915#7975] / [i915#8213])
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@gem_exec_suspend@basic-s4-devices@smem.html
* igt@gem_fenced_exec_thrash@no-spare-fences-busy-interruptible:
- shard-dg2: NOTRUN -> [SKIP][43] ([i915#4860]) +1 other test skip
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_fenced_exec_thrash@no-spare-fences-busy-interruptible.html
* igt@gem_lmem_swapping@heavy-random:
- shard-glk: NOTRUN -> [SKIP][44] ([fdo#109271] / [i915#4613]) +1 other test skip
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk8/igt@gem_lmem_swapping@heavy-random.html
* igt@gem_lmem_swapping@random-engines:
- shard-mtlp: NOTRUN -> [SKIP][45] ([i915#4613])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_lmem_swapping@random-engines.html
* igt@gem_madvise@dontneed-before-pwrite:
- shard-mtlp: NOTRUN -> [SKIP][46] ([i915#3282])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_madvise@dontneed-before-pwrite.html
* igt@gem_mmap@basic:
- shard-dg1: NOTRUN -> [SKIP][47] ([i915#4083])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@gem_mmap@basic.html
* igt@gem_mmap@big-bo:
- shard-mtlp: NOTRUN -> [SKIP][48] ([i915#4083])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_mmap@big-bo.html
* igt@gem_mmap_gtt@basic-small-bo:
- shard-dg2: NOTRUN -> [SKIP][49] ([i915#4077]) +21 other tests skip
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_mmap_gtt@basic-small-bo.html
* igt@gem_mmap_gtt@big-copy-odd:
- shard-mtlp: NOTRUN -> [SKIP][50] ([i915#4077]) +1 other test skip
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_mmap_gtt@big-copy-odd.html
* igt@gem_mmap_wc@bad-size:
- shard-dg2: NOTRUN -> [SKIP][51] ([i915#4083]) +6 other tests skip
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_mmap_wc@bad-size.html
* igt@gem_partial_pwrite_pread@reads-uncached:
- shard-dg2: NOTRUN -> [SKIP][52] ([i915#3282]) +7 other tests skip
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_partial_pwrite_pread@reads-uncached.html
* igt@gem_partial_pwrite_pread@writes-after-reads-display:
- shard-rkl: NOTRUN -> [SKIP][53] ([i915#3282]) +3 other tests skip
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@gem_partial_pwrite_pread@writes-after-reads-display.html
* igt@gem_pwrite@basic-exhaustion:
- shard-glk: NOTRUN -> [WARN][54] ([i915#2658])
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk9/igt@gem_pwrite@basic-exhaustion.html
* igt@gem_pxp@create-regular-context-1:
- shard-mtlp: NOTRUN -> [SKIP][55] ([i915#4270]) +1 other test skip
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gem_pxp@create-regular-context-1.html
* igt@gem_pxp@dmabuf-shared-protected-dst-is-context-refcounted:
- shard-dg2: NOTRUN -> [SKIP][56] ([i915#4270]) +9 other tests skip
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_pxp@dmabuf-shared-protected-dst-is-context-refcounted.html
* igt@gem_pxp@protected-encrypted-src-copy-not-readible:
- shard-rkl: NOTRUN -> [SKIP][57] ([i915#4270]) +2 other tests skip
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@gem_pxp@protected-encrypted-src-copy-not-readible.html
* igt@gem_render_copy@y-tiled-ccs-to-y-tiled:
- shard-dg2: NOTRUN -> [SKIP][58] ([i915#5190]) +9 other tests skip
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_render_copy@y-tiled-ccs-to-y-tiled.html
* igt@gem_set_tiling_vs_blt@tiled-to-untiled:
- shard-dg2: NOTRUN -> [SKIP][59] ([i915#4079]) +3 other tests skip
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_set_tiling_vs_blt@tiled-to-untiled.html
* igt@gem_softpin@evict-snoop-interruptible:
- shard-dg2: NOTRUN -> [SKIP][60] ([i915#4885])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@gem_softpin@evict-snoop-interruptible.html
- shard-rkl: NOTRUN -> [SKIP][61] ([fdo#109312])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@gem_softpin@evict-snoop-interruptible.html
* igt@gem_userptr_blits@create-destroy-unsync:
- shard-dg2: NOTRUN -> [SKIP][62] ([i915#3297]) +3 other tests skip
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@gem_userptr_blits@create-destroy-unsync.html
* igt@gem_userptr_blits@forbidden-operations:
- shard-dg2: NOTRUN -> [SKIP][63] ([i915#3282] / [i915#3297])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gem_userptr_blits@forbidden-operations.html
* igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy:
- shard-dg2: NOTRUN -> [SKIP][64] ([i915#3297] / [i915#4880]) +1 other test skip
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html
* igt@gem_userptr_blits@unsync-unmap:
- shard-rkl: NOTRUN -> [SKIP][65] ([i915#3297])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@gem_userptr_blits@unsync-unmap.html
* igt@gem_workarounds@suspend-resume-fd:
- shard-tglu: [PASS][66] -> [ABORT][67] ([i915#8213])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-9/igt@gem_workarounds@suspend-resume-fd.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-9/igt@gem_workarounds@suspend-resume-fd.html
* igt@gen7_exec_parse@basic-allowed:
- shard-mtlp: NOTRUN -> [SKIP][68] ([fdo#109289]) +1 other test skip
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gen7_exec_parse@basic-allowed.html
* igt@gen7_exec_parse@chained-batch:
- shard-rkl: NOTRUN -> [SKIP][69] ([fdo#109289])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@gen7_exec_parse@chained-batch.html
* igt@gen9_exec_parse@batch-invalid-length:
- shard-rkl: NOTRUN -> [SKIP][70] ([i915#2527]) +1 other test skip
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@gen9_exec_parse@batch-invalid-length.html
* igt@gen9_exec_parse@bb-start-cmd:
- shard-mtlp: NOTRUN -> [SKIP][71] ([i915#2856])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@gen9_exec_parse@bb-start-cmd.html
* igt@gen9_exec_parse@unaligned-access:
- shard-dg2: NOTRUN -> [SKIP][72] ([i915#2856]) +8 other tests skip
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@gen9_exec_parse@unaligned-access.html
* igt@i915_fb_tiling:
- shard-dg2: NOTRUN -> [SKIP][73] ([i915#4881])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@i915_fb_tiling.html
* igt@i915_module_load@reload-with-fault-injection:
- shard-dg2: NOTRUN -> [INCOMPLETE][74] ([i915#10137] / [i915#9820] / [i915#9849])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pipe_stress@stress-xrgb8888-ytiled:
- shard-dg2: NOTRUN -> [SKIP][75] ([i915#7091])
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@i915_pipe_stress@stress-xrgb8888-ytiled.html
* igt@i915_pm_rc6_residency@rc6-idle@gt0-rcs0:
- shard-dg1: [PASS][76] -> [FAIL][77] ([i915#3591])
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-13/igt@i915_pm_rc6_residency@rc6-idle@gt0-rcs0.html
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-16/igt@i915_pm_rc6_residency@rc6-idle@gt0-rcs0.html
* igt@i915_pm_rpm@gem-execbuf-stress-pc8:
- shard-dg2: NOTRUN -> [SKIP][78] ([fdo#109293] / [fdo#109506])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html
* igt@i915_pm_rps@basic-api:
- shard-mtlp: NOTRUN -> [SKIP][79] ([i915#6621])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@i915_pm_rps@basic-api.html
* igt@i915_pm_rps@min-max-config-loaded:
- shard-dg2: NOTRUN -> [SKIP][80] ([i915#6621]) +1 other test skip
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@i915_pm_rps@min-max-config-loaded.html
* igt@i915_pm_rps@thresholds-park@gt0:
- shard-mtlp: NOTRUN -> [SKIP][81] ([i915#8925])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@i915_pm_rps@thresholds-park@gt0.html
- shard-dg2: NOTRUN -> [SKIP][82] ([i915#8925])
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@i915_pm_rps@thresholds-park@gt0.html
* igt@i915_pm_rps@thresholds-park@gt1:
- shard-mtlp: NOTRUN -> [SKIP][83] ([i915#3555] / [i915#8925])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@i915_pm_rps@thresholds-park@gt1.html
* igt@i915_pm_sseu@full-enable:
- shard-dg2: NOTRUN -> [SKIP][84] ([i915#4387])
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@i915_pm_sseu@full-enable.html
* igt@i915_power@sanity:
- shard-mtlp: [PASS][85] -> [SKIP][86] ([i915#7984])
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-3/igt@i915_power@sanity.html
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-8/igt@i915_power@sanity.html
* igt@i915_query@query-topology-coherent-slice-mask:
- shard-dg2: NOTRUN -> [SKIP][87] ([i915#6188])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@i915_query@query-topology-coherent-slice-mask.html
* igt@kms_addfb_basic@basic-x-tiled-legacy:
- shard-dg2: NOTRUN -> [SKIP][88] ([i915#4212]) +1 other test skip
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_addfb_basic@basic-x-tiled-legacy.html
* igt@kms_addfb_basic@basic-y-tiled-legacy:
- shard-dg2: NOTRUN -> [SKIP][89] ([i915#4215] / [i915#5190])
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_addfb_basic@basic-y-tiled-legacy.html
* igt@kms_async_flips@invalid-async-flip:
- shard-dg2: NOTRUN -> [SKIP][90] ([i915#6228])
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_async_flips@invalid-async-flip.html
* igt@kms_atomic@plane-primary-overlay-mutable-zpos:
- shard-dg2: NOTRUN -> [SKIP][91] ([i915#9531])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_atomic@plane-primary-overlay-mutable-zpos.html
* igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels:
- shard-dg2: NOTRUN -> [SKIP][92] ([i915#1769] / [i915#3555]) +1 other test skip
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels.html
* igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-0:
- shard-rkl: NOTRUN -> [SKIP][93] ([i915#5286]) +3 other tests skip
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-0.html
* igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-async-flip:
- shard-dg1: NOTRUN -> [SKIP][94] ([i915#4538] / [i915#5286])
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-async-flip.html
* igt@kms_big_fb@linear-64bpp-rotate-90:
- shard-mtlp: NOTRUN -> [SKIP][95] ([fdo#111614])
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_big_fb@linear-64bpp-rotate-90.html
* igt@kms_big_fb@x-tiled-32bpp-rotate-270:
- shard-dg2: NOTRUN -> [SKIP][96] ([fdo#111614]) +6 other tests skip
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_big_fb@x-tiled-32bpp-rotate-270.html
- shard-rkl: NOTRUN -> [SKIP][97] ([fdo#111614] / [i915#3638]) +1 other test skip
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_big_fb@x-tiled-32bpp-rotate-270.html
* igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
- shard-tglu: [PASS][98] -> [FAIL][99] ([i915#3743])
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-8/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-6/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
* igt@kms_big_fb@yf-tiled-32bpp-rotate-180:
- shard-rkl: NOTRUN -> [SKIP][100] ([fdo#110723]) +2 other tests skip
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_big_fb@yf-tiled-32bpp-rotate-180.html
* igt@kms_big_fb@yf-tiled-32bpp-rotate-90:
- shard-dg1: NOTRUN -> [SKIP][101] ([i915#4538])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_big_fb@yf-tiled-32bpp-rotate-90.html
* igt@kms_big_fb@yf-tiled-64bpp-rotate-0:
- shard-dg2: NOTRUN -> [SKIP][102] ([i915#4538] / [i915#5190]) +15 other tests skip
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_big_fb@yf-tiled-64bpp-rotate-0.html
* igt@kms_big_fb@yf-tiled-addfb:
- shard-mtlp: NOTRUN -> [SKIP][103] ([i915#6187])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_big_fb@yf-tiled-addfb.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180:
- shard-mtlp: NOTRUN -> [SKIP][104] ([fdo#111615]) +1 other test skip
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180.html
* igt@kms_big_joiner@invalid-modeset:
- shard-rkl: NOTRUN -> [SKIP][105] ([i915#2705])
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_big_joiner@invalid-modeset.html
* igt@kms_ccs@pipe-a-crc-primary-rotation-180-4-tiled-mtl-rc-ccs:
- shard-dg2: NOTRUN -> [SKIP][106] ([i915#5354]) +129 other tests skip
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_ccs@pipe-a-crc-primary-rotation-180-4-tiled-mtl-rc-ccs.html
* igt@kms_ccs@pipe-b-bad-pixel-format-y-tiled-gen12-mc-ccs:
- shard-dg1: NOTRUN -> [SKIP][107] ([i915#5354] / [i915#6095]) +6 other tests skip
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_ccs@pipe-b-bad-pixel-format-y-tiled-gen12-mc-ccs.html
* igt@kms_ccs@pipe-b-crc-primary-rotation-180-4-tiled-dg2-rc-ccs:
- shard-rkl: NOTRUN -> [SKIP][108] ([i915#5354] / [i915#6095]) +9 other tests skip
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_ccs@pipe-b-crc-primary-rotation-180-4-tiled-dg2-rc-ccs.html
* igt@kms_ccs@pipe-c-bad-rotation-90-yf-tiled-ccs:
- shard-mtlp: NOTRUN -> [SKIP][109] ([i915#5354] / [i915#6095]) +10 other tests skip
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_ccs@pipe-c-bad-rotation-90-yf-tiled-ccs.html
* igt@kms_ccs@pipe-d-random-ccs-data-y-tiled-gen12-rc-ccs-cc:
- shard-rkl: NOTRUN -> [SKIP][110] ([i915#5354]) +10 other tests skip
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_ccs@pipe-d-random-ccs-data-y-tiled-gen12-rc-ccs-cc.html
* igt@kms_chamelium_audio@hdmi-audio:
- shard-dg2: NOTRUN -> [SKIP][111] ([i915#7828]) +14 other tests skip
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_chamelium_audio@hdmi-audio.html
* igt@kms_chamelium_color@ctm-blue-to-red:
- shard-dg2: NOTRUN -> [SKIP][112] ([fdo#111827]) +2 other tests skip
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_chamelium_color@ctm-blue-to-red.html
* igt@kms_chamelium_frames@hdmi-cmp-planar-formats:
- shard-dg1: NOTRUN -> [SKIP][113] ([i915#7828])
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_chamelium_frames@hdmi-cmp-planar-formats.html
* igt@kms_chamelium_hpd@dp-hpd:
- shard-rkl: NOTRUN -> [SKIP][114] ([i915#7828]) +3 other tests skip
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_chamelium_hpd@dp-hpd.html
* igt@kms_chamelium_hpd@hdmi-hpd-enable-disable-mode:
- shard-mtlp: NOTRUN -> [SKIP][115] ([i915#7828]) +2 other tests skip
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_chamelium_hpd@hdmi-hpd-enable-disable-mode.html
* igt@kms_content_protection@dp-mst-type-1:
- shard-mtlp: NOTRUN -> [SKIP][116] ([i915#3299])
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_content_protection@dp-mst-type-1.html
- shard-dg2: NOTRUN -> [SKIP][117] ([i915#3299])
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_content_protection@dp-mst-type-1.html
* igt@kms_content_protection@uevent:
- shard-dg2: NOTRUN -> [SKIP][118] ([i915#7118]) +1 other test skip
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_content_protection@uevent.html
* igt@kms_cursor_crc@cursor-onscreen-512x512:
- shard-rkl: NOTRUN -> [SKIP][119] ([i915#3359])
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_cursor_crc@cursor-onscreen-512x512.html
* igt@kms_cursor_crc@cursor-random-256x85:
- shard-mtlp: NOTRUN -> [SKIP][120] ([i915#8814])
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_cursor_crc@cursor-random-256x85.html
* igt@kms_cursor_crc@cursor-random-512x170:
- shard-dg2: NOTRUN -> [SKIP][121] ([i915#3359]) +3 other tests skip
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_cursor_crc@cursor-random-512x170.html
* igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic:
- shard-dg2: NOTRUN -> [SKIP][122] ([fdo#109274] / [i915#5354]) +9 other tests skip
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions:
- shard-snb: NOTRUN -> [SKIP][123] ([fdo#109271] / [fdo#111767])
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb6/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-toggle:
- shard-rkl: NOTRUN -> [SKIP][124] ([fdo#111767] / [fdo#111825])
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_cursor_legacy@cursorb-vs-flipb-toggle.html
- shard-dg2: NOTRUN -> [SKIP][125] ([fdo#109274] / [fdo#111767] / [i915#5354])
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_cursor_legacy@cursorb-vs-flipb-toggle.html
* igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot:
- shard-mtlp: NOTRUN -> [SKIP][126] ([i915#9067])
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot.html
- shard-dg2: NOTRUN -> [SKIP][127] ([i915#9067])
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot.html
* igt@kms_cursor_legacy@torture-move@pipe-a:
- shard-snb: [PASS][128] -> [DMESG-WARN][129] ([i915#10166])
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb1/igt@kms_cursor_legacy@torture-move@pipe-a.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb7/igt@kms_cursor_legacy@torture-move@pipe-a.html
* igt@kms_dirtyfb@fbc-dirtyfb-ioctl@a-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][130] ([fdo#110189] / [i915#9227])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_dirtyfb@fbc-dirtyfb-ioctl@a-hdmi-a-3.html
* igt@kms_dirtyfb@fbc-dirtyfb-ioctl@a-hdmi-a-4:
- shard-dg1: NOTRUN -> [SKIP][131] ([fdo#110189] / [i915#9723])
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-16/igt@kms_dirtyfb@fbc-dirtyfb-ioctl@a-hdmi-a-4.html
* igt@kms_display_modes@extended-mode-basic:
- shard-dg2: NOTRUN -> [SKIP][132] ([i915#3555]) +8 other tests skip
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_display_modes@extended-mode-basic.html
- shard-rkl: NOTRUN -> [SKIP][133] ([i915#3555]) +2 other tests skip
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_display_modes@extended-mode-basic.html
* igt@kms_draw_crc@draw-method-mmap-gtt:
- shard-dg2: NOTRUN -> [SKIP][134] ([i915#8812])
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_draw_crc@draw-method-mmap-gtt.html
* igt@kms_dsc@dsc-fractional-bpp:
- shard-dg2: NOTRUN -> [SKIP][135] ([i915#3840] / [i915#9688])
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_dsc@dsc-fractional-bpp.html
- shard-mtlp: NOTRUN -> [SKIP][136] ([i915#3840] / [i915#9688])
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_dsc@dsc-fractional-bpp.html
* igt@kms_dsc@dsc-with-bpc:
- shard-dg2: NOTRUN -> [SKIP][137] ([i915#3555] / [i915#3840])
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_dsc@dsc-with-bpc.html
* igt@kms_feature_discovery@display-2x:
- shard-dg2: NOTRUN -> [SKIP][138] ([i915#1839])
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_feature_discovery@display-2x.html
* igt@kms_feature_discovery@dp-mst:
- shard-dg2: NOTRUN -> [SKIP][139] ([i915#9337])
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_feature_discovery@dp-mst.html
- shard-mtlp: NOTRUN -> [SKIP][140] ([i915#9337])
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_feature_discovery@dp-mst.html
* igt@kms_feature_discovery@psr2:
- shard-dg2: NOTRUN -> [SKIP][141] ([i915#658])
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_feature_discovery@psr2.html
* igt@kms_flip@2x-absolute-wf_vblank:
- shard-dg1: NOTRUN -> [SKIP][142] ([fdo#111825] / [i915#9934])
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_flip@2x-absolute-wf_vblank.html
* igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
- shard-dg2: NOTRUN -> [SKIP][143] ([fdo#109274] / [fdo#111767])
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@2x-modeset-vs-vblank-race:
- shard-dg2: NOTRUN -> [SKIP][144] ([fdo#109274]) +15 other tests skip
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_flip@2x-modeset-vs-vblank-race.html
* igt@kms_flip@2x-wf_vblank-ts-check:
- shard-mtlp: NOTRUN -> [SKIP][145] ([i915#3637])
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_flip@2x-wf_vblank-ts-check.html
* igt@kms_flip@flip-vs-fences-interruptible:
- shard-dg2: NOTRUN -> [SKIP][146] ([i915#8381]) +2 other tests skip
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_flip@flip-vs-fences-interruptible.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling@pipe-a-valid-mode:
- shard-dg2: NOTRUN -> [SKIP][147] ([i915#2672]) +6 other tests skip
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-upscaling@pipe-a-valid-mode:
- shard-rkl: NOTRUN -> [SKIP][148] ([i915#2672]) +1 other test skip
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-upscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-linear-to-16bpp-linear-downscaling@pipe-a-default-mode:
- shard-mtlp: NOTRUN -> [SKIP][149] ([i915#3555] / [i915#8810])
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-16bpp-linear-downscaling@pipe-a-default-mode.html
* igt@kms_force_connector_basic@force-load-detect:
- shard-rkl: NOTRUN -> [SKIP][150] ([fdo#109285])
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-wc:
- shard-dg2: NOTRUN -> [SKIP][151] ([i915#8708]) +22 other tests skip
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-blt:
- shard-dg1: NOTRUN -> [SKIP][152] ([fdo#111767] / [fdo#111825])
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-wc:
- shard-snb: [PASS][153] -> [SKIP][154] ([fdo#109271]) +14 other tests skip
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb7/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-wc.html
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb1/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-indfb-fliptrack-mmap-gtt:
- shard-mtlp: NOTRUN -> [SKIP][155] ([i915#8708])
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_frontbuffer_tracking@fbcpsr-1p-indfb-fliptrack-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-pri-shrfb-draw-mmap-wc:
- shard-rkl: NOTRUN -> [SKIP][156] ([fdo#111825] / [i915#1825]) +14 other tests skip
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-pri-shrfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt:
- shard-rkl: NOTRUN -> [SKIP][157] ([fdo#111825]) +6 other tests skip
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-tiling-4:
- shard-dg1: NOTRUN -> [SKIP][158] ([i915#5439])
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_frontbuffer_tracking@fbcpsr-tiling-4.html
* igt@kms_frontbuffer_tracking@plane-fbc-rte:
- shard-dg1: NOTRUN -> [SKIP][159] ([i915#10070])
[159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_frontbuffer_tracking@plane-fbc-rte.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-mmap-gtt:
- shard-dg1: NOTRUN -> [SKIP][160] ([i915#8708])
[160]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-mmap-gtt:
- shard-glk: NOTRUN -> [SKIP][161] ([fdo#109271]) +135 other tests skip
[161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk9/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-pgflip-blt:
- shard-dg1: NOTRUN -> [SKIP][162] ([i915#3458]) +1 other test skip
[162]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-msflip-blt:
- shard-mtlp: NOTRUN -> [SKIP][163] ([i915#1825]) +12 other tests skip
[163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-msflip-blt.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-draw-blt:
- shard-dg1: NOTRUN -> [SKIP][164] ([fdo#111825]) +2 other tests skip
[164]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-draw-render:
- shard-rkl: NOTRUN -> [SKIP][165] ([fdo#111767] / [fdo#111825] / [i915#1825])
[165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-onoff:
- shard-dg2: NOTRUN -> [SKIP][166] ([fdo#111767] / [i915#5354]) +2 other tests skip
[166]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-onoff.html
* igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary:
- shard-dg2: NOTRUN -> [SKIP][167] ([i915#3458]) +28 other tests skip
[167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary.html
* igt@kms_frontbuffer_tracking@psr-suspend:
- shard-rkl: NOTRUN -> [SKIP][168] ([i915#3023]) +8 other tests skip
[168]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_frontbuffer_tracking@psr-suspend.html
* igt@kms_hdr@static-toggle-suspend:
- shard-dg2: NOTRUN -> [SKIP][169] ([i915#3555] / [i915#8228]) +1 other test skip
[169]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_hdr@static-toggle-suspend.html
* igt@kms_multipipe_modeset@basic-max-pipe-crc-check:
- shard-rkl: NOTRUN -> [SKIP][170] ([i915#4070] / [i915#4816])
[170]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_multipipe_modeset@basic-max-pipe-crc-check.html
* igt@kms_panel_fitting@legacy:
- shard-dg2: NOTRUN -> [SKIP][171] ([i915#6301])
[171]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_panel_fitting@legacy.html
- shard-rkl: NOTRUN -> [SKIP][172] ([i915#6301])
[172]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_panel_fitting@legacy.html
* igt@kms_pipe_b_c_ivb@disable-pipe-b-enable-pipe-c:
- shard-dg2: NOTRUN -> [SKIP][173] ([fdo#109289]) +3 other tests skip
[173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_pipe_b_c_ivb@disable-pipe-b-enable-pipe-c.html
* igt@kms_plane_scaling@intel-max-src-size:
- shard-mtlp: NOTRUN -> [SKIP][174] ([i915#6953])
[174]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_plane_scaling@intel-max-src-size.html
- shard-dg2: NOTRUN -> [SKIP][175] ([i915#6953] / [i915#9423])
[175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_plane_scaling@intel-max-src-size.html
* igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-3:
- shard-dg1: NOTRUN -> [FAIL][176] ([i915#8292])
[176]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-3.html
* igt@kms_plane_scaling@plane-downscale-factor-0-25-with-modifiers@pipe-a-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][177] ([i915#9423]) +3 other tests skip
[177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-modifiers@pipe-a-hdmi-a-3.html
* igt@kms_plane_scaling@plane-downscale-factor-0-25-with-modifiers@pipe-b-hdmi-a-2:
- shard-rkl: NOTRUN -> [SKIP][178] ([i915#9423]) +3 other tests skip
[178]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-6/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-modifiers@pipe-b-hdmi-a-2.html
* igt@kms_plane_scaling@plane-downscale-factor-0-25-with-modifiers@pipe-b-hdmi-a-4:
- shard-dg1: NOTRUN -> [SKIP][179] ([i915#9423]) +3 other tests skip
[179]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-16/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-modifiers@pipe-b-hdmi-a-4.html
* igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-a-hdmi-a-3:
- shard-dg1: NOTRUN -> [SKIP][180] ([i915#5176] / [i915#9423]) +3 other tests skip
[180]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-12/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-a-hdmi-a-3.html
* igt@kms_plane_scaling@planes-downscale-factor-0-25-unity-scaling@pipe-b-edp-1:
- shard-mtlp: NOTRUN -> [SKIP][181] ([i915#5235]) +2 other tests skip
[181]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_plane_scaling@planes-downscale-factor-0-25-unity-scaling@pipe-b-edp-1.html
* igt@kms_plane_scaling@planes-downscale-factor-0-25-unity-scaling@pipe-d-edp-1:
- shard-mtlp: NOTRUN -> [SKIP][182] ([i915#3555] / [i915#5235])
[182]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_plane_scaling@planes-downscale-factor-0-25-unity-scaling@pipe-d-edp-1.html
* igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-20x20@pipe-b-hdmi-a-2:
- shard-rkl: NOTRUN -> [SKIP][183] ([i915#5235]) +9 other tests skip
[183]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-20x20@pipe-b-hdmi-a-2.html
* igt@kms_plane_scaling@planes-downscale-factor-0-25@pipe-c-hdmi-a-4:
- shard-dg1: NOTRUN -> [SKIP][184] ([i915#5235]) +11 other tests skip
[184]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-15/igt@kms_plane_scaling@planes-downscale-factor-0-25@pipe-c-hdmi-a-4.html
* igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-25@pipe-a-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][185] ([i915#5235] / [i915#9423]) +11 other tests skip
[185]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-25@pipe-a-hdmi-a-3.html
* igt@kms_pm_dc@dc6-dpms:
- shard-dg2: NOTRUN -> [SKIP][186] ([i915#5978])
[186]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_pm_dc@dc6-dpms.html
* igt@kms_pm_lpsp@screens-disabled:
- shard-mtlp: NOTRUN -> [SKIP][187] ([i915#8430])
[187]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_pm_lpsp@screens-disabled.html
- shard-dg2: NOTRUN -> [SKIP][188] ([i915#8430])
[188]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_pm_lpsp@screens-disabled.html
* igt@kms_pm_rpm@modeset-lpsp-stress:
- shard-rkl: [PASS][189] -> [SKIP][190] ([i915#9519]) +2 other tests skip
[189]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-7/igt@kms_pm_rpm@modeset-lpsp-stress.html
[190]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_pm_rpm@modeset-lpsp-stress.html
* igt@kms_pm_rpm@modeset-non-lpsp-stress:
- shard-dg2: NOTRUN -> [SKIP][191] ([i915#9519])
[191]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_pm_rpm@modeset-non-lpsp-stress.html
* igt@kms_prime@d3hot:
- shard-rkl: NOTRUN -> [SKIP][192] ([i915#6524])
[192]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_prime@d3hot.html
- shard-dg2: NOTRUN -> [SKIP][193] ([i915#6524] / [i915#6805])
[193]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_prime@d3hot.html
* igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf:
- shard-dg2: NOTRUN -> [SKIP][194] ([i915#9683]) +4 other tests skip
[194]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html
* igt@kms_psr2_sf@overlay-plane-move-continuous-sf:
- shard-rkl: NOTRUN -> [SKIP][195] ([i915#9683])
[195]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_psr2_sf@overlay-plane-move-continuous-sf.html
* igt@kms_psr2_sf@overlay-plane-update-continuous-sf:
- shard-rkl: NOTRUN -> [SKIP][196] ([fdo#111068] / [i915#9683])
[196]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_psr2_sf@overlay-plane-update-continuous-sf.html
* igt@kms_psr2_su@page_flip-nv12:
- shard-mtlp: NOTRUN -> [SKIP][197] ([i915#4348])
[197]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_psr2_su@page_flip-nv12.html
* igt@kms_psr_stress_test@flip-primary-invalidate-overlay:
- shard-dg1: NOTRUN -> [SKIP][198] ([i915#9685])
[198]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_psr_stress_test@flip-primary-invalidate-overlay.html
* igt@kms_rotation_crc@primary-yf-tiled-reflect-x-0:
- shard-dg1: NOTRUN -> [SKIP][199] ([fdo#111615] / [i915#5289])
[199]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-0.html
* igt@kms_rotation_crc@sprite-rotation-90-pos-100-0:
- shard-dg2: NOTRUN -> [SKIP][200] ([i915#4235]) +1 other test skip
[200]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_rotation_crc@sprite-rotation-90-pos-100-0.html
* igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_build_fourcc_list:
- shard-rkl: NOTRUN -> [DMESG-FAIL][201] ([i915#10143])
[201]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_build_fourcc_list.html
- shard-dg2: NOTRUN -> [DMESG-FAIL][202] ([i915#10143])
[202]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_build_fourcc_list.html
* igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_clip_offset:
- shard-dg2: NOTRUN -> [DMESG-WARN][203] ([i915#10143])
[203]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-1/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_clip_offset.html
- shard-rkl: NOTRUN -> [DMESG-WARN][204] ([i915#10143])
[204]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-3/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_clip_offset.html
- shard-glk: [PASS][205] -> [DMESG-WARN][206] ([i915#10143] / [i915#10165])
[205]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-glk7/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_clip_offset.html
[206]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk1/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_clip_offset.html
* igt@kms_sysfs_edid_timing:
- shard-dg2: NOTRUN -> [FAIL][207] ([IGT#2])
[207]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_sysfs_edid_timing.html
* igt@kms_tiled_display@basic-test-pattern-with-chamelium:
- shard-rkl: NOTRUN -> [SKIP][208] ([i915#8623])
[208]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
* igt@kms_tv_load_detect@load-detect:
- shard-dg2: NOTRUN -> [SKIP][209] ([fdo#109309])
[209]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@kms_tv_load_detect@load-detect.html
* igt@kms_universal_plane@cursor-fb-leak@pipe-a-hdmi-a-1:
- shard-rkl: [PASS][210] -> [FAIL][211] ([i915#9196])
[210]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-7/igt@kms_universal_plane@cursor-fb-leak@pipe-a-hdmi-a-1.html
[211]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-4/igt@kms_universal_plane@cursor-fb-leak@pipe-a-hdmi-a-1.html
* igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-1:
- shard-tglu: [PASS][212] -> [FAIL][213] ([i915#9196]) +1 other test fail
[212]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-4/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-1.html
[213]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-8/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-1.html
* igt@kms_universal_plane@cursor-fb-leak@pipe-d-edp-1:
- shard-mtlp: [PASS][214] -> [FAIL][215] ([i915#9196])
[214]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-3/igt@kms_universal_plane@cursor-fb-leak@pipe-d-edp-1.html
[215]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-8/igt@kms_universal_plane@cursor-fb-leak@pipe-d-edp-1.html
* igt@kms_vrr@flip-basic:
- shard-mtlp: NOTRUN -> [SKIP][216] ([i915#3555] / [i915#8808])
[216]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@kms_vrr@flip-basic.html
* igt@kms_vrr@flip-basic-fastset:
- shard-dg2: NOTRUN -> [SKIP][217] ([i915#9906])
[217]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@kms_vrr@flip-basic-fastset.html
* igt@kms_writeback@writeback-fb-id:
- shard-rkl: NOTRUN -> [SKIP][218] ([i915#2437])
[218]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@kms_writeback@writeback-fb-id.html
* igt@kms_writeback@writeback-pixel-formats:
- shard-dg2: NOTRUN -> [SKIP][219] ([i915#2437]) +1 other test skip
[219]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_writeback@writeback-pixel-formats.html
* igt@perf@gen8-unprivileged-single-ctx-counters:
- shard-dg2: NOTRUN -> [SKIP][220] ([i915#2436])
[220]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@perf@gen8-unprivileged-single-ctx-counters.html
* igt@perf@global-sseu-config:
- shard-dg2: NOTRUN -> [SKIP][221] ([i915#7387])
[221]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@perf@global-sseu-config.html
* igt@perf_pmu@rc6@other-idle-gt0:
- shard-dg2: NOTRUN -> [SKIP][222] ([i915#8516])
[222]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@perf_pmu@rc6@other-idle-gt0.html
* igt@prime_udl:
- shard-dg2: NOTRUN -> [SKIP][223] ([fdo#109291])
[223]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@prime_udl.html
* igt@prime_vgem@basic-write:
- shard-dg2: NOTRUN -> [SKIP][224] ([i915#3291] / [i915#3708])
[224]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@prime_vgem@basic-write.html
- shard-mtlp: NOTRUN -> [SKIP][225] ([i915#3708])
[225]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@prime_vgem@basic-write.html
* igt@prime_vgem@coherency-gtt:
- shard-dg2: NOTRUN -> [SKIP][226] ([i915#3708] / [i915#4077])
[226]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@prime_vgem@coherency-gtt.html
* igt@prime_vgem@fence-read-hang:
- shard-dg2: NOTRUN -> [SKIP][227] ([i915#3708])
[227]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@prime_vgem@fence-read-hang.html
* igt@syncobj_wait@invalid-wait-zero-handles:
- shard-mtlp: NOTRUN -> [FAIL][228] ([i915#9779])
[228]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@syncobj_wait@invalid-wait-zero-handles.html
- shard-dg2: NOTRUN -> [FAIL][229] ([i915#9779])
[229]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@syncobj_wait@invalid-wait-zero-handles.html
* igt@v3d/v3d_job_submission@multiple-singlesync-to-multisync:
- shard-mtlp: NOTRUN -> [SKIP][230] ([i915#2575]) +2 other tests skip
[230]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@v3d/v3d_job_submission@multiple-singlesync-to-multisync.html
* igt@v3d/v3d_submit_cl@bad-pad:
- shard-rkl: NOTRUN -> [SKIP][231] ([fdo#109315]) +4 other tests skip
[231]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@v3d/v3d_submit_cl@bad-pad.html
* igt@v3d/v3d_submit_csd@bad-bo:
- shard-dg1: NOTRUN -> [SKIP][232] ([i915#2575])
[232]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@v3d/v3d_submit_csd@bad-bo.html
* igt@v3d/v3d_submit_csd@single-out-sync:
- shard-dg2: NOTRUN -> [SKIP][233] ([i915#2575]) +20 other tests skip
[233]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@v3d/v3d_submit_csd@single-out-sync.html
* igt@vc4/vc4_perfmon@create-perfmon-invalid-events:
- shard-rkl: NOTRUN -> [SKIP][234] ([i915#7711]) +1 other test skip
[234]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-5/igt@vc4/vc4_perfmon@create-perfmon-invalid-events.html
* igt@vc4/vc4_purgeable_bo@mark-unpurgeable-check-retained:
- shard-dg2: NOTRUN -> [SKIP][235] ([i915#7711]) +12 other tests skip
[235]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@vc4/vc4_purgeable_bo@mark-unpurgeable-check-retained.html
* igt@vc4/vc4_tiling@get-after-free:
- shard-mtlp: NOTRUN -> [SKIP][236] ([i915#7711]) +2 other tests skip
[236]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-7/igt@vc4/vc4_tiling@get-after-free.html
#### Possible fixes ####
* igt@drm_fdinfo@most-busy-check-all@rcs0:
- shard-rkl: [FAIL][237] ([i915#7742]) -> [PASS][238]
[237]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-4/igt@drm_fdinfo@most-busy-check-all@rcs0.html
[238]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-1/igt@drm_fdinfo@most-busy-check-all@rcs0.html
* igt@gem_ccs@suspend-resume@linear-compressed-compfmt0-smem-lmem0:
- shard-dg2: [INCOMPLETE][239] ([i915#7297]) -> [PASS][240]
[239]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-1/igt@gem_ccs@suspend-resume@linear-compressed-compfmt0-smem-lmem0.html
[240]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@gem_ccs@suspend-resume@linear-compressed-compfmt0-smem-lmem0.html
* igt@gem_exec_fair@basic-none@vecs0:
- shard-rkl: [FAIL][241] ([i915#2842]) -> [PASS][242] +5 other tests pass
[241]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-5/igt@gem_exec_fair@basic-none@vecs0.html
[242]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-6/igt@gem_exec_fair@basic-none@vecs0.html
* igt@gem_exec_suspend@basic-s0@smem:
- shard-dg2: [INCOMPLETE][243] ([i915#9275]) -> [PASS][244]
[243]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@gem_exec_suspend@basic-s0@smem.html
[244]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-6/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gen9_exec_parse@allowed-single:
- shard-glk: [ABORT][245] ([i915#5566]) -> [PASS][246]
[245]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-glk1/igt@gen9_exec_parse@allowed-single.html
[246]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk7/igt@gen9_exec_parse@allowed-single.html
* igt@i915_module_load@reload-with-fault-injection:
- shard-dg1: [INCOMPLETE][247] ([i915#10137] / [i915#9820] / [i915#9849]) -> [PASS][248]
[247]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-12/igt@i915_module_load@reload-with-fault-injection.html
[248]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-13/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pm_rpm@system-suspend-execbuf:
- shard-dg1: [DMESG-WARN][249] ([i915#4423]) -> [PASS][250] +1 other test pass
[249]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@i915_pm_rpm@system-suspend-execbuf.html
[250]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-19/igt@i915_pm_rpm@system-suspend-execbuf.html
* igt@kms_big_fb@4-tiled-64bpp-rotate-180:
- shard-mtlp: [FAIL][251] ([i915#5138]) -> [PASS][252] +1 other test pass
[251]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-8/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
[252]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-5/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
* igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:
- shard-tglu: [FAIL][253] ([i915#3743]) -> [PASS][254] +1 other test pass
[253]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-8/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
[254]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-6/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
* igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic:
- shard-snb: [SKIP][255] ([fdo#109271] / [fdo#111767]) -> [PASS][256]
[255]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb1/igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic.html
[256]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb7/igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic:
- shard-dg2: [TIMEOUT][257] -> [PASS][258]
[257]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@kms_cursor_legacy@flip-vs-cursor-atomic.html
[258]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_cursor_legacy@flip-vs-cursor-atomic.html
* igt@kms_cursor_legacy@torture-bo@pipe-a:
- shard-tglu: [DMESG-WARN][259] ([i915#10166]) -> [PASS][260]
[259]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-3/igt@kms_cursor_legacy@torture-bo@pipe-a.html
[260]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-2/igt@kms_cursor_legacy@torture-bo@pipe-a.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-dg2: [FAIL][261] ([i915#4767]) -> [PASS][262]
[261]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-5/igt@kms_fbcon_fbt@fbc-suspend.html
[262]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-5/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_flip@2x-flip-vs-rmfb@ab-vga1-hdmi-a1:
- shard-snb: [INCOMPLETE][263] -> [PASS][264]
[263]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb1/igt@kms_flip@2x-flip-vs-rmfb@ab-vga1-hdmi-a1.html
[264]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb4/igt@kms_flip@2x-flip-vs-rmfb@ab-vga1-hdmi-a1.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-plflip-blt:
- shard-snb: [SKIP][265] ([fdo#109271]) -> [PASS][266] +5 other tests pass
[265]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb1/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-plflip-blt.html
[266]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb7/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-plflip-blt.html
* igt@kms_pm_rpm@modeset-non-lpsp-stress:
- shard-rkl: [SKIP][267] ([i915#9519]) -> [PASS][268] +2 other tests pass
[267]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-5/igt@kms_pm_rpm@modeset-non-lpsp-stress.html
[268]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-1/igt@kms_pm_rpm@modeset-non-lpsp-stress.html
* igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_xrgb8888_to_xbgr8888:
- shard-glk: [DMESG-WARN][269] ([i915#10143] / [i915#10165]) -> [PASS][270]
[269]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-glk7/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_xrgb8888_to_xbgr8888.html
[270]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk1/igt@kms_selftest@drm_format_helper@drm_format_helper_test-drm_test_fb_xrgb8888_to_xbgr8888.html
* igt@kms_universal_plane@cursor-fb-leak@pipe-b-edp-1:
- shard-mtlp: [FAIL][271] ([i915#9196]) -> [PASS][272]
[271]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-3/igt@kms_universal_plane@cursor-fb-leak@pipe-b-edp-1.html
[272]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-8/igt@kms_universal_plane@cursor-fb-leak@pipe-b-edp-1.html
* igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-1:
- shard-snb: [FAIL][273] ([i915#9196]) -> [PASS][274]
[273]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb4/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-1.html
[274]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb1/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-1.html
* igt@perf_pmu@module-unload:
- shard-dg1: [DMESG-WARN][275] ([i915#4391] / [i915#4423]) -> [PASS][276] +1 other test pass
[275]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@perf_pmu@module-unload.html
[276]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-19/igt@perf_pmu@module-unload.html
#### Warnings ####
* igt@gem_fence_thrash@bo-copy:
- shard-dg2: [TIMEOUT][277] -> [SKIP][278] ([i915#4860])
[277]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@gem_fence_thrash@bo-copy.html
[278]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@gem_fence_thrash@bo-copy.html
* igt@gem_pread@exhaustion:
- shard-tglu: [INCOMPLETE][279] ([i915#10042] / [i915#10137]) -> [WARN][280] ([i915#2658])
[279]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-tglu-10/igt@gem_pread@exhaustion.html
[280]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-tglu-2/igt@gem_pread@exhaustion.html
- shard-glk: [INCOMPLETE][281] ([i915#10042] / [i915#10137]) -> [WARN][282] ([i915#2658])
[281]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-glk1/igt@gem_pread@exhaustion.html
[282]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-glk5/igt@gem_pread@exhaustion.html
* igt@i915_module_load@reload-with-fault-injection:
- shard-mtlp: [ABORT][283] ([i915#10131] / [i915#9697]) -> [ABORT][284] ([i915#10131] / [i915#9820])
[283]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-mtlp-7/igt@i915_module_load@reload-with-fault-injection.html
[284]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-mtlp-5/igt@i915_module_load@reload-with-fault-injection.html
* igt@kms_big_fb@x-tiled-16bpp-rotate-90:
- shard-dg2: [TIMEOUT][285] -> [SKIP][286] ([fdo#111614])
[285]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@kms_big_fb@x-tiled-16bpp-rotate-90.html
[286]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_big_fb@x-tiled-16bpp-rotate-90.html
* igt@kms_big_fb@y-tiled-16bpp-rotate-0:
- shard-dg2: [TIMEOUT][287] -> [SKIP][288] ([i915#4538] / [i915#5190])
[287]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@kms_big_fb@y-tiled-16bpp-rotate-0.html
[288]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_big_fb@y-tiled-16bpp-rotate-0.html
* igt@kms_ccs@pipe-a-crc-sprite-planes-basic-4-tiled-dg2-rc-ccs:
- shard-dg1: [SKIP][289] ([i915#4423] / [i915#5354] / [i915#6095]) -> [SKIP][290] ([i915#5354] / [i915#6095])
[289]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@kms_ccs@pipe-a-crc-sprite-planes-basic-4-tiled-dg2-rc-ccs.html
[290]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-19/igt@kms_ccs@pipe-a-crc-sprite-planes-basic-4-tiled-dg2-rc-ccs.html
* igt@kms_chamelium_audio@dp-audio:
- shard-snb: [ABORT][291] -> [SKIP][292] ([fdo#109271])
[291]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-snb7/igt@kms_chamelium_audio@dp-audio.html
[292]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-snb6/igt@kms_chamelium_audio@dp-audio.html
* igt@kms_content_protection@mei-interface:
- shard-dg1: [SKIP][293] ([i915#9424]) -> [SKIP][294] ([i915#9433])
[293]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@kms_content_protection@mei-interface.html
[294]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-12/igt@kms_content_protection@mei-interface.html
* igt@kms_cursor_legacy@cursorb-vs-flipa-toggle:
- shard-dg1: [SKIP][295] ([fdo#111825] / [i915#4423]) -> [SKIP][296] ([fdo#111825])
[295]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@kms_cursor_legacy@cursorb-vs-flipa-toggle.html
[296]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-19/igt@kms_cursor_legacy@cursorb-vs-flipa-toggle.html
* igt@kms_fbcon_fbt@psr:
- shard-rkl: [SKIP][297] ([i915#3955]) -> [SKIP][298] ([fdo#110189] / [i915#3955])
[297]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-4/igt@kms_fbcon_fbt@psr.html
[298]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-2/igt@kms_fbcon_fbt@psr.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-mmap-wc:
- shard-dg1: [SKIP][299] ([i915#4423] / [i915#8708]) -> [SKIP][300] ([i915#8708])
[299]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-mmap-wc.html
[300]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-19/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-indfb-draw-mmap-cpu:
- shard-dg2: [TIMEOUT][301] -> [SKIP][302] ([i915#5354])
[301]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-indfb-draw-mmap-cpu.html
[302]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-indfb-draw-mmap-cpu.html
* igt@kms_plane_scaling@plane-downscale-factor-0-25-with-pixel-format@pipe-a-hdmi-a-4:
- shard-dg1: [SKIP][303] ([i915#4423] / [i915#9423]) -> [SKIP][304] ([i915#9423])
[303]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg1-17/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-pixel-format@pipe-a-hdmi-a-4.html
[304]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg1-19/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-pixel-format@pipe-a-hdmi-a-4.html
* igt@kms_pm_dc@dc6-dpms:
- shard-rkl: [FAIL][305] ([i915#9295]) -> [SKIP][306] ([i915#3361])
[305]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-rkl-5/igt@kms_pm_dc@dc6-dpms.html
[306]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-rkl-6/igt@kms_pm_dc@dc6-dpms.html
* igt@kms_pm_rpm@dpms-mode-unset-non-lpsp:
- shard-dg2: [SKIP][307] -> [SKIP][308] ([i915#9519])
[307]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-10/igt@kms_pm_rpm@dpms-mode-unset-non-lpsp.html
[308]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-10/igt@kms_pm_rpm@dpms-mode-unset-non-lpsp.html
* igt@prime_mmap@test_aperture_limit@test_aperture_limit-smem:
- shard-dg2: [INCOMPLETE][309] ([i915#5493]) -> [CRASH][310] ([i915#9351])
[309]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14212/shard-dg2-3/igt@prime_mmap@test_aperture_limit@test_aperture_limit-smem.html
[310]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_129467v1/shard-dg2-7/igt@prime_mmap@test_aperture_limit@test_aperture_limit-smem.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[IGT#2]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/2
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
[fdo#109283]: https://bugs.freedesktop.org/show_bug.cgi?id=109283
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
[fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291
[fdo#109293]: https://bugs.freedesktop.org/show_bug.cgi?id=109293
[fdo#109309]: https://bugs.freedesktop.org/show_bug.cgi?id=109309
[fdo#109312]: https://bugs.freedesktop.org/show_bug.cgi?id=109312
[fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
[fdo#109506]: https://bugs.freedesktop.org/show_bug.cgi?id=109506
[fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189
[fdo#110723]: https://bugs.freedesktop.org/show_bug.cgi?id=110723
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111614]: https://bugs.freedesktop.org/show_bug.cgi?id=111614
[fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
[fdo#111767]: https://bugs.freedesktop.org/show_bug.cgi?id=111767
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#10042]: https://gitlab.freedesktop.org/drm/intel/issues/10042
[i915#10070]: https://gitlab.freedesktop.org/drm/intel/issues/10070
[i915#10131]: https://gitlab.freedesktop.org/drm/intel/issues/10131
[i915#10137]: https://gitlab.freedesktop.org/drm/intel/issues/10137
[i915#10140]: https://gitlab.freedesktop.org/drm/intel/issues/10140
[i915#10143]: https://gitlab.freedesktop.org/drm/intel/issues/10143
[i915#10165]: https://gitlab.freedesktop.org/drm/intel/issues/10165
[i915#10166]: https://gitlab.freedesktop.org/drm/intel/issues/10166
[i915#1769]: https://gitlab.freedesktop.org/drm/intel/issues/1769
[i915#1825]: https://gitlab.freedesktop.org/drm/intel/issues/1825
[i915#1839]: https://gitlab.freedesktop.org/drm/intel/issues/1839
[i915#2436]: https://gitlab.freedesktop.org/drm/intel/issues/2436
[i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
[i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527
[i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575
[i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#2705]: https://gitlab.freedesktop.org/drm/intel/issues/2705
[i915#280]: https://gitlab.freedesktop.org/drm/intel/issues/280
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2856]: https://gitlab.freedesktop.org/drm/intel/issues/2856
[i915#3023]: https://gitlab.freedesktop.org/drm/intel/issues/3023
[i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281
[i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
[i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
[i915#3297]: https://gitlab.freedesktop.org/drm/intel/issues/3297
[i915#3299]: https://gitlab.freedesktop.org/drm/intel/issues/3299
[i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359
[i915#3361]: https://gitlab.freedesktop.org/drm/intel/issues/3361
[i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458
[i915#3539]: https://gitlab.freedesktop.org/drm/intel/issues/3539
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3591]: https://gitlab.freedesktop.org/drm/intel/issues/3591
[i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
[i915#3638]: https://gitlab.freedesktop.org/drm/intel/issues/3638
[i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
[i915#3743]: https://gitlab.freedesktop.org/drm/intel/issues/3743
[i915#3778]: https://gitlab.freedesktop.org/drm/intel/issues/3778
[i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
[i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955
[i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070
[i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
[i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
[i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
[i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
[i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
[i915#4235]: https://gitlab.freedesktop.org/drm/intel/issues/4235
[i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270
[i915#4348]: https://gitlab.freedesktop.org/drm/intel/issues/4348
[i915#4387]: https://gitlab.freedesktop.org/drm/intel/issues/4387
[i915#4391]: https://gitlab.freedesktop.org/drm/intel/issues/4391
[i915#4423]: https://gitlab.freedesktop.org/drm/intel/issues/4423
[i915#4475]: https://gitlab.freedesktop.org/drm/intel/issues/4475
[i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
[i915#4537]: https://gitlab.freedesktop.org/drm/intel/issues/4537
[i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4767]: https://gitlab.freedesktop.org/drm/intel/issues/4767
[i915#4771]: https://gitlab.freedesktop.org/drm/intel/issues/4771
[i915#4812]: https://gitlab.freedesktop.org/drm/intel/issues/4812
[i915#4816]: https://gitlab.freedesktop.org/drm/intel/issues/4816
[i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852
[i915#4860]: https://gitlab.freedesktop.org/drm/intel/issues/4860
[i915#4880]: https://gitlab.freedesktop.org/drm/intel/issues/4880
[i915#4881]: https://gitlab.freedesktop.org/drm/intel/issues/4881
[i915#4885]: https://gitlab.freedesktop.org/drm/intel/issues/4885
[i915#5107]: https://gitlab.freedesktop.org/drm/intel/issues/5107
[i915#5138]: https://gitlab.freedesktop.org/drm/intel/issues/5138
[i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
[i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
[i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
[i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286
[i915#5289]: https://gitlab.freedesktop.org/drm/intel/issues/5289
[i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
[i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439
[i915#5493]: https://gitlab.freedesktop.org/drm/intel/issues/5493
[i915#5566]: https://gitlab.freedesktop.org/drm/intel/issues/5566
[i915#5591]: https://gitlab.freedesktop.org/drm/intel/issues/5591
[i915#5978]: https://gitlab.freedesktop.org/drm/intel/issues/5978
[i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095
[i915#6187]: https://gitlab.freedesktop.org/drm/intel/issues/6187
[i915#6188]: https://gitlab.freedesktop.org/drm/intel/issues/6188
[i915#6228]: https://gitlab.freedesktop.org/drm/intel/issues/6228
[i915#6268]: https://gitlab.freedesktop.org/drm/intel/issues/6268
[i915#6301]: https://gitlab.freedesktop.org/drm/intel/issues/6301
[i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
[i915#6805]: https://gitlab.freedesktop.org/drm/intel/issues/6805
[i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953
[i915#7091]: https://gitlab.freedesktop.org/drm/intel/issues/7091
[i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118
[i915#7297]: https://gitlab.freedesktop.org/drm/intel/issues/7297
[i915#7387]: https://gitlab.freedesktop.org/drm/intel/issues/7387
[i915#7392]: https://gitlab.freedesktop.org/drm/intel/issues/7392
[i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697
[i915#7701]: https://gitlab.freedesktop.org/drm/intel/issues/7701
[i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711
[i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742
[i915#7765]: https://gitlab.freedesktop.org/drm/intel/issues/7765
[i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828
[i915#7975]: https://gitlab.freedesktop.org/drm/intel/issues/7975
[i915#7984]: https://gitlab.freedesktop.org/drm/intel/issues/7984
[i915#8213]: https://gitlab.freedesktop.org/drm/intel/issues/8213
[i915#8228]: https://gitlab.freedesktop.org/drm/intel/issues/8228
[i915#8292]: https://gitlab.freedesktop.org/drm/intel/issues/8292
[i915#8381]: https://gitlab.freedesktop.org/drm/intel/issues/8381
[i915#8411]: https://gitlab.freedesktop.org/drm/intel/issues/8411
[i915#8414]: https://gitlab.freedesktop.org/drm/intel/issues/8414
[i915#8430]: https://gitlab.freedesktop.org/drm/intel/issues/8430
[i915#8516]: https://gitlab.freedesktop.org/drm/intel/issues/8516
[i915#8555]: https://gitlab.freedesktop.org/drm/intel/issues/8555
[i915#8562]: https://gitlab.freedesktop.org/drm/intel/issues/8562
[i915#8623]: https://gitlab.freedesktop.org/drm/intel/issues/8623
[i915#8708]: https://gitlab.freedesktop.org/drm/intel/issues/8708
[i915#8808]: https://gitlab.freedesktop.org/drm/intel/issues/8808
[i915#8810]: https://gitlab.freedesktop.org/drm/intel/issues/8810
[i915#8812]: https://gitlab.freedesktop.org/drm/intel/issues/8812
[i915#8814]: https://gitlab.freedesktop.org/drm/intel/issues/8814
[i915#8925]: https://gitlab.freedesktop.org/drm/intel/issues/8925
[i915#9067]: https://gitlab.freedesktop.org/drm/intel/issues/9067
[i915#9196]: https://gitlab.freedesktop.org/drm/intel/issues/9196
[i915#9227]: https://gitlab.freedesktop.org/drm/intel/issues/9227
[i915#9275]: https://gitlab.freedesktop.org/drm/intel/issues/9275
[i915#9295]: https://gitlab.freedesktop.org/drm/intel/issues/9295
[i915#9337]: https://gitlab.freedesktop.org/drm/intel/issues/9337
[i915#9351]: https://gitlab.freedesktop.org/drm/intel/issues/9351
[i915#9423]: https://gitlab.freedesktop.org/drm/intel/issues/9423
[i915#9424]: https://gitlab.freedesktop.org/drm/intel/issues/9424
[i915#9433]: https://gitlab.freedesktop.org/drm/intel/issues/9433
[i915#9519]: https://gitlab.freedesktop.org/drm/intel/issues/9519
[i915#9531]: https://gitlab.freedesktop.org/drm/intel/issues/9531
[i915#9606]: https://gitlab.freedesktop.org/drm/intel/issues/9606
[i915#9683]: https://gitlab.freedesktop.org/drm/intel/issues/9683
[i915#9685]: https://gitlab.freedesktop.org/drm/intel/issues/9685
[i915#9688]: https://gitlab.freedesktop.org/drm/intel/issues/9688
[i915#9697]: https://gitlab.freedesktop.org/drm/intel/issues/9697
[i915#9723]: https://gitlab.freedesktop.org/drm/intel/issues/9723
[i915#9732]: https://gitlab.freedesktop.org/drm/intel/issues/9732
[i915#9779]: https://gitlab.freedesktop.org/drm/intel/issues/9779
[i915#9808]: https://gitlab.freedesktop.org/drm/intel/issues/9808
[i915#9820]: https://gitlab.freedesktop.org/drm/intel/issues/9820
[i915#9849]: https://gitlab.freedesktop.org/drm/intel/issues/9849
[i915#9906]: https://gitlab.freedesktop.org/drm/intel/issues/9906
[i915#9934]: https://gitlab.freedesktop.org/drm/intel/issues/9934
Build changes
-------------
* Linux: CI_DRM_14212 -> Patchwork_129467v1
CI-20190529: 20190529
CI_DRM_14212: 1dd92467500a5ead3e44bbdfe15e064dd79b65ef @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7702: bf075a74ece1956fc0e554291591b9da3eab54cf @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_129467v1: 1dd92467500a5ead3e44bbdfe15e064dd79b65ef @ 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_129467v1/index.html
[-- Attachment #2: Type: text/html, Size: 102294 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-02 19:58 ` [PATCH] " Ville Syrjälä
2024-02-02 20:06 ` Ville Syrjälä
@ 2024-02-03 13:25 ` Gustavo Sousa
2024-02-05 15:34 ` Ville Syrjälä
1 sibling, 1 reply; 13+ messages in thread
From: Gustavo Sousa @ 2024-02-03 13:25 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
Quoting Ville Syrjälä (2024-02-02 16:58:37-03:00)
>On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
>> Looks like the name and description of intel_cdclk_needs_modeset()
>> became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
>> cdclk changes if possible"), when it became possible to update the cdclk
>> without requiring disabling the pipes when only changing the cd2x
>> divider was enough.
>>
>> Later on we also added the same type of support with squash and crawling
>> with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
>> when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
>> squasher to be reconfigured live") and commit d62686ba3b54
>> ("drm/i915/adl_p: CDCLK crawl support for ADL").
>>
>> As such, update that function's name and documentation to something more
>> appropriate, since the real checks for requiring modeset are done
>> elsewhere.
>>
>> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> ---
>>
>> One thing worth noting here is that, with this change, we are left with an
>> awkward situation where two function names related to checking changes in cdclk:
>>
>> intel_cdclk_params_changed() and intel_cdclk_changed()
>>
>> ,
>>
>> and I find it weird that we have intel_cdclk_changed(), which checks for the
>> voltage level as well. Shouldn't the voltage level be a function of cdclk and
>> ddi clock? Why do we need that?
>>
>> drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
>> drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
>> .../drm/i915/display/intel_display_power_well.c | 4 ++--
>> 3 files changed, 11 insertions(+), 12 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> index 26200ee3e23f..caadd880865f 100644
>> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
>> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
>> }
>>
>> /**
>> - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
>> - * configurations requires a modeset on all pipes
>> + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
>> * @a: first CDCLK configuration
>> * @b: second CDCLK configuration
>> *
>> * Returns:
>> - * True if changing between the two CDCLK configurations
>> - * requires all pipes to be off, false if not.
>> + * True if parameters changed in a way that requires programming the CDCLK
>> + * and False otherwise.
>> */
>> -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
>> - const struct intel_cdclk_config *b)
>> +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
>> + const struct intel_cdclk_config *b)
>
>The new name isn't very descriptive either.
Yeah... I would much rather use intel_cdclk_changed(), but that one is
already taken.
>
>Outside the cd2x/crawl/squash cases we stil have to consider
>two cases:
>1. cdclk frequency/pll changes (voltage level can change or not)
>2. cdclk frequency/pll doesn't change, but voltage level needs to change
>
>And that difference is what intel_cdclk_needs_modeset() is trying
>convey. And intel_cdclk_changed() tells us whether anything at all
>is changing.
I might be missing something, but, by going through the specs, it looked
to me that voltage level was dependent on cdclk (as well as on ddi
clock) and not the other way around. That's why I find it odd that we
need an intel_cdclk_changed() that, besides looking for changes in
cdclk, also checks for the voltage level.
In intel_set_cdclk(), we check intel_cdclk_changed() before continuing.
If, for example, there is a change in ddi clock that requires a change
in voltage level but no changes in cdclk, intel_cdclk_changed() would
return true, right? Wouldn't that make us unnecessarily go through
intel_set_cdclk()?
--
Gustavo Sousa
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-02 20:06 ` Ville Syrjälä
@ 2024-02-03 13:32 ` Gustavo Sousa
2024-02-14 20:08 ` Gustavo Sousa
0 siblings, 1 reply; 13+ messages in thread
From: Gustavo Sousa @ 2024-02-03 13:32 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
Quoting Ville Syrjälä (2024-02-02 17:06:02-03:00)
>On Fri, Feb 02, 2024 at 09:58:37PM +0200, Ville Syrjälä wrote:
>> On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
>> > Looks like the name and description of intel_cdclk_needs_modeset()
>> > became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
>> > cdclk changes if possible"), when it became possible to update the cdclk
>> > without requiring disabling the pipes when only changing the cd2x
>> > divider was enough.
>> >
>> > Later on we also added the same type of support with squash and crawling
>> > with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
>> > when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
>> > squasher to be reconfigured live") and commit d62686ba3b54
>> > ("drm/i915/adl_p: CDCLK crawl support for ADL").
>> >
>> > As such, update that function's name and documentation to something more
>> > appropriate, since the real checks for requiring modeset are done
>> > elsewhere.
>> >
>> > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> > ---
>> >
>> > One thing worth noting here is that, with this change, we are left with an
>> > awkward situation where two function names related to checking changes in cdclk:
>> >
>> > intel_cdclk_params_changed() and intel_cdclk_changed()
>> >
>> > ,
>> >
>> > and I find it weird that we have intel_cdclk_changed(), which checks for the
>> > voltage level as well. Shouldn't the voltage level be a function of cdclk and
>> > ddi clock? Why do we need that?
>> >
>> > drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
>> > drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
>> > .../drm/i915/display/intel_display_power_well.c | 4 ++--
>> > 3 files changed, 11 insertions(+), 12 deletions(-)
>> >
>> > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> > index 26200ee3e23f..caadd880865f 100644
>> > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
>> > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> > @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
>> > }
>> >
>> > /**
>> > - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
>> > - * configurations requires a modeset on all pipes
>> > + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
>> > * @a: first CDCLK configuration
>> > * @b: second CDCLK configuration
>> > *
>> > * Returns:
>> > - * True if changing between the two CDCLK configurations
>> > - * requires all pipes to be off, false if not.
>> > + * True if parameters changed in a way that requires programming the CDCLK
>> > + * and False otherwise.
>> > */
>> > -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
>> > - const struct intel_cdclk_config *b)
>> > +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
>> > + const struct intel_cdclk_config *b)
>>
>> The new name isn't very descriptive either.
>>
>> Outside the cd2x/crawl/squash cases we stil have to consider
>> two cases:
>> 1. cdclk frequency/pll changes (voltage level can change or not)
>> 2. cdclk frequency/pll doesn't change, but voltage level needs to change
>>
>> And that difference is what intel_cdclk_needs_modeset() is trying
>> convey. And intel_cdclk_changed() tells us whether anything at all
>> is changing.
>
>intel_cdclk_clock_changed() is perhaps the best name I can
>think of, off the top of my head. But I'm notoriously bad at
>naming things so take that with a pinch of salt.
Sounds better indeed. And I think intel_cdclk_changed() would be more
concise, but that one is already taken internally.
--
Gustavo Sousa
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-03 13:25 ` Gustavo Sousa
@ 2024-02-05 15:34 ` Ville Syrjälä
2024-02-14 19:56 ` Gustavo Sousa
0 siblings, 1 reply; 13+ messages in thread
From: Ville Syrjälä @ 2024-02-05 15:34 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
On Sat, Feb 03, 2024 at 10:25:18AM -0300, Gustavo Sousa wrote:
> Quoting Ville Syrjälä (2024-02-02 16:58:37-03:00)
> >On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
> >> Looks like the name and description of intel_cdclk_needs_modeset()
> >> became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
> >> cdclk changes if possible"), when it became possible to update the cdclk
> >> without requiring disabling the pipes when only changing the cd2x
> >> divider was enough.
> >>
> >> Later on we also added the same type of support with squash and crawling
> >> with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
> >> when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
> >> squasher to be reconfigured live") and commit d62686ba3b54
> >> ("drm/i915/adl_p: CDCLK crawl support for ADL").
> >>
> >> As such, update that function's name and documentation to something more
> >> appropriate, since the real checks for requiring modeset are done
> >> elsewhere.
> >>
> >> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> >> ---
> >>
> >> One thing worth noting here is that, with this change, we are left with an
> >> awkward situation where two function names related to checking changes in cdclk:
> >>
> >> intel_cdclk_params_changed() and intel_cdclk_changed()
> >>
> >> ,
> >>
> >> and I find it weird that we have intel_cdclk_changed(), which checks for the
> >> voltage level as well. Shouldn't the voltage level be a function of cdclk and
> >> ddi clock? Why do we need that?
> >>
> >> drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
> >> drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
> >> .../drm/i915/display/intel_display_power_well.c | 4 ++--
> >> 3 files changed, 11 insertions(+), 12 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> >> index 26200ee3e23f..caadd880865f 100644
> >> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> >> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> >> @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
> >> }
> >>
> >> /**
> >> - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
> >> - * configurations requires a modeset on all pipes
> >> + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
> >> * @a: first CDCLK configuration
> >> * @b: second CDCLK configuration
> >> *
> >> * Returns:
> >> - * True if changing between the two CDCLK configurations
> >> - * requires all pipes to be off, false if not.
> >> + * True if parameters changed in a way that requires programming the CDCLK
> >> + * and False otherwise.
> >> */
> >> -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
> >> - const struct intel_cdclk_config *b)
> >> +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
> >> + const struct intel_cdclk_config *b)
> >
> >The new name isn't very descriptive either.
>
> Yeah... I would much rather use intel_cdclk_changed(), but that one is
> already taken.
>
> >
> >Outside the cd2x/crawl/squash cases we stil have to consider
> >two cases:
> >1. cdclk frequency/pll changes (voltage level can change or not)
> >2. cdclk frequency/pll doesn't change, but voltage level needs to change
> >
> >And that difference is what intel_cdclk_needs_modeset() is trying
> >convey. And intel_cdclk_changed() tells us whether anything at all
> >is changing.
>
> I might be missing something, but, by going through the specs, it looked
> to me that voltage level was dependent on cdclk (as well as on ddi
> clock) and not the other way around. That's why I find it odd that we
> need an intel_cdclk_changed() that, besides looking for changes in
> cdclk, also checks for the voltage level.
>
> In intel_set_cdclk(), we check intel_cdclk_changed() before continuing.
> If, for example, there is a change in ddi clock that requires a change
> in voltage level but no changes in cdclk, intel_cdclk_changed() would
> return true, right? Wouldn't that make us unnecessarily go through
> intel_set_cdclk()?
intel_set_cdclk() is the thing that does the voltage change.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-05 15:34 ` Ville Syrjälä
@ 2024-02-14 19:56 ` Gustavo Sousa
2024-02-14 20:08 ` Ville Syrjälä
0 siblings, 1 reply; 13+ messages in thread
From: Gustavo Sousa @ 2024-02-14 19:56 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
Hi, Ville.
Sorry for taking long to get back to this.
Quoting Ville Syrjälä (2024-02-05 12:34:57-03:00)
>On Sat, Feb 03, 2024 at 10:25:18AM -0300, Gustavo Sousa wrote:
>> Quoting Ville Syrjälä (2024-02-02 16:58:37-03:00)
>> >On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
>> >> Looks like the name and description of intel_cdclk_needs_modeset()
>> >> became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
>> >> cdclk changes if possible"), when it became possible to update the cdclk
>> >> without requiring disabling the pipes when only changing the cd2x
>> >> divider was enough.
>> >>
>> >> Later on we also added the same type of support with squash and crawling
>> >> with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
>> >> when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
>> >> squasher to be reconfigured live") and commit d62686ba3b54
>> >> ("drm/i915/adl_p: CDCLK crawl support for ADL").
>> >>
>> >> As such, update that function's name and documentation to something more
>> >> appropriate, since the real checks for requiring modeset are done
>> >> elsewhere.
>> >>
>> >> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> >> ---
>> >>
>> >> One thing worth noting here is that, with this change, we are left with an
>> >> awkward situation where two function names related to checking changes in cdclk:
>> >>
>> >> intel_cdclk_params_changed() and intel_cdclk_changed()
>> >>
>> >> ,
>> >>
>> >> and I find it weird that we have intel_cdclk_changed(), which checks for the
>> >> voltage level as well. Shouldn't the voltage level be a function of cdclk and
>> >> ddi clock? Why do we need that?
>> >>
>> >> drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
>> >> drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
>> >> .../drm/i915/display/intel_display_power_well.c | 4 ++--
>> >> 3 files changed, 11 insertions(+), 12 deletions(-)
>> >>
>> >> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> >> index 26200ee3e23f..caadd880865f 100644
>> >> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
>> >> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> >> @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
>> >> }
>> >>
>> >> /**
>> >> - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
>> >> - * configurations requires a modeset on all pipes
>> >> + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
>> >> * @a: first CDCLK configuration
>> >> * @b: second CDCLK configuration
>> >> *
>> >> * Returns:
>> >> - * True if changing between the two CDCLK configurations
>> >> - * requires all pipes to be off, false if not.
>> >> + * True if parameters changed in a way that requires programming the CDCLK
>> >> + * and False otherwise.
>> >> */
>> >> -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
>> >> - const struct intel_cdclk_config *b)
>> >> +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
>> >> + const struct intel_cdclk_config *b)
>> >
>> >The new name isn't very descriptive either.
>>
>> Yeah... I would much rather use intel_cdclk_changed(), but that one is
>> already taken.
>>
>> >
>> >Outside the cd2x/crawl/squash cases we stil have to consider
>> >two cases:
>> >1. cdclk frequency/pll changes (voltage level can change or not)
>> >2. cdclk frequency/pll doesn't change, but voltage level needs to change
>> >
>> >And that difference is what intel_cdclk_needs_modeset() is trying
>> >convey. And intel_cdclk_changed() tells us whether anything at all
>> >is changing.
>>
>> I might be missing something, but, by going through the specs, it looked
>> to me that voltage level was dependent on cdclk (as well as on ddi
>> clock) and not the other way around. That's why I find it odd that we
>> need an intel_cdclk_changed() that, besides looking for changes in
>> cdclk, also checks for the voltage level.
>>
>> In intel_set_cdclk(), we check intel_cdclk_changed() before continuing.
>> If, for example, there is a change in ddi clock that requires a change
>> in voltage level but no changes in cdclk, intel_cdclk_changed() would
>> return true, right? Wouldn't that make us unnecessarily go through
>> intel_set_cdclk()?
>
>intel_set_cdclk() is the thing that does the voltage change.
Yep and perhaps I provided an incomplete response above. Sorry.
I was wondering if handling voltage level should really be
intel_set_cdclk()'s responsibility.
I might be missing the big picture here, but, at least for the recent
platforms, I get the understanding that voltage level handling should be
a separate step in the hardware commit process.
Would it be possible to have a commit containing (i) update(s) to ddi
clk and (ii) no update to cdclk such that (i) require an update to
voltage level, right?
--
Gustavo Sousa
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-14 19:56 ` Gustavo Sousa
@ 2024-02-14 20:08 ` Ville Syrjälä
0 siblings, 0 replies; 13+ messages in thread
From: Ville Syrjälä @ 2024-02-14 20:08 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
On Wed, Feb 14, 2024 at 04:56:50PM -0300, Gustavo Sousa wrote:
> Hi, Ville.
>
> Sorry for taking long to get back to this.
>
> Quoting Ville Syrjälä (2024-02-05 12:34:57-03:00)
> >On Sat, Feb 03, 2024 at 10:25:18AM -0300, Gustavo Sousa wrote:
> >> Quoting Ville Syrjälä (2024-02-02 16:58:37-03:00)
> >> >On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
> >> >> Looks like the name and description of intel_cdclk_needs_modeset()
> >> >> became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
> >> >> cdclk changes if possible"), when it became possible to update the cdclk
> >> >> without requiring disabling the pipes when only changing the cd2x
> >> >> divider was enough.
> >> >>
> >> >> Later on we also added the same type of support with squash and crawling
> >> >> with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
> >> >> when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
> >> >> squasher to be reconfigured live") and commit d62686ba3b54
> >> >> ("drm/i915/adl_p: CDCLK crawl support for ADL").
> >> >>
> >> >> As such, update that function's name and documentation to something more
> >> >> appropriate, since the real checks for requiring modeset are done
> >> >> elsewhere.
> >> >>
> >> >> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> >> >> ---
> >> >>
> >> >> One thing worth noting here is that, with this change, we are left with an
> >> >> awkward situation where two function names related to checking changes in cdclk:
> >> >>
> >> >> intel_cdclk_params_changed() and intel_cdclk_changed()
> >> >>
> >> >> ,
> >> >>
> >> >> and I find it weird that we have intel_cdclk_changed(), which checks for the
> >> >> voltage level as well. Shouldn't the voltage level be a function of cdclk and
> >> >> ddi clock? Why do we need that?
> >> >>
> >> >> drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
> >> >> drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
> >> >> .../drm/i915/display/intel_display_power_well.c | 4 ++--
> >> >> 3 files changed, 11 insertions(+), 12 deletions(-)
> >> >>
> >> >> diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> >> >> index 26200ee3e23f..caadd880865f 100644
> >> >> --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> >> >> +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> >> >> @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
> >> >> }
> >> >>
> >> >> /**
> >> >> - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
> >> >> - * configurations requires a modeset on all pipes
> >> >> + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
> >> >> * @a: first CDCLK configuration
> >> >> * @b: second CDCLK configuration
> >> >> *
> >> >> * Returns:
> >> >> - * True if changing between the two CDCLK configurations
> >> >> - * requires all pipes to be off, false if not.
> >> >> + * True if parameters changed in a way that requires programming the CDCLK
> >> >> + * and False otherwise.
> >> >> */
> >> >> -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
> >> >> - const struct intel_cdclk_config *b)
> >> >> +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
> >> >> + const struct intel_cdclk_config *b)
> >> >
> >> >The new name isn't very descriptive either.
> >>
> >> Yeah... I would much rather use intel_cdclk_changed(), but that one is
> >> already taken.
> >>
> >> >
> >> >Outside the cd2x/crawl/squash cases we stil have to consider
> >> >two cases:
> >> >1. cdclk frequency/pll changes (voltage level can change or not)
> >> >2. cdclk frequency/pll doesn't change, but voltage level needs to change
> >> >
> >> >And that difference is what intel_cdclk_needs_modeset() is trying
> >> >convey. And intel_cdclk_changed() tells us whether anything at all
> >> >is changing.
> >>
> >> I might be missing something, but, by going through the specs, it looked
> >> to me that voltage level was dependent on cdclk (as well as on ddi
> >> clock) and not the other way around. That's why I find it odd that we
> >> need an intel_cdclk_changed() that, besides looking for changes in
> >> cdclk, also checks for the voltage level.
> >>
> >> In intel_set_cdclk(), we check intel_cdclk_changed() before continuing.
> >> If, for example, there is a change in ddi clock that requires a change
> >> in voltage level but no changes in cdclk, intel_cdclk_changed() would
> >> return true, right? Wouldn't that make us unnecessarily go through
> >> intel_set_cdclk()?
> >
> >intel_set_cdclk() is the thing that does the voltage change.
>
> Yep and perhaps I provided an incomplete response above. Sorry.
>
> I was wondering if handling voltage level should really be
> intel_set_cdclk()'s responsibility.
>
> I might be missing the big picture here, but, at least for the recent
> platforms, I get the understanding that voltage level handling should be
> a separate step in the hardware commit process.
>
> Would it be possible to have a commit containing (i) update(s) to ddi
> clk and (ii) no update to cdclk such that (i) require an update to
> voltage level, right?
That is possible yes. But I don't think there's much
point in complicating things by splitting the voltage
level stuff into a completely separate thing.
What I think we could do is split .set_cdclk() into more
fine grained steps so that:
- it's easier to reuse individual pieces across platforms without
ugly if ladders
- perhaps make it a bit easier to skip unnecessary steps
although the actual cdclk progrmaming in the nop case really
just ends up being a CDLCK_CTL/etc. register write
so not a big deal in practice.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-03 13:32 ` Gustavo Sousa
@ 2024-02-14 20:08 ` Gustavo Sousa
2024-02-14 20:15 ` Ville Syrjälä
0 siblings, 1 reply; 13+ messages in thread
From: Gustavo Sousa @ 2024-02-14 20:08 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
Quoting Gustavo Sousa (2024-02-03 10:32:18-03:00)
>Quoting Ville Syrjälä (2024-02-02 17:06:02-03:00)
>>On Fri, Feb 02, 2024 at 09:58:37PM +0200, Ville Syrjälä wrote:
>>> On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
>>> > Looks like the name and description of intel_cdclk_needs_modeset()
>>> > became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
>>> > cdclk changes if possible"), when it became possible to update the cdclk
>>> > without requiring disabling the pipes when only changing the cd2x
>>> > divider was enough.
>>> >
>>> > Later on we also added the same type of support with squash and crawling
>>> > with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
>>> > when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
>>> > squasher to be reconfigured live") and commit d62686ba3b54
>>> > ("drm/i915/adl_p: CDCLK crawl support for ADL").
>>> >
>>> > As such, update that function's name and documentation to something more
>>> > appropriate, since the real checks for requiring modeset are done
>>> > elsewhere.
>>> >
>>> > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>>> > ---
>>> >
>>> > One thing worth noting here is that, with this change, we are left with an
>>> > awkward situation where two function names related to checking changes in cdclk:
>>> >
>>> > intel_cdclk_params_changed() and intel_cdclk_changed()
>>> >
>>> > ,
>>> >
>>> > and I find it weird that we have intel_cdclk_changed(), which checks for the
>>> > voltage level as well. Shouldn't the voltage level be a function of cdclk and
>>> > ddi clock? Why do we need that?
>>> >
>>> > drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
>>> > drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
>>> > .../drm/i915/display/intel_display_power_well.c | 4 ++--
>>> > 3 files changed, 11 insertions(+), 12 deletions(-)
>>> >
>>> > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
>>> > index 26200ee3e23f..caadd880865f 100644
>>> > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
>>> > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
>>> > @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
>>> > }
>>> >
>>> > /**
>>> > - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
>>> > - * configurations requires a modeset on all pipes
>>> > + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
>>> > * @a: first CDCLK configuration
>>> > * @b: second CDCLK configuration
>>> > *
>>> > * Returns:
>>> > - * True if changing between the two CDCLK configurations
>>> > - * requires all pipes to be off, false if not.
>>> > + * True if parameters changed in a way that requires programming the CDCLK
>>> > + * and False otherwise.
>>> > */
>>> > -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
>>> > - const struct intel_cdclk_config *b)
>>> > +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
>>> > + const struct intel_cdclk_config *b)
>>>
>>> The new name isn't very descriptive either.
>>>
>>> Outside the cd2x/crawl/squash cases we stil have to consider
>>> two cases:
>>> 1. cdclk frequency/pll changes (voltage level can change or not)
>>> 2. cdclk frequency/pll doesn't change, but voltage level needs to change
>>>
>>> And that difference is what intel_cdclk_needs_modeset() is trying
>>> convey. And intel_cdclk_changed() tells us whether anything at all
>>> is changing.
>>
>>intel_cdclk_clock_changed() is perhaps the best name I can
>>think of, off the top of my head. But I'm notoriously bad at
>>naming things so take that with a pinch of salt.
>
>Sounds better indeed. And I think intel_cdclk_changed() would be more
>concise, but that one is already taken internally.
Hm... On second thought, I'm not sure intel_cdclk_clock_changed() would
be very accurate, at least with the current implementation. We are
checking not only on the resulting cdclk frequency, but also on
ref and vco values.
Although I'm not sure if that could happen in real life, but the current
implementation gives me the interpretation that, in theory, a change in
ref or vco could happen even if the cdclk value did not change. Or are
we safe to say that checking on cdclk should be enough and looking at
vco and ref is actually redundant?
--
Gustavo Sousa
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-14 20:08 ` Gustavo Sousa
@ 2024-02-14 20:15 ` Ville Syrjälä
2024-02-14 20:30 ` Gustavo Sousa
0 siblings, 1 reply; 13+ messages in thread
From: Ville Syrjälä @ 2024-02-14 20:15 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
On Wed, Feb 14, 2024 at 05:08:48PM -0300, Gustavo Sousa wrote:
> Quoting Gustavo Sousa (2024-02-03 10:32:18-03:00)
> >Quoting Ville Syrjälä (2024-02-02 17:06:02-03:00)
> >>On Fri, Feb 02, 2024 at 09:58:37PM +0200, Ville Syrjälä wrote:
> >>> On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
> >>> > Looks like the name and description of intel_cdclk_needs_modeset()
> >>> > became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
> >>> > cdclk changes if possible"), when it became possible to update the cdclk
> >>> > without requiring disabling the pipes when only changing the cd2x
> >>> > divider was enough.
> >>> >
> >>> > Later on we also added the same type of support with squash and crawling
> >>> > with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
> >>> > when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
> >>> > squasher to be reconfigured live") and commit d62686ba3b54
> >>> > ("drm/i915/adl_p: CDCLK crawl support for ADL").
> >>> >
> >>> > As such, update that function's name and documentation to something more
> >>> > appropriate, since the real checks for requiring modeset are done
> >>> > elsewhere.
> >>> >
> >>> > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> >>> > ---
> >>> >
> >>> > One thing worth noting here is that, with this change, we are left with an
> >>> > awkward situation where two function names related to checking changes in cdclk:
> >>> >
> >>> > intel_cdclk_params_changed() and intel_cdclk_changed()
> >>> >
> >>> > ,
> >>> >
> >>> > and I find it weird that we have intel_cdclk_changed(), which checks for the
> >>> > voltage level as well. Shouldn't the voltage level be a function of cdclk and
> >>> > ddi clock? Why do we need that?
> >>> >
> >>> > drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
> >>> > drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
> >>> > .../drm/i915/display/intel_display_power_well.c | 4 ++--
> >>> > 3 files changed, 11 insertions(+), 12 deletions(-)
> >>> >
> >>> > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
> >>> > index 26200ee3e23f..caadd880865f 100644
> >>> > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
> >>> > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
> >>> > @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
> >>> > }
> >>> >
> >>> > /**
> >>> > - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
> >>> > - * configurations requires a modeset on all pipes
> >>> > + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
> >>> > * @a: first CDCLK configuration
> >>> > * @b: second CDCLK configuration
> >>> > *
> >>> > * Returns:
> >>> > - * True if changing between the two CDCLK configurations
> >>> > - * requires all pipes to be off, false if not.
> >>> > + * True if parameters changed in a way that requires programming the CDCLK
> >>> > + * and False otherwise.
> >>> > */
> >>> > -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
> >>> > - const struct intel_cdclk_config *b)
> >>> > +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
> >>> > + const struct intel_cdclk_config *b)
> >>>
> >>> The new name isn't very descriptive either.
> >>>
> >>> Outside the cd2x/crawl/squash cases we stil have to consider
> >>> two cases:
> >>> 1. cdclk frequency/pll changes (voltage level can change or not)
> >>> 2. cdclk frequency/pll doesn't change, but voltage level needs to change
> >>>
> >>> And that difference is what intel_cdclk_needs_modeset() is trying
> >>> convey. And intel_cdclk_changed() tells us whether anything at all
> >>> is changing.
> >>
> >>intel_cdclk_clock_changed() is perhaps the best name I can
> >>think of, off the top of my head. But I'm notoriously bad at
> >>naming things so take that with a pinch of salt.
> >
> >Sounds better indeed. And I think intel_cdclk_changed() would be more
> >concise, but that one is already taken internally.
>
> Hm... On second thought, I'm not sure intel_cdclk_clock_changed() would
> be very accurate, at least with the current implementation. We are
> checking not only on the resulting cdclk frequency, but also on
> ref and vco values.
That's all about the clock.
>
> Although I'm not sure if that could happen in real life, but the current
> implementation gives me the interpretation that, in theory, a change in
> ref or vco could happen even if the cdclk value did not change. Or are
> we safe to say that checking on cdclk should be enough and looking at
> vco and ref is actually redundant?
ref never changes, vco could in theory change such that the
resulting cdclk stays the same, but probably no real world
examples of that.
But I don't see any point in putting those kinds of assumptions
into the code given that it wouldn't provide any actual benefits.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed
2024-02-14 20:15 ` Ville Syrjälä
@ 2024-02-14 20:30 ` Gustavo Sousa
0 siblings, 0 replies; 13+ messages in thread
From: Gustavo Sousa @ 2024-02-14 20:30 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
Quoting Ville Syrjälä (2024-02-14 17:15:51-03:00)
>On Wed, Feb 14, 2024 at 05:08:48PM -0300, Gustavo Sousa wrote:
>> Quoting Gustavo Sousa (2024-02-03 10:32:18-03:00)
>> >Quoting Ville Syrjälä (2024-02-02 17:06:02-03:00)
>> >>On Fri, Feb 02, 2024 at 09:58:37PM +0200, Ville Syrjälä wrote:
>> >>> On Fri, Feb 02, 2024 at 10:12:08AM -0300, Gustavo Sousa wrote:
>> >>> > Looks like the name and description of intel_cdclk_needs_modeset()
>> >>> > became inacurate as of commit 59f9e9cab3a1 ("drm/i915: Skip modeset for
>> >>> > cdclk changes if possible"), when it became possible to update the cdclk
>> >>> > without requiring disabling the pipes when only changing the cd2x
>> >>> > divider was enough.
>> >>> >
>> >>> > Later on we also added the same type of support with squash and crawling
>> >>> > with commit 25e0e5ae5610 ("drm/i915/display: Do both crawl and squash
>> >>> > when changing cdclk"), commit d4a23930490d ("drm/i915: Allow cdclk
>> >>> > squasher to be reconfigured live") and commit d62686ba3b54
>> >>> > ("drm/i915/adl_p: CDCLK crawl support for ADL").
>> >>> >
>> >>> > As such, update that function's name and documentation to something more
>> >>> > appropriate, since the real checks for requiring modeset are done
>> >>> > elsewhere.
>> >>> >
>> >>> > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> >>> > ---
>> >>> >
>> >>> > One thing worth noting here is that, with this change, we are left with an
>> >>> > awkward situation where two function names related to checking changes in cdclk:
>> >>> >
>> >>> > intel_cdclk_params_changed() and intel_cdclk_changed()
>> >>> >
>> >>> > ,
>> >>> >
>> >>> > and I find it weird that we have intel_cdclk_changed(), which checks for the
>> >>> > voltage level as well. Shouldn't the voltage level be a function of cdclk and
>> >>> > ddi clock? Why do we need that?
>> >>> >
>> >>> > drivers/gpu/drm/i915/display/intel_cdclk.c | 15 +++++++--------
>> >>> > drivers/gpu/drm/i915/display/intel_cdclk.h | 4 ++--
>> >>> > .../drm/i915/display/intel_display_power_well.c | 4 ++--
>> >>> > 3 files changed, 11 insertions(+), 12 deletions(-)
>> >>> >
>> >>> > diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> >>> > index 26200ee3e23f..caadd880865f 100644
>> >>> > --- a/drivers/gpu/drm/i915/display/intel_cdclk.c
>> >>> > +++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
>> >>> > @@ -2233,17 +2233,16 @@ static bool intel_cdclk_can_squash(struct drm_i915_private *dev_priv,
>> >>> > }
>> >>> >
>> >>> > /**
>> >>> > - * intel_cdclk_needs_modeset - Determine if changong between the CDCLK
>> >>> > - * configurations requires a modeset on all pipes
>> >>> > + * intel_cdclk_params_changed - Check whether CDCLK parameters changed
>> >>> > * @a: first CDCLK configuration
>> >>> > * @b: second CDCLK configuration
>> >>> > *
>> >>> > * Returns:
>> >>> > - * True if changing between the two CDCLK configurations
>> >>> > - * requires all pipes to be off, false if not.
>> >>> > + * True if parameters changed in a way that requires programming the CDCLK
>> >>> > + * and False otherwise.
>> >>> > */
>> >>> > -bool intel_cdclk_needs_modeset(const struct intel_cdclk_config *a,
>> >>> > - const struct intel_cdclk_config *b)
>> >>> > +bool intel_cdclk_params_changed(const struct intel_cdclk_config *a,
>> >>> > + const struct intel_cdclk_config *b)
>> >>>
>> >>> The new name isn't very descriptive either.
>> >>>
>> >>> Outside the cd2x/crawl/squash cases we stil have to consider
>> >>> two cases:
>> >>> 1. cdclk frequency/pll changes (voltage level can change or not)
>> >>> 2. cdclk frequency/pll doesn't change, but voltage level needs to change
>> >>>
>> >>> And that difference is what intel_cdclk_needs_modeset() is trying
>> >>> convey. And intel_cdclk_changed() tells us whether anything at all
>> >>> is changing.
>> >>
>> >>intel_cdclk_clock_changed() is perhaps the best name I can
>> >>think of, off the top of my head. But I'm notoriously bad at
>> >>naming things so take that with a pinch of salt.
>> >
>> >Sounds better indeed. And I think intel_cdclk_changed() would be more
>> >concise, but that one is already taken internally.
>>
>> Hm... On second thought, I'm not sure intel_cdclk_clock_changed() would
>> be very accurate, at least with the current implementation. We are
>> checking not only on the resulting cdclk frequency, but also on
>> ref and vco values.
>
>That's all about the clock.
>
>>
>> Although I'm not sure if that could happen in real life, but the current
>> implementation gives me the interpretation that, in theory, a change in
>> ref or vco could happen even if the cdclk value did not change. Or are
>> we safe to say that checking on cdclk should be enough and looking at
>> vco and ref is actually redundant?
>
>ref never changes, vco could in theory change such that the
>resulting cdclk stays the same, but probably no real world
>examples of that.
>
>But I don't see any point in putting those kinds of assumptions
>into the code given that it wouldn't provide any actual benefits.
Alright. Thanks!
I've sent a v2 using the proposed name:
https://patchwork.freedesktop.org/series/129908/
--
Gustavo Sousa
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2024-02-14 20:31 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-02 13:12 [PATCH] drm/i915/cdclk: Rename intel_cdclk_needs_modeset to intel_cdclk_params_changed Gustavo Sousa
2024-02-02 18:47 ` ✓ Fi.CI.BAT: success for " Patchwork
2024-02-02 19:58 ` [PATCH] " Ville Syrjälä
2024-02-02 20:06 ` Ville Syrjälä
2024-02-03 13:32 ` Gustavo Sousa
2024-02-14 20:08 ` Gustavo Sousa
2024-02-14 20:15 ` Ville Syrjälä
2024-02-14 20:30 ` Gustavo Sousa
2024-02-03 13:25 ` Gustavo Sousa
2024-02-05 15:34 ` Ville Syrjälä
2024-02-14 19:56 ` Gustavo Sousa
2024-02-14 20:08 ` Ville Syrjälä
2024-02-02 20:17 ` ✓ Fi.CI.IGT: success for " 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.