* [PATCH v2 0/2] Fix for two GuC issues
@ 2022-11-02 19:21 ` John.C.Harrison
0 siblings, 0 replies; 19+ messages in thread
From: John.C.Harrison @ 2022-11-02 19:21 UTC (permalink / raw)
To: Intel-GFX; +Cc: John Harrison, DRI-Devel
From: John Harrison <John.C.Harrison@Intel.com>
Fix for a deadlock issue between the GuC busyness stats worker and GT
resets. Also fix kernel contexts not getting the correct scheduling
priority at start of day.
v2: Rename existing uses of _trylock rather than adding a _noretry
version. Also improve the comment a bit.
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
John Harrison (2):
drm/i915/guc: Properly initialise kernel contexts
drm/i915/guc: Don't deadlock busyness stats vs reset
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
.../gpu/drm/i915/gt/uc/intel_guc_submission.c | 7 ++++++-
4 files changed, 24 insertions(+), 4 deletions(-)
--
2.37.3
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-gfx] [PATCH v2 0/2] Fix for two GuC issues
@ 2022-11-02 19:21 ` John.C.Harrison
0 siblings, 0 replies; 19+ messages in thread
From: John.C.Harrison @ 2022-11-02 19:21 UTC (permalink / raw)
To: Intel-GFX; +Cc: DRI-Devel
From: John Harrison <John.C.Harrison@Intel.com>
Fix for a deadlock issue between the GuC busyness stats worker and GT
resets. Also fix kernel contexts not getting the correct scheduling
priority at start of day.
v2: Rename existing uses of _trylock rather than adding a _noretry
version. Also improve the comment a bit.
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
John Harrison (2):
drm/i915/guc: Properly initialise kernel contexts
drm/i915/guc: Don't deadlock busyness stats vs reset
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
.../gpu/drm/i915/gt/uc/intel_guc_submission.c | 7 ++++++-
4 files changed, 24 insertions(+), 4 deletions(-)
--
2.37.3
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
@ 2022-11-02 19:21 ` John.C.Harrison
-1 siblings, 0 replies; 19+ messages in thread
From: John.C.Harrison @ 2022-11-02 19:21 UTC (permalink / raw)
To: Intel-GFX; +Cc: John Harrison, DRI-Devel
From: John Harrison <John.C.Harrison@Intel.com>
If a context has already been registered prior to first submission
then context init code was not being called. The noticeable effect of
that was the scheduling priority was left at zero (meaning super high
priority) instead of being set to normal. This would occur with
kernel contexts at start of day as they are manually pinned up front
rather than on first submission. So add a call to initialise those
when they are pinned.
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
---
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index 4ccb29f9ac55c..941613be3b9dd 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -4111,6 +4111,9 @@ static inline void guc_kernel_context_pin(struct intel_guc *guc,
if (context_guc_id_invalid(ce))
pin_guc_id(guc, ce);
+ if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
+ guc_context_init(ce);
+
try_context_registration(ce, true);
}
--
2.37.3
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Intel-gfx] [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
@ 2022-11-02 19:21 ` John.C.Harrison
0 siblings, 0 replies; 19+ messages in thread
From: John.C.Harrison @ 2022-11-02 19:21 UTC (permalink / raw)
To: Intel-GFX; +Cc: DRI-Devel
From: John Harrison <John.C.Harrison@Intel.com>
If a context has already been registered prior to first submission
then context init code was not being called. The noticeable effect of
that was the scheduling priority was left at zero (meaning super high
priority) instead of being set to normal. This would occur with
kernel contexts at start of day as they are manually pinned up front
rather than on first submission. So add a call to initialise those
when they are pinned.
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
---
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index 4ccb29f9ac55c..941613be3b9dd 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -4111,6 +4111,9 @@ static inline void guc_kernel_context_pin(struct intel_guc *guc,
if (context_guc_id_invalid(ce))
pin_guc_id(guc, ce);
+ if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
+ guc_context_init(ce);
+
try_context_registration(ce, true);
}
--
2.37.3
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
@ 2022-11-02 19:21 ` John.C.Harrison
-1 siblings, 0 replies; 19+ messages in thread
From: John.C.Harrison @ 2022-11-02 19:21 UTC (permalink / raw)
To: Intel-GFX; +Cc: John Harrison, DRI-Devel
From: John Harrison <John.C.Harrison@Intel.com>
The engine busyness stats has a worker function to do things like
64bit extend the 32bit hardware counters. The GuC's reset prepare
function flushes out this worker function to ensure no corruption
happens during the reset. Unforunately, the worker function has an
infinite wait for active resets to finish before doing its work. Thus
a deadlock would occur if the worker function had actually started
just as the reset starts.
The function being used to lock the reset-in-progress mutex is called
intel_gt_reset_trylock(). However, as noted it does not follow
standard 'trylock' conventions and exit if already locked. So rename
the current _trylock function to intel_gt_reset_lock_interruptible(),
which is the behaviour it actually provides. In addition, add a new
implementation of _trylock and call that from the busyness stats
worker instead.
v2: Rename existing trylock to interruptible rather than trying to
preserve the existing (confusing) naming scheme (review comments from
Tvrtko).
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
.../gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +++-
4 files changed, 21 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
index e63329bc80659..c29efdef8313a 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
@@ -330,7 +330,7 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
if (ret)
goto err_rpm;
- ret = intel_gt_reset_trylock(ggtt->vm.gt, &srcu);
+ ret = intel_gt_reset_lock_interruptible(ggtt->vm.gt, &srcu);
if (ret)
goto err_pages;
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
index 3159df6cdd492..24736ebee17c2 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.c
+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
@@ -1407,15 +1407,19 @@ void intel_gt_handle_error(struct intel_gt *gt,
intel_runtime_pm_put(gt->uncore->rpm, wakeref);
}
-int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
+static int _intel_gt_reset_lock(struct intel_gt *gt, int *srcu, bool retry)
{
might_lock(>->reset.backoff_srcu);
- might_sleep();
+ if (retry)
+ might_sleep();
rcu_read_lock();
while (test_bit(I915_RESET_BACKOFF, >->reset.flags)) {
rcu_read_unlock();
+ if (!retry)
+ return -EBUSY;
+
if (wait_event_interruptible(gt->reset.queue,
!test_bit(I915_RESET_BACKOFF,
>->reset.flags)))
@@ -1429,6 +1433,16 @@ int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
return 0;
}
+int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
+{
+ return _intel_gt_reset_lock(gt, srcu, false);
+}
+
+int intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu)
+{
+ return _intel_gt_reset_lock(gt, srcu, true);
+}
+
void intel_gt_reset_unlock(struct intel_gt *gt, int tag)
__releases(>->reset.backoff_srcu)
{
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.h b/drivers/gpu/drm/i915/gt/intel_reset.h
index adc734e673870..25c975b6e8fc0 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.h
+++ b/drivers/gpu/drm/i915/gt/intel_reset.h
@@ -39,6 +39,7 @@ int __intel_engine_reset_bh(struct intel_engine_cs *engine,
void __i915_request_reset(struct i915_request *rq, bool guilty);
int __must_check intel_gt_reset_trylock(struct intel_gt *gt, int *srcu);
+int __must_check intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu);
void intel_gt_reset_unlock(struct intel_gt *gt, int tag);
void intel_gt_set_wedged(struct intel_gt *gt);
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index 941613be3b9dd..92e514061d20b 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -1401,7 +1401,9 @@ static void guc_timestamp_ping(struct work_struct *wrk)
/*
* Synchronize with gt reset to make sure the worker does not
- * corrupt the engine/guc stats.
+ * corrupt the engine/guc stats. NB: can't actually block waiting
+ * for a reset to complete as the reset requires flushing out
+ * this worker thread if started. So waiting would deadlock.
*/
ret = intel_gt_reset_trylock(gt, &srcu);
if (ret)
--
2.37.3
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Intel-gfx] [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset
@ 2022-11-02 19:21 ` John.C.Harrison
0 siblings, 0 replies; 19+ messages in thread
From: John.C.Harrison @ 2022-11-02 19:21 UTC (permalink / raw)
To: Intel-GFX; +Cc: DRI-Devel
From: John Harrison <John.C.Harrison@Intel.com>
The engine busyness stats has a worker function to do things like
64bit extend the 32bit hardware counters. The GuC's reset prepare
function flushes out this worker function to ensure no corruption
happens during the reset. Unforunately, the worker function has an
infinite wait for active resets to finish before doing its work. Thus
a deadlock would occur if the worker function had actually started
just as the reset starts.
The function being used to lock the reset-in-progress mutex is called
intel_gt_reset_trylock(). However, as noted it does not follow
standard 'trylock' conventions and exit if already locked. So rename
the current _trylock function to intel_gt_reset_lock_interruptible(),
which is the behaviour it actually provides. In addition, add a new
implementation of _trylock and call that from the busyness stats
worker instead.
v2: Rename existing trylock to interruptible rather than trying to
preserve the existing (confusing) naming scheme (review comments from
Tvrtko).
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
.../gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +++-
4 files changed, 21 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
index e63329bc80659..c29efdef8313a 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
@@ -330,7 +330,7 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
if (ret)
goto err_rpm;
- ret = intel_gt_reset_trylock(ggtt->vm.gt, &srcu);
+ ret = intel_gt_reset_lock_interruptible(ggtt->vm.gt, &srcu);
if (ret)
goto err_pages;
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
index 3159df6cdd492..24736ebee17c2 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.c
+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
@@ -1407,15 +1407,19 @@ void intel_gt_handle_error(struct intel_gt *gt,
intel_runtime_pm_put(gt->uncore->rpm, wakeref);
}
-int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
+static int _intel_gt_reset_lock(struct intel_gt *gt, int *srcu, bool retry)
{
might_lock(>->reset.backoff_srcu);
- might_sleep();
+ if (retry)
+ might_sleep();
rcu_read_lock();
while (test_bit(I915_RESET_BACKOFF, >->reset.flags)) {
rcu_read_unlock();
+ if (!retry)
+ return -EBUSY;
+
if (wait_event_interruptible(gt->reset.queue,
!test_bit(I915_RESET_BACKOFF,
>->reset.flags)))
@@ -1429,6 +1433,16 @@ int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
return 0;
}
+int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
+{
+ return _intel_gt_reset_lock(gt, srcu, false);
+}
+
+int intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu)
+{
+ return _intel_gt_reset_lock(gt, srcu, true);
+}
+
void intel_gt_reset_unlock(struct intel_gt *gt, int tag)
__releases(>->reset.backoff_srcu)
{
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.h b/drivers/gpu/drm/i915/gt/intel_reset.h
index adc734e673870..25c975b6e8fc0 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.h
+++ b/drivers/gpu/drm/i915/gt/intel_reset.h
@@ -39,6 +39,7 @@ int __intel_engine_reset_bh(struct intel_engine_cs *engine,
void __i915_request_reset(struct i915_request *rq, bool guilty);
int __must_check intel_gt_reset_trylock(struct intel_gt *gt, int *srcu);
+int __must_check intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu);
void intel_gt_reset_unlock(struct intel_gt *gt, int tag);
void intel_gt_set_wedged(struct intel_gt *gt);
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index 941613be3b9dd..92e514061d20b 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -1401,7 +1401,9 @@ static void guc_timestamp_ping(struct work_struct *wrk)
/*
* Synchronize with gt reset to make sure the worker does not
- * corrupt the engine/guc stats.
+ * corrupt the engine/guc stats. NB: can't actually block waiting
+ * for a reset to complete as the reset requires flushing out
+ * this worker thread if started. So waiting would deadlock.
*/
ret = intel_gt_reset_trylock(gt, &srcu);
if (ret)
--
2.37.3
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for Fix for two GuC issues (rev2)
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
` (2 preceding siblings ...)
(?)
@ 2022-11-02 20:56 ` Patchwork
-1 siblings, 0 replies; 19+ messages in thread
From: Patchwork @ 2022-11-02 20:56 UTC (permalink / raw)
To: john.c.harrison; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 4427 bytes --]
== Series Details ==
Series: Fix for two GuC issues (rev2)
URL : https://patchwork.freedesktop.org/series/110269/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12332 -> Patchwork_110269v2
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/index.html
Participating hosts (40 -> 28)
------------------------------
Missing (12): fi-adl-ddr5 bat-dg2-8 bat-adlm-1 bat-dg2-9 bat-adlp-6 bat-adlp-4 bat-adln-1 bat-rplp-1 bat-rpls-1 bat-rpls-2 bat-dg2-11 bat-jsl-1
Known issues
------------
Here are the changes found in Patchwork_110269v2 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s3@smem:
- fi-bdw-5557u: [PASS][1] -> [INCOMPLETE][2] ([i915#146])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html
* igt@gem_lmem_swapping@random-engines:
- fi-icl-u2: NOTRUN -> [SKIP][3] ([i915#4613]) +3 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-icl-u2/igt@gem_lmem_swapping@random-engines.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-hsw-g3258: NOTRUN -> [SKIP][4] ([fdo#109271] / [fdo#111827])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-hsw-g3258/igt@kms_chamelium@common-hpd-after-suspend.html
- fi-icl-u2: NOTRUN -> [SKIP][5] ([fdo#111827])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@prime_vgem@basic-userptr:
- fi-icl-u2: NOTRUN -> [SKIP][6] ([fdo#109295] / [i915#3301])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-icl-u2/igt@prime_vgem@basic-userptr.html
#### Possible fixes ####
* igt@i915_pm_rpm@basic-rte:
- fi-icl-u2: [DMESG-WARN][7] ([i915#4890]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/fi-icl-u2/igt@i915_pm_rpm@basic-rte.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-icl-u2/igt@i915_pm_rpm@basic-rte.html
* igt@i915_selftest@live@gt_heartbeat:
- fi-apl-guc: [DMESG-FAIL][9] ([i915#5334]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/fi-apl-guc/igt@i915_selftest@live@gt_heartbeat.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-apl-guc/igt@i915_selftest@live@gt_heartbeat.html
* igt@i915_selftest@live@hangcheck:
- fi-hsw-g3258: [INCOMPLETE][11] ([i915#3303] / [i915#4785]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146
[i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
[i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
[i915#4890]: https://gitlab.freedesktop.org/drm/intel/issues/4890
[i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
Build changes
-------------
* Linux: CI_DRM_12332 -> Patchwork_110269v2
CI-20190529: 20190529
CI_DRM_12332: 601b2ef606e4b83d5518aa6a5011bb2b1c5954d9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7038: 5389b3f3b9b75df6bd8506e4aa3da357fd0c0ab1 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_110269v2: 601b2ef606e4b83d5518aa6a5011bb2b1c5954d9 @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
1b6ad032eb2d drm/i915/guc: Don't deadlock busyness stats vs reset
b0ccd4733351 drm/i915/guc: Properly initialise kernel contexts
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/index.html
[-- Attachment #2: Type: text/html, Size: 5286 bytes --]
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for Fix for two GuC issues (rev2)
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
` (3 preceding siblings ...)
(?)
@ 2022-11-03 4:45 ` Patchwork
2022-11-04 19:40 ` John Harrison
-1 siblings, 1 reply; 19+ messages in thread
From: Patchwork @ 2022-11-03 4:45 UTC (permalink / raw)
To: john.c.harrison; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 32296 bytes --]
== Series Details ==
Series: Fix for two GuC issues (rev2)
URL : https://patchwork.freedesktop.org/series/110269/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12332_full -> Patchwork_110269v2_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_110269v2_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_110269v2_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Participating hosts (9 -> 9)
------------------------------
No changes in participating hosts
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_110269v2_full:
### CI changes ###
#### Possible regressions ####
* boot:
- shard-iclb: ([PASS][1], [PASS][2], [PASS][3], [PASS][4], [PASS][5], [PASS][6], [PASS][7], [PASS][8], [PASS][9], [PASS][10], [PASS][11], [PASS][12], [PASS][13], [PASS][14], [PASS][15], [PASS][16], [PASS][17], [PASS][18], [PASS][19], [PASS][20], [PASS][21], [PASS][22], [PASS][23], [PASS][24], [PASS][25]) -> ([PASS][26], [PASS][27], [PASS][28], [PASS][29], [PASS][30], [PASS][31], [PASS][32], [PASS][33], [PASS][34], [PASS][35], [PASS][36], [PASS][37], [PASS][38], [PASS][39], [PASS][40], [PASS][41], [PASS][42], [PASS][43], [PASS][44], [PASS][45], [PASS][46], [FAIL][47], [PASS][48], [PASS][49], [PASS][50])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb8/boot.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb8/boot.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb8/boot.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb7/boot.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb7/boot.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb7/boot.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/boot.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/boot.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/boot.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/boot.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/boot.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/boot.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/boot.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/boot.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/boot.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb5/boot.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb5/boot.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb5/boot.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/boot.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/boot.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/boot.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html
### IGT changes ###
#### Possible regressions ####
* igt@i915_suspend@fence-restore-untiled:
- shard-skl: [PASS][51] -> [INCOMPLETE][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl10/igt@i915_suspend@fence-restore-untiled.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl9/igt@i915_suspend@fence-restore-untiled.html
* igt@kms_cursor_legacy@cursor-vs-flip@atomic:
- shard-skl: NOTRUN -> [INCOMPLETE][53]
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_cursor_legacy@cursor-vs-flip@atomic.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-wc:
- shard-iclb: [PASS][54] -> [FAIL][55]
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-wc.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-wc.html
Known issues
------------
Here are the changes found in Patchwork_110269v2_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@feature_discovery@psr2:
- shard-iclb: [PASS][56] -> [SKIP][57] ([i915#658])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@feature_discovery@psr2.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@feature_discovery@psr2.html
* igt@gem_ctx_isolation@preservation-s3@vecs0:
- shard-skl: [PASS][58] -> [INCOMPLETE][59] ([i915#4793])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl6/igt@gem_ctx_isolation@preservation-s3@vecs0.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl9/igt@gem_ctx_isolation@preservation-s3@vecs0.html
* igt@gem_exec_balancer@parallel-keep-in-fence:
- shard-iclb: [PASS][60] -> [SKIP][61] ([i915#4525]) +2 similar issues
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@gem_exec_balancer@parallel-keep-in-fence.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@gem_exec_balancer@parallel-keep-in-fence.html
* igt@gem_exec_fair@basic-deadline:
- shard-glk: [PASS][62] -> [FAIL][63] ([i915#2846])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk8/igt@gem_exec_fair@basic-deadline.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk6/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-pace@vcs1:
- shard-iclb: NOTRUN -> [FAIL][64] ([i915#2842])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@gem_exec_fair@basic-pace@vcs1.html
* igt@gem_exec_suspend@basic-s3@smem:
- shard-apl: [PASS][65] -> [DMESG-WARN][66] ([i915#180]) +3 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl1/igt@gem_exec_suspend@basic-s3@smem.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl3/igt@gem_exec_suspend@basic-s3@smem.html
* igt@gem_lmem_swapping@basic:
- shard-skl: NOTRUN -> [SKIP][67] ([fdo#109271] / [i915#4613]) +2 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@heavy-verify-multi:
- shard-glk: NOTRUN -> [SKIP][68] ([fdo#109271] / [i915#4613])
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk3/igt@gem_lmem_swapping@heavy-verify-multi.html
* igt@gem_lmem_swapping@massive:
- shard-apl: NOTRUN -> [SKIP][69] ([fdo#109271] / [i915#4613]) +1 similar issue
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl8/igt@gem_lmem_swapping@massive.html
* igt@gem_pread@exhaustion:
- shard-apl: NOTRUN -> [INCOMPLETE][70] ([i915#7248])
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl7/igt@gem_pread@exhaustion.html
* igt@gem_tiled_wb:
- shard-skl: NOTRUN -> [TIMEOUT][71] ([i915#6990])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@gem_tiled_wb.html
* igt@i915_selftest@live@gt_heartbeat:
- shard-skl: [PASS][72] -> [DMESG-FAIL][73] ([i915#5334])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl7/igt@i915_selftest@live@gt_heartbeat.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@i915_selftest@live@gt_heartbeat.html
* igt@i915_selftest@live@hangcheck:
- shard-tglb: [PASS][74] -> [DMESG-WARN][75] ([i915#5591])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-tglb6/igt@i915_selftest@live@hangcheck.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-tglb2/igt@i915_selftest@live@hangcheck.html
* igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc:
- shard-apl: NOTRUN -> [SKIP][76] ([fdo#109271] / [i915#3886]) +3 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl8/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_chamelium@hdmi-edid-change-during-suspend:
- shard-apl: NOTRUN -> [SKIP][77] ([fdo#109271] / [fdo#111827]) +1 similar issue
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl8/igt@kms_chamelium@hdmi-edid-change-during-suspend.html
* igt@kms_color_chamelium@gamma:
- shard-skl: NOTRUN -> [SKIP][78] ([fdo#109271] / [fdo#111827])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_color_chamelium@gamma.html
* igt@kms_cursor_legacy@cursor-vs-flip@varying-size:
- shard-iclb: [PASS][79] -> [FAIL][80] ([i915#5072])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/igt@kms_cursor_legacy@cursor-vs-flip@varying-size.html
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/igt@kms_cursor_legacy@cursor-vs-flip@varying-size.html
* igt@kms_flip@flip-vs-expired-vblank@a-hdmi-a1:
- shard-glk: [PASS][81] -> [FAIL][82] ([i915#79])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk5/igt@kms_flip@flip-vs-expired-vblank@a-hdmi-a1.html
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk1/igt@kms_flip@flip-vs-expired-vblank@a-hdmi-a1.html
* igt@kms_flip@flip-vs-suspend@b-dp1:
- shard-apl: NOTRUN -> [DMESG-WARN][83] ([i915#180]) +1 similar issue
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@kms_flip@flip-vs-suspend@b-dp1.html
* igt@kms_flip@plain-flip-ts-check@c-edp1:
- shard-skl: [PASS][84] -> [FAIL][85] ([i915#2122]) +2 similar issues
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl6/igt@kms_flip@plain-flip-ts-check@c-edp1.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_flip@plain-flip-ts-check@c-edp1.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode:
- shard-iclb: NOTRUN -> [SKIP][86] ([i915#2587] / [i915#2672])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-upscaling@pipe-a-default-mode:
- shard-iclb: NOTRUN -> [SKIP][87] ([i915#2672]) +3 similar issues
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-upscaling@pipe-a-default-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilercccs-downscaling@pipe-a-default-mode:
- shard-iclb: NOTRUN -> [SKIP][88] ([i915#2672] / [i915#3555])
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilercccs-downscaling@pipe-a-default-mode.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt:
- shard-apl: NOTRUN -> [SKIP][89] ([fdo#109271]) +35 similar issues
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite:
- shard-iclb: [PASS][90] -> [FAIL][91] ([i915#2546])
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite.html
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite.html
* igt@kms_frontbuffer_tracking@fbcpsr-suspend:
- shard-skl: NOTRUN -> [SKIP][92] ([fdo#109271]) +59 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move:
- shard-skl: [PASS][93] -> [DMESG-WARN][94] ([i915#1982])
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl10/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move.html
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-render:
- shard-glk: NOTRUN -> [SKIP][95] ([fdo#109271]) +4 similar issues
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk3/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-render.html
* igt@kms_psr2_sf@cursor-plane-move-continuous-sf:
- shard-skl: NOTRUN -> [SKIP][96] ([fdo#109271] / [i915#658]) +1 similar issue
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_psr2_sf@cursor-plane-move-continuous-sf.html
* igt@kms_psr@psr2_cursor_blt:
- shard-iclb: [PASS][97] -> [SKIP][98] ([fdo#109441]) +2 similar issues
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_psr@psr2_cursor_blt.html
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_psr@psr2_cursor_blt.html
* igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
- shard-tglb: [PASS][99] -> [SKIP][100] ([i915#5519])
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-tglb3/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-tglb1/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
* igt@kms_writeback@writeback-invalid-parameters:
- shard-skl: NOTRUN -> [SKIP][101] ([fdo#109271] / [i915#2437])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_writeback@writeback-invalid-parameters.html
* igt@sysfs_clients@busy:
- shard-glk: NOTRUN -> [SKIP][102] ([fdo#109271] / [i915#2994])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk3/igt@sysfs_clients@busy.html
* igt@sysfs_clients@split-10:
- shard-skl: NOTRUN -> [SKIP][103] ([fdo#109271] / [i915#2994])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl9/igt@sysfs_clients@split-10.html
#### Possible fixes ####
* igt@gem_exec_balancer@parallel-contexts:
- shard-iclb: [SKIP][104] ([i915#4525]) -> [PASS][105]
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/igt@gem_exec_balancer@parallel-contexts.html
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@gem_exec_balancer@parallel-contexts.html
* igt@gem_exec_fair@basic-flow@rcs0:
- shard-tglb: [FAIL][106] ([i915#2842]) -> [PASS][107]
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-tglb2/igt@gem_exec_fair@basic-flow@rcs0.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-tglb5/igt@gem_exec_fair@basic-flow@rcs0.html
* igt@gem_mmap_offset@bad-object:
- shard-skl: [DMESG-WARN][108] ([i915#1982]) -> [PASS][109] +3 similar issues
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl4/igt@gem_mmap_offset@bad-object.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl7/igt@gem_mmap_offset@bad-object.html
* igt@i915_pm_rc6_residency@rc6-idle@vcs0:
- shard-skl: [WARN][110] ([i915#1804]) -> [PASS][111]
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl7/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
* igt@i915_pm_rpm@basic-pci-d3-state:
- shard-iclb: [FAIL][112] -> [PASS][113]
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/igt@i915_pm_rpm@basic-pci-d3-state.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@i915_pm_rpm@basic-pci-d3-state.html
* igt@kms_cursor_legacy@cursor-vs-flip@toggle:
- shard-skl: [INCOMPLETE][114] -> [PASS][115]
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl6/igt@kms_cursor_legacy@cursor-vs-flip@toggle.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_cursor_legacy@cursor-vs-flip@toggle.html
* igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size:
- shard-glk: [FAIL][116] ([i915#2346]) -> [PASS][117] +1 similar issue
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk8/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk6/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
* igt@kms_flip@flip-vs-absolute-wf_vblank@a-edp1:
- shard-skl: [FAIL][118] ([i915#2122]) -> [PASS][119] +1 similar issue
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl5/igt@kms_flip@flip-vs-absolute-wf_vblank@a-edp1.html
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl7/igt@kms_flip@flip-vs-absolute-wf_vblank@a-edp1.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
- shard-skl: [FAIL][120] ([i915#79]) -> [PASS][121] +2 similar issues
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl3/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
* igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode:
- shard-iclb: [SKIP][122] ([i915#3555]) -> [PASS][123] +4 similar issues
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode.html
* igt@kms_plane_lowres@tiling-y@pipe-a-hdmi-a-1:
- shard-glk: [DMESG-FAIL][124] ([i915#118]) -> [PASS][125]
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk9/igt@kms_plane_lowres@tiling-y@pipe-a-hdmi-a-1.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk8/igt@kms_plane_lowres@tiling-y@pipe-a-hdmi-a-1.html
* igt@kms_plane_lowres@tiling-y@pipe-c-hdmi-a-1:
- shard-glk: [FAIL][126] ([i915#7307]) -> [PASS][127] +1 similar issue
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk9/igt@kms_plane_lowres@tiling-y@pipe-c-hdmi-a-1.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk8/igt@kms_plane_lowres@tiling-y@pipe-c-hdmi-a-1.html
* igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1:
- shard-iclb: [SKIP][128] ([i915#5176]) -> [PASS][129] +2 similar issues
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1.html
* igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1:
- shard-iclb: [SKIP][130] ([i915#5235]) -> [PASS][131] +2 similar issues
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html
* igt@kms_psr@psr2_sprite_blt:
- shard-iclb: [SKIP][132] ([fdo#109441]) -> [PASS][133] +2 similar issues
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/igt@kms_psr@psr2_sprite_blt.html
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html
* igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
- shard-iclb: [SKIP][134] ([i915#5519]) -> [PASS][135]
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
* igt@kms_vblank@pipe-b-ts-continuation-suspend:
- shard-apl: [DMESG-WARN][136] ([i915#180]) -> [PASS][137] +3 similar issues
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl2/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl7/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
* igt@perf@polling:
- shard-skl: [FAIL][138] ([i915#1542]) -> [PASS][139]
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl9/igt@perf@polling.html
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@perf@polling.html
* igt@prime_mmap_coherency@ioctl-errors:
- shard-skl: [INCOMPLETE][140] ([i915#2295]) -> [PASS][141]
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl3/igt@prime_mmap_coherency@ioctl-errors.html
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@prime_mmap_coherency@ioctl-errors.html
* igt@syncobj_wait@wait-all-for-submit-snapshot:
- shard-skl: [FAIL][142] ([i915#7109]) -> [PASS][143]
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl10/igt@syncobj_wait@wait-all-for-submit-snapshot.html
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@syncobj_wait@wait-all-for-submit-snapshot.html
#### Warnings ####
* igt@gem_pwrite@basic-exhaustion:
- shard-glk: [WARN][144] ([i915#2658]) -> [INCOMPLETE][145] ([i915#7248])
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk5/igt@gem_pwrite@basic-exhaustion.html
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk1/igt@gem_pwrite@basic-exhaustion.html
- shard-skl: [TIMEOUT][146] ([i915#7248]) -> [INCOMPLETE][147] ([i915#7248])
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl3/igt@gem_pwrite@basic-exhaustion.html
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@gem_pwrite@basic-exhaustion.html
* igt@i915_pm_dc@dc3co-vpb-simulation:
- shard-iclb: [SKIP][148] ([i915#588]) -> [SKIP][149] ([i915#658])
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@i915_pm_dc@dc3co-vpb-simulation.html
* igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf:
- shard-iclb: [SKIP][150] ([i915#2920]) -> [SKIP][151] ([i915#658])
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf.html
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf.html
* igt@kms_psr2_sf@cursor-plane-update-sf:
- shard-iclb: [SKIP][152] ([fdo#111068] / [i915#658]) -> [SKIP][153] ([i915#2920])
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/igt@kms_psr2_sf@cursor-plane-update-sf.html
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_psr2_sf@cursor-plane-update-sf.html
* igt@runner@aborted:
- shard-apl: ([FAIL][154], [FAIL][155], [FAIL][156], [FAIL][157], [FAIL][158], [FAIL][159]) ([fdo#109271] / [i915#180] / [i915#3002] / [i915#4312]) -> ([FAIL][160], [FAIL][161], [FAIL][162], [FAIL][163], [FAIL][164]) ([i915#180] / [i915#3002] / [i915#4312])
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl2/igt@runner@aborted.html
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl3/igt@runner@aborted.html
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl3/igt@runner@aborted.html
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl8/igt@runner@aborted.html
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl1/igt@runner@aborted.html
[159]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl6/igt@runner@aborted.html
[160]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl3/igt@runner@aborted.html
[161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl3/igt@runner@aborted.html
[162]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@runner@aborted.html
[163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@runner@aborted.html
[164]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@runner@aborted.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
[i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2295]: https://gitlab.freedesktop.org/drm/intel/issues/2295
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
[i915#2546]: https://gitlab.freedesktop.org/drm/intel/issues/2546
[i915#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587
[i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846
[i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920
[i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
[i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4793]: https://gitlab.freedesktop.org/drm/intel/issues/4793
[i915#5072]: https://gitlab.freedesktop.org/drm/intel/issues/5072
[i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
[i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
[i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
[i915#5519]: https://gitlab.freedesktop.org/drm/intel/issues/5519
[i915#5591]: https://gitlab.freedesktop.org/drm/intel/issues/5591
[i915#588]: https://gitlab.freedesktop.org/drm/intel/issues/588
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#6990]: https://gitlab.freedesktop.org/drm/intel/issues/6990
[i915#7109]: https://gitlab.freedesktop.org/drm/intel/issues/7109
[i915#7248]: https://gitlab.freedesktop.org/drm/intel/issues/7248
[i915#7307]: https://gitlab.freedesktop.org/drm/intel/issues/7307
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
Build changes
-------------
* Linux: CI_DRM_12332 -> Patchwork_110269v2
CI-20190529: 20190529
CI_DRM_12332: 601b2ef606e4b83d5518aa6a5011bb2b1c5954d9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7038: 5389b3f3b9b75df6bd8506e4aa3da357fd0c0ab1 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_110269v2: 601b2ef606e4b83d5518aa6a5011bb2b1c5954d9 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/index.html
[-- Attachment #2: Type: text/html, Size: 37492 bytes --]
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
(?)
@ 2022-11-03 11:31 ` Tvrtko Ursulin
2022-11-03 18:45 ` John Harrison
-1 siblings, 1 reply; 19+ messages in thread
From: Tvrtko Ursulin @ 2022-11-03 11:31 UTC (permalink / raw)
To: John.C.Harrison, Intel-GFX; +Cc: DRI-Devel
On 02/11/2022 19:21, John.C.Harrison@Intel.com wrote:
> From: John Harrison <John.C.Harrison@Intel.com>
>
> The engine busyness stats has a worker function to do things like
> 64bit extend the 32bit hardware counters. The GuC's reset prepare
> function flushes out this worker function to ensure no corruption
> happens during the reset. Unforunately, the worker function has an
> infinite wait for active resets to finish before doing its work. Thus
> a deadlock would occur if the worker function had actually started
> just as the reset starts.
>
> The function being used to lock the reset-in-progress mutex is called
> intel_gt_reset_trylock(). However, as noted it does not follow
> standard 'trylock' conventions and exit if already locked. So rename
> the current _trylock function to intel_gt_reset_lock_interruptible(),
> which is the behaviour it actually provides. In addition, add a new
> implementation of _trylock and call that from the busyness stats
> worker instead.
>
> v2: Rename existing trylock to interruptible rather than trying to
> preserve the existing (confusing) naming scheme (review comments from
> Tvrtko).
>
> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> ---
> drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
> drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
> drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
> .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +++-
> 4 files changed, 21 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
> index e63329bc80659..c29efdef8313a 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
> @@ -330,7 +330,7 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
> if (ret)
> goto err_rpm;
>
> - ret = intel_gt_reset_trylock(ggtt->vm.gt, &srcu);
> + ret = intel_gt_reset_lock_interruptible(ggtt->vm.gt, &srcu);
> if (ret)
> goto err_pages;
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
> index 3159df6cdd492..24736ebee17c2 100644
> --- a/drivers/gpu/drm/i915/gt/intel_reset.c
> +++ b/drivers/gpu/drm/i915/gt/intel_reset.c
> @@ -1407,15 +1407,19 @@ void intel_gt_handle_error(struct intel_gt *gt,
> intel_runtime_pm_put(gt->uncore->rpm, wakeref);
> }
>
> -int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
> +static int _intel_gt_reset_lock(struct intel_gt *gt, int *srcu, bool retry)
> {
> might_lock(>->reset.backoff_srcu);
> - might_sleep();
> + if (retry)
> + might_sleep();
>
> rcu_read_lock();
> while (test_bit(I915_RESET_BACKOFF, >->reset.flags)) {
> rcu_read_unlock();
>
> + if (!retry)
> + return -EBUSY;
> +
> if (wait_event_interruptible(gt->reset.queue,
> !test_bit(I915_RESET_BACKOFF,
> >->reset.flags)))
> @@ -1429,6 +1433,16 @@ int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
> return 0;
> }
>
> +int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
> +{
> + return _intel_gt_reset_lock(gt, srcu, false);
> +}
> +
> +int intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu)
> +{
> + return _intel_gt_reset_lock(gt, srcu, true);
> +}
> +
> void intel_gt_reset_unlock(struct intel_gt *gt, int tag)
> __releases(>->reset.backoff_srcu)
> {
> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.h b/drivers/gpu/drm/i915/gt/intel_reset.h
> index adc734e673870..25c975b6e8fc0 100644
> --- a/drivers/gpu/drm/i915/gt/intel_reset.h
> +++ b/drivers/gpu/drm/i915/gt/intel_reset.h
> @@ -39,6 +39,7 @@ int __intel_engine_reset_bh(struct intel_engine_cs *engine,
> void __i915_request_reset(struct i915_request *rq, bool guilty);
>
> int __must_check intel_gt_reset_trylock(struct intel_gt *gt, int *srcu);
> +int __must_check intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu);
> void intel_gt_reset_unlock(struct intel_gt *gt, int tag);
>
> void intel_gt_set_wedged(struct intel_gt *gt);
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> index 941613be3b9dd..92e514061d20b 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> @@ -1401,7 +1401,9 @@ static void guc_timestamp_ping(struct work_struct *wrk)
>
> /*
> * Synchronize with gt reset to make sure the worker does not
> - * corrupt the engine/guc stats.
> + * corrupt the engine/guc stats. NB: can't actually block waiting
> + * for a reset to complete as the reset requires flushing out
> + * this worker thread if started. So waiting would deadlock.
> */
> ret = intel_gt_reset_trylock(gt, &srcu);
> if (ret)
LGTM but I don't remember fully how ping worker and reset interact so
I'll let Umesh r-b. Like is it okay to skip the ping or we'd need to
re-schedule it ASAP due wrap issues? Maybe reset makes that pointless, I
don't remember.
Regards,
Tvrtko
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset
2022-11-03 11:31 ` Tvrtko Ursulin
@ 2022-11-03 18:45 ` John Harrison
2022-11-03 18:54 ` Umesh Nerlige Ramappa
0 siblings, 1 reply; 19+ messages in thread
From: John Harrison @ 2022-11-03 18:45 UTC (permalink / raw)
To: Tvrtko Ursulin, Intel-GFX, Umesh Nerlige Ramappa; +Cc: DRI-Devel
On 11/3/2022 04:31, Tvrtko Ursulin wrote:
> On 02/11/2022 19:21, John.C.Harrison@Intel.com wrote:
>> From: John Harrison <John.C.Harrison@Intel.com>
>>
>> The engine busyness stats has a worker function to do things like
>> 64bit extend the 32bit hardware counters. The GuC's reset prepare
>> function flushes out this worker function to ensure no corruption
>> happens during the reset. Unforunately, the worker function has an
>> infinite wait for active resets to finish before doing its work. Thus
>> a deadlock would occur if the worker function had actually started
>> just as the reset starts.
>>
>> The function being used to lock the reset-in-progress mutex is called
>> intel_gt_reset_trylock(). However, as noted it does not follow
>> standard 'trylock' conventions and exit if already locked. So rename
>> the current _trylock function to intel_gt_reset_lock_interruptible(),
>> which is the behaviour it actually provides. In addition, add a new
>> implementation of _trylock and call that from the busyness stats
>> worker instead.
>>
>> v2: Rename existing trylock to interruptible rather than trying to
>> preserve the existing (confusing) naming scheme (review comments from
>> Tvrtko).
>>
>> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
>> ---
>> drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
>> drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
>> drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
>> .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +++-
>> 4 files changed, 21 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>> b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>> index e63329bc80659..c29efdef8313a 100644
>> --- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>> +++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>> @@ -330,7 +330,7 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
>> if (ret)
>> goto err_rpm;
>> - ret = intel_gt_reset_trylock(ggtt->vm.gt, &srcu);
>> + ret = intel_gt_reset_lock_interruptible(ggtt->vm.gt, &srcu);
>> if (ret)
>> goto err_pages;
>> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c
>> b/drivers/gpu/drm/i915/gt/intel_reset.c
>> index 3159df6cdd492..24736ebee17c2 100644
>> --- a/drivers/gpu/drm/i915/gt/intel_reset.c
>> +++ b/drivers/gpu/drm/i915/gt/intel_reset.c
>> @@ -1407,15 +1407,19 @@ void intel_gt_handle_error(struct intel_gt *gt,
>> intel_runtime_pm_put(gt->uncore->rpm, wakeref);
>> }
>> -int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
>> +static int _intel_gt_reset_lock(struct intel_gt *gt, int *srcu, bool
>> retry)
>> {
>> might_lock(>->reset.backoff_srcu);
>> - might_sleep();
>> + if (retry)
>> + might_sleep();
>> rcu_read_lock();
>> while (test_bit(I915_RESET_BACKOFF, >->reset.flags)) {
>> rcu_read_unlock();
>> + if (!retry)
>> + return -EBUSY;
>> +
>> if (wait_event_interruptible(gt->reset.queue,
>> !test_bit(I915_RESET_BACKOFF,
>> >->reset.flags)))
>> @@ -1429,6 +1433,16 @@ int intel_gt_reset_trylock(struct intel_gt
>> *gt, int *srcu)
>> return 0;
>> }
>> +int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
>> +{
>> + return _intel_gt_reset_lock(gt, srcu, false);
>> +}
>> +
>> +int intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu)
>> +{
>> + return _intel_gt_reset_lock(gt, srcu, true);
>> +}
>> +
>> void intel_gt_reset_unlock(struct intel_gt *gt, int tag)
>> __releases(>->reset.backoff_srcu)
>> {
>> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.h
>> b/drivers/gpu/drm/i915/gt/intel_reset.h
>> index adc734e673870..25c975b6e8fc0 100644
>> --- a/drivers/gpu/drm/i915/gt/intel_reset.h
>> +++ b/drivers/gpu/drm/i915/gt/intel_reset.h
>> @@ -39,6 +39,7 @@ int __intel_engine_reset_bh(struct intel_engine_cs
>> *engine,
>> void __i915_request_reset(struct i915_request *rq, bool guilty);
>> int __must_check intel_gt_reset_trylock(struct intel_gt *gt, int
>> *srcu);
>> +int __must_check intel_gt_reset_lock_interruptible(struct intel_gt
>> *gt, int *srcu);
>> void intel_gt_reset_unlock(struct intel_gt *gt, int tag);
>> void intel_gt_set_wedged(struct intel_gt *gt);
>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> index 941613be3b9dd..92e514061d20b 100644
>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> @@ -1401,7 +1401,9 @@ static void guc_timestamp_ping(struct
>> work_struct *wrk)
>> /*
>> * Synchronize with gt reset to make sure the worker does not
>> - * corrupt the engine/guc stats.
>> + * corrupt the engine/guc stats. NB: can't actually block waiting
>> + * for a reset to complete as the reset requires flushing out
>> + * this worker thread if started. So waiting would deadlock.
>> */
>> ret = intel_gt_reset_trylock(gt, &srcu);
>> if (ret)
>
> LGTM but I don't remember fully how ping worker and reset interact so
> I'll let Umesh r-b. Like is it okay to skip the ping or we'd need to
> re-schedule it ASAP due wrap issues? Maybe reset makes that pointless,
> I don't remember.
The reset is cancelling the worker anyway. And it will then be
rescheduled once the reset is done. And the ping time is defined as
1/8th the wrap time (being approx 223 seconds on current platforms). So
as long as the reset doesn't take longer than about 200s, there is no
issue. And if the reset did take longer than that then we have bigger
issues than the busyness stats (which can't actually be counting anyway
because nothing is running if the GT is in reset) being slightly off.
John.
>
> Regards,
>
> Tvrtko
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset
2022-11-03 18:45 ` John Harrison
@ 2022-11-03 18:54 ` Umesh Nerlige Ramappa
0 siblings, 0 replies; 19+ messages in thread
From: Umesh Nerlige Ramappa @ 2022-11-03 18:54 UTC (permalink / raw)
To: John Harrison; +Cc: Tvrtko Ursulin, Intel-GFX, DRI-Devel
On Thu, Nov 03, 2022 at 11:45:57AM -0700, John Harrison wrote:
>On 11/3/2022 04:31, Tvrtko Ursulin wrote:
>>On 02/11/2022 19:21, John.C.Harrison@Intel.com wrote:
>>>From: John Harrison <John.C.Harrison@Intel.com>
>>>
>>>The engine busyness stats has a worker function to do things like
>>>64bit extend the 32bit hardware counters. The GuC's reset prepare
>>>function flushes out this worker function to ensure no corruption
>>>happens during the reset. Unforunately, the worker function has an
>>>infinite wait for active resets to finish before doing its work. Thus
>>>a deadlock would occur if the worker function had actually started
>>>just as the reset starts.
>>>
>>>The function being used to lock the reset-in-progress mutex is called
>>>intel_gt_reset_trylock(). However, as noted it does not follow
>>>standard 'trylock' conventions and exit if already locked. So rename
>>>the current _trylock function to intel_gt_reset_lock_interruptible(),
>>>which is the behaviour it actually provides. In addition, add a new
>>>implementation of _trylock and call that from the busyness stats
>>>worker instead.
>>>
>>>v2: Rename existing trylock to interruptible rather than trying to
>>>preserve the existing (confusing) naming scheme (review comments from
>>>Tvrtko).
>>>
>>>Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
>>>---
>>> drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
>>> drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
>>> drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
>>> .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +++-
>>> 4 files changed, 21 insertions(+), 4 deletions(-)
>>>
>>>diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>index e63329bc80659..c29efdef8313a 100644
>>>--- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>+++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>@@ -330,7 +330,7 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
>>> if (ret)
>>> goto err_rpm;
>>> - ret = intel_gt_reset_trylock(ggtt->vm.gt, &srcu);
>>>+ ret = intel_gt_reset_lock_interruptible(ggtt->vm.gt, &srcu);
>>> if (ret)
>>> goto err_pages;
>>> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c
>>>b/drivers/gpu/drm/i915/gt/intel_reset.c
>>>index 3159df6cdd492..24736ebee17c2 100644
>>>--- a/drivers/gpu/drm/i915/gt/intel_reset.c
>>>+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
>>>@@ -1407,15 +1407,19 @@ void intel_gt_handle_error(struct intel_gt *gt,
>>> intel_runtime_pm_put(gt->uncore->rpm, wakeref);
>>> }
>>> -int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
>>>+static int _intel_gt_reset_lock(struct intel_gt *gt, int *srcu,
>>>bool retry)
>>> {
>>> might_lock(>->reset.backoff_srcu);
>>>- might_sleep();
>>>+ if (retry)
>>>+ might_sleep();
>>> rcu_read_lock();
>>> while (test_bit(I915_RESET_BACKOFF, >->reset.flags)) {
>>> rcu_read_unlock();
>>> + if (!retry)
>>>+ return -EBUSY;
>>>+
>>> if (wait_event_interruptible(gt->reset.queue,
>>> !test_bit(I915_RESET_BACKOFF,
>>> >->reset.flags)))
>>>@@ -1429,6 +1433,16 @@ int intel_gt_reset_trylock(struct intel_gt
>>>*gt, int *srcu)
>>> return 0;
>>> }
>>> +int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
>>>+{
>>>+ return _intel_gt_reset_lock(gt, srcu, false);
>>>+}
>>>+
>>>+int intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu)
>>>+{
>>>+ return _intel_gt_reset_lock(gt, srcu, true);
>>>+}
>>>+
>>> void intel_gt_reset_unlock(struct intel_gt *gt, int tag)
>>> __releases(>->reset.backoff_srcu)
>>> {
>>>diff --git a/drivers/gpu/drm/i915/gt/intel_reset.h
>>>b/drivers/gpu/drm/i915/gt/intel_reset.h
>>>index adc734e673870..25c975b6e8fc0 100644
>>>--- a/drivers/gpu/drm/i915/gt/intel_reset.h
>>>+++ b/drivers/gpu/drm/i915/gt/intel_reset.h
>>>@@ -39,6 +39,7 @@ int __intel_engine_reset_bh(struct
>>>intel_engine_cs *engine,
>>> void __i915_request_reset(struct i915_request *rq, bool guilty);
>>> int __must_check intel_gt_reset_trylock(struct intel_gt *gt,
>>>int *srcu);
>>>+int __must_check intel_gt_reset_lock_interruptible(struct
>>>intel_gt *gt, int *srcu);
>>> void intel_gt_reset_unlock(struct intel_gt *gt, int tag);
>>> void intel_gt_set_wedged(struct intel_gt *gt);
>>>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>index 941613be3b9dd..92e514061d20b 100644
>>>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>@@ -1401,7 +1401,9 @@ static void guc_timestamp_ping(struct
>>>work_struct *wrk)
>>> /*
>>> * Synchronize with gt reset to make sure the worker does not
>>>- * corrupt the engine/guc stats.
>>>+ * corrupt the engine/guc stats. NB: can't actually block waiting
>>>+ * for a reset to complete as the reset requires flushing out
>>>+ * this worker thread if started. So waiting would deadlock.
>>> */
>>> ret = intel_gt_reset_trylock(gt, &srcu);
>>> if (ret)
>>
>>LGTM but I don't remember fully how ping worker and reset interact
>>so I'll let Umesh r-b. Like is it okay to skip the ping or we'd need
>>to re-schedule it ASAP due wrap issues? Maybe reset makes that
>>pointless, I don't remember.
>The reset is cancelling the worker anyway. And it will then be
>rescheduled once the reset is done. And the ping time is defined as
>1/8th the wrap time (being approx 223 seconds on current platforms).
>So as long as the reset doesn't take longer than about 200s, there is
>no issue. And if the reset did take longer than that then we have
>bigger issues than the busyness stats (which can't actually be
>counting anyway because nothing is running if the GT is in reset)
>being slightly off.
In addition to canceling the ping worker, __reset_guc_busyness_stats is
performing the same activities that the ping-worker would do if it were
to run, so we should be safe to skip the worker when a reset is in
progress, so lgtm,
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Thanks,
Umesh
>
>John.
>
>>
>>Regards,
>>
>>Tvrtko
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset
@ 2022-11-03 18:54 ` Umesh Nerlige Ramappa
0 siblings, 0 replies; 19+ messages in thread
From: Umesh Nerlige Ramappa @ 2022-11-03 18:54 UTC (permalink / raw)
To: John Harrison; +Cc: Intel-GFX, DRI-Devel
On Thu, Nov 03, 2022 at 11:45:57AM -0700, John Harrison wrote:
>On 11/3/2022 04:31, Tvrtko Ursulin wrote:
>>On 02/11/2022 19:21, John.C.Harrison@Intel.com wrote:
>>>From: John Harrison <John.C.Harrison@Intel.com>
>>>
>>>The engine busyness stats has a worker function to do things like
>>>64bit extend the 32bit hardware counters. The GuC's reset prepare
>>>function flushes out this worker function to ensure no corruption
>>>happens during the reset. Unforunately, the worker function has an
>>>infinite wait for active resets to finish before doing its work. Thus
>>>a deadlock would occur if the worker function had actually started
>>>just as the reset starts.
>>>
>>>The function being used to lock the reset-in-progress mutex is called
>>>intel_gt_reset_trylock(). However, as noted it does not follow
>>>standard 'trylock' conventions and exit if already locked. So rename
>>>the current _trylock function to intel_gt_reset_lock_interruptible(),
>>>which is the behaviour it actually provides. In addition, add a new
>>>implementation of _trylock and call that from the busyness stats
>>>worker instead.
>>>
>>>v2: Rename existing trylock to interruptible rather than trying to
>>>preserve the existing (confusing) naming scheme (review comments from
>>>Tvrtko).
>>>
>>>Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
>>>---
>>> drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
>>> drivers/gpu/drm/i915/gt/intel_reset.c | 18 ++++++++++++++++--
>>> drivers/gpu/drm/i915/gt/intel_reset.h | 1 +
>>> .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +++-
>>> 4 files changed, 21 insertions(+), 4 deletions(-)
>>>
>>>diff --git a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>index e63329bc80659..c29efdef8313a 100644
>>>--- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>+++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
>>>@@ -330,7 +330,7 @@ static vm_fault_t vm_fault_gtt(struct vm_fault *vmf)
>>> if (ret)
>>> goto err_rpm;
>>> - ret = intel_gt_reset_trylock(ggtt->vm.gt, &srcu);
>>>+ ret = intel_gt_reset_lock_interruptible(ggtt->vm.gt, &srcu);
>>> if (ret)
>>> goto err_pages;
>>> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c
>>>b/drivers/gpu/drm/i915/gt/intel_reset.c
>>>index 3159df6cdd492..24736ebee17c2 100644
>>>--- a/drivers/gpu/drm/i915/gt/intel_reset.c
>>>+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
>>>@@ -1407,15 +1407,19 @@ void intel_gt_handle_error(struct intel_gt *gt,
>>> intel_runtime_pm_put(gt->uncore->rpm, wakeref);
>>> }
>>> -int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
>>>+static int _intel_gt_reset_lock(struct intel_gt *gt, int *srcu,
>>>bool retry)
>>> {
>>> might_lock(>->reset.backoff_srcu);
>>>- might_sleep();
>>>+ if (retry)
>>>+ might_sleep();
>>> rcu_read_lock();
>>> while (test_bit(I915_RESET_BACKOFF, >->reset.flags)) {
>>> rcu_read_unlock();
>>> + if (!retry)
>>>+ return -EBUSY;
>>>+
>>> if (wait_event_interruptible(gt->reset.queue,
>>> !test_bit(I915_RESET_BACKOFF,
>>> >->reset.flags)))
>>>@@ -1429,6 +1433,16 @@ int intel_gt_reset_trylock(struct intel_gt
>>>*gt, int *srcu)
>>> return 0;
>>> }
>>> +int intel_gt_reset_trylock(struct intel_gt *gt, int *srcu)
>>>+{
>>>+ return _intel_gt_reset_lock(gt, srcu, false);
>>>+}
>>>+
>>>+int intel_gt_reset_lock_interruptible(struct intel_gt *gt, int *srcu)
>>>+{
>>>+ return _intel_gt_reset_lock(gt, srcu, true);
>>>+}
>>>+
>>> void intel_gt_reset_unlock(struct intel_gt *gt, int tag)
>>> __releases(>->reset.backoff_srcu)
>>> {
>>>diff --git a/drivers/gpu/drm/i915/gt/intel_reset.h
>>>b/drivers/gpu/drm/i915/gt/intel_reset.h
>>>index adc734e673870..25c975b6e8fc0 100644
>>>--- a/drivers/gpu/drm/i915/gt/intel_reset.h
>>>+++ b/drivers/gpu/drm/i915/gt/intel_reset.h
>>>@@ -39,6 +39,7 @@ int __intel_engine_reset_bh(struct
>>>intel_engine_cs *engine,
>>> void __i915_request_reset(struct i915_request *rq, bool guilty);
>>> int __must_check intel_gt_reset_trylock(struct intel_gt *gt,
>>>int *srcu);
>>>+int __must_check intel_gt_reset_lock_interruptible(struct
>>>intel_gt *gt, int *srcu);
>>> void intel_gt_reset_unlock(struct intel_gt *gt, int tag);
>>> void intel_gt_set_wedged(struct intel_gt *gt);
>>>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>index 941613be3b9dd..92e514061d20b 100644
>>>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>>@@ -1401,7 +1401,9 @@ static void guc_timestamp_ping(struct
>>>work_struct *wrk)
>>> /*
>>> * Synchronize with gt reset to make sure the worker does not
>>>- * corrupt the engine/guc stats.
>>>+ * corrupt the engine/guc stats. NB: can't actually block waiting
>>>+ * for a reset to complete as the reset requires flushing out
>>>+ * this worker thread if started. So waiting would deadlock.
>>> */
>>> ret = intel_gt_reset_trylock(gt, &srcu);
>>> if (ret)
>>
>>LGTM but I don't remember fully how ping worker and reset interact
>>so I'll let Umesh r-b. Like is it okay to skip the ping or we'd need
>>to re-schedule it ASAP due wrap issues? Maybe reset makes that
>>pointless, I don't remember.
>The reset is cancelling the worker anyway. And it will then be
>rescheduled once the reset is done. And the ping time is defined as
>1/8th the wrap time (being approx 223 seconds on current platforms).
>So as long as the reset doesn't take longer than about 200s, there is
>no issue. And if the reset did take longer than that then we have
>bigger issues than the busyness stats (which can't actually be
>counting anyway because nothing is running if the GT is in reset)
>being slightly off.
In addition to canceling the ping worker, __reset_guc_busyness_stats is
performing the same activities that the ping-worker would do if it were
to run, so we should be safe to skip the worker when a reset is in
progress, so lgtm,
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Thanks,
Umesh
>
>John.
>
>>
>>Regards,
>>
>>Tvrtko
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
@ 2022-11-04 18:53 ` Ceraolo Spurio, Daniele
-1 siblings, 0 replies; 19+ messages in thread
From: Ceraolo Spurio, Daniele @ 2022-11-04 18:53 UTC (permalink / raw)
To: John.C.Harrison, Intel-GFX; +Cc: DRI-Devel
On 11/2/2022 12:21 PM, John.C.Harrison@Intel.com wrote:
> From: John Harrison <John.C.Harrison@Intel.com>
>
> If a context has already been registered prior to first submission
> then context init code was not being called. The noticeable effect of
> that was the scheduling priority was left at zero (meaning super high
> priority) instead of being set to normal. This would occur with
> kernel contexts at start of day as they are manually pinned up front
> rather than on first submission. So add a call to initialise those
> when they are pinned.
Does this need a fixes tag? on one side, we were leaving the priority to
the wrong value, but on the other there were no actual consequences.
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Daniele
> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> ---
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> index 4ccb29f9ac55c..941613be3b9dd 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> @@ -4111,6 +4111,9 @@ static inline void guc_kernel_context_pin(struct intel_guc *guc,
> if (context_guc_id_invalid(ce))
> pin_guc_id(guc, ce);
>
> + if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
> + guc_context_init(ce);
> +
> try_context_registration(ce, true);
> }
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
@ 2022-11-04 18:53 ` Ceraolo Spurio, Daniele
0 siblings, 0 replies; 19+ messages in thread
From: Ceraolo Spurio, Daniele @ 2022-11-04 18:53 UTC (permalink / raw)
To: John.C.Harrison, Intel-GFX; +Cc: DRI-Devel
On 11/2/2022 12:21 PM, John.C.Harrison@Intel.com wrote:
> From: John Harrison <John.C.Harrison@Intel.com>
>
> If a context has already been registered prior to first submission
> then context init code was not being called. The noticeable effect of
> that was the scheduling priority was left at zero (meaning super high
> priority) instead of being set to normal. This would occur with
> kernel contexts at start of day as they are manually pinned up front
> rather than on first submission. So add a call to initialise those
> when they are pinned.
Does this need a fixes tag? on one side, we were leaving the priority to
the wrong value, but on the other there were no actual consequences.
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Daniele
> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> ---
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> index 4ccb29f9ac55c..941613be3b9dd 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> @@ -4111,6 +4111,9 @@ static inline void guc_kernel_context_pin(struct intel_guc *guc,
> if (context_guc_id_invalid(ce))
> pin_guc_id(guc, ce);
>
> + if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
> + guc_context_init(ce);
> +
> try_context_registration(ce, true);
> }
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
2022-11-04 18:53 ` [Intel-gfx] " Ceraolo Spurio, Daniele
@ 2022-11-04 18:58 ` John Harrison
-1 siblings, 0 replies; 19+ messages in thread
From: John Harrison @ 2022-11-04 18:58 UTC (permalink / raw)
To: Ceraolo Spurio, Daniele, Intel-GFX; +Cc: DRI-Devel
On 11/4/2022 11:53, Ceraolo Spurio, Daniele wrote:
> On 11/2/2022 12:21 PM, John.C.Harrison@Intel.com wrote:
>> From: John Harrison <John.C.Harrison@Intel.com>
>>
>> If a context has already been registered prior to first submission
>> then context init code was not being called. The noticeable effect of
>> that was the scheduling priority was left at zero (meaning super high
>> priority) instead of being set to normal. This would occur with
>> kernel contexts at start of day as they are manually pinned up front
>> rather than on first submission. So add a call to initialise those
>> when they are pinned.
>
> Does this need a fixes tag? on one side, we were leaving the priority
> to the wrong value, but on the other there were no actual consequences.
>
I think that's the point. There was no actual issue, it's just a
theoretical problem. So there is nothing to be gained by pushing this as
a fix. It it seems like it would be a lot of unnecessary effort to push
it all the way back to 5.17.
John.
> Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>
> Daniele
>
>> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
>> ---
>> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> index 4ccb29f9ac55c..941613be3b9dd 100644
>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> @@ -4111,6 +4111,9 @@ static inline void
>> guc_kernel_context_pin(struct intel_guc *guc,
>> if (context_guc_id_invalid(ce))
>> pin_guc_id(guc, ce);
>> + if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
>> + guc_context_init(ce);
>> +
>> try_context_registration(ce, true);
>> }
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
@ 2022-11-04 18:58 ` John Harrison
0 siblings, 0 replies; 19+ messages in thread
From: John Harrison @ 2022-11-04 18:58 UTC (permalink / raw)
To: Ceraolo Spurio, Daniele, Intel-GFX; +Cc: DRI-Devel
On 11/4/2022 11:53, Ceraolo Spurio, Daniele wrote:
> On 11/2/2022 12:21 PM, John.C.Harrison@Intel.com wrote:
>> From: John Harrison <John.C.Harrison@Intel.com>
>>
>> If a context has already been registered prior to first submission
>> then context init code was not being called. The noticeable effect of
>> that was the scheduling priority was left at zero (meaning super high
>> priority) instead of being set to normal. This would occur with
>> kernel contexts at start of day as they are manually pinned up front
>> rather than on first submission. So add a call to initialise those
>> when they are pinned.
>
> Does this need a fixes tag? on one side, we were leaving the priority
> to the wrong value, but on the other there were no actual consequences.
>
I think that's the point. There was no actual issue, it's just a
theoretical problem. So there is nothing to be gained by pushing this as
a fix. It it seems like it would be a lot of unnecessary effort to push
it all the way back to 5.17.
John.
> Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>
> Daniele
>
>> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
>> ---
>> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> index 4ccb29f9ac55c..941613be3b9dd 100644
>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>> @@ -4111,6 +4111,9 @@ static inline void
>> guc_kernel_context_pin(struct intel_guc *guc,
>> if (context_guc_id_invalid(ce))
>> pin_guc_id(guc, ce);
>> + if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
>> + guc_context_init(ce);
>> +
>> try_context_registration(ce, true);
>> }
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for Fix for two GuC issues (rev2)
2022-11-03 4:45 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
@ 2022-11-04 19:40 ` John Harrison
0 siblings, 0 replies; 19+ messages in thread
From: John Harrison @ 2022-11-04 19:40 UTC (permalink / raw)
To: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 36509 bytes --]
On 11/2/2022 21:45, Patchwork wrote:
> Project List - Patchwork *Patch Details*
> *Series:* Fix for two GuC issues (rev2)
> *URL:* https://patchwork.freedesktop.org/series/110269/
> *State:* failure
> *Details:*
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/index.html
>
>
> CI Bug Log - changes from CI_DRM_12332_full -> Patchwork_110269v2_full
>
>
> Summary
>
> *FAILURE*
>
> Serious unknown changes coming with Patchwork_110269v2_full absolutely
> need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_110269v2_full, please notify your bug team to
> allow them
> to document this new failure mode, which will reduce false positives
> in CI.
>
>
> Participating hosts (9 -> 9)
>
> No changes in participating hosts
>
>
> Possible new issues
>
> Here are the unknown changes that may have been introduced in
> Patchwork_110269v2_full:
>
>
> CI changes
>
>
> Possible regressions
>
> * boot:
> o shard-iclb: (PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb7/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb7/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb7/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/boot.html>)
> -> (PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb6/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb5/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/boot.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html>,
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/boot.html>)
>
I have no idea what I supposed to make of this. Clicking the 'FAIL' link
just produces an 'Access Denied' XML file, it's not even HTML.
Also, this patch has no functional change for any non GuC submission
platform and ICL is not GuC submission enabled. So assuming this issue
is not related.
> *
>
>
> IGT changes
>
>
> Possible regressions
>
> *
>
> igt@i915_suspend@fence-restore-untiled:
>
> o shard-skl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl10/igt@i915_suspend@fence-restore-untiled.html>
> -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl9/igt@i915_suspend@fence-restore-untiled.html>
> *
>
> igt@kms_cursor_legacy@cursor-vs-flip@atomic:
>
> o shard-skl: NOTRUN -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_cursor_legacy@cursor-vs-flip@atomic.html>
> *
>
> igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-wc:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-wc.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-wc.html>
>
As above, this is a GuC only change so will not affect SKL or ICL.
John.
> *
>
>
> Known issues
>
> Here are the changes found in Patchwork_110269v2_full that come from
> known issues:
>
>
> IGT changes
>
>
> Issues hit
>
> *
>
> igt@feature_discovery@psr2:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@feature_discovery@psr2.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@feature_discovery@psr2.html>
> (i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>)
> *
>
> igt@gem_ctx_isolation@preservation-s3@vecs0:
>
> o shard-skl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl6/igt@gem_ctx_isolation@preservation-s3@vecs0.html>
> -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl9/igt@gem_ctx_isolation@preservation-s3@vecs0.html>
> (i915#4793 <https://gitlab.freedesktop.org/drm/intel/issues/4793>)
> *
>
> igt@gem_exec_balancer@parallel-keep-in-fence:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@gem_exec_balancer@parallel-keep-in-fence.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@gem_exec_balancer@parallel-keep-in-fence.html>
> (i915#4525
> <https://gitlab.freedesktop.org/drm/intel/issues/4525>) +2
> similar issues
> *
>
> igt@gem_exec_fair@basic-deadline:
>
> o shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk8/igt@gem_exec_fair@basic-deadline.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk6/igt@gem_exec_fair@basic-deadline.html>
> (i915#2846 <https://gitlab.freedesktop.org/drm/intel/issues/2846>)
> *
>
> igt@gem_exec_fair@basic-pace@vcs1:
>
> o shard-iclb: NOTRUN -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@gem_exec_fair@basic-pace@vcs1.html>
> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>)
> *
>
> igt@gem_exec_suspend@basic-s3@smem:
>
> o shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl1/igt@gem_exec_suspend@basic-s3@smem.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl3/igt@gem_exec_suspend@basic-s3@smem.html>
> (i915#180
> <https://gitlab.freedesktop.org/drm/intel/issues/180>) +3
> similar issues
> *
>
> igt@gem_lmem_swapping@basic:
>
> o shard-skl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@gem_lmem_swapping@basic.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#4613
> <https://gitlab.freedesktop.org/drm/intel/issues/4613>) +2
> similar issues
> *
>
> igt@gem_lmem_swapping@heavy-verify-multi:
>
> o shard-glk: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk3/igt@gem_lmem_swapping@heavy-verify-multi.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#4613 <https://gitlab.freedesktop.org/drm/intel/issues/4613>)
> *
>
> igt@gem_lmem_swapping@massive:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl8/igt@gem_lmem_swapping@massive.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#4613
> <https://gitlab.freedesktop.org/drm/intel/issues/4613>) +1
> similar issue
> *
>
> igt@gem_pread@exhaustion:
>
> o shard-apl: NOTRUN -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl7/igt@gem_pread@exhaustion.html>
> (i915#7248 <https://gitlab.freedesktop.org/drm/intel/issues/7248>)
> *
>
> igt@gem_tiled_wb:
>
> o shard-skl: NOTRUN -> TIMEOUT
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@gem_tiled_wb.html>
> (i915#6990 <https://gitlab.freedesktop.org/drm/intel/issues/6990>)
> *
>
> igt@i915_selftest@live@gt_heartbeat:
>
> o shard-skl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl7/igt@i915_selftest@live@gt_heartbeat.html>
> -> DMESG-FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@i915_selftest@live@gt_heartbeat.html>
> (i915#5334 <https://gitlab.freedesktop.org/drm/intel/issues/5334>)
> *
>
> igt@i915_selftest@live@hangcheck:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-tglb6/igt@i915_selftest@live@hangcheck.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-tglb2/igt@i915_selftest@live@hangcheck.html>
> (i915#5591 <https://gitlab.freedesktop.org/drm/intel/issues/5591>)
> *
>
> igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl8/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#3886
> <https://gitlab.freedesktop.org/drm/intel/issues/3886>) +3
> similar issues
> *
>
> igt@kms_chamelium@hdmi-edid-change-during-suspend:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl8/igt@kms_chamelium@hdmi-edid-change-during-suspend.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> fdo#111827
> <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +1
> similar issue
> *
>
> igt@kms_color_chamelium@gamma:
>
> o shard-skl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_color_chamelium@gamma.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> fdo#111827 <https://bugs.freedesktop.org/show_bug.cgi?id=111827>)
> *
>
> igt@kms_cursor_legacy@cursor-vs-flip@varying-size:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/igt@kms_cursor_legacy@cursor-vs-flip@varying-size.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb7/igt@kms_cursor_legacy@cursor-vs-flip@varying-size.html>
> (i915#5072 <https://gitlab.freedesktop.org/drm/intel/issues/5072>)
> *
>
> igt@kms_flip@flip-vs-expired-vblank@a-hdmi-a1:
>
> o shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk5/igt@kms_flip@flip-vs-expired-vblank@a-hdmi-a1.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk1/igt@kms_flip@flip-vs-expired-vblank@a-hdmi-a1.html>
> (i915#79 <https://gitlab.freedesktop.org/drm/intel/issues/79>)
> *
>
> igt@kms_flip@flip-vs-suspend@b-dp1:
>
> o shard-apl: NOTRUN -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@kms_flip@flip-vs-suspend@b-dp1.html>
> (i915#180
> <https://gitlab.freedesktop.org/drm/intel/issues/180>) +1
> similar issue
> *
>
> igt@kms_flip@plain-flip-ts-check@c-edp1:
>
> o shard-skl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl6/igt@kms_flip@plain-flip-ts-check@c-edp1.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_flip@plain-flip-ts-check@c-edp1.html>
> (i915#2122
> <https://gitlab.freedesktop.org/drm/intel/issues/2122>) +2
> similar issues
> *
>
> igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode:
>
> o shard-iclb: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb8/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-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>)
> *
>
> igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-upscaling@pipe-a-default-mode:
>
> o shard-iclb: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-32bpp-yftile-upscaling@pipe-a-default-mode.html>
> (i915#2672
> <https://gitlab.freedesktop.org/drm/intel/issues/2672>) +3
> similar issues
> *
>
> igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilercccs-downscaling@pipe-a-default-mode:
>
> o shard-iclb: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytilercccs-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_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +35
> similar issues
> *
>
> igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite.html>
> (i915#2546 <https://gitlab.freedesktop.org/drm/intel/issues/2546>)
> *
>
> igt@kms_frontbuffer_tracking@fbcpsr-suspend:
>
> o shard-skl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +59
> similar issues
> *
>
> igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move:
>
> o shard-skl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl10/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move.html>
> (i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>)
> *
>
> igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-render:
>
> o shard-glk: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk3/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-render.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +4
> similar issues
> *
>
> igt@kms_psr2_sf@cursor-plane-move-continuous-sf:
>
> o shard-skl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_psr2_sf@cursor-plane-move-continuous-sf.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#658
> <https://gitlab.freedesktop.org/drm/intel/issues/658>) +1
> similar issue
> *
>
> igt@kms_psr@psr2_cursor_blt:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_psr@psr2_cursor_blt.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_psr@psr2_cursor_blt.html>
> (fdo#109441
> <https://bugs.freedesktop.org/show_bug.cgi?id=109441>) +2
> similar issues
> *
>
> igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-tglb3/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-tglb1/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html>
> (i915#5519 <https://gitlab.freedesktop.org/drm/intel/issues/5519>)
> *
>
> igt@kms_writeback@writeback-invalid-parameters:
>
> o shard-skl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_writeback@writeback-invalid-parameters.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#2437 <https://gitlab.freedesktop.org/drm/intel/issues/2437>)
> *
>
> igt@sysfs_clients@busy:
>
> o shard-glk: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk3/igt@sysfs_clients@busy.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#2994 <https://gitlab.freedesktop.org/drm/intel/issues/2994>)
> *
>
> igt@sysfs_clients@split-10:
>
> o shard-skl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl9/igt@sysfs_clients@split-10.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#2994 <https://gitlab.freedesktop.org/drm/intel/issues/2994>)
>
>
> Possible fixes
>
> *
>
> igt@gem_exec_balancer@parallel-contexts:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb6/igt@gem_exec_balancer@parallel-contexts.html>
> (i915#4525
> <https://gitlab.freedesktop.org/drm/intel/issues/4525>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@gem_exec_balancer@parallel-contexts.html>
> *
>
> igt@gem_exec_fair@basic-flow@rcs0:
>
> o shard-tglb: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-tglb2/igt@gem_exec_fair@basic-flow@rcs0.html>
> (i915#2842
> <https://gitlab.freedesktop.org/drm/intel/issues/2842>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-tglb5/igt@gem_exec_fair@basic-flow@rcs0.html>
> *
>
> igt@gem_mmap_offset@bad-object:
>
> o shard-skl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl4/igt@gem_mmap_offset@bad-object.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl7/igt@gem_mmap_offset@bad-object.html>
> +3 similar issues
> *
>
> igt@i915_pm_rc6_residency@rc6-idle@vcs0:
>
> o shard-skl: WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl7/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html>
> (i915#1804
> <https://gitlab.freedesktop.org/drm/intel/issues/1804>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html>
> *
>
> igt@i915_pm_rpm@basic-pci-d3-state:
>
> o shard-iclb: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb3/igt@i915_pm_rpm@basic-pci-d3-state.html>
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@i915_pm_rpm@basic-pci-d3-state.html>
> *
>
> igt@kms_cursor_legacy@cursor-vs-flip@toggle:
>
> o shard-skl: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl6/igt@kms_cursor_legacy@cursor-vs-flip@toggle.html>
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@kms_cursor_legacy@cursor-vs-flip@toggle.html>
> *
>
> igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size:
>
> o shard-glk: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/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_110269v2/shard-glk6/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html>
> +1 similar issue
> *
>
> igt@kms_flip@flip-vs-absolute-wf_vblank@a-edp1:
>
> o shard-skl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl5/igt@kms_flip@flip-vs-absolute-wf_vblank@a-edp1.html>
> (i915#2122
> <https://gitlab.freedesktop.org/drm/intel/issues/2122>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl7/igt@kms_flip@flip-vs-absolute-wf_vblank@a-edp1.html>
> +1 similar issue
> *
>
> igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
>
> o shard-skl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl3/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html>
> (i915#79 <https://gitlab.freedesktop.org/drm/intel/issues/79>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html>
> +2 similar issues
> *
>
> igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode.html>
> (i915#3555
> <https://gitlab.freedesktop.org/drm/intel/issues/3555>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode.html>
> +4 similar issues
> *
>
> igt@kms_plane_lowres@tiling-y@pipe-a-hdmi-a-1:
>
> o shard-glk: DMESG-FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk9/igt@kms_plane_lowres@tiling-y@pipe-a-hdmi-a-1.html>
> (i915#118
> <https://gitlab.freedesktop.org/drm/intel/issues/118>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk8/igt@kms_plane_lowres@tiling-y@pipe-a-hdmi-a-1.html>
> *
>
> igt@kms_plane_lowres@tiling-y@pipe-c-hdmi-a-1:
>
> o shard-glk: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk9/igt@kms_plane_lowres@tiling-y@pipe-c-hdmi-a-1.html>
> (i915#7307
> <https://gitlab.freedesktop.org/drm/intel/issues/7307>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk8/igt@kms_plane_lowres@tiling-y@pipe-c-hdmi-a-1.html>
> +1 similar issue
> *
>
> igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1.html>
> (i915#5176
> <https://gitlab.freedesktop.org/drm/intel/issues/5176>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_plane_scaling@plane-downscale-with-pixel-format-factor-0-5@pipe-b-edp-1.html>
> +2 similar issues
> *
>
> igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html>
> (i915#5235
> <https://gitlab.freedesktop.org/drm/intel/issues/5235>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html>
> +2 similar issues
> *
>
> igt@kms_psr@psr2_sprite_blt:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/igt@kms_psr@psr2_sprite_blt.html>
> (fdo#109441
> <https://bugs.freedesktop.org/show_bug.cgi?id=109441>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html>
> +2 similar issues
> *
>
> igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb5/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html>
> (i915#5519
> <https://gitlab.freedesktop.org/drm/intel/issues/5519>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb1/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html>
> *
>
> igt@kms_vblank@pipe-b-ts-continuation-suspend:
>
> o shard-apl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl2/igt@kms_vblank@pipe-b-ts-continuation-suspend.html>
> (i915#180
> <https://gitlab.freedesktop.org/drm/intel/issues/180>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl7/igt@kms_vblank@pipe-b-ts-continuation-suspend.html>
> +3 similar issues
> *
>
> igt@perf@polling:
>
> o shard-skl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl9/igt@perf@polling.html>
> (i915#1542
> <https://gitlab.freedesktop.org/drm/intel/issues/1542>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@perf@polling.html>
> *
>
> igt@prime_mmap_coherency@ioctl-errors:
>
> o shard-skl: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl3/igt@prime_mmap_coherency@ioctl-errors.html>
> (i915#2295
> <https://gitlab.freedesktop.org/drm/intel/issues/2295>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl4/igt@prime_mmap_coherency@ioctl-errors.html>
> *
>
> igt@syncobj_wait@wait-all-for-submit-snapshot:
>
> o shard-skl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl10/igt@syncobj_wait@wait-all-for-submit-snapshot.html>
> (i915#7109
> <https://gitlab.freedesktop.org/drm/intel/issues/7109>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl6/igt@syncobj_wait@wait-all-for-submit-snapshot.html>
>
>
> Warnings
>
> *
>
> igt@gem_pwrite@basic-exhaustion:
>
> o
>
> shard-glk: WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-glk5/igt@gem_pwrite@basic-exhaustion.html>
> (i915#2658
> <https://gitlab.freedesktop.org/drm/intel/issues/2658>) ->
> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-glk1/igt@gem_pwrite@basic-exhaustion.html>
> (i915#7248 <https://gitlab.freedesktop.org/drm/intel/issues/7248>)
>
> o
>
> shard-skl: TIMEOUT
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-skl3/igt@gem_pwrite@basic-exhaustion.html>
> (i915#7248
> <https://gitlab.freedesktop.org/drm/intel/issues/7248>) ->
> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-skl1/igt@gem_pwrite@basic-exhaustion.html>
> (i915#7248 <https://gitlab.freedesktop.org/drm/intel/issues/7248>)
>
> *
>
> igt@i915_pm_dc@dc3co-vpb-simulation:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html>
> (i915#588
> <https://gitlab.freedesktop.org/drm/intel/issues/588>) -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@i915_pm_dc@dc3co-vpb-simulation.html>
> (i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>)
> *
>
> igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb2/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf.html>
> (i915#2920
> <https://gitlab.freedesktop.org/drm/intel/issues/2920>) ->
> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb3/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-fully-sf.html>
> (i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>)
> *
>
> igt@kms_psr2_sf@cursor-plane-update-sf:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-iclb1/igt@kms_psr2_sf@cursor-plane-update-sf.html>
> (fdo#111068
> <https://bugs.freedesktop.org/show_bug.cgi?id=111068> /
> i915#658
> <https://gitlab.freedesktop.org/drm/intel/issues/658>) -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-iclb2/igt@kms_psr2_sf@cursor-plane-update-sf.html>
> (i915#2920 <https://gitlab.freedesktop.org/drm/intel/issues/2920>)
> *
>
> igt@runner@aborted:
>
> o shard-apl: (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl2/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl3/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl3/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl8/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl1/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12332/shard-apl6/igt@runner@aborted.html>)
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#180 <https://gitlab.freedesktop.org/drm/intel/issues/180>
> / i915#3002
> <https://gitlab.freedesktop.org/drm/intel/issues/3002> /
> i915#4312
> <https://gitlab.freedesktop.org/drm/intel/issues/4312>) ->
> (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl3/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl3/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_110269v2/shard-apl6/igt@runner@aborted.html>)
> (i915#180
> <https://gitlab.freedesktop.org/drm/intel/issues/180> /
> i915#3002
> <https://gitlab.freedesktop.org/drm/intel/issues/3002> /
> i915#4312 <https://gitlab.freedesktop.org/drm/intel/issues/4312>)
>
>
> Build changes
>
> * Linux: CI_DRM_12332 -> Patchwork_110269v2
>
> CI-20190529: 20190529
> CI_DRM_12332: 601b2ef606e4b83d5518aa6a5011bb2b1c5954d9 @
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_7038: 5389b3f3b9b75df6bd8506e4aa3da357fd0c0ab1 @
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_110269v2: 601b2ef606e4b83d5518aa6a5011bb2b1c5954d9 @
> git://anongit.freedesktop.org/gfx-ci/linux
> piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @
> git://anongit.freedesktop.org/piglit
>
[-- Attachment #2: Type: text/html, Size: 48363 bytes --]
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
@ 2022-11-05 5:18 ` Lucas De Marchi
-1 siblings, 0 replies; 19+ messages in thread
From: Lucas De Marchi @ 2022-11-05 5:18 UTC (permalink / raw)
To: John.C.Harrison; +Cc: Intel-GFX, DRI-Devel
On Wed, Nov 02, 2022 at 12:21:08PM -0700, John.C.Harrison@Intel.com wrote:
>From: John Harrison <John.C.Harrison@Intel.com>
>
>If a context has already been registered prior to first submission
>then context init code was not being called. The noticeable effect of
>that was the scheduling priority was left at zero (meaning super high
>priority) instead of being set to normal. This would occur with
>kernel contexts at start of day as they are manually pinned up front
>rather than on first submission. So add a call to initialise those
>when they are pinned.
>
>Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi <lucas.demarchi@intel.com>
>---
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>index 4ccb29f9ac55c..941613be3b9dd 100644
>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>@@ -4111,6 +4111,9 @@ static inline void guc_kernel_context_pin(struct intel_guc *guc,
> if (context_guc_id_invalid(ce))
> pin_guc_id(guc, ce);
>
>+ if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
>+ guc_context_init(ce);
>+
> try_context_registration(ce, true);
> }
>
>--
>2.37.3
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Intel-gfx] [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts
@ 2022-11-05 5:18 ` Lucas De Marchi
0 siblings, 0 replies; 19+ messages in thread
From: Lucas De Marchi @ 2022-11-05 5:18 UTC (permalink / raw)
To: John.C.Harrison; +Cc: Intel-GFX, DRI-Devel
On Wed, Nov 02, 2022 at 12:21:08PM -0700, John.C.Harrison@Intel.com wrote:
>From: John Harrison <John.C.Harrison@Intel.com>
>
>If a context has already been registered prior to first submission
>then context init code was not being called. The noticeable effect of
>that was the scheduling priority was left at zero (meaning super high
>priority) instead of being set to normal. This would occur with
>kernel contexts at start of day as they are manually pinned up front
>rather than on first submission. So add a call to initialise those
>when they are pinned.
>
>Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi <lucas.demarchi@intel.com>
>---
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>index 4ccb29f9ac55c..941613be3b9dd 100644
>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>@@ -4111,6 +4111,9 @@ static inline void guc_kernel_context_pin(struct intel_guc *guc,
> if (context_guc_id_invalid(ce))
> pin_guc_id(guc, ce);
>
>+ if (!test_bit(CONTEXT_GUC_INIT, &ce->flags))
>+ guc_context_init(ce);
>+
> try_context_registration(ce, true);
> }
>
>--
>2.37.3
>
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2022-11-05 5:18 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-02 19:21 [PATCH v2 0/2] Fix for two GuC issues John.C.Harrison
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
2022-11-02 19:21 ` [PATCH v2 1/2] drm/i915/guc: Properly initialise kernel contexts John.C.Harrison
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
2022-11-04 18:53 ` Ceraolo Spurio, Daniele
2022-11-04 18:53 ` [Intel-gfx] " Ceraolo Spurio, Daniele
2022-11-04 18:58 ` John Harrison
2022-11-04 18:58 ` [Intel-gfx] " John Harrison
2022-11-05 5:18 ` Lucas De Marchi
2022-11-05 5:18 ` [Intel-gfx] " Lucas De Marchi
2022-11-02 19:21 ` [PATCH v2 2/2] drm/i915/guc: Don't deadlock busyness stats vs reset John.C.Harrison
2022-11-02 19:21 ` [Intel-gfx] " John.C.Harrison
2022-11-03 11:31 ` Tvrtko Ursulin
2022-11-03 18:45 ` John Harrison
2022-11-03 18:54 ` Umesh Nerlige Ramappa
2022-11-03 18:54 ` Umesh Nerlige Ramappa
2022-11-02 20:56 ` [Intel-gfx] ✓ Fi.CI.BAT: success for Fix for two GuC issues (rev2) Patchwork
2022-11-03 4:45 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2022-11-04 19:40 ` John Harrison
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.