diff mbox series

[v2,1/6] drm/i915: move the pre_pin earlier

Message ID 20211117142024.1043017-1-matthew.auld@intel.com (mailing list archive)
State New, archived
Headers show
Series [v2,1/6] drm/i915: move the pre_pin earlier | expand

Commit Message

Matthew Auld Nov. 17, 2021, 2:20 p.m. UTC
In intel_context_do_pin_ww, when calling into the pre_pin hook(which is
passed the ww context) it could in theory return -EDEADLK(which is very
likely with debug kernels), once we start adding more ww locking in there,
like in the next patch. If so then we need to be mindful of having to
restart the do_pin at this point.

If this is the kernel_context, or some other early in-kernel context
where we have yet to setup the default_state, then we always inhibit the
context restore, and instead rely on the delayed active_release to set
the CONTEXT_VALID_BIT for us(if we even care), which should indicate
that we have context switched away, and that our newly saved context
state should now be valid. However, since we currently grab the active
reference before the potential ww dance, we can end up setting the
CONTEXT_VALID_BIT much too early, if we need to backoff, and then upon
re-trying the do_pin, we could potentially cause the hardware to
incorrectly load some garbage context state when later context switching
to that context, but at the very least this will trigger the
GEM_BUG_ON() in __engine_unpark. For now let's just move any ww dance
stuff prior to arming the active reference.

For normal user contexts this shouldn't be a concern, since we should
already have the default_state ready when initialising the lrc state,
and so there should be no concern with active_release somehow
prematurely setting the CONTEXT_VALID_BIT.

v2(Thomas):
  - Also re-order the union unwind

Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/i915/gt/intel_context.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Thomas Hellstrom Nov. 17, 2021, 6:49 p.m. UTC | #1
On 11/17/21 15:20, Matthew Auld wrote:
> In intel_context_do_pin_ww, when calling into the pre_pin hook(which is
> passed the ww context) it could in theory return -EDEADLK(which is very
> likely with debug kernels), once we start adding more ww locking in there,
> like in the next patch. If so then we need to be mindful of having to
> restart the do_pin at this point.
>
> If this is the kernel_context, or some other early in-kernel context
> where we have yet to setup the default_state, then we always inhibit the
> context restore, and instead rely on the delayed active_release to set
> the CONTEXT_VALID_BIT for us(if we even care), which should indicate
> that we have context switched away, and that our newly saved context
> state should now be valid. However, since we currently grab the active
> reference before the potential ww dance, we can end up setting the
> CONTEXT_VALID_BIT much too early, if we need to backoff, and then upon
> re-trying the do_pin, we could potentially cause the hardware to
> incorrectly load some garbage context state when later context switching
> to that context, but at the very least this will trigger the
> GEM_BUG_ON() in __engine_unpark. For now let's just move any ww dance
> stuff prior to arming the active reference.
>
> For normal user contexts this shouldn't be a concern, since we should
> already have the default_state ready when initialising the lrc state,
> and so there should be no concern with active_release somehow
> prematurely setting the CONTEXT_VALID_BIT.
>
> v2(Thomas):
>    - Also re-order the union unwind
>
> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>

Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>


