* [Intel-gfx] [Intel-gfx v2 0/1] drm/i915/guc: Don't update engine busyness stats too frequently
@ 2022-06-18 5:43 Alan Previn
2022-06-18 5:43 ` [Intel-gfx] [Intel-gfx v2 1/1] " Alan Previn
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Alan Previn @ 2022-06-18 5:43 UTC (permalink / raw)
To: intel-gfx
This change ensures we don't collect new stat counters too soon
after the last sample.
Changes from prior revs:
v1: - Move the location of the new logic to higher up
the callstack in intel_guc_busyness_park (Umesh).
- Fix typo threshold of jiffies-since-last from double
to half of ping_delay (Umesh)
Alan Previn (1):
drm/i915/guc: Don't update engine busyness stats too frequently
drivers/gpu/drm/i915/gt/uc/intel_guc.h | 8 ++++++++
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 12 ++++++++++++
2 files changed, 20 insertions(+)
base-commit: ac17a5249380aaabe5d1eaebd9b3a2eedc08ccdc
--
2.25.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] [Intel-gfx v2 1/1] drm/i915/guc: Don't update engine busyness stats too frequently
2022-06-18 5:43 [Intel-gfx] [Intel-gfx v2 0/1] drm/i915/guc: Don't update engine busyness stats too frequently Alan Previn
@ 2022-06-18 5:43 ` Alan Previn
2022-06-20 8:46 ` Tvrtko Ursulin
2022-06-22 23:00 ` Umesh Nerlige Ramappa
2022-06-18 6:12 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/guc: Don't update engine busyness stats too frequently (rev3) Patchwork
` (2 subsequent siblings)
3 siblings, 2 replies; 10+ messages in thread
From: Alan Previn @ 2022-06-18 5:43 UTC (permalink / raw)
To: intel-gfx
Using igt's gem-create and with additional patches to track object
creation time, it was measured that guc_update_engine_gt_clks was
getting called over 188 thousand times in the span of 15 seconds
(running the test three times).
Get a jiffies sample on every trigger and ensure we skip sampling
if we are being called too soon. Use half of the ping_delay as a
safe threshold.
NOTE: with this change, the number of calls went down to just 14
over the same span of time (matching the original intent of running
about once every 24 seconds, at 19.2Mhz GT freq, per engine).
Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
---
drivers/gpu/drm/i915/gt/uc/intel_guc.h | 8 ++++++++
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 12 ++++++++++++
2 files changed, 20 insertions(+)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
index 966e69a8b1c1..26f3e4403de7 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.h
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
@@ -230,6 +230,14 @@ struct intel_guc {
* @shift: Right shift value for the gpm timestamp
*/
u32 shift;
+
+ /**
+ * @last_jiffies: jiffies at last actual stats collection time
+ * We use this timestamp to ensure we don't oversample the
+ * stats because runtime power management events can trigger
+ * stats collection at much higher rates than required.
+ */
+ u64 last_stat_jiffs;
} timestamp;
#ifdef CONFIG_DRM_I915_SELFTEST
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 e62ea35513ea..05c945f14ef5 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -1314,6 +1314,8 @@ static void __update_guc_busyness_stats(struct intel_guc *guc)
unsigned long flags;
ktime_t unused;
+ guc->timestamp.last_stat_jiffs = get_jiffies_64();
+
spin_lock_irqsave(&guc->timestamp.lock, flags);
guc_update_pm_timestamp(guc, &unused);
@@ -1386,6 +1388,16 @@ void intel_guc_busyness_park(struct intel_gt *gt)
return;
cancel_delayed_work(&guc->timestamp.work);
+
+ /*
+ * Before parking, we should sample engine busyness stats if we need to.
+ * We can skip it if we are less than half a ping from the last time we
+ * sampled the business stats.
+ */
+ if (guc->timestamp.last_stat_jiffs && (get_jiffies_64() - guc->timestamp.last_stat_jiffs <
+ (guc->timestamp.ping_delay >> 1)))
+ return;
+
__update_guc_busyness_stats(guc);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/guc: Don't update engine busyness stats too frequently (rev3)
2022-06-18 5:43 [Intel-gfx] [Intel-gfx v2 0/1] drm/i915/guc: Don't update engine busyness stats too frequently Alan Previn
2022-06-18 5:43 ` [Intel-gfx] [Intel-gfx v2 1/1] " Alan Previn
@ 2022-06-18 6:12 ` Patchwork
2022-06-18 6:33 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-06-18 20:35 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
3 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2022-06-18 6:12 UTC (permalink / raw)
To: Alan Previn; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/guc: Don't update engine busyness stats too frequently (rev3)
URL : https://patchwork.freedesktop.org/series/105023/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Don't update engine busyness stats too frequently (rev3)
2022-06-18 5:43 [Intel-gfx] [Intel-gfx v2 0/1] drm/i915/guc: Don't update engine busyness stats too frequently Alan Previn
2022-06-18 5:43 ` [Intel-gfx] [Intel-gfx v2 1/1] " Alan Previn
2022-06-18 6:12 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/guc: Don't update engine busyness stats too frequently (rev3) Patchwork
@ 2022-06-18 6:33 ` Patchwork
2022-06-18 20:35 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
3 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2022-06-18 6:33 UTC (permalink / raw)
To: Alan Previn; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 11400 bytes --]
== Series Details ==
Series: drm/i915/guc: Don't update engine busyness stats too frequently (rev3)
URL : https://patchwork.freedesktop.org/series/105023/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11776 -> Patchwork_105023v3
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/index.html
Participating hosts (35 -> 35)
------------------------------
Additional (2): fi-cml-u2 fi-icl-u2
Missing (2): fi-rkl-11600 fi-bdw-samus
Known issues
------------
Here are the changes found in Patchwork_105023v3 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_fence@basic-busy@bcs0:
- fi-cml-u2: NOTRUN -> [SKIP][1] ([i915#1208]) +1 similar issue
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@gem_exec_fence@basic-busy@bcs0.html
* igt@gem_exec_suspend@basic-s0@smem:
- fi-cml-u2: NOTRUN -> [DMESG-WARN][2] ([i915#5122])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gem_huc_copy@huc-copy:
- fi-cml-u2: NOTRUN -> [SKIP][3] ([i915#2190])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@gem_huc_copy@huc-copy.html
- fi-icl-u2: NOTRUN -> [SKIP][4] ([i915#2190])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@random-engines:
- fi-icl-u2: NOTRUN -> [SKIP][5] ([i915#4613]) +3 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@gem_lmem_swapping@random-engines.html
* igt@gem_lmem_swapping@verify-random:
- fi-cml-u2: NOTRUN -> [SKIP][6] ([i915#4613]) +3 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@gem_lmem_swapping@verify-random.html
* igt@i915_pm_rpm@module-reload:
- fi-cfl-8109u: [PASS][7] -> [DMESG-FAIL][8] ([i915#62])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-cfl-8109u/igt@i915_pm_rpm@module-reload.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cfl-8109u/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live@requests:
- fi-pnv-d510: [PASS][9] -> [DMESG-FAIL][10] ([i915#4528])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-pnv-d510/igt@i915_selftest@live@requests.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-pnv-d510/igt@i915_selftest@live@requests.html
* igt@i915_selftest@live@ring_submission:
- fi-cfl-8109u: [PASS][11] -> [DMESG-WARN][12] ([i915#5904]) +11 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-cfl-8109u/igt@i915_selftest@live@ring_submission.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cfl-8109u/igt@i915_selftest@live@ring_submission.html
* igt@i915_suspend@basic-s2idle-without-i915:
- fi-cfl-8109u: [PASS][13] -> [DMESG-WARN][14] ([i915#5904] / [i915#62])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-cfl-8109u/igt@i915_suspend@basic-s2idle-without-i915.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cfl-8109u/igt@i915_suspend@basic-s2idle-without-i915.html
* igt@i915_suspend@basic-s3-without-i915:
- fi-icl-u2: NOTRUN -> [SKIP][15] ([i915#5903])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@i915_suspend@basic-s3-without-i915.html
* igt@kms_addfb_basic@invalid-set-prop:
- fi-cfl-8109u: [PASS][16] -> [DMESG-WARN][17] ([i915#62]) +78 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-cfl-8109u/igt@kms_addfb_basic@invalid-set-prop.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cfl-8109u/igt@kms_addfb_basic@invalid-set-prop.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-icl-u2: NOTRUN -> [SKIP][18] ([fdo#111827]) +8 similar issues
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_chamelium@vga-hpd-fast:
- fi-cml-u2: NOTRUN -> [SKIP][19] ([fdo#109284] / [fdo#111827]) +7 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@kms_chamelium@vga-hpd-fast.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-cml-u2: NOTRUN -> [SKIP][20] ([fdo#109278]) +1 similar issue
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
- fi-icl-u2: NOTRUN -> [SKIP][21] ([fdo#109278] / [i915#4103]) +1 similar issue
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_flip@basic-flip-vs-wf_vblank@a-edp1:
- fi-tgl-u2: [PASS][22] -> [DMESG-WARN][23] ([i915#402])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-tgl-u2/igt@kms_flip@basic-flip-vs-wf_vblank@a-edp1.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-tgl-u2/igt@kms_flip@basic-flip-vs-wf_vblank@a-edp1.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-icl-u2: NOTRUN -> [WARN][24] ([i915#6008])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@kms_force_connector_basic@force-connector-state.html
* igt@kms_force_connector_basic@force-load-detect:
- fi-cml-u2: NOTRUN -> [SKIP][25] ([fdo#109285])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@kms_force_connector_basic@force-load-detect.html
- fi-icl-u2: NOTRUN -> [SKIP][26] ([fdo#109285])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_frontbuffer_tracking@basic:
- fi-cml-u2: NOTRUN -> [DMESG-WARN][27] ([i915#4269])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
- fi-cml-u2: NOTRUN -> [SKIP][28] ([fdo#109278] / [i915#533])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
- fi-icl-u2: NOTRUN -> [SKIP][29] ([fdo#109278])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
* igt@kms_setmode@basic-clone-single-crtc:
- fi-icl-u2: NOTRUN -> [SKIP][30] ([i915#3555])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@kms_setmode@basic-clone-single-crtc.html
- fi-cml-u2: NOTRUN -> [SKIP][31] ([i915#3555])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@kms_setmode@basic-clone-single-crtc.html
* igt@prime_vgem@basic-userptr:
- fi-cml-u2: NOTRUN -> [SKIP][32] ([fdo#109295] / [i915#3301])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@prime_vgem@basic-userptr.html
- fi-icl-u2: NOTRUN -> [SKIP][33] ([fdo#109295] / [i915#3301])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-icl-u2/igt@prime_vgem@basic-userptr.html
* igt@runner@aborted:
- fi-cml-u2: NOTRUN -> [FAIL][34] ([i915#4312] / [i915#5257])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-cml-u2/igt@runner@aborted.html
- fi-pnv-d510: NOTRUN -> [FAIL][35] ([fdo#109271] / [i915#2403] / [i915#4312])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-pnv-d510/igt@runner@aborted.html
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s0@smem:
- {fi-ehl-2}: [DMESG-WARN][36] ([i915#5122]) -> [PASS][37]
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html
#### Warnings ####
* igt@i915_selftest@live@hangcheck:
- fi-hsw-4770: [INCOMPLETE][38] ([i915#3303] / [i915#4785]) -> [INCOMPLETE][39] ([i915#4785])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1208]: https://gitlab.freedesktop.org/drm/intel/issues/1208
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2403]: https://gitlab.freedesktop.org/drm/intel/issues/2403
[i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
[i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
[i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
[i915#4269]: https://gitlab.freedesktop.org/drm/intel/issues/4269
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
[i915#5122]: https://gitlab.freedesktop.org/drm/intel/issues/5122
[i915#5257]: https://gitlab.freedesktop.org/drm/intel/issues/5257
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#5903]: https://gitlab.freedesktop.org/drm/intel/issues/5903
[i915#5904]: https://gitlab.freedesktop.org/drm/intel/issues/5904
[i915#6008]: https://gitlab.freedesktop.org/drm/intel/issues/6008
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
Build changes
-------------
* Linux: CI_DRM_11776 -> Patchwork_105023v3
CI-20190529: 20190529
CI_DRM_11776: ac17a5249380aaabe5d1eaebd9b3a2eedc08ccdc @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6536: e3de4d32b7a509635fbff4d5131c05a7767699f7 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_105023v3: ac17a5249380aaabe5d1eaebd9b3a2eedc08ccdc @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
d37bcde04f20 drm/i915/guc: Don't update engine busyness stats too frequently
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/index.html
[-- Attachment #2: Type: text/html, Size: 13806 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/guc: Don't update engine busyness stats too frequently (rev3)
2022-06-18 5:43 [Intel-gfx] [Intel-gfx v2 0/1] drm/i915/guc: Don't update engine busyness stats too frequently Alan Previn
` (2 preceding siblings ...)
2022-06-18 6:33 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
@ 2022-06-18 20:35 ` Patchwork
3 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2022-06-18 20:35 UTC (permalink / raw)
To: Teres Alexis, Alan Previn; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 36702 bytes --]
== Series Details ==
Series: drm/i915/guc: Don't update engine busyness stats too frequently (rev3)
URL : https://patchwork.freedesktop.org/series/105023/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11776_full -> Patchwork_105023v3_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_105023v3_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_105023v3_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_105023v3_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], [FAIL][46], [PASS][47], [PASS][48], [PASS][49], [PASS][50])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb1/boot.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb1/boot.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb1/boot.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/boot.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/boot.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/boot.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb3/boot.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb3/boot.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb3/boot.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb4/boot.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb4/boot.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb4/boot.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb5/boot.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb5/boot.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb5/boot.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb5/boot.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb6/boot.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb6/boot.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb6/boot.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb7/boot.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb7/boot.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb7/boot.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb8/boot.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb8/boot.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb8/boot.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/boot.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/boot.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb1/boot.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb1/boot.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb1/boot.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb7/boot.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb8/boot.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb7/boot.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb8/boot.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb7/boot.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb6/boot.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb6/boot.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/boot.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb3/boot.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb3/boot.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb3/boot.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb6/boot.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb5/boot.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb5/boot.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb5/boot.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb5/boot.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb4/boot.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb8/boot.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb4/boot.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb4/boot.html
### IGT changes ###
#### Possible regressions ####
* igt@api_intel_allocator@two-level-inception-interruptible:
- shard-tglb: [PASS][51] -> [INCOMPLETE][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb1/igt@api_intel_allocator@two-level-inception-interruptible.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb8/igt@api_intel_allocator@two-level-inception-interruptible.html
* igt@gem_busy@close-race:
- shard-skl: [PASS][53] -> [TIMEOUT][54]
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl1/igt@gem_busy@close-race.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl1/igt@gem_busy@close-race.html
Known issues
------------
Here are the changes found in Patchwork_105023v3_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_balancer@parallel-keep-in-fence:
- shard-iclb: [PASS][55] -> [SKIP][56] ([i915#4525]) +1 similar issue
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb4/igt@gem_exec_balancer@parallel-keep-in-fence.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb6/igt@gem_exec_balancer@parallel-keep-in-fence.html
* igt@gem_exec_balancer@parallel-ordering:
- shard-kbl: NOTRUN -> [FAIL][57] ([i915#6117])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl1/igt@gem_exec_balancer@parallel-ordering.html
* igt@gem_exec_fair@basic-deadline:
- shard-apl: NOTRUN -> [FAIL][58] ([i915#6141])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl1/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-iclb: [PASS][59] -> [FAIL][60] ([i915#2842])
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb6/igt@gem_exec_fair@basic-none-share@rcs0.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb1/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none-vip@rcs0:
- shard-glk: [PASS][61] -> [FAIL][62] ([i915#2842])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-glk5/igt@gem_exec_fair@basic-none-vip@rcs0.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-glk4/igt@gem_exec_fair@basic-none-vip@rcs0.html
* igt@gem_exec_fair@basic-none@bcs0:
- shard-tglb: [PASS][63] -> [FAIL][64] ([i915#2842]) +1 similar issue
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb2/igt@gem_exec_fair@basic-none@bcs0.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb7/igt@gem_exec_fair@basic-none@bcs0.html
* igt@gem_lmem_swapping@parallel-multi:
- shard-kbl: NOTRUN -> [SKIP][65] ([fdo#109271] / [i915#4613]) +1 similar issue
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl7/igt@gem_lmem_swapping@parallel-multi.html
* igt@gem_lmem_swapping@random:
- shard-apl: NOTRUN -> [SKIP][66] ([fdo#109271] / [i915#4613])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl4/igt@gem_lmem_swapping@random.html
* igt@gem_softpin@noreloc-s3:
- shard-apl: [PASS][67] -> [DMESG-WARN][68] ([i915#180]) +2 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl2/igt@gem_softpin@noreloc-s3.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl8/igt@gem_softpin@noreloc-s3.html
* igt@i915_pm_dc@dc6-psr:
- shard-iclb: [PASS][69] -> [FAIL][70] ([i915#454])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb8/igt@i915_pm_dc@dc6-psr.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb3/igt@i915_pm_dc@dc6-psr.html
* igt@i915_selftest@live@hangcheck:
- shard-snb: [PASS][71] -> [INCOMPLETE][72] ([i915#3921])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-snb2/igt@i915_selftest@live@hangcheck.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-snb5/igt@i915_selftest@live@hangcheck.html
* igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_mc_ccs:
- shard-apl: NOTRUN -> [SKIP][73] ([fdo#109271] / [i915#3886])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl1/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-b-bad-pixel-format-4_tiled_dg2_rc_ccs:
- shard-kbl: NOTRUN -> [SKIP][74] ([fdo#109271]) +126 similar issues
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl1/igt@kms_ccs@pipe-b-bad-pixel-format-4_tiled_dg2_rc_ccs.html
* igt@kms_ccs@pipe-c-crc-primary-basic-y_tiled_gen12_rc_ccs_cc:
- shard-kbl: NOTRUN -> [SKIP][75] ([fdo#109271] / [i915#3886]) +8 similar issues
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@kms_ccs@pipe-c-crc-primary-basic-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_chamelium@dp-hpd-storm-disable:
- shard-apl: NOTRUN -> [SKIP][76] ([fdo#109271] / [fdo#111827]) +2 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl1/igt@kms_chamelium@dp-hpd-storm-disable.html
* igt@kms_chamelium@hdmi-mode-timings:
- shard-kbl: NOTRUN -> [SKIP][77] ([fdo#109271] / [fdo#111827]) +6 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl1/igt@kms_chamelium@hdmi-mode-timings.html
* igt@kms_color_chamelium@pipe-a-ctm-limited-range:
- shard-skl: NOTRUN -> [SKIP][78] ([fdo#109271] / [fdo#111827])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl6/igt@kms_color_chamelium@pipe-a-ctm-limited-range.html
* igt@kms_cursor_crc@pipe-a-cursor-suspend:
- shard-kbl: [PASS][79] -> [DMESG-WARN][80] ([i915#180])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl3/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size:
- shard-iclb: [PASS][81] -> [FAIL][82] ([i915#2346])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb1/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb7/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@pipe-d-single-bo:
- shard-kbl: NOTRUN -> [SKIP][83] ([fdo#109271] / [i915#533])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl7/igt@kms_cursor_legacy@pipe-d-single-bo.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-apl: [PASS][84] -> [INCOMPLETE][85] ([i915#180] / [i915#1982] / [i915#4939])
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl4/igt@kms_fbcon_fbt@fbc-suspend.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl3/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2:
- shard-glk: [PASS][86] -> [FAIL][87] ([i915#2122])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-glk2/igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2.html
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-glk4/igt@kms_flip@2x-flip-vs-expired-vblank@bc-hdmi-a1-hdmi-a2.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
- shard-skl: [PASS][88] -> [FAIL][89] ([i915#79])
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl10/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl4/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
* igt@kms_flip@flip-vs-suspend@a-dp1:
- shard-kbl: NOTRUN -> [DMESG-WARN][90] ([i915#180]) +2 similar issues
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@kms_flip@flip-vs-suspend@a-dp1.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
- shard-kbl: NOTRUN -> [FAIL][91] ([fdo#108145] / [i915#265])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl7/igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb.html
* igt@kms_plane_scaling@plane-upscale-with-rotation-factor-0-25@pipe-c-edp-1:
- shard-skl: NOTRUN -> [SKIP][92] ([fdo#109271]) +3 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl6/igt@kms_plane_scaling@plane-upscale-with-rotation-factor-0-25@pipe-c-edp-1.html
* igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-5@pipe-c-edp-1:
- shard-iclb: [PASS][93] -> [SKIP][94] ([i915#5235]) +2 similar issues
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb1/igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-5@pipe-c-edp-1.html
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-5@pipe-c-edp-1.html
* igt@kms_psr2_sf@overlay-plane-update-continuous-sf:
- shard-kbl: NOTRUN -> [SKIP][95] ([fdo#109271] / [i915#658])
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl1/igt@kms_psr2_sf@overlay-plane-update-continuous-sf.html
* igt@kms_psr@psr2_sprite_plane_onoff:
- shard-iclb: [PASS][96] -> [SKIP][97] ([fdo#109441]) +2 similar issues
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb4/igt@kms_psr@psr2_sprite_plane_onoff.html
* igt@kms_writeback@writeback-check-output:
- shard-kbl: NOTRUN -> [SKIP][98] ([fdo#109271] / [i915#2437])
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl7/igt@kms_writeback@writeback-check-output.html
* igt@perf@short-reads:
- shard-skl: [PASS][99] -> [FAIL][100] ([i915#51])
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl4/igt@perf@short-reads.html
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl7/igt@perf@short-reads.html
* igt@prime_nv_pcopy@test3_4:
- shard-apl: NOTRUN -> [SKIP][101] ([fdo#109271]) +51 similar issues
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl4/igt@prime_nv_pcopy@test3_4.html
* igt@sysfs_clients@create:
- shard-apl: NOTRUN -> [SKIP][102] ([fdo#109271] / [i915#2994])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl4/igt@sysfs_clients@create.html
* igt@sysfs_clients@fair-7:
- shard-kbl: NOTRUN -> [SKIP][103] ([fdo#109271] / [i915#2994]) +1 similar issue
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl7/igt@sysfs_clients@fair-7.html
#### Possible fixes ####
* igt@gem_ctx_persistence@many-contexts:
- shard-kbl: [INCOMPLETE][104] ([i915#3593]) -> [PASS][105]
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl3/igt@gem_ctx_persistence@many-contexts.html
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@gem_ctx_persistence@many-contexts.html
* igt@gem_exec_balancer@parallel-contexts:
- shard-iclb: [SKIP][106] ([i915#4525]) -> [PASS][107]
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb6/igt@gem_exec_balancer@parallel-contexts.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb1/igt@gem_exec_balancer@parallel-contexts.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-apl: [FAIL][108] ([i915#2842]) -> [PASS][109] +1 similar issue
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl7/igt@gem_exec_fair@basic-none@vcs0.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl2/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_exec_fair@basic-pace@bcs0:
- shard-tglb: [FAIL][110] ([i915#2842]) -> [PASS][111] +1 similar issue
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb1/igt@gem_exec_fair@basic-pace@bcs0.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb8/igt@gem_exec_fair@basic-pace@bcs0.html
* igt@gem_exec_fair@basic-pace@vcs1:
- shard-kbl: [FAIL][112] ([i915#2842]) -> [PASS][113] +1 similar issue
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl1/igt@gem_exec_fair@basic-pace@vcs1.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl1/igt@gem_exec_fair@basic-pace@vcs1.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-iclb: [FAIL][114] ([i915#2849]) -> [PASS][115]
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/igt@gem_exec_fair@basic-throttle@rcs0.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb8/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gem_exec_whisper@basic-queues-all:
- shard-glk: [DMESG-WARN][116] ([i915#118]) -> [PASS][117] +1 similar issue
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-glk4/igt@gem_exec_whisper@basic-queues-all.html
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-glk9/igt@gem_exec_whisper@basic-queues-all.html
* igt@gem_huc_copy@huc-copy:
- shard-tglb: [SKIP][118] ([i915#2190]) -> [PASS][119]
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb7/igt@gem_huc_copy@huc-copy.html
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb3/igt@gem_huc_copy@huc-copy.html
* igt@i915_pm_dc@dc6-dpms:
- shard-iclb: [FAIL][120] ([i915#454]) -> [PASS][121]
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb3/igt@i915_pm_dc@dc6-dpms.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/igt@i915_pm_dc@dc6-dpms.html
* igt@i915_selftest@live@hangcheck:
- shard-tglb: [DMESG-WARN][122] ([i915#5591]) -> [PASS][123]
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb2/igt@i915_selftest@live@hangcheck.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb3/igt@i915_selftest@live@hangcheck.html
* igt@i915_suspend@fence-restore-tiled2untiled:
- shard-apl: [DMESG-WARN][124] ([i915#180]) -> [PASS][125] +1 similar issue
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl8/igt@i915_suspend@fence-restore-tiled2untiled.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl4/igt@i915_suspend@fence-restore-tiled2untiled.html
* igt@i915_suspend@fence-restore-untiled:
- shard-kbl: [DMESG-WARN][126] ([i915#180]) -> [PASS][127] +2 similar issues
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@i915_suspend@fence-restore-untiled.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl4/igt@i915_suspend@fence-restore-untiled.html
* igt@kms_async_flips@alternate-sync-async-flip@pipe-c-edp-1:
- shard-skl: [FAIL][128] ([i915#2521]) -> [PASS][129]
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl10/igt@kms_async_flips@alternate-sync-async-flip@pipe-c-edp-1.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl4/igt@kms_async_flips@alternate-sync-async-flip@pipe-c-edp-1.html
* igt@kms_async_flips@crc@pipe-a-dp-1:
- shard-kbl: [FAIL][130] -> [PASS][131]
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl4/igt@kms_async_flips@crc@pipe-a-dp-1.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl3/igt@kms_async_flips@crc@pipe-a-dp-1.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions:
- shard-glk: [FAIL][132] ([i915#2346]) -> [PASS][133]
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-glk4/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-glk7/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
* igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2:
- shard-glk: [FAIL][134] ([i915#2122]) -> [PASS][135]
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-glk9/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-glk6/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1:
- shard-skl: [FAIL][136] ([i915#79]) -> [PASS][137]
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl10/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl4/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html
* igt@kms_flip@plain-flip-fb-recreate@a-edp1:
- shard-skl: [FAIL][138] ([i915#2122]) -> [PASS][139] +3 similar issues
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl10/igt@kms_flip@plain-flip-fb-recreate@a-edp1.html
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl10/igt@kms_flip@plain-flip-fb-recreate@a-edp1.html
* igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-16bpp-ytile-downscaling:
- shard-iclb: [SKIP][140] ([i915#3701]) -> [PASS][141] +1 similar issue
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-16bpp-ytile-downscaling.html
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb8/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-16bpp-ytile-downscaling.html
* igt@kms_frontbuffer_tracking@psr-rgb101010-draw-mmap-gtt:
- shard-skl: [DMESG-WARN][142] ([i915#1982]) -> [PASS][143]
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl6/igt@kms_frontbuffer_tracking@psr-rgb101010-draw-mmap-gtt.html
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl4/igt@kms_frontbuffer_tracking@psr-rgb101010-draw-mmap-gtt.html
* igt@kms_hdr@bpc-switch-suspend@pipe-a-dp-1:
- shard-kbl: [FAIL][144] ([i915#1188]) -> [PASS][145]
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl1/igt@kms_hdr@bpc-switch-suspend@pipe-a-dp-1.html
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl7/igt@kms_hdr@bpc-switch-suspend@pipe-a-dp-1.html
* igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-pixel-formats@pipe-b-edp-1:
- shard-iclb: [SKIP][146] ([i915#5176]) -> [PASS][147] +1 similar issue
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb3/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-pixel-formats@pipe-b-edp-1.html
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-pixel-formats@pipe-b-edp-1.html
* igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1:
- shard-iclb: [SKIP][148] ([i915#5235]) -> [PASS][149] +2 similar issues
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb2/igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1.html
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb8/igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1.html
* igt@kms_psr@psr2_cursor_blt:
- shard-iclb: [SKIP][150] ([fdo#109441]) -> [PASS][151] +1 similar issue
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb1/igt@kms_psr@psr2_cursor_blt.html
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/igt@kms_psr@psr2_cursor_blt.html
* igt@perf@polling-parameterized:
- shard-skl: [FAIL][152] ([i915#5639]) -> [PASS][153]
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl6/igt@perf@polling-parameterized.html
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl4/igt@perf@polling-parameterized.html
#### Warnings ####
* igt@gem_eio@unwedge-stress:
- shard-tglb: [FAIL][154] ([i915#5784]) -> [TIMEOUT][155] ([i915#3063])
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb2/igt@gem_eio@unwedge-stress.html
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb7/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_fair@basic-none-rrul@rcs0:
- shard-iclb: [FAIL][156] ([i915#2842]) -> [FAIL][157] ([i915#2852])
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb5/igt@gem_exec_fair@basic-none-rrul@rcs0.html
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb7/igt@gem_exec_fair@basic-none-rrul@rcs0.html
* igt@gem_exec_fair@basic-none@rcs0:
- shard-tglb: [SKIP][158] ([i915#2848]) -> [FAIL][159] ([i915#2842])
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-tglb2/igt@gem_exec_fair@basic-none@rcs0.html
[159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-tglb7/igt@gem_exec_fair@basic-none@rcs0.html
* igt@i915_suspend@sysfs-reader:
- shard-kbl: [DMESG-WARN][160] ([i915#180]) -> [INCOMPLETE][161] ([i915#3614] / [i915#4817])
[160]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@i915_suspend@sysfs-reader.html
[161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl4/igt@i915_suspend@sysfs-reader.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-kbl: [FAIL][162] ([i915#4767]) -> [INCOMPLETE][163] ([i915#180])
[162]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl4/igt@kms_fbcon_fbt@fbc-suspend.html
[163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite:
- shard-skl: [SKIP][164] ([fdo#109271] / [i915#1888]) -> [SKIP][165] ([fdo#109271])
[164]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-skl7/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite.html
[165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-skl6/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite.html
* igt@kms_psr2_su@page_flip-nv12:
- shard-iclb: [SKIP][166] ([fdo#109642] / [fdo#111068] / [i915#658]) -> [FAIL][167] ([i915#5939])
[166]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-iclb3/igt@kms_psr2_su@page_flip-nv12.html
[167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-iclb2/igt@kms_psr2_su@page_flip-nv12.html
* igt@runner@aborted:
- shard-apl: ([FAIL][168], [FAIL][169], [FAIL][170], [FAIL][171], [FAIL][172]) ([fdo#109271] / [i915#180] / [i915#3002] / [i915#4312] / [i915#5257]) -> ([FAIL][173], [FAIL][174], [FAIL][175], [FAIL][176], [FAIL][177], [FAIL][178]) ([i915#180] / [i915#3002] / [i915#4312] / [i915#5257])
[168]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl7/igt@runner@aborted.html
[169]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl8/igt@runner@aborted.html
[170]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl4/igt@runner@aborted.html
[171]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl8/igt@runner@aborted.html
[172]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-apl7/igt@runner@aborted.html
[173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl7/igt@runner@aborted.html
[174]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl4/igt@runner@aborted.html
[175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl8/igt@runner@aborted.html
[176]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl3/igt@runner@aborted.html
[177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl4/igt@runner@aborted.html
[178]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-apl6/igt@runner@aborted.html
- shard-kbl: ([FAIL][179], [FAIL][180], [FAIL][181], [FAIL][182], [FAIL][183], [FAIL][184], [FAIL][185]) ([i915#180] / [i915#3002] / [i915#4312] / [i915#5257]) -> ([FAIL][186], [FAIL][187], [FAIL][188], [FAIL][189], [FAIL][190], [FAIL][191]) ([i915#180] / [i915#3002] / [i915#4312] / [i915#5257] / [i915#92])
[179]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl7/igt@runner@aborted.html
[180]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@runner@aborted.html
[181]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@runner@aborted.html
[182]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl7/igt@runner@aborted.html
[183]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@runner@aborted.html
[184]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@runner@aborted.html
[185]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11776/shard-kbl6/igt@runner@aborted.html
[186]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@runner@aborted.html
[187]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@runner@aborted.html
[188]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl1/igt@runner@aborted.html
[189]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@runner@aborted.html
[190]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl6/igt@runner@aborted.html
[191]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_105023v3/shard-kbl4/igt@runner@aborted.html
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
[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#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
[i915#2521]: https://gitlab.freedesktop.org/drm/intel/issues/2521
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2848]: https://gitlab.freedesktop.org/drm/intel/issues/2848
[i915#2849]: https://gitlab.freedesktop.org/drm/intel/issues/2849
[i915#2852]: https://gitlab.freedesktop.org/drm/intel/issues/2852
[i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
[i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002
[i915#3063]: https://gitlab.freedesktop.org/drm/intel/issues/3063
[i915#3593]: https://gitlab.freedesktop.org/drm/intel/issues/3593
[i915#3614]: https://gitlab.freedesktop.org/drm/intel/issues/3614
[i915#3701]: https://gitlab.freedesktop.org/drm/intel/issues/3701
[i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
[i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
[i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4767]: https://gitlab.freedesktop.org/drm/intel/issues/4767
[i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
[i915#4939]: https://gitlab.freedesktop.org/drm/intel/issues/4939
[i915#51]: https://gitlab.freedesktop.org/drm/intel/issues/51
[i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
[i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
[i915#5257]: https://gitlab.freedesktop.org/drm/intel/issues/5257
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#5591]: https://gitlab.freedesktop.org/drm/intel/issues/5591
[i915#5639]: https://gitlab.freedesktop.org/drm/intel/issues/5639
[i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784
[i915#5939]: https://gitlab.freedesktop.org/drm/intel/issues/5939
[i915#6117]: https://gitlab.freedesktop.org/drm/intel/issues/6117
[i915#6141]: https://gitlab.freedesktop.org/drm/intel/issues/6141
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92
Build changes
-------------
* Linux: CI_DRM_11776 -> Patchwork_105023v3
CI-20190529: 20190529
CI_DRM_11776: ac17a5249380aaabe5d1eaebd9b3a2eedc08ccdc @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6536: e3de4d32b7a509635fbff4d5131c05a7767699f7 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_105023v3: ac17a5249380aaabe5d1eaebd9b3a2eedc08ccdc @ 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_105023v3/index.html
[-- Attachment #2: Type: text/html, Size: 43210 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] [Intel-gfx v2 1/1] drm/i915/guc: Don't update engine busyness stats too frequently
2022-06-18 5:43 ` [Intel-gfx] [Intel-gfx v2 1/1] " Alan Previn
@ 2022-06-20 8:46 ` Tvrtko Ursulin
2022-06-22 22:36 ` Umesh Nerlige Ramappa
2022-06-23 0:25 ` Teres Alexis, Alan Previn
2022-06-22 23:00 ` Umesh Nerlige Ramappa
1 sibling, 2 replies; 10+ messages in thread
From: Tvrtko Ursulin @ 2022-06-20 8:46 UTC (permalink / raw)
To: Alan Previn, intel-gfx
On 18/06/2022 06:43, Alan Previn wrote:
> Using igt's gem-create and with additional patches to track object
> creation time, it was measured that guc_update_engine_gt_clks was
> getting called over 188 thousand times in the span of 15 seconds
> (running the test three times).
>
> Get a jiffies sample on every trigger and ensure we skip sampling
> if we are being called too soon. Use half of the ping_delay as a
> safe threshold.
>
> NOTE: with this change, the number of calls went down to just 14
> over the same span of time (matching the original intent of running
> about once every 24 seconds, at 19.2Mhz GT freq, per engine).
It would be beneficial to record the root cause. Both frequency of
parking/unparking caused by <insert what> and lmem access cost.
> Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
> ---
> drivers/gpu/drm/i915/gt/uc/intel_guc.h | 8 ++++++++
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 12 ++++++++++++
> 2 files changed, 20 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
> index 966e69a8b1c1..26f3e4403de7 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc.h
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
> @@ -230,6 +230,14 @@ struct intel_guc {
> * @shift: Right shift value for the gpm timestamp
> */
> u32 shift;
> +
> + /**
> + * @last_jiffies: jiffies at last actual stats collection time
> + * We use this timestamp to ensure we don't oversample the
> + * stats because runtime power management events can trigger
> + * stats collection at much higher rates than required.
> + */
> + u64 last_stat_jiffs;
Why the new "jiffs" naming and not the usual jiffies?
Otherwise a good comment - just align the member name with the kerneldoc
name.
> } timestamp;
>
> #ifdef CONFIG_DRM_I915_SELFTEST
> 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 e62ea35513ea..05c945f14ef5 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> @@ -1314,6 +1314,8 @@ static void __update_guc_busyness_stats(struct intel_guc *guc)
> unsigned long flags;
> ktime_t unused;
>
> + guc->timestamp.last_stat_jiffs = get_jiffies_64();
Why the 64 bit flavour? It's a first in i915 but it doesn't feel so special.
> +
> spin_lock_irqsave(&guc->timestamp.lock, flags);
>
> guc_update_pm_timestamp(guc, &unused);
> @@ -1386,6 +1388,16 @@ void intel_guc_busyness_park(struct intel_gt *gt)
> return;
>
> cancel_delayed_work(&guc->timestamp.work);
> +
> + /*
> + * Before parking, we should sample engine busyness stats if we need to.
> + * We can skip it if we are less than half a ping from the last time we
> + * sampled the business stats.
busyness
> + */
> + if (guc->timestamp.last_stat_jiffs && (get_jiffies_64() - guc->timestamp.last_stat_jiffs <
> + (guc->timestamp.ping_delay >> 1)))
> + return;
1)
Recommend a division instead of a shift.
2)
Is there a time_after() macro for this?
3)
Should the logic be contained/consolidated in __update_guc_busyness_stats?
There is cancel_delayed_work in there - is it okay for that to be
bypassed from here?
Regards,
Tvrtko
> +
> __update_guc_busyness_stats(guc);
> }
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] [Intel-gfx v2 1/1] drm/i915/guc: Don't update engine busyness stats too frequently
2022-06-20 8:46 ` Tvrtko Ursulin
@ 2022-06-22 22:36 ` Umesh Nerlige Ramappa
2022-06-23 0:25 ` Teres Alexis, Alan Previn
1 sibling, 0 replies; 10+ messages in thread
From: Umesh Nerlige Ramappa @ 2022-06-22 22:36 UTC (permalink / raw)
To: Tvrtko Ursulin; +Cc: intel-gfx, Alan Previn
On Mon, Jun 20, 2022 at 09:46:30AM +0100, Tvrtko Ursulin wrote:
>
>On 18/06/2022 06:43, Alan Previn wrote:
>>Using igt's gem-create and with additional patches to track object
>>creation time, it was measured that guc_update_engine_gt_clks was
>>getting called over 188 thousand times in the span of 15 seconds
>>(running the test three times).
>>
>>Get a jiffies sample on every trigger and ensure we skip sampling
>>if we are being called too soon. Use half of the ping_delay as a
>>safe threshold.
>>
>>NOTE: with this change, the number of calls went down to just 14
>>over the same span of time (matching the original intent of running
>>about once every 24 seconds, at 19.2Mhz GT freq, per engine).
>
>It would be beneficial to record the root cause. Both frequency of
>parking/unparking caused by <insert what> and lmem access cost.
>
>>Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
>>---
>> drivers/gpu/drm/i915/gt/uc/intel_guc.h | 8 ++++++++
>> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 12 ++++++++++++
>> 2 files changed, 20 insertions(+)
>>
>>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
>>index 966e69a8b1c1..26f3e4403de7 100644
>>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.h
>>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
>>@@ -230,6 +230,14 @@ struct intel_guc {
>> * @shift: Right shift value for the gpm timestamp
>> */
>> u32 shift;
>>+
>>+ /**
>>+ * @last_jiffies: jiffies at last actual stats collection time
>>+ * We use this timestamp to ensure we don't oversample the
>>+ * stats because runtime power management events can trigger
>>+ * stats collection at much higher rates than required.
>>+ */
>>+ u64 last_stat_jiffs;
>
>Why the new "jiffs" naming and not the usual jiffies?
>
>Otherwise a good comment - just align the member name with the
>kerneldoc name.
>
>> } timestamp;
>> #ifdef CONFIG_DRM_I915_SELFTEST
>>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 e62ea35513ea..05c945f14ef5 100644
>>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>>@@ -1314,6 +1314,8 @@ static void __update_guc_busyness_stats(struct intel_guc *guc)
>> unsigned long flags;
>> ktime_t unused;
>>+ guc->timestamp.last_stat_jiffs = get_jiffies_64();
>
>Why the 64 bit flavour? It's a first in i915 but it doesn't feel so special.
>
>>+
>> spin_lock_irqsave(&guc->timestamp.lock, flags);
>> guc_update_pm_timestamp(guc, &unused);
>>@@ -1386,6 +1388,16 @@ void intel_guc_busyness_park(struct intel_gt *gt)
>> return;
>> cancel_delayed_work(&guc->timestamp.work);
>>+
>>+ /*
>>+ * Before parking, we should sample engine busyness stats if we need to.
>>+ * We can skip it if we are less than half a ping from the last time we
>>+ * sampled the business stats.
>
>busyness
>
>>+ */
>>+ if (guc->timestamp.last_stat_jiffs && (get_jiffies_64() - guc->timestamp.last_stat_jiffs <
>>+ (guc->timestamp.ping_delay >> 1)))
>>+ return;
>
>1)
>Recommend a division instead of a shift.
>
>2)
>Is there a time_after() macro for this?
>
>3)
>Should the logic be contained/consolidated in __update_guc_busyness_stats?
>
It wouldn't hurt to have it in __update_guc_busyness_stats, but
__update_guc_busyness_stats is also called from the ping worker. Since
this is targeted for gt_park/gt_unpark events, I would leave it here.
>There is cancel_delayed_work in there - is it okay for that to be
>bypassed from here?
I don't see it being bypassed. cancel_delayed_work is called before the
jiffies logic here.
Regards,
Umesh
>
>Regards,
>
>Tvrtko
>
>>+
>> __update_guc_busyness_stats(guc);
>> }
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] [Intel-gfx v2 1/1] drm/i915/guc: Don't update engine busyness stats too frequently
2022-06-18 5:43 ` [Intel-gfx] [Intel-gfx v2 1/1] " Alan Previn
2022-06-20 8:46 ` Tvrtko Ursulin
@ 2022-06-22 23:00 ` Umesh Nerlige Ramappa
2022-06-22 23:55 ` Teres Alexis, Alan Previn
1 sibling, 1 reply; 10+ messages in thread
From: Umesh Nerlige Ramappa @ 2022-06-22 23:00 UTC (permalink / raw)
To: Alan Previn; +Cc: intel-gfx
On Fri, Jun 17, 2022 at 10:43:45PM -0700, Alan Previn wrote:
>Using igt's gem-create and with additional patches to track object
>creation time, it was measured that guc_update_engine_gt_clks was
>getting called over 188 thousand times in the span of 15 seconds
>(running the test three times).
>
>Get a jiffies sample on every trigger and ensure we skip sampling
>if we are being called too soon. Use half of the ping_delay as a
>safe threshold.
>
>NOTE: with this change, the number of calls went down to just 14
>over the same span of time (matching the original intent of running
>about once every 24 seconds, at 19.2Mhz GT freq, per engine).
>
>Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
>---
> drivers/gpu/drm/i915/gt/uc/intel_guc.h | 8 ++++++++
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 12 ++++++++++++
> 2 files changed, 20 insertions(+)
>
>diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
>index 966e69a8b1c1..26f3e4403de7 100644
>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.h
>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
>@@ -230,6 +230,14 @@ struct intel_guc {
> * @shift: Right shift value for the gpm timestamp
> */
> u32 shift;
>+
>+ /**
>+ * @last_jiffies: jiffies at last actual stats collection time
>+ * We use this timestamp to ensure we don't oversample the
>+ * stats because runtime power management events can trigger
>+ * stats collection at much higher rates than required.
>+ */
>+ u64 last_stat_jiffs;
> } timestamp;
>
> #ifdef CONFIG_DRM_I915_SELFTEST
>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 e62ea35513ea..05c945f14ef5 100644
>--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
>@@ -1314,6 +1314,8 @@ static void __update_guc_busyness_stats(struct intel_guc *guc)
> unsigned long flags;
> ktime_t unused;
>
>+ guc->timestamp.last_stat_jiffs = get_jiffies_64();
>+
> spin_lock_irqsave(&guc->timestamp.lock, flags);
>
> guc_update_pm_timestamp(guc, &unused);
>@@ -1386,6 +1388,16 @@ void intel_guc_busyness_park(struct intel_gt *gt)
> return;
>
> cancel_delayed_work(&guc->timestamp.work);
>+
>+ /*
>+ * Before parking, we should sample engine busyness stats if we need to.
>+ * We can skip it if we are less than half a ping from the last time we
>+ * sampled the business stats.
>+ */
>+ if (guc->timestamp.last_stat_jiffs && (get_jiffies_64() - guc->timestamp.last_stat_jiffs <
>+ (guc->timestamp.ping_delay >> 1)))
>+ return;
>+
I would just sample the jiffies here instead of inside
__update_guc_busyness_stats(), so all the logic is within this function
and easy to read.
Either ways, this should work too, so this is
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Regards,
Umesh
> __update_guc_busyness_stats(guc);
> }
>
>--
>2.25.1
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] [Intel-gfx v2 1/1] drm/i915/guc: Don't update engine busyness stats too frequently
2022-06-22 23:00 ` Umesh Nerlige Ramappa
@ 2022-06-22 23:55 ` Teres Alexis, Alan Previn
0 siblings, 0 replies; 10+ messages in thread
From: Teres Alexis, Alan Previn @ 2022-06-22 23:55 UTC (permalink / raw)
To: Nerlige Ramappa, Umesh; +Cc: intel-gfx@lists.freedesktop.org
Thanks Umesh. As per offline - i will address Tvrtko's comments too.
...alan
On Wed, 2022-06-22 at 16:00 -0700, Nerlige Ramappa, Umesh wrote:
> On Fri, Jun 17, 2022 at 10:43:45PM -0700, Alan Previn wrote:
> > Using igt's gem-create and with additional patches to track object
> > creation time, it was measured that guc_update_engine_gt_clks was
> > getting called over 188 thousand times in the span of 15 seconds
> > (running the test three times).
> >
> > Get a jiffies sample on every trigger and ensure we skip sampling
> > if we are being called too soon. Use half of the ping_delay as a
> > safe threshold.
> >
> > NOTE: with this change, the number of calls went down to just 14
> > over the same span of time (matching the original intent of running
> > about once every 24 seconds, at 19.2Mhz GT freq, per engine).
> >
> > Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
> > ---
> > drivers/gpu/drm/i915/gt/uc/intel_guc.h | 8 ++++++++
> > drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 12 ++++++++++++
> > 2 files changed, 20 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
> > index 966e69a8b1c1..26f3e4403de7 100644
> > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc.h
> > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
> > @@ -230,6 +230,14 @@ struct intel_guc {
> > * @shift: Right shift value for the gpm timestamp
> > */
> > u32 shift;
> > +
> > + /**
> > + * @last_jiffies: jiffies at last actual stats collection time
> > + * We use this timestamp to ensure we don't oversample the
> > + * stats because runtime power management events can trigger
> > + * stats collection at much higher rates than required.
> > + */
> > + u64 last_stat_jiffs;
> > } timestamp;
> >
> > #ifdef CONFIG_DRM_I915_SELFTEST
> > 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 e62ea35513ea..05c945f14ef5 100644
> > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> > @@ -1314,6 +1314,8 @@ static void __update_guc_busyness_stats(struct intel_guc *guc)
> > unsigned long flags;
> > ktime_t unused;
> >
> > + guc->timestamp.last_stat_jiffs = get_jiffies_64();
> > +
> > spin_lock_irqsave(&guc->timestamp.lock, flags);
> >
> > guc_update_pm_timestamp(guc, &unused);
> > @@ -1386,6 +1388,16 @@ void intel_guc_busyness_park(struct intel_gt *gt)
> > return;
> >
> > cancel_delayed_work(&guc->timestamp.work);
> > +
> > + /*
> > + * Before parking, we should sample engine busyness stats if we need to.
> > + * We can skip it if we are less than half a ping from the last time we
> > + * sampled the business stats.
> > + */
> > + if (guc->timestamp.last_stat_jiffs && (get_jiffies_64() - guc->timestamp.last_stat_jiffs <
> > + (guc->timestamp.ping_delay >> 1)))
> > + return;
> > +
>
> I would just sample the jiffies here instead of inside
> __update_guc_busyness_stats(), so all the logic is within this function
> and easy to read.
>
> Either ways, this should work too, so this is
>
> Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>
> Regards,
> Umesh
>
> > __update_guc_busyness_stats(guc);
> > }
> >
> > --
> > 2.25.1
> >
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] [Intel-gfx v2 1/1] drm/i915/guc: Don't update engine busyness stats too frequently
2022-06-20 8:46 ` Tvrtko Ursulin
2022-06-22 22:36 ` Umesh Nerlige Ramappa
@ 2022-06-23 0:25 ` Teres Alexis, Alan Previn
1 sibling, 0 replies; 10+ messages in thread
From: Teres Alexis, Alan Previn @ 2022-06-23 0:25 UTC (permalink / raw)
To: tvrtko.ursulin@linux.intel.com, intel-gfx@lists.freedesktop.org
> > + /**
> > + * @last_jiffies: jiffies at last actual stats collection time
> > + * We use this timestamp to ensure we don't oversample the
> > + * stats because runtime power management events can trigger
> > + * stats collection at much higher rates than required.
> > + */
> > + u64 last_stat_jiffs;
>
> Why the new "jiffs" naming and not the usual jiffies?
>
> Otherwise a good comment - just align the member name with the kerneldoc
> name.
>
my mistake - will align the names.
> > unsigned long flags;
> > ktime_t unused;
> >
> > + guc->timestamp.last_stat_jiffs = get_jiffies_64();
>
> Why the 64 bit flavour? It's a first in i915 but it doesn't feel so special.
>
sure - will use the regular jiffies
> > +
> > spin_lock_irqsave(&guc->timestamp.lock, flags);
> >
> > guc_update_pm_timestamp(guc, &unused);
> > @@ -1386,6 +1388,16 @@ void intel_guc_busyness_park(struct intel_gt *gt)
> > return;
> >
> > cancel_delayed_work(&guc->timestamp.work);
> > +
> > + /*
> > + * Before parking, we should sample engine busyness stats if we need to.
> > + * We can skip it if we are less than half a ping from the last time we
> > + * sampled the business stats.
>
> busyness
yup.
>
> > + */
> > + if (guc->timestamp.last_stat_jiffs && (get_jiffies_64() - guc->timestamp.last_stat_jiffs <
> > + (guc->timestamp.ping_delay >> 1)))
> > + return;
>
> 1)
> Recommend a division instead of a shift.
ok
>
> 2)
> Is there a time_after() macro for this?
>
yes there is - will do.
> 3)
> Should the logic be contained/consolidated in __update_guc_busyness_stats?
As Umesh mentioned, __update_guc_busyness_stats is called from the non __gt_park callers and in those cases we don't
want it to skip. I wanted avoid adding additional unnecessary params to signal if the caller would be okay with skipping
- so rather just make that decision at the caller's level. However, for the updating of the latest last_stat_jiffies, i
wanted to ensure that it got updated for all callers so we ensure the absolute minimal required busyness updates are
made when gt_park is called while other callers also got called in between.
>
> There is cancel_delayed_work in there - is it okay for that to be
> bypassed from here?
>
I believe Umesh addressed this.
> Regards,
>
> Tvrtko
>
> > +
> > __update_guc_busyness_stats(guc);
> > }
> >
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-06-23 0:25 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-06-18 5:43 [Intel-gfx] [Intel-gfx v2 0/1] drm/i915/guc: Don't update engine busyness stats too frequently Alan Previn
2022-06-18 5:43 ` [Intel-gfx] [Intel-gfx v2 1/1] " Alan Previn
2022-06-20 8:46 ` Tvrtko Ursulin
2022-06-22 22:36 ` Umesh Nerlige Ramappa
2022-06-23 0:25 ` Teres Alexis, Alan Previn
2022-06-22 23:00 ` Umesh Nerlige Ramappa
2022-06-22 23:55 ` Teres Alexis, Alan Previn
2022-06-18 6:12 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/guc: Don't update engine busyness stats too frequently (rev3) Patchwork
2022-06-18 6:33 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-06-18 20:35 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox