diff mbox series

[v3] drm/i915: handle uncore spinlock when not available

Message ID 20231023103328.1495942-1-luciano.coelho@intel.com (mailing list archive)
State New, archived
Headers show
Series [v3] drm/i915: handle uncore spinlock when not available | expand

Commit Message

Luca Coelho Oct. 23, 2023, 10:33 a.m. UTC
The uncore code may not always be available (e.g. when we build the
display code with Xe), so we can't always rely on having the uncore's
spinlock.

To handle this, split the spin_lock/unlock_irqsave/restore() into
spin_lock/unlock() followed by a call to local_irq_save/restore() and
create wrapper functions for locking and unlocking the uncore's
spinlock.  In these functions, we have a condition check and only
actually try to lock/unlock the spinlock when I915 is defined, and
thus uncore is available.

This keeps the ifdefs contained in these new functions and all such
logic inside the display code.

Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
---

In v2:

   * Renamed uncore_spin_*() to intel_spin_*()
   * Corrected the order: save, lock, unlock, restore

In v3:

   * Undid the change to pass drm_i915_private instead of the lock
     itself, since we would have to include i915_drv.h and that pulls
     in a truckload of other includes.

 drivers/gpu/drm/i915/display/intel_display.h | 20 ++++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_vblank.c  | 19 ++++++++++++-------
 2 files changed, 32 insertions(+), 7 deletions(-)

Comments

Tvrtko Ursulin Oct. 25, 2023, 10:18 a.m. UTC | #1
On 23/10/2023 11:33, Luca Coelho wrote:
> The uncore code may not always be available (e.g. when we build the
> display code with Xe), so we can't always rely on having the uncore's
> spinlock.
> 
> To handle this, split the spin_lock/unlock_irqsave/restore() into
> spin_lock/unlock() followed by a call to local_irq_save/restore() and
> create wrapper functions for locking and unlocking the uncore's
> spinlock.  In these functions, we have a condition check and only
> actually try to lock/unlock the spinlock when I915 is defined, and
> thus uncore is available.
> 
> This keeps the ifdefs contained in these new functions and all such
> logic inside the display code.
> 
> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
> ---
> 
> In v2:
> 
>     * Renamed uncore_spin_*() to intel_spin_*()
>     * Corrected the order: save, lock, unlock, restore
> 
> In v3:
> 
>     * Undid the change to pass drm_i915_private instead of the lock
>       itself, since we would have to include i915_drv.h and that pulls
>       in a truckload of other includes.
> 
>   drivers/gpu/drm/i915/display/intel_display.h | 20 ++++++++++++++++++++
>   drivers/gpu/drm/i915/display/intel_vblank.c  | 19 ++++++++++++-------
>   2 files changed, 32 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h
> index 0e5dffe8f018..2a33fcc8ce68 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.h
> +++ b/drivers/gpu/drm/i915/display/intel_display.h
> @@ -559,4 +559,24 @@ bool assert_port_valid(struct drm_i915_private *i915, enum port port);
>   
>   bool intel_scanout_needs_vtd_wa(struct drm_i915_private *i915);
>   
> +/*
> + * The uncore version of the spin lock functions is used to decide
> + * whether we need to lock the uncore lock or not.  This is only
> + * needed in i915, not in Xe.  Keep the decision-making centralized
> + * here.
> + */
> +static inline void intel_spin_lock(spinlock_t *lock)
> +{
> +#ifdef I915
> +	spin_lock(lock);
> +#endif
> +}
> +
> +static inline void intel_spin_unlock(spinlock_t *lock)
> +{
> +#ifdef I915
> +	spin_unlock(lock);
> +#endif
> +}
> +
>   #endif
> diff --git a/drivers/gpu/drm/i915/display/intel_vblank.c b/drivers/gpu/drm/i915/display/intel_vblank.c
> index 2cec2abf9746..9b482d648762 100644
> --- a/drivers/gpu/drm/i915/display/intel_vblank.c
> +++ b/drivers/gpu/drm/i915/display/intel_vblank.c
> @@ -306,7 +306,8 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
>   	 * register reads, potentially with preemption disabled, so the
>   	 * following code must not block on uncore.lock.
>   	 */
> -	spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> +	local_irq_save(irqflags);

Does Xe needs interrupts off?

> +	intel_spin_lock(&dev_priv->uncore.lock);

My 2p/c is that intel_spin_lock as a name does not work when it is 
specifically about the single and specific (uncore) lock. One cannot 
call intel_spin_lock(some->other->lock) etc.

Perhaps call it i915_uncore_lock_irqsave(i915, flags) so it is clear it 
is only for i915.

Regards,

Tvrtko

>   	/* preempt_disable_rt() should go right here in PREEMPT_RT patchset. */
>   
> @@ -374,7 +375,8 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
>   
>   	/* preempt_enable_rt() should go right here in PREEMPT_RT patchset. */
>   
> -	spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
> +	intel_spin_unlock(&dev_priv->uncore.lock);
> +	local_irq_restore(irqflags);
>   
>   	/*
>   	 * While in vblank, position will be negative
> @@ -412,9 +414,13 @@ int intel_get_crtc_scanline(struct intel_crtc *crtc)
>   	unsigned long irqflags;
>   	int position;
>   
> -	spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> +	local_irq_save(irqflags);
> +	intel_spin_lock(&dev_priv->uncore.lock);
> +
>   	position = __intel_get_crtc_scanline(crtc);
> -	spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
> +
> +	intel_spin_unlock(&dev_priv->uncore.lock);
> +	local_irq_restore(irqflags);
>   
>   	return position;
>   }
> @@ -537,7 +543,7 @@ void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
>   	 * Need to audit everything to make sure it's safe.
>   	 */
>   	spin_lock_irqsave(&i915->drm.vblank_time_lock, irqflags);
> -	spin_lock(&i915->uncore.lock);
> +	intel_spin_lock(&i915->uncore.lock);
>   
>   	drm_calc_timestamping_constants(&crtc->base, &adjusted_mode);
>   
> @@ -546,7 +552,6 @@ void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
>   	crtc->mode_flags = mode_flags;
>   
>   	crtc->scanline_offset = intel_crtc_scanline_offset(crtc_state);
> -
> -	spin_unlock(&i915->uncore.lock);
> +	intel_spin_unlock(&i915->uncore.lock);
>   	spin_unlock_irqrestore(&i915->drm.vblank_time_lock, irqflags);
>   }
Tvrtko Ursulin Oct. 25, 2023, 10:25 a.m. UTC | #2
On 25/10/2023 11:18, Tvrtko Ursulin wrote:
> 
> On 23/10/2023 11:33, Luca Coelho wrote:
>> The uncore code may not always be available (e.g. when we build the
>> display code with Xe), so we can't always rely on having the uncore's
>> spinlock.
>>
>> To handle this, split the spin_lock/unlock_irqsave/restore() into
>> spin_lock/unlock() followed by a call to local_irq_save/restore() and
>> create wrapper functions for locking and unlocking the uncore's
>> spinlock.  In these functions, we have a condition check and only
>> actually try to lock/unlock the spinlock when I915 is defined, and
>> thus uncore is available.
>>
>> This keeps the ifdefs contained in these new functions and all such
>> logic inside the display code.
>>
>> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
>> ---
>>
>> In v2:
>>
>>     * Renamed uncore_spin_*() to intel_spin_*()
>>     * Corrected the order: save, lock, unlock, restore
>>
>> In v3:
>>
>>     * Undid the change to pass drm_i915_private instead of the lock
>>       itself, since we would have to include i915_drv.h and that pulls
>>       in a truckload of other includes.
>>
>>   drivers/gpu/drm/i915/display/intel_display.h | 20 ++++++++++++++++++++
>>   drivers/gpu/drm/i915/display/intel_vblank.c  | 19 ++++++++++++-------
>>   2 files changed, 32 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_display.h 
>> b/drivers/gpu/drm/i915/display/intel_display.h
>> index 0e5dffe8f018..2a33fcc8ce68 100644
>> --- a/drivers/gpu/drm/i915/display/intel_display.h
>> +++ b/drivers/gpu/drm/i915/display/intel_display.h
>> @@ -559,4 +559,24 @@ bool assert_port_valid(struct drm_i915_private 
>> *i915, enum port port);
>>   bool intel_scanout_needs_vtd_wa(struct drm_i915_private *i915);
>> +/*
>> + * The uncore version of the spin lock functions is used to decide
>> + * whether we need to lock the uncore lock or not.  This is only
>> + * needed in i915, not in Xe.  Keep the decision-making centralized
>> + * here.
>> + */
>> +static inline void intel_spin_lock(spinlock_t *lock)
>> +{
>> +#ifdef I915
>> +    spin_lock(lock);
>> +#endif
>> +}
>> +
>> +static inline void intel_spin_unlock(spinlock_t *lock)
>> +{
>> +#ifdef I915
>> +    spin_unlock(lock);
>> +#endif
>> +}
>> +
>>   #endif
>> diff --git a/drivers/gpu/drm/i915/display/intel_vblank.c 
>> b/drivers/gpu/drm/i915/display/intel_vblank.c
>> index 2cec2abf9746..9b482d648762 100644
>> --- a/drivers/gpu/drm/i915/display/intel_vblank.c
>> +++ b/drivers/gpu/drm/i915/display/intel_vblank.c
>> @@ -306,7 +306,8 @@ static bool i915_get_crtc_scanoutpos(struct 
>> drm_crtc *_crtc,
>>        * register reads, potentially with preemption disabled, so the
>>        * following code must not block on uncore.lock.
>>        */
>> -    spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
>> +    local_irq_save(irqflags);
> 
> Does Xe needs interrupts off?
> 
>> +    intel_spin_lock(&dev_priv->uncore.lock);
> 
> My 2p/c is that intel_spin_lock as a name does not work when it is 
> specifically about the single and specific (uncore) lock. One cannot 
> call intel_spin_lock(some->other->lock) etc.
> 
> Perhaps call it i915_uncore_lock_irqsave(i915, flags) so it is clear it 
> is only for i915.

Or, if the implementation will later gain the #else block for Xe, 
perhaps intel_uncore_lock_...?

Regards,

Tvrtko

>>       /* preempt_disable_rt() should go right here in PREEMPT_RT 
>> patchset. */
>> @@ -374,7 +375,8 @@ static bool i915_get_crtc_scanoutpos(struct 
>> drm_crtc *_crtc,
>>       /* preempt_enable_rt() should go right here in PREEMPT_RT 
>> patchset. */
>> -    spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
>> +    intel_spin_unlock(&dev_priv->uncore.lock);
>> +    local_irq_restore(irqflags);
>>       /*
>>        * While in vblank, position will be negative
>> @@ -412,9 +414,13 @@ int intel_get_crtc_scanline(struct intel_crtc *crtc)
>>       unsigned long irqflags;
>>       int position;
>> -    spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
>> +    local_irq_save(irqflags);
>> +    intel_spin_lock(&dev_priv->uncore.lock);
>> +
>>       position = __intel_get_crtc_scanline(crtc);
>> -    spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
>> +
>> +    intel_spin_unlock(&dev_priv->uncore.lock);
>> +    local_irq_restore(irqflags);
>>       return position;
>>   }
>> @@ -537,7 +543,7 @@ void intel_crtc_update_active_timings(const struct 
>> intel_crtc_state *crtc_state,
>>        * Need to audit everything to make sure it's safe.
>>        */
>>       spin_lock_irqsave(&i915->drm.vblank_time_lock, irqflags);
>> -    spin_lock(&i915->uncore.lock);
>> +    intel_spin_lock(&i915->uncore.lock);
>>       drm_calc_timestamping_constants(&crtc->base, &adjusted_mode);
>> @@ -546,7 +552,6 @@ void intel_crtc_update_active_timings(const struct 
>> intel_crtc_state *crtc_state,
>>       crtc->mode_flags = mode_flags;
>>       crtc->scanline_offset = intel_crtc_scanline_offset(crtc_state);
>> -
>> -    spin_unlock(&i915->uncore.lock);
>> +    intel_spin_unlock(&i915->uncore.lock);
>>       spin_unlock_irqrestore(&i915->drm.vblank_time_lock, irqflags);
>>   }
Luca Coelho Oct. 25, 2023, 10:32 a.m. UTC | #3
On Wed, 2023-10-25 at 11:25 +0100, Tvrtko Ursulin wrote:
> On 25/10/2023 11:18, Tvrtko Ursulin wrote:
> > 
> > On 23/10/2023 11:33, Luca Coelho wrote:
> > > The uncore code may not always be available (e.g. when we build the
> > > display code with Xe), so we can't always rely on having the uncore's
> > > spinlock.
> > > 
> > > To handle this, split the spin_lock/unlock_irqsave/restore() into
> > > spin_lock/unlock() followed by a call to local_irq_save/restore() and
> > > create wrapper functions for locking and unlocking the uncore's
> > > spinlock.  In these functions, we have a condition check and only
> > > actually try to lock/unlock the spinlock when I915 is defined, and
> > > thus uncore is available.
> > > 
> > > This keeps the ifdefs contained in these new functions and all such
> > > logic inside the display code.
> > > 
> > > Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
> > > ---
> > > 
> > > In v2:
> > > 
> > >     * Renamed uncore_spin_*() to intel_spin_*()
> > >     * Corrected the order: save, lock, unlock, restore
> > > 
> > > In v3:
> > > 
> > >     * Undid the change to pass drm_i915_private instead of the lock
> > >       itself, since we would have to include i915_drv.h and that pulls
> > >       in a truckload of other includes.
> > > 
> > >   drivers/gpu/drm/i915/display/intel_display.h | 20 ++++++++++++++++++++
> > >   drivers/gpu/drm/i915/display/intel_vblank.c  | 19 ++++++++++++-------
> > >   2 files changed, 32 insertions(+), 7 deletions(-)
> > > 
> > > diff --git a/drivers/gpu/drm/i915/display/intel_display.h 
> > > b/drivers/gpu/drm/i915/display/intel_display.h
> > > index 0e5dffe8f018..2a33fcc8ce68 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_display.h
> > > +++ b/drivers/gpu/drm/i915/display/intel_display.h
> > > @@ -559,4 +559,24 @@ bool assert_port_valid(struct drm_i915_private 
> > > *i915, enum port port);
> > >   bool intel_scanout_needs_vtd_wa(struct drm_i915_private *i915);
> > > +/*
> > > + * The uncore version of the spin lock functions is used to decide
> > > + * whether we need to lock the uncore lock or not.  This is only
> > > + * needed in i915, not in Xe.  Keep the decision-making centralized
> > > + * here.
> > > + */
> > > +static inline void intel_spin_lock(spinlock_t *lock)
> > > +{
> > > +#ifdef I915
> > > +    spin_lock(lock);
> > > +#endif
> > > +}
> > > +
> > > +static inline void intel_spin_unlock(spinlock_t *lock)
> > > +{
> > > +#ifdef I915
> > > +    spin_unlock(lock);
> > > +#endif
> > > +}
> > > +
> > >   #endif
> > > diff --git a/drivers/gpu/drm/i915/display/intel_vblank.c 
> > > b/drivers/gpu/drm/i915/display/intel_vblank.c
> > > index 2cec2abf9746..9b482d648762 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_vblank.c
> > > +++ b/drivers/gpu/drm/i915/display/intel_vblank.c
> > > @@ -306,7 +306,8 @@ static bool i915_get_crtc_scanoutpos(struct 
> > > drm_crtc *_crtc,
> > >        * register reads, potentially with preemption disabled, so the
> > >        * following code must not block on uncore.lock.
> > >        */
> > > -    spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
> > > +    local_irq_save(irqflags);
> > 
> > Does Xe needs interrupts off?

I'm actually not sure, but this is how it was in the Xe driver code, so
I kept it.


> > > +    intel_spin_lock(&dev_priv->uncore.lock);
> > 
> > My 2p/c is that intel_spin_lock as a name does not work when it is 
> > specifically about the single and specific (uncore) lock. One cannot 
> > call intel_spin_lock(some->other->lock) etc.

Right, this was changed when I was passing only dev_priv, but I
couldn't do that wihtout adding i915_drv.h, which was not good
either... But yeah, this is too generic, while the actual case is not.


> > Perhaps call it i915_uncore_lock_irqsave(i915, flags) so it is clear it 
> > is only for i915.

I wanted to avoid "i915", since we also call it when the display is
used with xe...


> Or, if the implementation will later gain the #else block for Xe, 
> perhaps intel_uncore_lock_...?

But still, uncore doesn't exist in Xe, so this is still not good...

Any other suggestions?

--
Cheers,
Luca.
Luca Coelho Oct. 25, 2023, 10:52 a.m. UTC | #4
Hi,

Possible regressions

  *   igt@kms<mailto:igt@kms>_flip@flip-vs-absolute-wf<mailto:_flip@flip-vs-absolute-wf>_vblank@a-vga1<mailto:_vblank@a-vga1>:
     *   shard-snb: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-snb4/igt@kms_flip@flip-vs-absolute-wf_vblank@a-vga1.html> -> ABORT<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb2/igt@kms_flip@flip-vs-absolute-wf_vblank@a-vga1.html>

FTR, this was a lockdep splat happening in block devices, so it has nothing to do with my change.

I won't retrigger because I may send a new version after Tvrtko's comments, though.

--
Cheers,
Luca.


On Wed, 2023-10-25 at 09:50 +0000, Patchwork wrote:
Patch Details
Series: drm/i915: handle uncore spinlock when not available (rev2)
URL:    https://patchwork.freedesktop.org/series/125442/
State:  failure
Details:        https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/index.html
CI Bug Log - changes from CI_DRM_13781_full -> Patchwork_125442v2_full
Summary

FAILURE

Serious unknown changes coming with Patchwork_125442v2_full absolutely need to be
verified manually.

If you think the reported changes have nothing to do with the changes
introduced in Patchwork_125442v2_full, please notify your bug team (lgci.bug.filing@intel.com) to allow them
to document this new failure mode, which will reduce false positives in CI.

External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/index.html

Participating hosts (11 -> 12)

Additional (1): shard-rkl0

Possible new issues

Here are the unknown changes that may have been introduced in Patchwork_125442v2_full:

IGT changes
Possible regressions

  *   igt@kms_flip@flip-vs-absolute-wf_vblank@a-vga1:
     *   shard-snb: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-snb4/igt@kms_flip@flip-vs-absolute-wf_vblank@a-vga1.html> -> ABORT<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb2/igt@kms_flip@flip-vs-absolute-wf_vblank@a-vga1.html>

Known issues

Here are the changes found in Patchwork_125442v2_full that come from known issues:

CI changes
Issues hit

  *   boot:
     *   shard-apl: (PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl1/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl1/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl1/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl2/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl2/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl2/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl3/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl3/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl3/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-apl6/boot.html>) -> (PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl1/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl1/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl1/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl2/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl2/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl2/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/boot.html>, FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl6/boot.html>, PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl6/boot.html>) (i915#8293<https://gitlab.freedesktop.org/drm/intel/issues/8293>)

IGT changes
Issues hit

  *   igt@api_intel_bb@blit-reloc-purge-cache:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@api_intel_bb@blit-reloc-purge-cache.html> (i915#8411<https://gitlab.freedesktop.org/drm/intel/issues/8411>) +2 other tests skip
  *   igt@drm_fdinfo@busy@ccs0:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@drm_fdinfo@busy@ccs0.html> (i915#8414<https://gitlab.freedesktop.org/drm/intel/issues/8414>) +30 other tests skip
  *   igt@drm_fdinfo@most-busy-idle-check-all@ccs0:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@drm_fdinfo@most-busy-idle-check-all@ccs0.html> (i915#8414<https://gitlab.freedesktop.org/drm/intel/issues/8414>) +5 other tests skip
  *   igt@drm_fdinfo@most-busy-idle-check-all@rcs0:

     *   shard-rkl: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-1/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-1/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html> (i915#7742<https://gitlab.freedesktop.org/drm/intel/issues/7742>)
  *   igt@drm_fdinfo@virtual-busy-idle:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@drm_fdinfo@virtual-busy-idle.html> (i915#8414<https://gitlab.freedesktop.org/drm/intel/issues/8414>)
  *   igt@gem_basic@multigpu-create-close:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_basic@multigpu-create-close.html> (i915#7697<https://gitlab.freedesktop.org/drm/intel/issues/7697>)
  *   igt@gem_ccs@ctrl-surf-copy-new-ctx:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_ccs@ctrl-surf-copy-new-ctx.html> (i915#9323<https://gitlab.freedesktop.org/drm/intel/issues/9323>)
  *   igt@gem_ccs@suspend-resume@tile4-compressed-compfmt0-smem-lmem0:

     *   shard-dg2: NOTRUN -> INCOMPLETE<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@gem_ccs@suspend-resume@tile4-compressed-compfmt0-smem-lmem0.html> (i915#7297<https://gitlab.freedesktop.org/drm/intel/issues/7297>)
  *   igt@gem_ctx_exec@basic-nohangcheck:

     *   shard-rkl: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-7/igt@gem_ctx_exec@basic-nohangcheck.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-7/igt@gem_ctx_exec@basic-nohangcheck.html> (i915#6268<https://gitlab.freedesktop.org/drm/intel/issues/6268>)
  *   igt@gem_ctx_persistence@engines-persistence:

     *   shard-snb: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb2/igt@gem_ctx_persistence@engines-persistence.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271> / i915#1099<https://gitlab.freedesktop.org/drm/intel/issues/1099>)
  *   igt@gem_ctx_persistence@heartbeat-hang:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@gem_ctx_persistence@heartbeat-hang.html> (i915#8555<https://gitlab.freedesktop.org/drm/intel/issues/8555>) +2 other tests skip
  *   igt@gem_ctx_persistence@heartbeat-many:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_ctx_persistence@heartbeat-many.html> (i915#8555<https://gitlab.freedesktop.org/drm/intel/issues/8555>)
  *   igt@gem_ctx_persistence@saturated-hostile-nopreempt@ccs0:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_ctx_persistence@saturated-hostile-nopreempt@ccs0.html> (i915#5882<https://gitlab.freedesktop.org/drm/intel/issues/5882>) +9 other tests skip
  *   igt@gem_ctx_sseu@invalid-args:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_ctx_sseu@invalid-args.html> (i915#280<https://gitlab.freedesktop.org/drm/intel/issues/280>)
  *   igt@gem_eio@unwedge-stress:

     *   shard-dg1: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg1-12/igt@gem_eio@unwedge-stress.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-17/igt@gem_eio@unwedge-stress.html> (i915#5784<https://gitlab.freedesktop.org/drm/intel/issues/5784>) +1 other test fail
  *   igt@gem_exec_balancer@bonded-false-hang:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_exec_balancer@bonded-false-hang.html> (i915#4812<https://gitlab.freedesktop.org/drm/intel/issues/4812>)
  *   igt@gem_exec_balancer@bonded-pair:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_exec_balancer@bonded-pair.html> (i915#4771<https://gitlab.freedesktop.org/drm/intel/issues/4771>)
  *   igt@gem_exec_balancer@bonded-sync:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_exec_balancer@bonded-sync.html> (i915#4771<https://gitlab.freedesktop.org/drm/intel/issues/4771>)
  *   igt@gem_exec_balancer@invalid-bonds:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_exec_balancer@invalid-bonds.html> (i915#4036<https://gitlab.freedesktop.org/drm/intel/issues/4036>)
  *   igt@gem_exec_capture@capture-invisible@lmem0:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@gem_exec_capture@capture-invisible@lmem0.html> (i915#6334<https://gitlab.freedesktop.org/drm/intel/issues/6334>) +1 other test skip
  *   igt@gem_exec_fair@basic-none:

     *   shard-snb: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb2/igt@gem_exec_fair@basic-none.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +130 other tests skip
  *   igt@gem_exec_fair@basic-pace-share@rcs0:

     *   shard-glk: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-glk5/igt@gem_exec_fair@basic-pace-share@rcs0.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-glk5/igt@gem_exec_fair@basic-pace-share@rcs0.html> (i915#2842<https://gitlab.freedesktop.org/drm/intel/issues/2842>)
  *   igt@gem_exec_fair@basic-pace-solo@rcs0:

     *   shard-rkl: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-1/igt@gem_exec_fair@basic-pace-solo@rcs0.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@gem_exec_fair@basic-pace-solo@rcs0.html> (i915#2842<https://gitlab.freedesktop.org/drm/intel/issues/2842>)
  *   igt@gem_exec_fair@basic-throttle:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@gem_exec_fair@basic-throttle.html> (i915#3539<https://gitlab.freedesktop.org/drm/intel/issues/3539>) +2 other tests skip
  *   igt@gem_exec_fence@submit3:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@gem_exec_fence@submit3.html> (i915#4812<https://gitlab.freedesktop.org/drm/intel/issues/4812>) +3 other tests skip
  *   igt@gem_exec_flush@basic-uc-pro-default:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_exec_flush@basic-uc-pro-default.html> (i915#3539<https://gitlab.freedesktop.org/drm/intel/issues/3539> / i915#4852<https://gitlab.freedesktop.org/drm/intel/issues/4852>) +6 other tests skip
  *   igt@gem_exec_flush@basic-uc-ro-default:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_exec_flush@basic-uc-ro-default.html> (i915#3539<https://gitlab.freedesktop.org/drm/intel/issues/3539> / i915#4852<https://gitlab.freedesktop.org/drm/intel/issues/4852>)
  *   igt@gem_exec_params@rsvd2-dirt:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@gem_exec_params@rsvd2-dirt.html> (fdo#109283<https://bugs.freedesktop.org/show_bug.cgi?id=109283> / i915#5107<https://gitlab.freedesktop.org/drm/intel/issues/5107>)
  *   igt@gem_exec_params@secure-non-root:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_exec_params@secure-non-root.html> (fdo#112283<https://bugs.freedesktop.org/show_bug.cgi?id=112283>)
  *   igt@gem_exec_reloc@basic-cpu-read-active:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_exec_reloc@basic-cpu-read-active.html> (i915#3281<https://gitlab.freedesktop.org/drm/intel/issues/3281>)
  *   igt@gem_exec_reloc@basic-wc-gtt-noreloc:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_exec_reloc@basic-wc-gtt-noreloc.html> (i915#3281<https://gitlab.freedesktop.org/drm/intel/issues/3281>) +2 other tests skip
  *   igt@gem_exec_reloc@basic-write-read-active:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_exec_reloc@basic-write-read-active.html> (i915#3281<https://gitlab.freedesktop.org/drm/intel/issues/3281>) +19 other tests skip
  *   igt@gem_exec_schedule@preempt-queue-chain:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_exec_schedule@preempt-queue-chain.html> (i915#4537<https://gitlab.freedesktop.org/drm/intel/issues/4537> / i915#4812<https://gitlab.freedesktop.org/drm/intel/issues/4812>)
  *   igt@gem_exec_schedule@preempt-queue-contexts-chain:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_exec_schedule@preempt-queue-contexts-chain.html> (i915#4537<https://gitlab.freedesktop.org/drm/intel/issues/4537> / i915#4812<https://gitlab.freedesktop.org/drm/intel/issues/4812>) +1 other test skip
  *   igt@gem_fence_thrash@bo-write-verify-x:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_fence_thrash@bo-write-verify-x.html> (i915#4860<https://gitlab.freedesktop.org/drm/intel/issues/4860>) +5 other tests skip
  *   igt@gem_fenced_exec_thrash@too-many-fences:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_fenced_exec_thrash@too-many-fences.html> (i915#4860<https://gitlab.freedesktop.org/drm/intel/issues/4860>)
  *   igt@gem_gtt_cpu_tlb:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_gtt_cpu_tlb.html> (i915#4077<https://gitlab.freedesktop.org/drm/intel/issues/4077>) +2 other tests skip
  *   igt@gem_lmem_swapping@massive:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_lmem_swapping@massive.html> (i915#4613<https://gitlab.freedesktop.org/drm/intel/issues/4613>)
  *   igt@gem_lmem_swapping@parallel-random-verify-ccs@lmem0:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_lmem_swapping@parallel-random-verify-ccs@lmem0.html> (i915#4565<https://gitlab.freedesktop.org/drm/intel/issues/4565>)
  *   igt@gem_lmem_swapping@smem-oom@lmem0:

     *   shard-dg1: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg1-18/igt@gem_lmem_swapping@smem-oom@lmem0.html> -> DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-16/igt@gem_lmem_swapping@smem-oom@lmem0.html> (i915#4936<https://gitlab.freedesktop.org/drm/intel/issues/4936> / i915#5493<https://gitlab.freedesktop.org/drm/intel/issues/5493>)
  *   igt@gem_lmem_swapping@verify-random:

     *   shard-apl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/igt@gem_lmem_swapping@verify-random.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271> / i915#4613<https://gitlab.freedesktop.org/drm/intel/issues/4613>) +4 other tests skip
  *   igt@gem_madvise@dontneed-before-pwrite:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_madvise@dontneed-before-pwrite.html> (i915#3282<https://gitlab.freedesktop.org/drm/intel/issues/3282>) +1 other test skip
  *   igt@gem_media_fill@media-fill:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_media_fill@media-fill.html> (i915#8289<https://gitlab.freedesktop.org/drm/intel/issues/8289>)
  *   igt@gem_mmap@short-mmap:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_mmap@short-mmap.html> (i915#4083<https://gitlab.freedesktop.org/drm/intel/issues/4083>) +9 other tests skip
  *   igt@gem_mmap_gtt@fault-concurrent:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_mmap_gtt@fault-concurrent.html> (i915#4077<https://gitlab.freedesktop.org/drm/intel/issues/4077>) +3 other tests skip
  *   igt@gem_mmap_wc@close:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_mmap_wc@close.html> (i915#4083<https://gitlab.freedesktop.org/drm/intel/issues/4083>)
  *   igt@gem_partial_pwrite_pread@reads:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@gem_partial_pwrite_pread@reads.html> (i915#3282<https://gitlab.freedesktop.org/drm/intel/issues/3282>) +7 other tests skip
  *   igt@gem_partial_pwrite_pread@reads-display:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_partial_pwrite_pread@reads-display.html> (i915#3282<https://gitlab.freedesktop.org/drm/intel/issues/3282>)
  *   igt@gem_pwrite@basic-exhaustion:

     *   shard-apl: NOTRUN -> WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl2/igt@gem_pwrite@basic-exhaustion.html> (i915#2658<https://gitlab.freedesktop.org/drm/intel/issues/2658>)
  *   igt@gem_pxp@regular-baseline-src-copy-readible:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_pxp@regular-baseline-src-copy-readible.html> (i915#4270<https://gitlab.freedesktop.org/drm/intel/issues/4270>) +1 other test skip
  *   igt@gem_pxp@reject-modify-context-protection-off-3:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_pxp@reject-modify-context-protection-off-3.html> (i915#4270<https://gitlab.freedesktop.org/drm/intel/issues/4270>) +5 other tests skip
  *   igt@gem_render_copy@y-tiled-mc-ccs-to-yf-tiled-ccs:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_render_copy@y-tiled-mc-ccs-to-yf-tiled-ccs.html> (i915#8428<https://gitlab.freedesktop.org/drm/intel/issues/8428>)
  *   igt@gem_set_tiling_vs_blt@tiled-to-untiled:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_set_tiling_vs_blt@tiled-to-untiled.html> (i915#4079<https://gitlab.freedesktop.org/drm/intel/issues/4079>) +5 other tests skip
  *   igt@gem_softpin@evict-snoop:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_softpin@evict-snoop.html> (i915#4885<https://gitlab.freedesktop.org/drm/intel/issues/4885>)
  *   igt@gem_softpin@noreloc-s3:

     *   shard-mtlp: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-5/igt@gem_softpin@noreloc-s3.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_softpin@noreloc-s3.html> (fdo#103375<https://bugs.freedesktop.org/show_bug.cgi?id=103375>)
  *   igt@gem_tiled_swapping@non-threaded:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_tiled_swapping@non-threaded.html> (i915#4077<https://gitlab.freedesktop.org/drm/intel/issues/4077>) +24 other tests skip
  *   igt@gem_unfence_active_buffers:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@gem_unfence_active_buffers.html> (i915#4879<https://gitlab.freedesktop.org/drm/intel/issues/4879>)
  *   igt@gem_userptr_blits@invalid-mmap-offset-unsync:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gem_userptr_blits@invalid-mmap-offset-unsync.html> (i915#3297<https://gitlab.freedesktop.org/drm/intel/issues/3297>)
  *   igt@gem_userptr_blits@map-fixed-invalidate-overlap:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gem_userptr_blits@map-fixed-invalidate-overlap.html> (i915#3297<https://gitlab.freedesktop.org/drm/intel/issues/3297> / i915#4880<https://gitlab.freedesktop.org/drm/intel/issues/4880>)
  *   igt@gem_userptr_blits@mmap-offset-banned@gtt:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@gem_userptr_blits@mmap-offset-banned@gtt.html> (i915#3297<https://gitlab.freedesktop.org/drm/intel/issues/3297>) +1 other test skip
  *   igt@gem_userptr_blits@unsync-unmap-after-close:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@gem_userptr_blits@unsync-unmap-after-close.html> (i915#3297<https://gitlab.freedesktop.org/drm/intel/issues/3297>) +1 other test skip
  *   igt@gem_userptr_blits@vma-merge:

     *   shard-apl: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/igt@gem_userptr_blits@vma-merge.html> (i915#3318<https://gitlab.freedesktop.org/drm/intel/issues/3318>)
  *   igt@gen7_exec_parse@chained-batch:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gen7_exec_parse@chained-batch.html> (fdo#109289<https://bugs.freedesktop.org/show_bug.cgi?id=109289>) +4 other tests skip
  *   igt@gen9_exec_parse@bb-start-far:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@gen9_exec_parse@bb-start-far.html> (i915#2856<https://gitlab.freedesktop.org/drm/intel/issues/2856>) +5 other tests skip
  *   igt@gen9_exec_parse@secure-batches:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@gen9_exec_parse@secure-batches.html> (i915#2527<https://gitlab.freedesktop.org/drm/intel/issues/2527>)
  *   igt@i915_hangman@engine-engine-error@vcs0:

     *   shard-mtlp: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@i915_hangman@engine-engine-error@vcs0.html> (i915#7069<https://gitlab.freedesktop.org/drm/intel/issues/7069>) +1 other test fail
  *   igt@i915_module_load@reload-with-fault-injection:

     *   shard-dg2: NOTRUN -> DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@i915_module_load@reload-with-fault-injection.html> (i915#9559<https://gitlab.freedesktop.org/drm/intel/issues/9559>)
  *   igt@i915_module_load@resize-bar:

     *   shard-dg2: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg2-2/igt@i915_module_load@resize-bar.html> -> DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@i915_module_load@resize-bar.html> (i915#1982<https://gitlab.freedesktop.org/drm/intel/issues/1982>)
  *   igt@i915_pm_rpm@gem-execbuf-stress-pc8:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html> (fdo#109293<https://bugs.freedesktop.org/show_bug.cgi?id=109293>)
  *   igt@i915_pm_rpm@system-suspend-execbuf:

     *   shard-rkl: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-1/igt@i915_pm_rpm@system-suspend-execbuf.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@i915_pm_rpm@system-suspend-execbuf.html> (fdo#103375<https://bugs.freedesktop.org/show_bug.cgi?id=103375>)
  *   igt@i915_pm_rps@engine-order:

     *   shard-apl: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl6/igt@i915_pm_rps@engine-order.html> (i915#6537<https://gitlab.freedesktop.org/drm/intel/issues/6537>)
  *   igt@i915_selftest@mock@memory_region:

     *   shard-dg1: NOTRUN -> DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@i915_selftest@mock@memory_region.html> (i915#9311<https://gitlab.freedesktop.org/drm/intel/issues/9311>)
  *   igt@kms_addfb_basic@tile-pitch-mismatch:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_addfb_basic@tile-pitch-mismatch.html> (i915#4212<https://gitlab.freedesktop.org/drm/intel/issues/4212> / i915#5608<https://gitlab.freedesktop.org/drm/intel/issues/5608>)
  *   igt@kms_async_flips@crc@pipe-a-hdmi-a-2:

     *   shard-dg2: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_async_flips@crc@pipe-a-hdmi-a-2.html> (i915#8247<https://gitlab.freedesktop.org/drm/intel/issues/8247>) +3 other tests fail
  *   igt@kms_big_fb@4-tiled-16bpp-rotate-270:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_big_fb@4-tiled-16bpp-rotate-270.html> (fdo#111614<https://bugs.freedesktop.org/show_bug.cgi?id=111614>)
  *   igt@kms_big_fb@4-tiled-64bpp-rotate-180:

     *   shard-mtlp: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-1/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-1/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html> (i915#5138<https://gitlab.freedesktop.org/drm/intel/issues/5138>)
  *   igt@kms_big_fb@4-tiled-addfb:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_big_fb@4-tiled-addfb.html> (i915#5286<https://gitlab.freedesktop.org/drm/intel/issues/5286>)
  *   igt@kms_big_fb@linear-32bpp-rotate-270:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_big_fb@linear-32bpp-rotate-270.html> (i915#3638<https://gitlab.freedesktop.org/drm/intel/issues/3638>)
  *   igt@kms_big_fb@x-tiled-32bpp-rotate-270:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_big_fb@x-tiled-32bpp-rotate-270.html> (fdo#111614<https://bugs.freedesktop.org/show_bug.cgi?id=111614>) +9 other tests skip
  *   igt@kms_big_fb@y-tiled-8bpp-rotate-180:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_big_fb@y-tiled-8bpp-rotate-180.html> (i915#5190<https://gitlab.freedesktop.org/drm/intel/issues/5190>) +26 other tests skip
  *   igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip:

     *   shard-tglu: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-tglu-3/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-tglu-2/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip.html> (i915#3743<https://gitlab.freedesktop.org/drm/intel/issues/3743>)
  *   igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html> (fdo#111615<https://bugs.freedesktop.org/show_bug.cgi?id=111615>) +1 other test skip
  *   igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180.html> (i915#4538<https://gitlab.freedesktop.org/drm/intel/issues/4538>)
  *   igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0.html> (i915#4538<https://gitlab.freedesktop.org/drm/intel/issues/4538> / i915#5190<https://gitlab.freedesktop.org/drm/intel/issues/5190>) +10 other tests skip
  *   igt@kms_big_joiner@invalid-modeset:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_big_joiner@invalid-modeset.html> (i915#2705<https://gitlab.freedesktop.org/drm/intel/issues/2705>) +2 other tests skip
  *   igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3.html> (i915#4087<https://gitlab.freedesktop.org/drm/intel/issues/4087> / i915#7213<https://gitlab.freedesktop.org/drm/intel/issues/7213>) +4 other tests skip
  *   igt@kms_cdclk@plane-scaling@pipe-c-hdmi-a-3:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@kms_cdclk@plane-scaling@pipe-c-hdmi-a-3.html> (i915#4087<https://gitlab.freedesktop.org/drm/intel/issues/4087>) +3 other tests skip
  *   igt@kms_chamelium_color@ctm-green-to-red:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_chamelium_color@ctm-green-to-red.html> (fdo#111827<https://bugs.freedesktop.org/show_bug.cgi?id=111827>)
  *   igt@kms_chamelium_color@ctm-negative:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_chamelium_color@ctm-negative.html> (fdo#111827<https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +3 other tests skip
  *   igt@kms_chamelium_frames@dp-crc-fast:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_chamelium_frames@dp-crc-fast.html> (i915#7828<https://gitlab.freedesktop.org/drm/intel/issues/7828>) +1 other test skip
  *   igt@kms_chamelium_frames@hdmi-cmp-planar-formats:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_chamelium_frames@hdmi-cmp-planar-formats.html> (i915#7828<https://gitlab.freedesktop.org/drm/intel/issues/7828>) +15 other tests skip
  *   igt@kms_chamelium_hpd@common-hpd-after-suspend:

     *   shard-apl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/igt@kms_chamelium_hpd@common-hpd-after-suspend.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +220 other tests skip
  *   igt@kms_chamelium_hpd@hdmi-hpd-storm-disable:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_chamelium_hpd@hdmi-hpd-storm-disable.html> (i915#7828<https://gitlab.freedesktop.org/drm/intel/issues/7828>) +1 other test skip
  *   igt@kms_color@deep-color@pipe-a-hdmi-a-2-gamma:

     *   shard-rkl: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@kms_color@deep-color@pipe-a-hdmi-a-2-gamma.html> (i915#6892<https://gitlab.freedesktop.org/drm/intel/issues/6892>) +1 other test fail
  *   igt@kms_content_protection@dp-mst-lic-type-1:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_content_protection@dp-mst-lic-type-1.html> (i915#3299<https://gitlab.freedesktop.org/drm/intel/issues/3299>)
  *   igt@kms_content_protection@dp-mst-type-1:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_content_protection@dp-mst-type-1.html> (i915#3299<https://gitlab.freedesktop.org/drm/intel/issues/3299>)
  *   igt@kms_content_protection@legacy:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_content_protection@legacy.html> (i915#6944<https://gitlab.freedesktop.org/drm/intel/issues/6944>)
  *   igt@kms_content_protection@lic@pipe-a-dp-1:

     *   shard-apl: NOTRUN -> TIMEOUT<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl2/igt@kms_content_protection@lic@pipe-a-dp-1.html> (i915#7173<https://gitlab.freedesktop.org/drm/intel/issues/7173>)
  *   igt@kms_content_protection@lic@pipe-a-dp-4:

     *   shard-dg2: NOTRUN -> TIMEOUT<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_content_protection@lic@pipe-a-dp-4.html> (i915#7173<https://gitlab.freedesktop.org/drm/intel/issues/7173>)
  *   igt@kms_content_protection@type1:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_content_protection@type1.html> (i915#7118<https://gitlab.freedesktop.org/drm/intel/issues/7118>) +1 other test skip
  *   igt@kms_content_protection@uevent@pipe-a-dp-4:

     *   shard-dg2: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_content_protection@uevent@pipe-a-dp-4.html> (i915#1339<https://gitlab.freedesktop.org/drm/intel/issues/1339>)
  *   igt@kms_cursor_crc@cursor-offscreen-512x170:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_cursor_crc@cursor-offscreen-512x170.html> (i915#3359<https://gitlab.freedesktop.org/drm/intel/issues/3359>)
  *   igt@kms_cursor_crc@cursor-random-512x170:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_cursor_crc@cursor-random-512x170.html> (i915#3359<https://gitlab.freedesktop.org/drm/intel/issues/3359>) +3 other tests skip
  *   igt@kms_cursor_crc@cursor-rapid-movement-32x32:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_cursor_crc@cursor-rapid-movement-32x32.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#8814<https://gitlab.freedesktop.org/drm/intel/issues/8814>)
  *   igt@kms_cursor_crc@cursor-sliding-32x10:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_cursor_crc@cursor-sliding-32x10.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555>) +8 other tests skip
  *   igt@kms_cursor_crc@cursor-sliding-512x170:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_cursor_crc@cursor-sliding-512x170.html> (i915#3359<https://gitlab.freedesktop.org/drm/intel/issues/3359>)
  *   igt@kms_cursor_legacy@2x-long-cursor-vs-flip-legacy:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-legacy.html> (i915#3546<https://gitlab.freedesktop.org/drm/intel/issues/3546>)
  *   igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html> (i915#4103<https://gitlab.freedesktop.org/drm/intel/issues/4103> / i915#4213<https://gitlab.freedesktop.org/drm/intel/issues/4213> / i915#5608<https://gitlab.freedesktop.org/drm/intel/issues/5608>)
  *   igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html> (fdo#109274<https://bugs.freedesktop.org/show_bug.cgi?id=109274> / i915#5354<https://gitlab.freedesktop.org/drm/intel/issues/5354>) +4 other tests skip
  *   igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions:

     *   shard-snb: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb2/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271> / fdo#111767<https://bugs.freedesktop.org/show_bug.cgi?id=111767>)
  *   igt@kms_cursor_legacy@cursorb-vs-flipb-toggle:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_cursor_legacy@cursorb-vs-flipb-toggle.html> (fdo#109274<https://bugs.freedesktop.org/show_bug.cgi?id=109274> / fdo#111767<https://bugs.freedesktop.org/show_bug.cgi?id=111767> / i915#5354<https://gitlab.freedesktop.org/drm/intel/issues/5354>)
  *   igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size.html> (i915#4103<https://gitlab.freedesktop.org/drm/intel/issues/4103> / i915#4213<https://gitlab.freedesktop.org/drm/intel/issues/4213>) +1 other test skip
  *   igt@kms_dirtyfb@dirtyfb-ioctl@drrs-hdmi-a-2:

     *   shard-rkl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@kms_dirtyfb@dirtyfb-ioctl@drrs-hdmi-a-2.html> (i915#9226<https://gitlab.freedesktop.org/drm/intel/issues/9226> / i915#9261<https://gitlab.freedesktop.org/drm/intel/issues/9261>) +1 other test skip
  *   igt@kms_dirtyfb@dirtyfb-ioctl@fbc-hdmi-a-2:

     *   shard-rkl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@kms_dirtyfb@dirtyfb-ioctl@fbc-hdmi-a-2.html> (i915#9227<https://gitlab.freedesktop.org/drm/intel/issues/9227>)
  *   igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1:

     *   shard-rkl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-7/igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1.html> (i915#3804<https://gitlab.freedesktop.org/drm/intel/issues/3804>)
  *   igt@kms_dsc@dsc-basic:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_dsc@dsc-basic.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#3840<https://gitlab.freedesktop.org/drm/intel/issues/3840>) +2 other tests skip
  *   igt@kms_flip@2x-absolute-wf_vblank-interruptible:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_flip@2x-absolute-wf_vblank-interruptible.html> (i915#3637<https://gitlab.freedesktop.org/drm/intel/issues/3637>) +1 other test skip
  *   igt@kms_flip@2x-flip-vs-blocking-wf-vblank:

     *   shard-apl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/igt@kms_flip@2x-flip-vs-blocking-wf-vblank.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271> / fdo#111767<https://bugs.freedesktop.org/show_bug.cgi?id=111767>)
  *   igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html> (fdo#109274<https://bugs.freedesktop.org/show_bug.cgi?id=109274> / fdo#111767<https://bugs.freedesktop.org/show_bug.cgi?id=111767>)
  *   igt@kms_flip@2x-flip-vs-fences:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_flip@2x-flip-vs-fences.html> (i915#8381<https://gitlab.freedesktop.org/drm/intel/issues/8381>) +2 other tests skip
  *   igt@kms_flip@2x-flip-vs-rmfb-interruptible:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_flip@2x-flip-vs-rmfb-interruptible.html> (fdo#111767<https://bugs.freedesktop.org/show_bug.cgi?id=111767> / fdo#111825<https://bugs.freedesktop.org/show_bug.cgi?id=111825>)
  *   igt@kms_flip@2x-modeset-vs-vblank-race:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@kms_flip@2x-modeset-vs-vblank-race.html> (fdo#109274<https://bugs.freedesktop.org/show_bug.cgi?id=109274>) +14 other tests skip
  *   igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling@pipe-a-valid-mode:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling@pipe-a-valid-mode.html> (i915#2587<https://gitlab.freedesktop.org/drm/intel/issues/2587> / i915#2672<https://gitlab.freedesktop.org/drm/intel/issues/2672>) +1 other test skip
  *   igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling@pipe-a-default-mode:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling@pipe-a-default-mode.html> (i915#2672<https://gitlab.freedesktop.org/drm/intel/issues/2672> / i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555>)
  *   igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-upscaling@pipe-a-valid-mode:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-upscaling@pipe-a-valid-mode.html> (i915#2672<https://gitlab.freedesktop.org/drm/intel/issues/2672>) +5 other tests skip
  *   igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilegen12rcccs-upscaling@pipe-a-valid-mode:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilegen12rcccs-upscaling@pipe-a-valid-mode.html> (i915#2672<https://gitlab.freedesktop.org/drm/intel/issues/2672> / i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555>)
  *   igt@kms_force_connector_basic@prune-stale-modes:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_force_connector_basic@prune-stale-modes.html> (i915#5274<https://gitlab.freedesktop.org/drm/intel/issues/5274>)
  *   igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render:

     *   shard-dg2: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg2-10/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-render.html> (i915#6880<https://gitlab.freedesktop.org/drm/intel/issues/6880>)
  *   igt@kms_frontbuffer_tracking@fbcpsr-1p-indfb-fliptrack-mmap-gtt:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_frontbuffer_tracking@fbcpsr-1p-indfb-fliptrack-mmap-gtt.html> (i915#8708<https://gitlab.freedesktop.org/drm/intel/issues/8708>) +1 other test skip
  *   igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-mmap-wc:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-mmap-wc.html> (i915#8708<https://gitlab.freedesktop.org/drm/intel/issues/8708>) +3 other tests skip
  *   igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-pri-indfb-draw-mmap-wc:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-pri-indfb-draw-mmap-wc.html> (i915#8708<https://gitlab.freedesktop.org/drm/intel/issues/8708>) +27 other tests skip
  *   igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-pri-shrfb-draw-render:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-pri-shrfb-draw-render.html> (i915#5354<https://gitlab.freedesktop.org/drm/intel/issues/5354>) +54 other tests skip
  *   igt@kms_frontbuffer_tracking@fbcpsr-tiling-y:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_frontbuffer_tracking@fbcpsr-tiling-y.html> (i915#5460<https://gitlab.freedesktop.org/drm/intel/issues/5460>)
  *   igt@kms_frontbuffer_tracking@psr-1p-primscrn-indfb-plflip-blt:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_frontbuffer_tracking@psr-1p-primscrn-indfb-plflip-blt.html> (i915#3458<https://gitlab.freedesktop.org/drm/intel/issues/3458>) +3 other tests skip
  *   igt@kms_frontbuffer_tracking@psr-1p-rte:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_frontbuffer_tracking@psr-1p-rte.html> (i915#3458<https://gitlab.freedesktop.org/drm/intel/issues/3458>) +34 other tests skip
  *   igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-render:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-render.html> (i915#1825<https://gitlab.freedesktop.org/drm/intel/issues/1825>) +6 other tests skip
  *   igt@kms_frontbuffer_tracking@psr-2p-scndscrn-indfb-pgflip-blt:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-indfb-pgflip-blt.html> (fdo#111825<https://bugs.freedesktop.org/show_bug.cgi?id=111825>) +6 other tests skip
  *   igt@kms_frontbuffer_tracking@psr-suspend:

     *   shard-mtlp: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-2/igt@kms_frontbuffer_tracking@psr-suspend.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-2/igt@kms_frontbuffer_tracking@psr-suspend.html> (i915#8909<https://gitlab.freedesktop.org/drm/intel/issues/8909>)
  *   igt@kms_getfb@getfb-reject-ccs:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_getfb@getfb-reject-ccs.html> (i915#6118<https://gitlab.freedesktop.org/drm/intel/issues/6118>)
  *   igt@kms_hdr@bpc-switch:

     *   shard-rkl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-2/igt@kms_hdr@bpc-switch.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#8228<https://gitlab.freedesktop.org/drm/intel/issues/8228>) +1 other test skip
  *   igt@kms_hdr@invalid-metadata-sizes:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_hdr@invalid-metadata-sizes.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#8228<https://gitlab.freedesktop.org/drm/intel/issues/8228>)
  *   igt@kms_pipe_b_c_ivb@disable-pipe-b-enable-pipe-c:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_pipe_b_c_ivb@disable-pipe-b-enable-pipe-c.html> (fdo#109289<https://bugs.freedesktop.org/show_bug.cgi?id=109289>)
  *   igt@kms_pipe_b_c_ivb@from-pipe-c-to-b-with-3-lanes:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_pipe_b_c_ivb@from-pipe-c-to-b-with-3-lanes.html> (fdo#109289<https://bugs.freedesktop.org/show_bug.cgi?id=109289>) +1 other test skip
  *   igt@kms_plane_alpha_blend@alpha-transparent-fb@pipe-c-dp-1:

     *   shard-apl: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl3/igt@kms_plane_alpha_blend@alpha-transparent-fb@pipe-c-dp-1.html> (i915#4573<https://gitlab.freedesktop.org/drm/intel/issues/4573>) +1 other test fail
  *   igt@kms_plane_lowres@tiling-y:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_plane_lowres@tiling-y.html> (i915#8821<https://gitlab.freedesktop.org/drm/intel/issues/8821>)
  *   igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-2:

     *   shard-rkl: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-2.html> (i915#8292<https://gitlab.freedesktop.org/drm/intel/issues/8292>)
  *   igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-d-hdmi-a-1:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-19/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-d-hdmi-a-1.html> (i915#5176<https://gitlab.freedesktop.org/drm/intel/issues/5176> / i915#9423<https://gitlab.freedesktop.org/drm/intel/issues/9423>) +3 other tests skip
  *   igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-25@pipe-b-hdmi-a-2:

     *   shard-rkl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-1/igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-25@pipe-b-hdmi-a-2.html> (i915#5235<https://gitlab.freedesktop.org/drm/intel/issues/5235>) +3 other tests skip
  *   igt@kms_plane_scaling@planes-downscale-factor-0-25@pipe-d-hdmi-a-3:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_plane_scaling@planes-downscale-factor-0-25@pipe-d-hdmi-a-3.html> (i915#5235<https://gitlab.freedesktop.org/drm/intel/issues/5235>) +15 other tests skip
  *   igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-75@pipe-c-edp-1:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-75@pipe-c-edp-1.html> (i915#5235<https://gitlab.freedesktop.org/drm/intel/issues/5235>) +5 other tests skip
  *   igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-25@pipe-d-hdmi-a-4:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-17/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-25@pipe-d-hdmi-a-4.html> (i915#5235<https://gitlab.freedesktop.org/drm/intel/issues/5235>) +11 other tests skip
  *   igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-d-edp-1:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-d-edp-1.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#5235<https://gitlab.freedesktop.org/drm/intel/issues/5235>) +1 other test skip
  *   igt@kms_prime@basic-modeset-hybrid:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@kms_prime@basic-modeset-hybrid.html> (i915#6524<https://gitlab.freedesktop.org/drm/intel/issues/6524> / i915#6805<https://gitlab.freedesktop.org/drm/intel/issues/6805>)
  *   igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area:

     *   shard-apl: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-apl6/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area.html> (fdo#109271<https://bugs.freedesktop.org/show_bug.cgi?id=109271> / i915#658<https://gitlab.freedesktop.org/drm/intel/issues/658>) +3 other tests skip
  *   igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-big-fb:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-big-fb.html> (i915#658<https://gitlab.freedesktop.org/drm/intel/issues/658>)
  *   igt@kms_psr2_su@frontbuffer-xrgb8888:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_psr2_su@frontbuffer-xrgb8888.html> (i915#658<https://gitlab.freedesktop.org/drm/intel/issues/658>) +3 other tests skip
  *   igt@kms_psr@psr2_sprite_mmap_gtt:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@kms_psr@psr2_sprite_mmap_gtt.html> (i915#1072<https://gitlab.freedesktop.org/drm/intel/issues/1072> / i915#4078<https://gitlab.freedesktop.org/drm/intel/issues/4078>)
  *   igt@kms_psr@psr2_sprite_plane_move:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-3/igt@kms_psr@psr2_sprite_plane_move.html> (i915#1072<https://gitlab.freedesktop.org/drm/intel/issues/1072>) +13 other tests skip
  *   igt@kms_psr_stress_test@invalidate-primary-flip-overlay:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html> (i915#5461<https://gitlab.freedesktop.org/drm/intel/issues/5461> / i915#658<https://gitlab.freedesktop.org/drm/intel/issues/658>)
  *   igt@kms_rotation_crc@primary-rotation-270:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@kms_rotation_crc@primary-rotation-270.html> (i915#4235<https://gitlab.freedesktop.org/drm/intel/issues/4235>) +1 other test skip
  *   igt@kms_rotation_crc@primary-y-tiled-reflect-x-90:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_rotation_crc@primary-y-tiled-reflect-x-90.html> (i915#4235<https://gitlab.freedesktop.org/drm/intel/issues/4235>)
  *   igt@kms_rotation_crc@primary-yf-tiled-reflect-x-270:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-270.html> (i915#4235<https://gitlab.freedesktop.org/drm/intel/issues/4235> / i915#5190<https://gitlab.freedesktop.org/drm/intel/issues/5190>)
  *   igt@kms_setmode@clone-exclusive-crtc:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@kms_setmode@clone-exclusive-crtc.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#4098<https://gitlab.freedesktop.org/drm/intel/issues/4098>) +2 other tests skip

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_setmode@clone-exclusive-crtc.html> (i915#3555<https://gitlab.freedesktop.org/drm/intel/issues/3555> / i915#8809<https://gitlab.freedesktop.org/drm/intel/issues/8809>)

  *   igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-2:

     *   shard-rkl: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-1/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-2.html> (i915#9196<https://gitlab.freedesktop.org/drm/intel/issues/9196>)
  *   igt@kms_vblank@ts-continuation-suspend@pipe-a-vga-1:

     *   shard-snb: NOTRUN -> DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb2/igt@kms_vblank@ts-continuation-suspend@pipe-a-vga-1.html> (i915#8841<https://gitlab.freedesktop.org/drm/intel/issues/8841>) +3 other tests dmesg-warn
  *   igt@kms_writeback@writeback-check-output:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@kms_writeback@writeback-check-output.html> (i915#2437<https://gitlab.freedesktop.org/drm/intel/issues/2437>)
  *   igt@kms_writeback@writeback-pixel-formats:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_writeback@writeback-pixel-formats.html> (i915#2437<https://gitlab.freedesktop.org/drm/intel/issues/2437>)
  *   igt@perf@enable-disable@0-rcs0:

     *   shard-dg2: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@perf@enable-disable@0-rcs0.html> (i915#8724<https://gitlab.freedesktop.org/drm/intel/issues/8724>)
  *   igt@perf@global-sseu-config-invalid:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@perf@global-sseu-config-invalid.html> (i915#7387<https://gitlab.freedesktop.org/drm/intel/issues/7387>)
  *   igt@perf@mi-rpc:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@perf@mi-rpc.html> (i915#2434<https://gitlab.freedesktop.org/drm/intel/issues/2434>)
  *   igt@perf_pmu@busy-double-start@rcs0:

     *   shard-mtlp: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-2/igt@perf_pmu@busy-double-start@rcs0.html> -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-2/igt@perf_pmu@busy-double-start@rcs0.html> (i915#4349<https://gitlab.freedesktop.org/drm/intel/issues/4349>)
  *   igt@perf_pmu@busy-double-start@vecs1:

     *   shard-dg2: NOTRUN -> FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-2/igt@perf_pmu@busy-double-start@vecs1.html> (i915#4349<https://gitlab.freedesktop.org/drm/intel/issues/4349>) +3 other tests fail
  *   igt@perf_pmu@cpu-hotplug:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@perf_pmu@cpu-hotplug.html> (i915#8850<https://gitlab.freedesktop.org/drm/intel/issues/8850>)
  *   igt@prime_mmap@test_aperture_limit@test_aperture_limit-smem:

     *   shard-dg2: NOTRUN -> CRASH<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@prime_mmap@test_aperture_limit@test_aperture_limit-smem.html> (i915#9351<https://gitlab.freedesktop.org/drm/intel/issues/9351>)
  *   igt@prime_vgem@basic-fence-mmap:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@prime_vgem@basic-fence-mmap.html> (i915#3708<https://gitlab.freedesktop.org/drm/intel/issues/3708> / i915#4077<https://gitlab.freedesktop.org/drm/intel/issues/4077>)
  *   igt@prime_vgem@basic-read:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@prime_vgem@basic-read.html> (i915#3291<https://gitlab.freedesktop.org/drm/intel/issues/3291> / i915#3708<https://gitlab.freedesktop.org/drm/intel/issues/3708>)
  *   igt@prime_vgem@coherency-gtt:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@prime_vgem@coherency-gtt.html> (i915#3708<https://gitlab.freedesktop.org/drm/intel/issues/3708> / i915#4077<https://gitlab.freedesktop.org/drm/intel/issues/4077>)
  *   igt@prime_vgem@fence-flip-hang:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@prime_vgem@fence-flip-hang.html> (i915#3708<https://gitlab.freedesktop.org/drm/intel/issues/3708>)
  *   igt@sysfs_heartbeat_interval@nopreempt@ccs0:

     *   shard-mtlp: PASS<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-8/igt@sysfs_heartbeat_interval@nopreempt@ccs0.html> -> ABORT<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@sysfs_heartbeat_interval@nopreempt@ccs0.html> (i915#9414<https://gitlab.freedesktop.org/drm/intel/issues/9414>)
  *   igt@v3d/v3d_submit_cl@bad-bo:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-11/igt@v3d/v3d_submit_cl@bad-bo.html> (i915#2575<https://gitlab.freedesktop.org/drm/intel/issues/2575>) +24 other tests skip
  *   igt@v3d/v3d_submit_cl@bad-perfmon:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@v3d/v3d_submit_cl@bad-perfmon.html> (i915#2575<https://gitlab.freedesktop.org/drm/intel/issues/2575>) +2 other tests skip
  *   igt@v3d/v3d_submit_csd@bad-bo:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@v3d/v3d_submit_csd@bad-bo.html> (i915#2575<https://gitlab.freedesktop.org/drm/intel/issues/2575>) +3 other tests skip
  *   igt@vc4/vc4_dmabuf_poll@poll-read-waits-until-write-done:

     *   shard-dg1: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-12/igt@vc4/vc4_dmabuf_poll@poll-read-waits-until-write-done.html> (i915#7711<https://gitlab.freedesktop.org/drm/intel/issues/7711>) +1 other test skip
  *   igt@vc4/vc4_purgeable_bo@mark-purgeable-twice:

     *   shard-mtlp: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@vc4/vc4_purgeable_bo@mark-purgeable-twice.html> (i915#7711<https://gitlab.freedesktop.org/drm/intel/issues/7711>) +1 other test skip
  *   igt@vc4/vc4_wait_seqno@bad-seqno-1ns:

     *   shard-dg2: NOTRUN -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@vc4/vc4_wait_seqno@bad-seqno-1ns.html> (i915#7711<https://gitlab.freedesktop.org/drm/intel/issues/7711>) +16 other tests skip

Possible fixes

  *   igt@drm_fdinfo@most-busy-check-all@rcs0:

     *   shard-rkl: FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-7/igt@drm_fdinfo@most-busy-check-all@rcs0.html> (i915#7742<https://gitlab.freedesktop.org/drm/intel/issues/7742>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-4/igt@drm_fdinfo@most-busy-check-all@rcs0.html>
  *   igt@i915_pm_freq_api@freq-suspend@gt0:

     *   shard-dg2: INCOMPLETE<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg2-6/igt@i915_pm_freq_api@freq-suspend@gt0.html> (i915#9407<https://gitlab.freedesktop.org/drm/intel/issues/9407>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-1/igt@i915_pm_freq_api@freq-suspend@gt0.html>
  *   igt@i915_power@sanity:

     *   shard-mtlp: SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-4/igt@i915_power@sanity.html> (i915#7984<https://gitlab.freedesktop.org/drm/intel/issues/7984>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-4/igt@i915_power@sanity.html>
  *   igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip:

     *   shard-tglu: FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-tglu-7/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip.html> (i915#3743<https://gitlab.freedesktop.org/drm/intel/issues/3743>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-tglu-3/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip.html>
  *   igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:

     *   shard-glk: FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-glk8/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html> (i915#2346<https://gitlab.freedesktop.org/drm/intel/issues/2346>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-glk4/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html>
  *   igt@kms_cursor_legacy@single-move@all-pipes:

     *   shard-mtlp: DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-4/igt@kms_cursor_legacy@single-move@all-pipes.html> (i915#2017<https://gitlab.freedesktop.org/drm/intel/issues/2017>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-7/igt@kms_cursor_legacy@single-move@all-pipes.html>
  *   igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2:

     *   shard-glk: FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-glk8/igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2.html> (i915#79<https://gitlab.freedesktop.org/drm/intel/issues/79>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-glk7/igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2.html>
  *   {igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a}:

     *   shard-mtlp: ABORT<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-8/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a.html> (i915#9414<https://gitlab.freedesktop.org/drm/intel/issues/9414>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-5/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a.html>
  *   {igt@kms_pm_rpm@dpms-non-lpsp}:

     *   shard-dg1: SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg1-19/igt@kms_pm_rpm@dpms-non-lpsp.html> (i915#9519<https://gitlab.freedesktop.org/drm/intel/issues/9519>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg1-15/igt@kms_pm_rpm@dpms-non-lpsp.html>
  *   {igt@kms_pm_rpm@modeset-non-lpsp}:

     *   shard-dg2: SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-dg2-10/igt@kms_pm_rpm@modeset-non-lpsp.html> (i915#9519<https://gitlab.freedesktop.org/drm/intel/issues/9519>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-dg2-6/igt@kms_pm_rpm@modeset-non-lpsp.html>

     *   shard-rkl: SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-7/igt@kms_pm_rpm@modeset-non-lpsp.html> (i915#9519<https://gitlab.freedesktop.org/drm/intel/issues/9519>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-1/igt@kms_pm_rpm@modeset-non-lpsp.html> +1 other test pass

  *   igt@kms_universal_plane@cursor-fb-leak@pipe-d-edp-1:

     *   shard-mtlp: FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-5/igt@kms_universal_plane@cursor-fb-leak@pipe-d-edp-1.html> (i915#9196<https://gitlab.freedesktop.org/drm/intel/issues/9196>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-4/igt@kms_universal_plane@cursor-fb-leak@pipe-d-edp-1.html> +2 other tests pass
  *   igt@perf_pmu@busy-double-start@bcs0:

     *   shard-mtlp: FAIL<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-mtlp-2/igt@perf_pmu@busy-double-start@bcs0.html> (i915#4349<https://gitlab.freedesktop.org/drm/intel/issues/4349>) -> PASS<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-mtlp-2/igt@perf_pmu@busy-double-start@bcs0.html>

Warnings

  *   igt@i915_suspend@basic-s3-without-i915:

     *   shard-snb: DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-snb1/igt@i915_suspend@basic-s3-without-i915.html> (i915#8841<https://gitlab.freedesktop.org/drm/intel/issues/8841>) -> DMESG-WARN<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-snb1/igt@i915_suspend@basic-s3-without-i915.html> (i915#1982<https://gitlab.freedesktop.org/drm/intel/issues/1982> / i915#8841<https://gitlab.freedesktop.org/drm/intel/issues/8841>)
  *   igt@kms_force_connector_basic@force-load-detect:

     *   shard-rkl: SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13781/shard-rkl-7/igt@kms_force_connector_basic@force-load-detect.html> (fdo#109285<https://bugs.freedesktop.org/show_bug.cgi?id=109285>) -> SKIP<https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_125442v2/shard-rkl-3/igt@kms_force_connector_basic@force-load-detect.html> (fdo#109285<https://bugs.freedesktop.org/show_bug.cgi?id=109285> / i915#4098<https://gitlab.freedesktop.org/drm/intel/issues/4098>)

{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).

Build changes

  *   Linux: CI_DRM_13781 -> Patchwork_125442v2

CI-20190529: 20190529
CI_DRM_13781: a983c752eb74b4af7f72fe09008a1169d315a77f @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7552: 557856802dfee103802f1157f97c65bb476d5468 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_125442v2: a983c752eb74b4af7f72fe09008a1169d315a77f @ git://anongit.freedesktop.org/gfx-ci/linux
Tvrtko Ursulin Oct. 25, 2023, 10:58 a.m. UTC | #5
On 25/10/2023 11:32, Coelho, Luciano wrote:
> On Wed, 2023-10-25 at 11:25 +0100, Tvrtko Ursulin wrote:
>> On 25/10/2023 11:18, Tvrtko Ursulin wrote:
>>>
>>> On 23/10/2023 11:33, Luca Coelho wrote:
>>>> The uncore code may not always be available (e.g. when we build the
>>>> display code with Xe), so we can't always rely on having the uncore's
>>>> spinlock.
>>>>
>>>> To handle this, split the spin_lock/unlock_irqsave/restore() into
>>>> spin_lock/unlock() followed by a call to local_irq_save/restore() and
>>>> create wrapper functions for locking and unlocking the uncore's
>>>> spinlock.  In these functions, we have a condition check and only
>>>> actually try to lock/unlock the spinlock when I915 is defined, and
>>>> thus uncore is available.
>>>>
>>>> This keeps the ifdefs contained in these new functions and all such
>>>> logic inside the display code.
>>>>
>>>> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
>>>> ---
>>>>
>>>> In v2:
>>>>
>>>>      * Renamed uncore_spin_*() to intel_spin_*()
>>>>      * Corrected the order: save, lock, unlock, restore
>>>>
>>>> In v3:
>>>>
>>>>      * Undid the change to pass drm_i915_private instead of the lock
>>>>        itself, since we would have to include i915_drv.h and that pulls
>>>>        in a truckload of other includes.
>>>>
>>>>    drivers/gpu/drm/i915/display/intel_display.h | 20 ++++++++++++++++++++
>>>>    drivers/gpu/drm/i915/display/intel_vblank.c  | 19 ++++++++++++-------
>>>>    2 files changed, 32 insertions(+), 7 deletions(-)
>>>>
>>>> diff --git a/drivers/gpu/drm/i915/display/intel_display.h
>>>> b/drivers/gpu/drm/i915/display/intel_display.h
>>>> index 0e5dffe8f018..2a33fcc8ce68 100644
>>>> --- a/drivers/gpu/drm/i915/display/intel_display.h
>>>> +++ b/drivers/gpu/drm/i915/display/intel_display.h
>>>> @@ -559,4 +559,24 @@ bool assert_port_valid(struct drm_i915_private
>>>> *i915, enum port port);
>>>>    bool intel_scanout_needs_vtd_wa(struct drm_i915_private *i915);
>>>> +/*
>>>> + * The uncore version of the spin lock functions is used to decide
>>>> + * whether we need to lock the uncore lock or not.  This is only
>>>> + * needed in i915, not in Xe.  Keep the decision-making centralized
>>>> + * here.
>>>> + */
>>>> +static inline void intel_spin_lock(spinlock_t *lock)
>>>> +{
>>>> +#ifdef I915
>>>> +    spin_lock(lock);
>>>> +#endif
>>>> +}
>>>> +
>>>> +static inline void intel_spin_unlock(spinlock_t *lock)
>>>> +{
>>>> +#ifdef I915
>>>> +    spin_unlock(lock);
>>>> +#endif
>>>> +}
>>>> +
>>>>    #endif
>>>> diff --git a/drivers/gpu/drm/i915/display/intel_vblank.c
>>>> b/drivers/gpu/drm/i915/display/intel_vblank.c
>>>> index 2cec2abf9746..9b482d648762 100644
>>>> --- a/drivers/gpu/drm/i915/display/intel_vblank.c
>>>> +++ b/drivers/gpu/drm/i915/display/intel_vblank.c
>>>> @@ -306,7 +306,8 @@ static bool i915_get_crtc_scanoutpos(struct
>>>> drm_crtc *_crtc,
>>>>         * register reads, potentially with preemption disabled, so the
>>>>         * following code must not block on uncore.lock.
>>>>         */
>>>> -    spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
>>>> +    local_irq_save(irqflags);
>>>
>>> Does Xe needs interrupts off?
> 
> I'm actually not sure, but this is how it was in the Xe driver code, so
> I kept it.
> 
> 
>>>> +    intel_spin_lock(&dev_priv->uncore.lock);
>>>
>>> My 2p/c is that intel_spin_lock as a name does not work when it is
>>> specifically about the single and specific (uncore) lock. One cannot
>>> call intel_spin_lock(some->other->lock) etc.
> 
> Right, this was changed when I was passing only dev_priv, but I
> couldn't do that wihtout adding i915_drv.h, which was not good
> either... But yeah, this is too generic, while the actual case is not.
> 
> 
>>> Perhaps call it i915_uncore_lock_irqsave(i915, flags) so it is clear it
>>> is only for i915.
> 
> I wanted to avoid "i915", since we also call it when the display is
> used with xe...
> 
> 
>> Or, if the implementation will later gain the #else block for Xe,
>> perhaps intel_uncore_lock_...?
> 
> But still, uncore doesn't exist in Xe, so this is still not good...
> 
> Any other suggestions?

I think it will boil down to the reason uncore lock is held over the 
respective sections ie. the comment in i915_get_crtc_scanoutpos.

If it is timing sensitive to the extent irq off was needed it may apply 
to Xe as well. Likewise the need to use mmio helpers which rely on the 
uncore lock already held. Question of whether there is conceptual 
commonality, will probably drive the best name, or the best approach in 
general.

Regards,

Tvrtko
Rodrigo Vivi Nov. 3, 2023, 3:31 a.m. UTC | #6
> > 
> > Any other suggestions?
> 
> I think it will boil down to the reason uncore lock is held over the 
> respective sections ie. the comment in i915_get_crtc_scanoutpos.
> 
> If it is timing sensitive to the extent irq off was needed it may
> apply 
> to Xe as well. Likewise the need to use mmio helpers which rely on
> the 
> uncore lock already held. Question of whether there is conceptual 
> commonality, will probably drive the best name, or the best approach
> in 
> general.

yeap, this is how I'm seeing this. If i915-display needs this global
lock around mmio operations, then we wound need to add it to the
xe_mmio as well and then solve the name, etc.

However, I don't believe that other users of the mmio would need
this lock. So I believe the right thing to do is to create a i915-
display only spin_lock, around the intel_de_mmio calls and here.

With this we entirely kill the dependency on someone-else's lock
and have something that is entirely inside display code so it
doesn't need to be ported to one or another driver core components.

> 
> Regards,
> 
> Tvrtko
Luca Coelho Nov. 3, 2023, 8:58 a.m. UTC | #7
On Fri, 2023-11-03 at 03:31 +0000, Vivi, Rodrigo wrote:
> > > 
> > > Any other suggestions?
> > 
> > I think it will boil down to the reason uncore lock is held over
> > the 
> > respective sections ie. the comment in i915_get_crtc_scanoutpos.
> > 
> > If it is timing sensitive to the extent irq off was needed it may
> > apply 
> > to Xe as well. Likewise the need to use mmio helpers which rely on
> > the 
> > uncore lock already held. Question of whether there is conceptual 
> > commonality, will probably drive the best name, or the best
> > approach
> > in 
> > general.
> 
> yeap, this is how I'm seeing this. If i915-display needs this global
> lock around mmio operations, then we wound need to add it to the
> xe_mmio as well and then solve the name, etc.
> 
> However, I don't believe that other users of the mmio would need
> this lock. So I believe the right thing to do is to create a i915-
> display only spin_lock, around the intel_de_mmio calls and here.
> 
> With this we entirely kill the dependency on someone-else's lock
> and have something that is entirely inside display code so it
> doesn't need to be ported to one or another driver core components.

Right, I agree too.

My patch was just trying to address the quick hack made for Xe, not the
actual implementation in Xe's side.  But it makes sense to implement
this new lock internally in the display so there are no dependencies or
wrappers needed.

I'll respin.

--
Cheers,
Luca.
Tvrtko Ursulin Nov. 3, 2023, 9:47 a.m. UTC | #8
On 03/11/2023 08:58, Coelho, Luciano wrote:
> On Fri, 2023-11-03 at 03:31 +0000, Vivi, Rodrigo wrote:
>>>>
>>>> Any other suggestions?
>>>
>>> I think it will boil down to the reason uncore lock is held over
>>> the
>>> respective sections ie. the comment in i915_get_crtc_scanoutpos.
>>>
>>> If it is timing sensitive to the extent irq off was needed it may
>>> apply
>>> to Xe as well. Likewise the need to use mmio helpers which rely on
>>> the
>>> uncore lock already held. Question of whether there is conceptual
>>> commonality, will probably drive the best name, or the best
>>> approach
>>> in
>>> general.
>>
>> yeap, this is how I'm seeing this. If i915-display needs this global
>> lock around mmio operations, then we wound need to add it to the
>> xe_mmio as well and then solve the name, etc.
>>
>> However, I don't believe that other users of the mmio would need
>> this lock. So I believe the right thing to do is to create a i915-
>> display only spin_lock, around the intel_de_mmio calls and here.
>>
>> With this we entirely kill the dependency on someone-else's lock
>> and have something that is entirely inside display code so it
>> doesn't need to be ported to one or another driver core components.
> 
> Right, I agree too.
> 
> My patch was just trying to address the quick hack made for Xe, not the
> actual implementation in Xe's side.  But it makes sense to implement
> this new lock internally in the display so there are no dependencies or
> wrappers needed.
> 
> I'll respin.

You could also make sure it needs to be a lock and not just say a 
preempt off or irq section?

Regards,

Tvrtko
Rodrigo Vivi Nov. 3, 2023, 1:25 p.m. UTC | #9
On Fri, 2023-11-03 at 09:47 +0000, Tvrtko Ursulin wrote:
> 
> On 03/11/2023 08:58, Coelho, Luciano wrote:
> > On Fri, 2023-11-03 at 03:31 +0000, Vivi, Rodrigo wrote:
> > > > > 
> > > > > Any other suggestions?
> > > > 
> > > > I think it will boil down to the reason uncore lock is held
> > > > over
> > > > the
> > > > respective sections ie. the comment in
> > > > i915_get_crtc_scanoutpos.
> > > > 
> > > > If it is timing sensitive to the extent irq off was needed it
> > > > may
> > > > apply
> > > > to Xe as well. Likewise the need to use mmio helpers which rely
> > > > on
> > > > the
> > > > uncore lock already held. Question of whether there is
> > > > conceptual
> > > > commonality, will probably drive the best name, or the best
> > > > approach
> > > > in
> > > > general.
> > > 
> > > yeap, this is how I'm seeing this. If i915-display needs this
> > > global
> > > lock around mmio operations, then we wound need to add it to the
> > > xe_mmio as well and then solve the name, etc.
> > > 
> > > However, I don't believe that other users of the mmio would need
> > > this lock. So I believe the right thing to do is to create a
> > > i915-
> > > display only spin_lock, around the intel_de_mmio calls and here.
> > > 
> > > With this we entirely kill the dependency on someone-else's lock
> > > and have something that is entirely inside display code so it
> > > doesn't need to be ported to one or another driver core
> > > components.
> > 
> > Right, I agree too.
> > 
> > My patch was just trying to address the quick hack made for Xe, not
> > the
> > actual implementation in Xe's side.  But it makes sense to
> > implement
> > this new lock internally in the display so there are no
> > dependencies or
> > wrappers needed.
> > 
> > I'll respin.
> 
> You could also make sure it needs to be a lock and not just say a 
> preempt off or irq section?

indeed a good question. maybe we don't need the lock at all there...

> 
> Regards,
> 
> Tvrtko
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h
index 0e5dffe8f018..2a33fcc8ce68 100644
--- a/drivers/gpu/drm/i915/display/intel_display.h
+++ b/drivers/gpu/drm/i915/display/intel_display.h
@@ -559,4 +559,24 @@  bool assert_port_valid(struct drm_i915_private *i915, enum port port);
 
 bool intel_scanout_needs_vtd_wa(struct drm_i915_private *i915);
 
+/*
+ * The uncore version of the spin lock functions is used to decide
+ * whether we need to lock the uncore lock or not.  This is only
+ * needed in i915, not in Xe.  Keep the decision-making centralized
+ * here.
+ */
+static inline void intel_spin_lock(spinlock_t *lock)
+{
+#ifdef I915
+	spin_lock(lock);
+#endif
+}
+
+static inline void intel_spin_unlock(spinlock_t *lock)
+{
+#ifdef I915
+	spin_unlock(lock);
+#endif
+}
+
 #endif
diff --git a/drivers/gpu/drm/i915/display/intel_vblank.c b/drivers/gpu/drm/i915/display/intel_vblank.c
index 2cec2abf9746..9b482d648762 100644
--- a/drivers/gpu/drm/i915/display/intel_vblank.c
+++ b/drivers/gpu/drm/i915/display/intel_vblank.c
@@ -306,7 +306,8 @@  static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
 	 * register reads, potentially with preemption disabled, so the
 	 * following code must not block on uncore.lock.
 	 */
-	spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
+	local_irq_save(irqflags);
+	intel_spin_lock(&dev_priv->uncore.lock);
 
 	/* preempt_disable_rt() should go right here in PREEMPT_RT patchset. */
 
@@ -374,7 +375,8 @@  static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
 
 	/* preempt_enable_rt() should go right here in PREEMPT_RT patchset. */
 
-	spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
+	intel_spin_unlock(&dev_priv->uncore.lock);
+	local_irq_restore(irqflags);
 
 	/*
 	 * While in vblank, position will be negative
@@ -412,9 +414,13 @@  int intel_get_crtc_scanline(struct intel_crtc *crtc)
 	unsigned long irqflags;
 	int position;
 
-	spin_lock_irqsave(&dev_priv->uncore.lock, irqflags);
+	local_irq_save(irqflags);
+	intel_spin_lock(&dev_priv->uncore.lock);
+
 	position = __intel_get_crtc_scanline(crtc);
-	spin_unlock_irqrestore(&dev_priv->uncore.lock, irqflags);
+
+	intel_spin_unlock(&dev_priv->uncore.lock);
+	local_irq_restore(irqflags);
 
 	return position;
 }
@@ -537,7 +543,7 @@  void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
 	 * Need to audit everything to make sure it's safe.
 	 */
 	spin_lock_irqsave(&i915->drm.vblank_time_lock, irqflags);
-	spin_lock(&i915->uncore.lock);
+	intel_spin_lock(&i915->uncore.lock);
 
 	drm_calc_timestamping_constants(&crtc->base, &adjusted_mode);
 
@@ -546,7 +552,6 @@  void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
 	crtc->mode_flags = mode_flags;
 
 	crtc->scanline_offset = intel_crtc_scanline_offset(crtc_state);
-
-	spin_unlock(&i915->uncore.lock);
+	intel_spin_unlock(&i915->uncore.lock);
 	spin_unlock_irqrestore(&i915->drm.vblank_time_lock, irqflags);
 }