> ---
>   drivers/gpu/drm/i915/gt/intel_context.c | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_context.c b/drivers/gpu/drm/i915/gt/intel_context.c
> index 5634d14052bc..4c296de1d67d 100644
> --- a/drivers/gpu/drm/i915/gt/intel_context.c
> +++ b/drivers/gpu/drm/i915/gt/intel_context.c
> @@ -228,17 +228,17 @@ int __intel_context_do_pin_ww(struct intel_context *ce,
>   	if (err)
>   		return err;
>   
> -	err = i915_active_acquire(&ce->active);
> +	err = ce->ops->pre_pin(ce, ww, &vaddr);
>   	if (err)
>   		goto err_ctx_unpin;
>   
> -	err = ce->ops->pre_pin(ce, ww, &vaddr);
> +	err = i915_active_acquire(&ce->active);
>   	if (err)
> -		goto err_release;
> +		goto err_post_unpin;
>   
>   	err = mutex_lock_interruptible(&ce->pin_mutex);
>   	if (err)
> -		goto err_post_unpin;
> +		goto err_release;
>   
>   	intel_engine_pm_might_get(ce->engine);
>   
> @@ -273,11 +273,11 @@ int __intel_context_do_pin_ww(struct intel_context *ce,
>   
>   err_unlock:
>   	mutex_unlock(&ce->pin_mutex);
> +err_release:
> +	i915_active_release(&ce->active);
>   err_post_unpin:
>   	if (!handoff)
>   		ce->ops->post_unpin(ce);
> -err_release:
> -	i915_active_release(&ce->active);
>   err_ctx_unpin:
>   	intel_context_post_unpin(ce);
>
Thomas Hellstrom Nov. 18, 2021, 6:57 a.m. UTC | #2
On Wed, 2021-11-17 at 19:49 +0100, Thomas Hellström wrote:
> 
> On 11/17/21 15:20, Matthew Auld wrote:
> > In intel_context_do_pin_ww, when calling into the pre_pin
> > hook(which is
> > passed the ww context) it could in theory return -EDEADLK(which is
> > very
> > likely with debug kernels), once we start adding more ww locking in
> > there,
> > like in the next patch. If so then we need to be mindful of having
> > to
> > restart the do_pin at this point.
> > 
> > If this is the kernel_context, or some other early in-kernel
> > context
> > where we have yet to setup the default_state, then we always
> > inhibit the
> > context restore, and instead rely on the delayed active_release to
> > set
> > the CONTEXT_VALID_BIT for us(if we even care), which should
> > indicate
> > that we have context switched away, and that our newly saved
> > context
> > state should now be valid. However, since we currently grab the
> > active
> > reference before the potential ww dance, we can end up setting the
> > CONTEXT_VALID_BIT much too early, if we need to backoff, and then
> > upon
> > re-trying the do_pin, we could potentially cause the hardware to
> > incorrectly load some garbage context state when later context
> > switching
> > to that context, but at the very least this will trigger the
> > GEM_BUG_ON() in __engine_unpark. For now let's just move any ww
> > dance
> > stuff prior to arming the active reference.
> > 
> > For normal user contexts this shouldn't be a concern, since we
> > should
> > already have the default_state ready when initialising the lrc
> > state,
> > and so there should be no concern with active_release somehow
> > prematurely setting the CONTEXT_VALID_BIT.
> > 
> > v2(Thomas):
> >    - Also re-order the union unwind

Oh should this be 

s/union/onion/ ?


> > 
> > Signed-off-by: Matthew Auld <matthew.auld@intel.com>
> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> 
> Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> 
> 
> > ---
> >   drivers/gpu/drm/i915/gt/intel_context.c | 12 ++++++------
> >   1 file changed, 6 insertions(+), 6 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/gt/intel_context.c
> > b/drivers/gpu/drm/i915/gt/intel_context.c
> > index 5634d14052bc..4c296de1d67d 100644
> > --- a/drivers/gpu/drm/i915/gt/intel_context.c
> > +++ b/drivers/gpu/drm/i915/gt/intel_context.c
> > @@ -228,17 +228,17 @@ int __intel_context_do_pin_ww(struct
> > intel_context *ce,
> >         if (err)
> >                 return err;
> >   
> > -       err = i915_active_acquire(&ce->active);
> > +       err = ce->ops->pre_pin(ce, ww, &vaddr);
> >         if (err)
> >                 goto err_ctx_unpin;
> >   
> > -       err = ce->ops->pre_pin(ce, ww, &vaddr);
> > +       err = i915_active_acquire(&ce->active);
> >         if (err)
> > -               goto err_release;
> > +               goto err_post_unpin;
> >   
> >         err = mutex_lock_interruptible(&ce->pin_mutex);
> >         if (err)
> > -               goto err_post_unpin;
> > +               goto err_release;
> >   
> >         intel_engine_pm_might_get(ce->engine);
> >   
> > @@ -273,11 +273,11 @@ int __intel_context_do_pin_ww(struct
> > intel_context *ce,
> >   
> >   err_unlock:
> >         mutex_unlock(&ce->pin_mutex);
> > +err_release:
> > +       i915_active_release(&ce->active);
> >   err_post_unpin:
> >         if (!handoff)
> >                 ce->ops->post_unpin(ce);
> > -err_release:
> > -       i915_active_release(&ce->active);
> >   err_ctx_unpin:
> >         intel_context_post_unpin(ce);
> >
Matthew Auld Nov. 18, 2021, 9:58 a.m. UTC | #3
On 18/11/2021 06:57, Thomas Hellström wrote:
> On Wed, 2021-11-17 at 19:49 +0100, Thomas Hellström wrote:
>>
>> On 11/17/21 15:20, Matthew Auld wrote:
>>> In intel_context_do_pin_ww, when calling into the pre_pin
>>> hook(which is
>>> passed the ww context) it could in theory return -EDEADLK(which is
>>> very
>>> likely with debug kernels), once we start adding more ww locking in
>>> there,
>>> like in the next patch. If so then we need to be mindful of having
>>> to
>>> restart the do_pin at this point.
>>>
>>> If this is the kernel_context, or some other early in-kernel
>>> context
>>> where we have yet to setup the default_state, then we always
>>> inhibit the
>>> context restore, and instead rely on the delayed active_release to
>>> set
>>> the CONTEXT_VALID_BIT for us(if we even care), which should
>>> indicate
>>> that we have context switched away, and that our newly saved
>>> context
>>> state should now be valid. However, since we currently grab the
>>> active
>>> reference before the potential ww dance, we can end up setting the
>>> CONTEXT_VALID_BIT much too early, if we need to backoff, and then
>>> upon
>>> re-trying the do_pin, we could potentially cause the hardware to
>>> incorrectly load some garbage context state when later context
>>> switching
>>> to that context, but at the very least this will trigger the
>>> GEM_BUG_ON() in __engine_unpark. For now let's just move any ww
>>> dance
>>> stuff prior to arming the active reference.
>>>
>>> For normal user contexts this shouldn't be a concern, since we
>>> should
>>> already have the default_state ready when initialising the lrc
>>> state,
>>> and so there should be no concern with active_release somehow
>>> prematurely setting the CONTEXT_VALID_BIT.
>>>
>>> v2(Thomas):
>>>     - Also re-order the union unwind
> 
> Oh should this be
> 
> s/union/onion/ ?

Oops, will fixup when pushing :)

> 
> 
>>>
>>> Signed-off-by: Matthew Auld <matthew.auld@intel.com>
>>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>>> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>>
>> Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>>
>>
>>> ---
>>>    drivers/gpu/drm/i915/gt/intel_context.c | 12 ++++++------
>>>    1 file changed, 6 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/i915/gt/intel_context.c
>>> b/drivers/gpu/drm/i915/gt/intel_context.c
>>> index 5634d14052bc..4c296de1d67d 100644
>>> --- a/drivers/gpu/drm/i915/gt/intel_context.c
>>> +++ b/drivers/gpu/drm/i915/gt/intel_context.c
>>> @@ -228,17 +228,17 @@ int __intel_context_do_pin_ww(struct
>>> intel_context *ce,
>>>          if (err)
>>>                  return err;
>>>    
>>> -       err = i915_active_acquire(&ce->active);
>>> +       err = ce->ops->pre_pin(ce, ww, &vaddr);
>>>          if (err)
>>>                  goto err_ctx_unpin;
>>>    
>>> -       err = ce->ops->pre_pin(ce, ww, &vaddr);
>>> +       err = i915_active_acquire(&ce->active);
>>>          if (err)
>>> -               goto err_release;
>>> +               goto err_post_unpin;
>>>    
>>>          err = mutex_lock_interruptible(&ce->pin_mutex);
>>>          if (err)
>>> -               goto err_post_unpin;
>>> +               goto err_release;
>>>    
>>>          intel_engine_pm_might_get(ce->engine);
>>>    
>>> @@ -273,11 +273,11 @@ int __intel_context_do_pin_ww(struct
>>> intel_context *ce,
>>>    
>>>    err_unlock:
>>>          mutex_unlock(&ce->pin_mutex);
>>> +err_release:
>>> +       i915_active_release(&ce->active);
>>>    err_post_unpin:
>>>          if (!handoff)
>>>                  ce->ops->post_unpin(ce);
>>> -err_release:
>>> -       i915_active_release(&ce->active);
>>>    err_ctx_unpin:
>>>          intel_context_post_unpin(ce);
>>>    
> 
>
Matthew Auld Nov. 18, 2021, 10:28 a.m. UTC | #4
On 17/11/2021 19:53, Patchwork wrote:
> *Patch Details*
> *Series:*	series starting with [v2,1/6] drm/i915: move the pre_pin earlier
> *URL:*	https://patchwork.freedesktop.org/series/97026/ 
> <https://patchwork.freedesktop.org/series/97026/>
> *State:*	failure
> *Details:* 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/index.html 
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/index.html>
> 
> 
>   CI Bug Log - changes from CI_DRM_10896 -> Patchwork_21617
> 
> 
>     Summary
> 
> *FAILURE*
> 
> Serious unknown changes coming with Patchwork_21617 absolutely need to be
> verified manually.
> 
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_21617, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
> 
> External URL: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/index.html
> 
> 
>     Participating hosts (40 -> 35)
> 
> Additional (1): fi-tgl-1115g4
> Missing (6): bat-dg1-6 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 bat-jsl-2 
> bat-jsl-1
> 
> 
>     Possible new issues
> 
> Here are the unknown changes that may have been introduced in 
> Patchwork_21617:
> 
> 
>       IGT changes
> 
> 
>         Possible regressions
> 
>   * igt@gem_lmem_swapping@verify-random:
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@gem_lmem_swapping@verify-random.html>
>         +3 similar issues

Looks to be unrelated.


> 
> 
>     Known issues
> 
> Here are the changes found in Patchwork_21617 that come from known issues:
> 
> 
>       IGT changes
> 
> 
>         Issues hit
> 
>   *
> 
>     igt@amdgpu/amd_basic@cs-gfx:
> 
>       o fi-rkl-guc: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-rkl-guc/igt@amdgpu/amd_basic@cs-gfx.html>
>         (fdo#109315
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109315>) +17
>         similar issues
>   *
> 
>     igt@amdgpu/amd_basic@query-info:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@amdgpu/amd_basic@query-info.html>
>         (fdo#109315 <https://bugs.freedesktop.org/show_bug.cgi?id=109315>)
>   *
> 
>     igt@amdgpu/amd_cs_nop@nop-gfx0:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@amdgpu/amd_cs_nop@nop-gfx0.html>
>         (fdo#109315
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109315> /
>         i915#2575
>         <https://gitlab.freedesktop.org/drm/intel/issues/2575>) +16
>         similar issues
>   *
> 
>     igt@amdgpu/amd_cs_nop@sync-fork-compute0:
> 
>       o fi-snb-2600: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +17
>         similar issues
>   *
> 
>     igt@gem_flink_basic@bad-flink:
> 
>       o fi-skl-6600u: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>         (i915#4547 <https://gitlab.freedesktop.org/drm/intel/issues/4547>)
>   *
> 
>     igt@gem_huc_copy@huc-copy:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@gem_huc_copy@huc-copy.html>
>         (i915#2190 <https://gitlab.freedesktop.org/drm/intel/issues/2190>)
>   *
> 
>     igt@i915_pm_backlight@basic-brightness:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@i915_pm_backlight@basic-brightness.html>
>         (i915#1155 <https://gitlab.freedesktop.org/drm/intel/issues/1155>)
>   *
> 
>     igt@kms_chamelium@common-hpd-after-suspend:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_chamelium@common-hpd-after-suspend.html>
>         (fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html>
>         (i915#4103
>         <https://gitlab.freedesktop.org/drm/intel/issues/4103>) +1
>         similar issue
>   *
> 
>     igt@kms_force_connector_basic@force-load-detect:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_force_connector_basic@force-load-detect.html>
>         (fdo#109285 <https://bugs.freedesktop.org/show_bug.cgi?id=109285>)
>   *
> 
>     igt@kms_psr@primary_mmap_gtt:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_psr@primary_mmap_gtt.html>
>         (i915#1072
>         <https://gitlab.freedesktop.org/drm/intel/issues/1072>) +3
>         similar issues
>   *
> 
>     igt@prime_vgem@basic-userptr:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@prime_vgem@basic-userptr.html>
>         (i915#3301 <https://gitlab.freedesktop.org/drm/intel/issues/3301>)
>   *
> 
>     igt@runner@aborted:
> 
>       o fi-skl-6600u: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-skl-6600u/igt@runner@aborted.html>
>         (i915#3363
>         <https://gitlab.freedesktop.org/drm/intel/issues/3363> /
>         i915#4312 <https://gitlab.freedesktop.org/drm/intel/issues/4312>)
> 
> 
>         Possible fixes
> 
>   *
> 
>     igt@i915_selftest@live@evict:
> 
>       o fi-kbl-soraka: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-kbl-soraka/igt@i915_selftest@live@evict.html>
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-kbl-soraka/igt@i915_selftest@live@evict.html>
>   *
> 
>     igt@i915_selftest@live@gt_engines:
> 
>       o fi-rkl-guc: DMESG-FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-rkl-guc/igt@i915_selftest@live@gt_engines.html>
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-rkl-guc/igt@i915_selftest@live@gt_engines.html>
>   *
> 
>     igt@i915_selftest@live@hangcheck:
> 
>       o fi-snb-2600: INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-snb-2600/igt@i915_selftest@live@hangcheck.html>
>         (i915#3921
>         <https://gitlab.freedesktop.org/drm/intel/issues/3921>) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-snb-2600/igt@i915_selftest@live@hangcheck.html>
> 
> {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_10896 -> Patchwork_21617
> 
> CI-20190529: 20190529
> CI_DRM_10896: bf11e5f354ac51635d032893d80a1d0015d277dc @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_6283: a2cd90a7c24bb7a4c19ca74c75ed8c950820dee2 @ 
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_21617: 7dc7916fd526389e070074e9e3895c12b0bcdb29 @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> 
> == Linux commits ==
> 
> 7dc7916fd526 drm/i915: Drain the ttm delayed workqueue too
> 283402ea4955 drm/i915: Remove resv from i915_vma
> 1a43e1835f7f drm/i915: vma is always backed by an object.
> b2328bc6d485 drm/i915: Create a full object for mock_ring, v2.
> 8864b863b878 drm/i915: Create a dummy object for gen6 ppgtt
> 2c4bb85ec07b drm/i915: move the pre_pin earlier
>
Vudum, Lakshminarayana Nov. 18, 2021, 4:31 p.m. UTC | #5
Just filed https://gitlab.freedesktop.org/drm/intel/-/issues/4565.
igt@gem_lmem_swapping@.*random.* - skip - No dynamic tests executed. SKIP

Rev 2 is already is there, so re-reporting is not required.

Lakshmi.

-----Original Message-----
From: Auld, Matthew <matthew.auld@intel.com> 
Sent: Thursday, November 18, 2021 2:28 AM
To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Subject: Re: ✗ Fi.CI.BAT: failure for series starting with [v2,1/6] drm/i915: move the pre_pin earlier

On 17/11/2021 19:53, Patchwork wrote:
> *Patch Details*
> *Series:*	series starting with [v2,1/6] drm/i915: move the pre_pin earlier
> *URL:*	https://patchwork.freedesktop.org/series/97026/ 
> <https://patchwork.freedesktop.org/series/97026/>
> *State:*	failure
> *Details:*
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/index.html
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/index.html>
> 
> 
>   CI Bug Log - changes from CI_DRM_10896 -> Patchwork_21617
> 
> 
>     Summary
> 
> *FAILURE*
> 
> Serious unknown changes coming with Patchwork_21617 absolutely need to 
> be verified manually.
> 
> If you think the reported changes have nothing to do with the changes 
> introduced in Patchwork_21617, please notify your bug team to allow 
> them to document this new failure mode, which will reduce false positives in CI.
> 
> External URL: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/index.html
> 
> 
>     Participating hosts (40 -> 35)
> 
> Additional (1): fi-tgl-1115g4
> Missing (6): bat-dg1-6 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 bat-jsl-2
> bat-jsl-1
> 
> 
>     Possible new issues
> 
> Here are the unknown changes that may have been introduced in
> Patchwork_21617:
> 
> 
>       IGT changes
> 
> 
>         Possible regressions
> 
>   * igt@gem_lmem_swapping@verify-random:
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@gem_lmem_swapping@verify-random.html>
>         +3 similar issues

Looks to be unrelated.


> 
> 
>     Known issues
> 
> Here are the changes found in Patchwork_21617 that come from known issues:
> 
> 
>       IGT changes
> 
> 
>         Issues hit
> 
>   *
> 
>     igt@amdgpu/amd_basic@cs-gfx:
> 
>       o fi-rkl-guc: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-rkl-guc/igt@amdgpu/amd_basic@cs-gfx.html>
>         (fdo#109315
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109315>) +17
>         similar issues
>   *
> 
>     igt@amdgpu/amd_basic@query-info:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@amdgpu/amd_basic@query-info.html>
>         (fdo#109315 <https://bugs.freedesktop.org/show_bug.cgi?id=109315>)
>   *
> 
>     igt@amdgpu/amd_cs_nop@nop-gfx0:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@amdgpu/amd_cs_nop@nop-gfx0.html>
>         (fdo#109315
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109315> /
>         i915#2575
>         <https://gitlab.freedesktop.org/drm/intel/issues/2575>) +16
>         similar issues
>   *
> 
>     igt@amdgpu/amd_cs_nop@sync-fork-compute0:
> 
>       o fi-snb-2600: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +17
>         similar issues
>   *
> 
>     igt@gem_flink_basic@bad-flink:
> 
>       o fi-skl-6600u: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>         (i915#4547 <https://gitlab.freedesktop.org/drm/intel/issues/4547>)
>   *
> 
>     igt@gem_huc_copy@huc-copy:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@gem_huc_copy@huc-copy.html>
>         (i915#2190 <https://gitlab.freedesktop.org/drm/intel/issues/2190>)
>   *
> 
>     igt@i915_pm_backlight@basic-brightness:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@i915_pm_backlight@basic-brightness.html>
>         (i915#1155 <https://gitlab.freedesktop.org/drm/intel/issues/1155>)
>   *
> 
>     igt@kms_chamelium@common-hpd-after-suspend:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_chamelium@common-hpd-after-suspend.html>
>         (fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html>
>         (i915#4103
>         <https://gitlab.freedesktop.org/drm/intel/issues/4103>) +1
>         similar issue
>   *
> 
>     igt@kms_force_connector_basic@force-load-detect:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_force_connector_basic@force-load-detect.html>
>         (fdo#109285 <https://bugs.freedesktop.org/show_bug.cgi?id=109285>)
>   *
> 
>     igt@kms_psr@primary_mmap_gtt:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@kms_psr@primary_mmap_gtt.html>
>         (i915#1072
>         <https://gitlab.freedesktop.org/drm/intel/issues/1072>) +3
>         similar issues
>   *
> 
>     igt@prime_vgem@basic-userptr:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-tgl-1115g4/igt@prime_vgem@basic-userptr.html>
>         (i915#3301 <https://gitlab.freedesktop.org/drm/intel/issues/3301>)
>   *
> 
>     igt@runner@aborted:
> 
>       o fi-skl-6600u: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-skl-6600u/igt@runner@aborted.html>
>         (i915#3363
>         <https://gitlab.freedesktop.org/drm/intel/issues/3363> /
>         i915#4312 
> <https://gitlab.freedesktop.org/drm/intel/issues/4312>)
> 
> 
>         Possible fixes
> 
>   *
> 
>     igt@i915_selftest@live@evict:
> 
>       o fi-kbl-soraka: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-kbl-soraka/igt@i915_selftest@live@evict.html>
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-kbl-soraka/igt@i915_selftest@live@evict.html>
>   *
> 
>     igt@i915_selftest@live@gt_engines:
> 
>       o fi-rkl-guc: DMESG-FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-rkl-guc/igt@i915_selftest@live@gt_engines.html>
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-rkl-guc/igt@i915_selftest@live@gt_engines.html>
>   *
> 
>     igt@i915_selftest@live@hangcheck:
> 
>       o fi-snb-2600: INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10896/fi-snb-2600/igt@i915_selftest@live@hangcheck.html>
>         (i915#3921
>         <https://gitlab.freedesktop.org/drm/intel/issues/3921>) -> PASS
>         
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21617/fi-snb-2600/
> igt@i915_selftest@live@hangcheck.html>
> 
> {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_10896 -> Patchwork_21617
> 
> CI-20190529: 20190529
> CI_DRM_10896: bf11e5f354ac51635d032893d80a1d0015d277dc @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_6283: a2cd90a7c24bb7a4c19ca74c75ed8c950820dee2 @ 
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_21617: 7dc7916fd526389e070074e9e3895c12b0bcdb29 @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> 
> == Linux commits ==
> 
> 7dc7916fd526 drm/i915: Drain the ttm delayed workqueue too
> 283402ea4955 drm/i915: Remove resv from i915_vma 1a43e1835f7f 
> drm/i915: vma is always backed by an object.
> b2328bc6d485 drm/i915: Create a full object for mock_ring, v2.
> 8864b863b878 drm/i915: Create a dummy object for gen6 ppgtt 
> 2c4bb85ec07b drm/i915: move the pre_pin earlier
>
Matthew Auld Nov. 19, 2021, 10:27 a.m. UTC | #6
On 18/11/2021 16:36, Patchwork wrote:
> *Patch Details*
> *Series:*	series starting with [v2,1/6] drm/i915: move the pre_pin 
> earlier (rev2)
> *URL:*	https://patchwork.freedesktop.org/series/97026/ 
> <https://patchwork.freedesktop.org/series/97026/>
> *State:*	failure
> *Details:* 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/index.html 
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/index.html>
> 
> 
>   CI Bug Log - changes from CI_DRM_10900 -> Patchwork_21631
> 
> 
>     Summary
> 
> *FAILURE*
> 
> Serious unknown changes coming with Patchwork_21631 absolutely need to be
> verified manually.
> 
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_21631, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
> 
> External URL: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/index.html
> 
> 
>     Participating hosts (38 -> 33)
> 
> Additional (1): fi-tgl-1115g4
> Missing (6): fi-kbl-soraka fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 
> bat-jsl-2 bat-jsl-1
> 
> 
>     Possible new issues
> 
> Here are the unknown changes that may have been introduced in 
> Patchwork_21631:
> 
> 
>       IGT changes
> 
> 
>         Possible regressions
> 
>   * igt@kms_psr@primary_page_flip:
>       o fi-skl-6600u: NOTRUN -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@kms_psr@primary_page_flip.html>

Looks like some ext4 specific issue.

> 
> 
>     Known issues
> 
> Here are the changes found in Patchwork_21631 that come from known issues:
> 
> 
>       IGT changes
> 
> 
>         Issues hit
> 
>   *
> 
>     igt@amdgpu/amd_basic@query-info:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@amdgpu/amd_basic@query-info.html>
>         (fdo#109315 <https://bugs.freedesktop.org/show_bug.cgi?id=109315>)
>   *
> 
>     igt@amdgpu/amd_basic@semaphore:
> 
>       o fi-bdw-5557u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-bdw-5557u/igt@amdgpu/amd_basic@semaphore.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +31
>         similar issues
>   *
> 
>     igt@amdgpu/amd_cs_nop@nop-gfx0:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@amdgpu/amd_cs_nop@nop-gfx0.html>
>         (fdo#109315
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109315> /
>         i915#2575
>         <https://gitlab.freedesktop.org/drm/intel/issues/2575>) +16
>         similar issues
>   *
> 
>     igt@core_hotunplug@unbind-rebind:
> 
>       o fi-tgl-u2: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-tgl-u2/igt@core_hotunplug@unbind-rebind.html>
>         -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-u2/igt@core_hotunplug@unbind-rebind.html>
>         (i915#4006 <https://gitlab.freedesktop.org/drm/intel/issues/4006>)
>   *
> 
>     igt@gem_exec_suspend@basic-s3:
> 
>       o fi-tgl-1115g4: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html>
>         (i915#1888 <https://gitlab.freedesktop.org/drm/intel/issues/1888>)
>   *
> 
>     igt@gem_huc_copy@huc-copy:
> 
>       o
> 
>         fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@gem_huc_copy@huc-copy.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
>         i915#2190 <https://gitlab.freedesktop.org/drm/intel/issues/2190>)
> 
>       o
> 
>         fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_huc_copy@huc-copy.html>
>         (i915#2190 <https://gitlab.freedesktop.org/drm/intel/issues/2190>)
> 
>   *
> 
>     igt@gem_lmem_swapping@basic:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_lmem_swapping@basic.html>
>         (i915#4555 <https://gitlab.freedesktop.org/drm/intel/issues/4555>)
>   *
> 
>     igt@gem_lmem_swapping@verify-random:
> 
>       o
> 
>         fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@gem_lmem_swapping@verify-random.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +6
>         similar issues
> 
>       o
> 
>         fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_lmem_swapping@verify-random.html>
>         (i915#4555
>         <https://gitlab.freedesktop.org/drm/intel/issues/4555> /
>         i915#4565
>         <https://gitlab.freedesktop.org/drm/intel/issues/4565>) +2
>         similar issues
> 
>   *
> 
>     igt@i915_pm_backlight@basic-brightness:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@i915_pm_backlight@basic-brightness.html>
>         (i915#1155 <https://gitlab.freedesktop.org/drm/intel/issues/1155>)
>   *
> 
>     igt@kms_chamelium@common-hpd-after-suspend:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_chamelium@common-hpd-after-suspend.html>
>         (fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_chamelium@dp-crc-fast:
> 
>       o fi-bdw-5557u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-bdw-5557u/igt@kms_chamelium@dp-crc-fast.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
>         fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_chamelium@vga-edid-read:
> 
>       o fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@kms_chamelium@vga-edid-read.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
>         fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html>
>         (i915#4103
>         <https://gitlab.freedesktop.org/drm/intel/issues/4103>) +1
>         similar issue
>   *
> 
>     igt@kms_force_connector_basic@force-load-detect:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_force_connector_basic@force-load-detect.html>
>         (fdo#109285 <https://bugs.freedesktop.org/show_bug.cgi?id=109285>)
>   *
> 
>     igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
> 
>       o fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> / i915#533
>         <https://gitlab.freedesktop.org/drm/intel/issues/533>)
>   *
> 
>     igt@kms_psr@primary_mmap_gtt:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_psr@primary_mmap_gtt.html>
>         (i915#1072
>         <https://gitlab.freedesktop.org/drm/intel/issues/1072>) +3
>         similar issues
>   *
> 
>     igt@prime_vgem@basic-userptr:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@prime_vgem@basic-userptr.html>
>         (i915#3301 <https://gitlab.freedesktop.org/drm/intel/issues/3301>)
>   *
> 
>     igt@runner@aborted:
> 
>       o fi-tgl-u2: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-u2/igt@runner@aborted.html>
>         (i915#1602
>         <https://gitlab.freedesktop.org/drm/intel/issues/1602> /
>         i915#2722 <https://gitlab.freedesktop.org/drm/intel/issues/2722>
>         / i915#4312 <https://gitlab.freedesktop.org/drm/intel/issues/4312>)
> 
> 
>         Possible fixes
> 
>   *
> 
>     igt@gem_exec_suspend@basic-s3:
> 
>       o fi-bdw-5557u: INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html>
>         (i915#146 <https://gitlab.freedesktop.org/drm/intel/issues/146>)
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html>
>   *
> 
>     igt@gem_flink_basic@bad-flink:
> 
>       o fi-skl-6600u: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>         (i915#4547
>         <https://gitlab.freedesktop.org/drm/intel/issues/4547>) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>   *
> 
>     igt@kms_frontbuffer_tracking@basic:
> 
>       o fi-cml-u2: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html>
>         (i915#4269
>         <https://gitlab.freedesktop.org/drm/intel/issues/4269>) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html>
> 
> 
>         Warnings
> 
>   * igt@gem_lmem_swapping@verify-random:
>       o fi-tgl-u2: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-tgl-u2/igt@gem_lmem_swapping@verify-random.html>
>         (i915#4555
>         <https://gitlab.freedesktop.org/drm/intel/issues/4555>) -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-u2/igt@gem_lmem_swapping@verify-random.html>
>         (i915#4555
>         <https://gitlab.freedesktop.org/drm/intel/issues/4555> /
>         i915#4565
>         <https://gitlab.freedesktop.org/drm/intel/issues/4565>) +2
>         similar issues
> 
> {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_10900 -> Patchwork_21631
> 
> CI-20190529: 20190529
> CI_DRM_10900: b50839f33180500c64a505623ab77829b869a57c @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_6285: 2e0355faad5c2e81cd6705b76e529ce526c7c9bf @ 
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_21631: 59b76c709a722b22b4ad4c6757b8472f42c33389 @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> 
> == Linux commits ==
> 
> 59b76c709a72 drm/i915: Drain the ttm delayed workqueue too
> 31d1314bebea drm/i915: Remove resv from i915_vma
> a3ec36b94293 drm/i915: vma is always backed by an object.
> af937f97b589 drm/i915: Create a full object for mock_ring, v2.
> 0e6d48163844 drm/i915: Create a dummy object for gen6 ppgtt
> ffd0efc92828 drm/i915: move the pre_pin earlier
>
Matthew Auld Nov. 19, 2021, 3:30 p.m. UTC | #7
On 19/11/2021 13:53, Patchwork wrote:
> *Patch Details*
> *Series:*	series starting with [v2,1/6] drm/i915: move the pre_pin 
> earlier (rev3)
> *URL:*	https://patchwork.freedesktop.org/series/97026/ 
> <https://patchwork.freedesktop.org/series/97026/>
> *State:*	failure
> *Details:* 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/index.html 
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/index.html>
> 
> 
>   CI Bug Log - changes from CI_DRM_10904_full -> Patchwork_21637_full
> 
> 
>     Summary
> 
> *FAILURE*
> 
> Serious unknown changes coming with Patchwork_21637_full absolutely need 
> to be
> verified manually.
> 
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_21637_full, please notify your bug team to allow 
> them
> to document this new failure mode, which will reduce false positives in CI.
> 
> 
>     Participating hosts (11 -> 10)
> 
> Missing (1): shard-rkl
> 
> 
>     Possible new issues
> 
> Here are the unknown changes that may have been introduced in 
> Patchwork_21637_full:
> 
> 
>       IGT changes
> 
> 
>         Possible regressions
> 
>   * igt@gem_ctx_shared@q-smoketest@vecs0:
>       o shard-tglb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb2/igt@gem_ctx_shared@q-smoketest@vecs0.html>
>         -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb7/igt@gem_ctx_shared@q-smoketest@vecs0.html>
> 

Looks like machine is killed by the runner(?), for some reason. Fairly 
sure this is not related. For the most part this series should only 
really impact some gen6/7 platforms.


> 
>     Known issues
> 
> Here are the changes found in Patchwork_21637_full that come from known 
> issues:
> 
> 
>       IGT changes
> 
> 
>         Issues hit
> 
>   *
> 
>     igt@gem_eio@unwedge-stress:
> 
>       o shard-tglb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb6/igt@gem_eio@unwedge-stress.html>
>         -> TIMEOUT
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb6/igt@gem_eio@unwedge-stress.html>
>         ([i915#2369] / [i915#3063] / [i915#3648])
>   *
> 
>     igt@gem_exec_capture@pi@bcs0:
> 
>       o shard-skl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl4/igt@gem_exec_capture@pi@bcs0.html>
>         -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl6/igt@gem_exec_capture@pi@bcs0.html>
>         ([i915#2369])
>   *
> 
>     igt@gem_exec_fair@basic-none-share@rcs0:
> 
>       o shard-apl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-apl8/igt@gem_exec_fair@basic-none-share@rcs0.html>
>         -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl6/igt@gem_exec_fair@basic-none-share@rcs0.html>
>         ([fdo#109271])
>   *
> 
>     igt@gem_exec_fair@basic-pace@vcs0:
> 
>       o shard-kbl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl2/igt@gem_exec_fair@basic-pace@vcs0.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@gem_exec_fair@basic-pace@vcs0.html>
>         ([i915#2842])
>   *
> 
>     igt@gem_exec_fair@basic-pace@vecs0:
> 
>       o
> 
>         shard-tglb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb2/igt@gem_exec_fair@basic-pace@vecs0.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@gem_exec_fair@basic-pace@vecs0.html>
>         ([i915#2842])
> 
>       o
> 
>         shard-iclb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb4/igt@gem_exec_fair@basic-pace@vecs0.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb4/igt@gem_exec_fair@basic-pace@vecs0.html>
>         ([i915#2842])
> 
>   *
> 
>     igt@gem_exec_suspend@basic-s3:
> 
>       o shard-tglb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb5/igt@gem_exec_suspend@basic-s3.html>
>         -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb7/igt@gem_exec_suspend@basic-s3.html>
>         ([i915#456])
>   *
> 
>     igt@gem_render_copy@y-tiled-to-vebox-y-tiled:
> 
>       o shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@gem_render_copy@y-tiled-to-vebox-y-tiled.html>
>         ([i915#768])
>   *
> 
>     igt@gem_userptr_blits@dmabuf-sync:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@gem_userptr_blits@dmabuf-sync.html>
>         ([fdo#109271] / [i915#3323])
>   *
> 
>     igt@gem_userptr_blits@vma-merge:
> 
>       o shard-skl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@gem_userptr_blits@vma-merge.html>
>         ([i915#3318])
>   *
> 
>     igt@gem_workarounds@suspend-resume-context:
> 
>       o shard-apl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-apl3/igt@gem_workarounds@suspend-resume-context.html>
>         -> DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl8/igt@gem_workarounds@suspend-resume-context.html>
>         ([i915#180]) +4 similar issues
>   *
> 
>     igt@gen9_exec_parse@allowed-all:
> 
>       o
> 
>         shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@gen9_exec_parse@allowed-all.html>
>         ([i915#2856])
> 
>       o
> 
>         shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb6/igt@gen9_exec_parse@allowed-all.html>
>         ([i915#2856])
> 
>       o
> 
>         shard-glk: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-glk9/igt@gen9_exec_parse@allowed-all.html>
>         -> DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-glk4/igt@gen9_exec_parse@allowed-all.html>
>         ([i915#1436] / [i915#716])
> 
>   *
> 
>     igt@i915_pm_dc@dc6-psr:
> 
>       o shard-iclb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb1/igt@i915_pm_dc@dc6-psr.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb7/igt@i915_pm_dc@dc6-psr.html>
>         ([i915#454])
>   *
> 
>     igt@i915_pm_rpm@modeset-lpsp-stress-no-wait:
> 
>       o shard-kbl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl4/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html>
>         ([fdo#109271]) +20 similar issues
>   *
> 
>     igt@i915_pm_rpm@system-suspend:
> 
>       o shard-kbl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl1/igt@i915_pm_rpm@system-suspend.html>
>         -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl2/igt@i915_pm_rpm@system-suspend.html>
>         ([i915#151])
>   *
> 
>     igt@kms_async_flips@crc:
> 
>       o shard-skl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl6/igt@kms_async_flips@crc.html>
>         ([i915#4272])
>   *
> 
>     igt@kms_big_fb@linear-8bpp-rotate-90:
> 
>       o
> 
>         shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb6/igt@kms_big_fb@linear-8bpp-rotate-90.html>
>         ([fdo#111614])
> 
>       o
> 
>         shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_big_fb@linear-8bpp-rotate-90.html>
>         ([fdo#110725] / [fdo#111614])
> 
>   *
> 
>     igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
> 
>       o shard-skl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl7/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html>
>         ([i915#3743]) +1 similar issue
>   *
> 
>     igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip:
> 
>       o shard-kbl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl4/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip.html>
>         ([fdo#109271] / [i915#3777])
>   *
> 
>     igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0.html>
>         ([fdo#111615])
>   *
> 
>     igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-hflip:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-hflip.html>
>         ([fdo#109271] / [i915#3777])
>   *
> 
>     igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_mc_ccs:
> 
>       o
> 
>         shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_mc_ccs.html>
>         ([fdo#109278] / [i915#3886])
> 
>       o
> 
>         shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb6/igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_mc_ccs.html>
>         ([i915#3689] / [i915#3886])
> 
>   *
> 
>     igt@kms_ccs@pipe-b-random-ccs-data-yf_tiled_ccs:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@kms_ccs@pipe-b-random-ccs-data-yf_tiled_ccs.html>
>         ([fdo#111615] / [i915#3689])
>   *
> 
>     igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_rc_ccs_cc:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_rc_ccs_cc.html>
>         ([fdo#109271] / [i915#3886]) +4 similar issues
>   *
> 
>     igt@kms_ccs@pipe-d-crc-primary-rotation-180-yf_tiled_ccs:
> 
>       o shard-apl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl6/igt@kms_ccs@pipe-d-crc-primary-rotation-180-yf_tiled_ccs.html>
>         ([fdo#109271]) +21 similar issues
>   *
> 
>     igt@kms_cdclk@plane-scaling:
> 
>       o shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_cdclk@plane-scaling.html>
>         ([i915#3742])
>   *
> 
>     igt@kms_chamelium@dp-crc-single:
> 
>       o shard-kbl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl4/igt@kms_chamelium@dp-crc-single.html>
>         ([fdo#109271] / [fdo#111827]) +2 similar issues
>   *
> 
>     igt@kms_chamelium@hdmi-crc-single:
> 
>       o
> 
>         shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_chamelium@hdmi-crc-single.html>
>         ([fdo#109284] / [fdo#111827])
> 
>       o
> 
>         shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb6/igt@kms_chamelium@hdmi-crc-single.html>
>         ([fdo#109284] / [fdo#111827])
> 
>   *
> 
>     igt@kms_color_chamelium@pipe-d-ctm-green-to-red:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@kms_color_chamelium@pipe-d-ctm-green-to-red.html>
>         ([fdo#109271] / [fdo#111827]) +9 similar issues
>   *
> 
>     igt@kms_color_chamelium@pipe-d-ctm-negative:
> 
>       o shard-apl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl6/igt@kms_color_chamelium@pipe-d-ctm-negative.html>
>         ([fdo#109271] / [fdo#111827]) +1 similar issue
>   *
> 
>     igt@kms_cursor_crc@pipe-a-cursor-32x32-onscreen:
> 
>       o shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_cursor_crc@pipe-a-cursor-32x32-onscreen.html>
>         ([fdo#109278]) +3 similar issues
>   *
> 
>     igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement.html>
>         ([i915#3319]) +1 similar issue
>   *
> 
>     igt@kms_cursor_crc@pipe-c-cursor-suspend:
> 
>       o shard-apl: NOTRUN -> DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl6/igt@kms_cursor_crc@pipe-c-cursor-suspend.html>
>         ([i915#180]) +1 similar issue
>   *
> 
>     igt@kms_cursor_crc@pipe-d-cursor-512x512-offscreen:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb6/igt@kms_cursor_crc@pipe-d-cursor-512x512-offscreen.html>
>         ([fdo#109279] / [i915#3359])
>   *
> 
>     igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:
> 
>       o shard-skl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl1/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl3/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html>
>         ([i915#2346] / [i915#533])
>   *
> 
>     igt@kms_flip@2x-flip-vs-expired-vblank:
> 
>       o shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_flip@2x-flip-vs-expired-vblank.html>
>         ([fdo#109274])
>   *
> 
>     igt@kms_flip@flip-vs-suspend-interruptible@a-dp1:
> 
>       o shard-kbl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl4/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html>
>         -> DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html>
>         ([i915#180]) +5 similar issues
>   *
> 
>     igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs:
> 
>       o shard-skl: NOTRUN -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs.html>
>         ([i915#3699])
>   *
> 
>     igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-blt:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-blt.html>
>         ([fdo#109271]) +149 similar issues
>   *
> 
>     igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-blt:
> 
>       o shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-blt.html>
>         ([fdo#109280]) +1 similar issue
>   *
> 
>     igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-onoff:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-onoff.html>
>         ([fdo#111825]) +9 similar issues
>   *
> 
>     igt@kms_hdr@static-swap:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@kms_hdr@static-swap.html>
>         ([i915#1187])
>   *
> 
>     igt@kms_pipe_crc_basic@nonblocking-crc-pipe-d-frame-sequence:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl6/igt@kms_pipe_crc_basic@nonblocking-crc-pipe-d-frame-sequence.html>
>         ([fdo#109271] / [i915#533]) +1 similar issue
>   *
> 
>     igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb:
> 
>       o shard-skl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html>
>         ([i915#265])
>   *
> 
>     igt@kms_plane_alpha_blend@pipe-b-alpha-basic:
> 
>       o shard-apl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl6/igt@kms_plane_alpha_blend@pipe-b-alpha-basic.html>
>         ([fdo#108145] / [i915#265])
>   *
> 
>     igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb:
> 
>       o shard-skl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html>
>         ([fdo#108145] / [i915#265])
>   *
> 
>     igt@kms_plane_alpha_blend@pipe-c-constant-alpha-max:
> 
>       o shard-kbl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl4/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-max.html>
>         ([fdo#108145] / [i915#265]) +1 similar issue
>   *
> 
>     igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min:
> 
>       o shard-skl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl6/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl4/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html>
>         ([fdo#108145] / [i915#265]) +1 similar issue
>   *
> 
>     igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping.html>
>         ([fdo#109271] / [i915#2733])
>   *
> 
>     igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3.html>
>         ([i915#2920])
>   *
> 
>     igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html>
>         ([fdo#109271] / [i915#658]) +1 similar issue
>   *
> 
>     igt@kms_psr2_su@frontbuffer:
> 
>       o shard-kbl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl4/igt@kms_psr2_su@frontbuffer.html>
>         ([fdo#109271] / [i915#658])
>   *
> 
>     igt@kms_psr@psr2_sprite_plane_move:
> 
>       o shard-iclb: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html>
>         -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb7/igt@kms_psr@psr2_sprite_plane_move.html>
>         ([fdo#109441]) +1 similar issue
>   *
> 
>     igt@kms_sysfs_edid_timing:
> 
>       o shard-skl: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl10/igt@kms_sysfs_edid_timing.html>
>         ([IGT#2])
>   *
> 
>     igt@nouveau_crc@pipe-c-ctx-flip-skip-current-frame:
> 
>       o shard-iclb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb6/igt@nouveau_crc@pipe-c-ctx-flip-skip-current-frame.html>
>         ([i915#2530])
>   *
> 
>     igt@nouveau_crc@pipe-c-source-outp-complete:
> 
>       o shard-tglb: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@nouveau_crc@pipe-c-source-outp-complete.html>
>         ([i915#2530]) +1 similar issue
>   *
> 
>     igt@perf@polling-parameterized:
> 
>       o shard-glk: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-glk6/igt@perf@polling-parameterized.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-glk6/igt@perf@polling-parameterized.html>
>         ([i915#1542])
>   *
> 
>     igt@perf@polling-small-buf:
> 
>       o shard-skl: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl7/igt@perf@polling-small-buf.html>
>         -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl6/igt@perf@polling-small-buf.html>
>         ([i915#1722])
>   *
> 
>     igt@perf_pmu@rc6-suspend:
> 
>       o shard-kbl: NOTRUN -> DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@perf_pmu@rc6-suspend.html>
>         ([i915#180])
>   *
> 
>     igt@sysfs_clients@fair-1:
> 
>       o shard-skl: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@sysfs_clients@fair-1.html>
>         ([fdo#109271] / [i915#2994]) +1 similar issue
> 
> 
>         Possible fixes
> 
>   *
> 
>     igt@gem_eio@in-flight-contexts-1us:
> 
>       o shard-tglb: TIMEOUT
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb3/igt@gem_eio@in-flight-contexts-1us.html>
>         ([i915#3063]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb8/igt@gem_eio@in-flight-contexts-1us.html>
>   *
> 
>     igt@gem_exec_fair@basic-none-share@rcs0:
> 
>       o shard-iclb: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb6/igt@gem_exec_fair@basic-none-share@rcs0.html>
>         ([i915#2842]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb2/igt@gem_exec_fair@basic-none-share@rcs0.html>
>   *
> 
>     igt@gem_exec_fair@basic-pace@vecs0:
> 
>       o shard-kbl: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl2/igt@gem_exec_fair@basic-pace@vecs0.html>
>         ([i915#2842]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@gem_exec_fair@basic-pace@vecs0.html>
>   *
> 
>     igt@gem_exec_fair@basic-sync@rcs0:
> 
>       o shard-kbl: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl4/igt@gem_exec_fair@basic-sync@rcs0.html>
>         ([fdo#109271]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl2/igt@gem_exec_fair@basic-sync@rcs0.html>
>   *
> 
>     igt@gem_exec_fair@basic-throttle@rcs0:
> 
>       o shard-iclb: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb3/igt@gem_exec_fair@basic-throttle@rcs0.html>
>         ([i915#2849]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb3/igt@gem_exec_fair@basic-throttle@rcs0.html>
>   *
> 
>     igt@gem_mmap_gtt@big-copy-xy:
> 
>       o shard-skl: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl4/igt@gem_mmap_gtt@big-copy-xy.html>
>         ([i915#1982]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl7/igt@gem_mmap_gtt@big-copy-xy.html>
>         +1 similar issue
>   *
> 
>     igt@i915_pm_rps@min-max-config-loaded:
> 
>       o shard-apl: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-apl2/igt@i915_pm_rps@min-max-config-loaded.html>
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl1/igt@i915_pm_rps@min-max-config-loaded.html>
>   *
> 
>     igt@i915_selftest@live@gem_contexts:
> 
>       o shard-tglb: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb2/igt@i915_selftest@live@gem_contexts.html>
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb3/igt@i915_selftest@live@gem_contexts.html>
>   *
> 
>     igt@i915_selftest@live@perf:
> 
>       o shard-tglb: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-tglb2/igt@i915_selftest@live@perf.html>
>         ([i915#2867]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-tglb3/igt@i915_selftest@live@perf.html>
>         +4 similar issues
>   *
> 
>     igt@i915_suspend@debugfs-reader:
> 
>       o shard-apl: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-apl8/igt@i915_suspend@debugfs-reader.html>
>         ([i915#180]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl6/igt@i915_suspend@debugfs-reader.html>
>         +1 similar issue
>   *
> 
>     igt@kms_cursor_legacy@flip-vs-cursor-varying-size:
> 
>       o shard-iclb: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb7/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html>
>         ([i915#2346]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb1/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html>
>   *
> 
>     igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1:
> 
>       o shard-skl: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl10/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html>
>         ([i915#2122]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl5/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html>
>   *
> 
>     igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
> 
>       o shard-skl: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl9/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html>
>         ([fdo#108145] / [i915#265]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html>
>         +1 similar issue
>   *
> 
>     igt@kms_psr@psr2_primary_mmap_cpu:
> 
>       o shard-iclb: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb4/igt@kms_psr@psr2_primary_mmap_cpu.html>
>         ([fdo#109441]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb2/igt@kms_psr@psr2_primary_mmap_cpu.html>
>         +1 similar issue
>   *
> 
>     igt@kms_vblank@pipe-c-ts-continuation-suspend:
> 
>       o shard-kbl: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl1/igt@kms_vblank@pipe-c-ts-continuation-suspend.html>
>         ([i915#180]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@kms_vblank@pipe-c-ts-continuation-suspend.html>
>         +1 similar issue
>   *
> 
>     igt@perf@polling-parameterized:
> 
>       o shard-skl: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl6/igt@perf@polling-parameterized.html>
>         ([i915#1542]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl4/igt@perf@polling-parameterized.html>
>   *
> 
>     igt@sysfs_heartbeat_interval@mixed@rcs0:
> 
>       o shard-skl: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-skl3/igt@sysfs_heartbeat_interval@mixed@rcs0.html>
>         ([i915#1731]) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-skl9/igt@sysfs_heartbeat_interval@mixed@rcs0.html>
>         +2 similar issues
> 
> 
>         Warnings
> 
>   *
> 
>     igt@i915_pm_dc@dc3co-vpb-simulation:
> 
>       o shard-iclb: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb4/igt@i915_pm_dc@dc3co-vpb-simulation.html>
>         ([i915#658]) -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html>
>         ([i915#588])
>   *
> 
>     igt@i915_pm_rc6_residency@rc6-fence:
> 
>       o shard-iclb: WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb4/igt@i915_pm_rc6_residency@rc6-fence.html>
>         ([i915#1804] / [i915#2684]) -> WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb2/igt@i915_pm_rc6_residency@rc6-fence.html>
>         ([i915#2684])
>   *
> 
>     igt@i915_pm_rc6_residency@rc6-idle:
> 
>       o shard-iclb: WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb8/igt@i915_pm_rc6_residency@rc6-idle.html>
>         ([i915#2684]) -> WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb3/igt@i915_pm_rc6_residency@rc6-idle.html>
>         ([i915#1804] / [i915#2684])
>   *
> 
>     igt@kms_content_protection@atomic-dpms:
> 
>       o shard-apl: DMESG-FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-apl4/igt@kms_content_protection@atomic-dpms.html>
>         ([fdo#110321]) -> TIMEOUT
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-apl4/igt@kms_content_protection@atomic-dpms.html>
>         ([i915#1319])
>   *
> 
>     igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-2:
> 
>       o shard-iclb: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb2/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-2.html>
>         ([i915#2920]) -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb7/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-2.html>
>         ([i915#658]) +2 similar issues
>   *
> 
>     igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1:
> 
>       o shard-iclb: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb6/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html>
>         ([i915#658]) -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html>
>         ([i915#2920]) +2 similar issues
>   *
> 
>     igt@kms_psr2_su@page_flip:
> 
>       o shard-iclb: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-iclb2/igt@kms_psr2_su@page_flip.html>
>         ([i915#4148]) -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-iclb3/igt@kms_psr2_su@page_flip.html>
>         ([fdo#109642] / [fdo#111068] / [i915#658])
>   *
> 
>     igt@runner@aborted:
> 
>       o
> 
>         shard-kbl: (FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl4/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl7/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl2/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl1/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl1/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10904/shard-kbl1/igt@runner@aborted.html>)
>         ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002] /
>         [i915#3363] / [i915#4312] / [i915#602]) -> (FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl7/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl1/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl1/igt@runner@aborted.html>,
>         FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21637/shard-kbl1/igt@runner@aborted.html>)
>         ([i915#180] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#4312])
> 
>       o
> 
>         shard-apl: ([FAIL][146], [FAIL][147], [FAIL][148], [FAIL][149],
>         [FAIL][150], [FAIL][151]) ([i915#180] / [i915#3002] /
>         [i915#3363] / [i915#4312]) -> ([FAIL][152], [FAIL][153],
>         [FAIL][154], [FAIL][155], [FAIL][156],
>
Vudum, Lakshminarayana Nov. 19, 2021, 5:05 p.m. UTC | #8
Filed https://gitlab.freedesktop.org/drm/intel/-/issues/4570 for the latest regression.
igt@gem_ctx_shared@q-smoketest@vecs0 - incomplete - No warnings/errors

Rev2 regression is related to https://gitlab.freedesktop.org/drm/intel/-/issues/4564
igt@gem_exec_suspend@basic-s3|igt@kms_psr@primary_page_flip - incomplete - INFO: task jbd2/nvme0n1p2-:161 blocked for more than 30 seconds.

Lakshmi.
-----Original Message-----
From: Auld, Matthew <matthew.auld@intel.com> 
Sent: Friday, November 19, 2021 2:27 AM
To: intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Subject: Re: ✗ Fi.CI.BAT: failure for series starting with [v2,1/6] drm/i915: move the pre_pin earlier (rev2)

On 18/11/2021 16:36, Patchwork wrote:
> *Patch Details*
> *Series:*	series starting with [v2,1/6] drm/i915: move the pre_pin 
> earlier (rev2)
> *URL:*	https://patchwork.freedesktop.org/series/97026/ 
> <https://patchwork.freedesktop.org/series/97026/>
> *State:*	failure
> *Details:*
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/index.html
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/index.html>
> 
> 
>   CI Bug Log - changes from CI_DRM_10900 -> Patchwork_21631
> 
> 
>     Summary
> 
> *FAILURE*
> 
> Serious unknown changes coming with Patchwork_21631 absolutely need to 
> be verified manually.
> 
> If you think the reported changes have nothing to do with the changes 
> introduced in Patchwork_21631, please notify your bug team to allow 
> them to document this new failure mode, which will reduce false positives in CI.
> 
> External URL: 
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/index.html
> 
> 
>     Participating hosts (38 -> 33)
> 
> Additional (1): fi-tgl-1115g4
> Missing (6): fi-kbl-soraka fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600
> bat-jsl-2 bat-jsl-1
> 
> 
>     Possible new issues
> 
> Here are the unknown changes that may have been introduced in
> Patchwork_21631:
> 
> 
>       IGT changes
> 
> 
>         Possible regressions
> 
>   * igt@kms_psr@primary_page_flip:
>       o fi-skl-6600u: NOTRUN -> INCOMPLETE
>         
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u
> /igt@kms_psr@primary_page_flip.html>

Looks like some ext4 specific issue.

> 
> 
>     Known issues
> 
> Here are the changes found in Patchwork_21631 that come from known issues:
> 
> 
>       IGT changes
> 
> 
>         Issues hit
> 
>   *
> 
>     igt@amdgpu/amd_basic@query-info:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@amdgpu/amd_basic@query-info.html>
>         (fdo#109315 <https://bugs.freedesktop.org/show_bug.cgi?id=109315>)
>   *
> 
>     igt@amdgpu/amd_basic@semaphore:
> 
>       o fi-bdw-5557u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-bdw-5557u/igt@amdgpu/amd_basic@semaphore.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +31
>         similar issues
>   *
> 
>     igt@amdgpu/amd_cs_nop@nop-gfx0:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@amdgpu/amd_cs_nop@nop-gfx0.html>
>         (fdo#109315
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109315> /
>         i915#2575
>         <https://gitlab.freedesktop.org/drm/intel/issues/2575>) +16
>         similar issues
>   *
> 
>     igt@core_hotunplug@unbind-rebind:
> 
>       o fi-tgl-u2: PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-tgl-u2/igt@core_hotunplug@unbind-rebind.html>
>         -> INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-u2/igt@core_hotunplug@unbind-rebind.html>
>         (i915#4006 <https://gitlab.freedesktop.org/drm/intel/issues/4006>)
>   *
> 
>     igt@gem_exec_suspend@basic-s3:
> 
>       o fi-tgl-1115g4: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html>
>         (i915#1888 <https://gitlab.freedesktop.org/drm/intel/issues/1888>)
>   *
> 
>     igt@gem_huc_copy@huc-copy:
> 
>       o
> 
>         fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@gem_huc_copy@huc-copy.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
>         i915#2190 
> <https://gitlab.freedesktop.org/drm/intel/issues/2190>)
> 
>       o
> 
>         fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_huc_copy@huc-copy.html>
>         (i915#2190 
> <https://gitlab.freedesktop.org/drm/intel/issues/2190>)
> 
>   *
> 
>     igt@gem_lmem_swapping@basic:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_lmem_swapping@basic.html>
>         (i915#4555 <https://gitlab.freedesktop.org/drm/intel/issues/4555>)
>   *
> 
>     igt@gem_lmem_swapping@verify-random:
> 
>       o
> 
>         fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@gem_lmem_swapping@verify-random.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +6
>         similar issues
> 
>       o
> 
>         fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@gem_lmem_swapping@verify-random.html>
>         (i915#4555
>         <https://gitlab.freedesktop.org/drm/intel/issues/4555> /
>         i915#4565
>         <https://gitlab.freedesktop.org/drm/intel/issues/4565>) +2
>         similar issues
> 
>   *
> 
>     igt@i915_pm_backlight@basic-brightness:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@i915_pm_backlight@basic-brightness.html>
>         (i915#1155 <https://gitlab.freedesktop.org/drm/intel/issues/1155>)
>   *
> 
>     igt@kms_chamelium@common-hpd-after-suspend:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_chamelium@common-hpd-after-suspend.html>
>         (fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_chamelium@dp-crc-fast:
> 
>       o fi-bdw-5557u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-bdw-5557u/igt@kms_chamelium@dp-crc-fast.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
>         fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_chamelium@vga-edid-read:
> 
>       o fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@kms_chamelium@vga-edid-read.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
>         fdo#111827
>         <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8
>         similar issues
>   *
> 
>     igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html>
>         (i915#4103
>         <https://gitlab.freedesktop.org/drm/intel/issues/4103>) +1
>         similar issue
>   *
> 
>     igt@kms_force_connector_basic@force-load-detect:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_force_connector_basic@force-load-detect.html>
>         (fdo#109285 <https://bugs.freedesktop.org/show_bug.cgi?id=109285>)
>   *
> 
>     igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
> 
>       o fi-skl-6600u: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html>
>         (fdo#109271
>         <https://bugs.freedesktop.org/show_bug.cgi?id=109271> / i915#533
>         <https://gitlab.freedesktop.org/drm/intel/issues/533>)
>   *
> 
>     igt@kms_psr@primary_mmap_gtt:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@kms_psr@primary_mmap_gtt.html>
>         (i915#1072
>         <https://gitlab.freedesktop.org/drm/intel/issues/1072>) +3
>         similar issues
>   *
> 
>     igt@prime_vgem@basic-userptr:
> 
>       o fi-tgl-1115g4: NOTRUN -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-1115g4/igt@prime_vgem@basic-userptr.html>
>         (i915#3301 <https://gitlab.freedesktop.org/drm/intel/issues/3301>)
>   *
> 
>     igt@runner@aborted:
> 
>       o fi-tgl-u2: NOTRUN -> FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-u2/igt@runner@aborted.html>
>         (i915#1602
>         <https://gitlab.freedesktop.org/drm/intel/issues/1602> /
>         i915#2722 <https://gitlab.freedesktop.org/drm/intel/issues/2722>
>         / i915#4312 
> <https://gitlab.freedesktop.org/drm/intel/issues/4312>)
> 
> 
>         Possible fixes
> 
>   *
> 
>     igt@gem_exec_suspend@basic-s3:
> 
>       o fi-bdw-5557u: INCOMPLETE
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html>
>         (i915#146 <https://gitlab.freedesktop.org/drm/intel/issues/146>)
>         -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html>
>   *
> 
>     igt@gem_flink_basic@bad-flink:
> 
>       o fi-skl-6600u: FAIL
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>         (i915#4547
>         <https://gitlab.freedesktop.org/drm/intel/issues/4547>) -> PASS
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-skl-6600u/igt@gem_flink_basic@bad-flink.html>
>   *
> 
>     igt@kms_frontbuffer_tracking@basic:
> 
>       o fi-cml-u2: DMESG-WARN
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html>
>         (i915#4269
>         <https://gitlab.freedesktop.org/drm/intel/issues/4269>) -> PASS
>         
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-cml-u2/ig
> t@kms_frontbuffer_tracking@basic.html>
> 
> 
>         Warnings
> 
>   * igt@gem_lmem_swapping@verify-random:
>       o fi-tgl-u2: SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10900/fi-tgl-u2/igt@gem_lmem_swapping@verify-random.html>
>         (i915#4555
>         <https://gitlab.freedesktop.org/drm/intel/issues/4555>) -> SKIP
>         <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21631/fi-tgl-u2/igt@gem_lmem_swapping@verify-random.html>
>         (i915#4555
>         <https://gitlab.freedesktop.org/drm/intel/issues/4555> /
>         i915#4565
>         <https://gitlab.freedesktop.org/drm/intel/issues/4565>) +2
>         similar issues
> 
> {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_10900 -> Patchwork_21631
> 
> CI-20190529: 20190529
> CI_DRM_10900: b50839f33180500c64a505623ab77829b869a57c @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_6285: 2e0355faad5c2e81cd6705b76e529ce526c7c9bf @ 
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_21631: 59b76c709a722b22b4ad4c6757b8472f42c33389 @ 
> git://anongit.freedesktop.org/gfx-ci/linux
> 
> == Linux commits ==
> 
> 59b76c709a72 drm/i915: Drain the ttm delayed workqueue too 
> 31d1314bebea drm/i915: Remove resv from i915_vma
> a3ec36b94293 drm/i915: vma is always backed by an object.
> af937f97b589 drm/i915: Create a full object for mock_ring, v2.
> 0e6d48163844 drm/i915: Create a dummy object for gen6 ppgtt
> ffd0efc92828 drm/i915: move the pre_pin earlier
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/gt/intel_context.c b/drivers/gpu/drm/i915/gt/intel_context.c
index 5634d14052bc..4c296de1d67d 100644
--- a/drivers/gpu/drm/i915/gt/intel_context.c
+++ b/drivers/gpu/drm/i915/gt/intel_context.c
@@ -228,17 +228,17 @@  int __intel_context_do_pin_ww(struct intel_context *ce,
 	if (err)
 		return err;
 
-	err = i915_active_acquire(&ce->active);
+	err = ce->ops->pre_pin(ce, ww, &vaddr);
 	if (err)
 		goto err_ctx_unpin;
 
-	err = ce->ops->pre_pin(ce, ww, &vaddr);
+	err = i915_active_acquire(&ce->active);
 	if (err)
-		goto err_release;
+		goto err_post_unpin;
 
 	err = mutex_lock_interruptible(&ce->pin_mutex);
 	if (err)
-		goto err_post_unpin;
+		goto err_release;
 
 	intel_engine_pm_might_get(ce->engine);
 
@@ -273,11 +273,11 @@  int __intel_context_do_pin_ww(struct intel_context *ce,
 
 err_unlock:
 	mutex_unlock(&ce->pin_mutex);
+err_release:
+	i915_active_release(&ce->active);
 err_post_unpin:
 	if (!handoff)
 		ce->ops->post_unpin(ce);
-err_release:
-	i915_active_release(&ce->active);
 err_ctx_unpin:
 	intel_context_post_unpin(ce);