* [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power
@ 2022-09-23 11:00 Riana Tauro
2022-09-23 11:00 ` [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles Riana Tauro
` (4 more replies)
0 siblings, 5 replies; 13+ messages in thread
From: Riana Tauro @ 2022-09-23 11:00 UTC (permalink / raw)
To: intel-gfx
live_slpc_power tests if running at low frequency saves power
Rev2 : Add multi-tile support
Riana Tauro (3):
drm/i915/guc/slpc: Run SLPC selftests on all tiles
drm/i915/selftests: Add helper function measure_power
drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
drivers/gpu/drm/i915/gt/selftest_rps.c | 12 +-
drivers/gpu/drm/i915/gt/selftest_slpc.c | 172 +++++++++++++++++++++---
2 files changed, 164 insertions(+), 20 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles
2022-09-23 11:00 [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power Riana Tauro
@ 2022-09-23 11:00 ` Riana Tauro
2022-09-26 16:02 ` Belgaumkar, Vinay
2022-09-23 11:00 ` [Intel-gfx] [PATCH 2/3] drm/i915/selftests: Add helper function measure_power Riana Tauro
` (3 subsequent siblings)
4 siblings, 1 reply; 13+ messages in thread
From: Riana Tauro @ 2022-09-23 11:00 UTC (permalink / raw)
To: intel-gfx
Run slpc selftests on all tiles
Signed-off-by: Riana Tauro <riana.tauro@intel.com>
---
drivers/gpu/drm/i915/gt/selftest_slpc.c | 45 ++++++++++++++++++++-----
1 file changed, 37 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c b/drivers/gpu/drm/i915/gt/selftest_slpc.c
index f8a1d27df272..928f74718881 100644
--- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
+++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
@@ -270,26 +270,50 @@ static int run_test(struct intel_gt *gt, int test_type)
static int live_slpc_vary_min(void *arg)
{
struct drm_i915_private *i915 = arg;
- struct intel_gt *gt = to_gt(i915);
+ struct intel_gt *gt;
+ unsigned int i;
+ int ret;
+
+ for_each_gt(gt, i915, i) {
+ ret = run_test(gt, VARY_MIN);
+ if (ret)
+ return ret;
+ }
- return run_test(gt, VARY_MIN);
+ return ret;
}
static int live_slpc_vary_max(void *arg)
{
struct drm_i915_private *i915 = arg;
- struct intel_gt *gt = to_gt(i915);
+ struct intel_gt *gt;
+ unsigned int i;
+ int ret;
+
+ for_each_gt(gt, i915, i) {
+ ret = run_test(gt, VARY_MAX);
+ if (ret)
+ return ret;
+ }
- return run_test(gt, VARY_MAX);
+ return ret;
}
/* check if pcode can grant RP0 */
static int live_slpc_max_granted(void *arg)
{
struct drm_i915_private *i915 = arg;
- struct intel_gt *gt = to_gt(i915);
+ struct intel_gt *gt;
+ unsigned int i;
+ int ret;
+
+ for_each_gt(gt, i915, i) {
+ ret = run_test(gt, MAX_GRANTED);
+ if (ret)
+ return ret;
+ }
- return run_test(gt, MAX_GRANTED);
+ return ret;
}
int intel_slpc_live_selftests(struct drm_i915_private *i915)
@@ -300,8 +324,13 @@ int intel_slpc_live_selftests(struct drm_i915_private *i915)
SUBTEST(live_slpc_max_granted),
};
- if (intel_gt_is_wedged(to_gt(i915)))
- return 0;
+ struct intel_gt *gt;
+ unsigned int i;
+
+ for_each_gt(gt, i915, i) {
+ if (intel_gt_is_wedged(gt))
+ return 0;
+ }
return i915_live_subtests(tests, i915);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [Intel-gfx] [PATCH 2/3] drm/i915/selftests: Add helper function measure_power
2022-09-23 11:00 [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power Riana Tauro
2022-09-23 11:00 ` [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles Riana Tauro
@ 2022-09-23 11:00 ` Riana Tauro
2022-09-26 15:57 ` Belgaumkar, Vinay
2022-09-23 11:00 ` [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power Riana Tauro
` (2 subsequent siblings)
4 siblings, 1 reply; 13+ messages in thread
From: Riana Tauro @ 2022-09-23 11:00 UTC (permalink / raw)
To: intel-gfx
move the power measurement and the triangle filter
to a different function. No functional changes.
Signed-off-by: Riana Tauro <riana.tauro@intel.com>
---
drivers/gpu/drm/i915/gt/selftest_rps.c | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/selftest_rps.c b/drivers/gpu/drm/i915/gt/selftest_rps.c
index cfb4708dd62e..99a372486fb7 100644
--- a/drivers/gpu/drm/i915/gt/selftest_rps.c
+++ b/drivers/gpu/drm/i915/gt/selftest_rps.c
@@ -1107,21 +1107,27 @@ static u64 __measure_power(int duration_ms)
return div64_u64(1000 * 1000 * dE, dt);
}
-static u64 measure_power_at(struct intel_rps *rps, int *freq)
+static u64 measure_power(struct intel_rps *rps, int *freq)
{
u64 x[5];
int i;
- *freq = rps_set_check(rps, *freq);
for (i = 0; i < 5; i++)
x[i] = __measure_power(5);
- *freq = (*freq + read_cagf(rps)) / 2;
+
+ *freq = (*freq + intel_rps_read_actual_frequency(rps)) / 2;
/* A simple triangle filter for better result stability */
sort(x, 5, sizeof(*x), cmp_u64, NULL);
return div_u64(x[1] + 2 * x[2] + x[3], 4);
}
+static u64 measure_power_at(struct intel_rps *rps, int *freq)
+{
+ *freq = rps_set_check(rps, *freq);
+ return measure_power(rps, freq);
+}
+
int live_rps_power(void *arg)
{
struct intel_gt *gt = arg;
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
2022-09-23 11:00 [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power Riana Tauro
2022-09-23 11:00 ` [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles Riana Tauro
2022-09-23 11:00 ` [Intel-gfx] [PATCH 2/3] drm/i915/selftests: Add helper function measure_power Riana Tauro
@ 2022-09-23 11:00 ` Riana Tauro
2022-09-26 16:04 ` Belgaumkar, Vinay
2022-09-23 13:44 ` [Intel-gfx] ✓ Fi.CI.BAT: success for Add SLPC selftest live_slpc_power (rev2) Patchwork
2022-09-24 2:01 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
4 siblings, 1 reply; 13+ messages in thread
From: Riana Tauro @ 2022-09-23 11:00 UTC (permalink / raw)
To: intel-gfx
A fundamental assumption is that at lower frequencies,
not only do we run slower, but we save power compared to
higher frequencies.
live_slpc_power checks if running at low frequency saves power
v2: re-use code to measure power
fixed cosmetic review comments (Vinay)
Signed-off-by: Riana Tauro <riana.tauro@intel.com>
---
drivers/gpu/drm/i915/gt/selftest_slpc.c | 127 ++++++++++++++++++++++--
1 file changed, 118 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c b/drivers/gpu/drm/i915/gt/selftest_slpc.c
index 928f74718881..4c6e9257e593 100644
--- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
+++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
@@ -11,7 +11,8 @@
enum test_type {
VARY_MIN,
VARY_MAX,
- MAX_GRANTED
+ MAX_GRANTED,
+ SLPC_POWER,
};
static int slpc_set_min_freq(struct intel_guc_slpc *slpc, u32 freq)
@@ -41,6 +42,39 @@ static int slpc_set_max_freq(struct intel_guc_slpc *slpc, u32 freq)
return ret;
}
+static int slpc_set_freq(struct intel_gt *gt, u32 freq)
+{
+ int err;
+ struct intel_guc_slpc *slpc = >->uc.guc.slpc;
+
+ err = slpc_set_max_freq(slpc, freq);
+ if (err) {
+ pr_err("Unable to update max freq");
+ return err;
+ }
+
+ err = slpc_set_min_freq(slpc, freq);
+ if (err) {
+ pr_err("Unable to update min freq");
+ return err;
+ }
+
+ return err;
+}
+
+static u64 measure_power_at_freq(struct intel_gt *gt, int *freq, u64 *power)
+{
+ int err = 0;
+
+ err = slpc_set_freq(gt, *freq);
+ if (err)
+ return err;
+ *freq = intel_rps_read_actual_frequency(>->rps);
+ *power = measure_power(>->rps, freq);
+
+ return err;
+}
+
static int vary_max_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
u32 *max_act_freq)
{
@@ -113,6 +147,58 @@ static int vary_min_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
return err;
}
+static int slpc_power(struct intel_gt *gt, struct intel_engine_cs *engine)
+{
+ struct intel_guc_slpc *slpc = >->uc.guc.slpc;
+ struct {
+ u64 power;
+ int freq;
+ } min, max;
+ int err = 0;
+
+ /*
+ * Our fundamental assumption is that running at lower frequency
+ * actually saves power. Let's see if our RAPL measurement supports
+ * that theory.
+ */
+ if (!librapl_supported(gt->i915))
+ return 0;
+
+ min.freq = slpc->min_freq;
+ err = measure_power_at_freq(gt, &min.freq, &min.power);
+
+ if (err)
+ return err;
+
+ max.freq = slpc->rp0_freq;
+ err = measure_power_at_freq(gt, &max.freq, &max.power);
+
+ if (err)
+ return err;
+
+ pr_info("%s: min:%llumW @ %uMHz, max:%llumW @ %uMHz\n",
+ engine->name,
+ min.power, min.freq,
+ max.power, max.freq);
+
+ if (10 * min.freq >= 9 * max.freq) {
+ pr_notice("Could not control frequency, ran at [%uMHz, %uMhz]\n",
+ min.freq, max.freq);
+ }
+
+ if (11 * min.power > 10 * max.power) {
+ pr_err("%s: did not conserve power when setting lower frequency!\n",
+ engine->name);
+ err = -EINVAL;
+ }
+
+ /* Restore min/max frequencies */
+ slpc_set_max_freq(slpc, slpc->rp0_freq);
+ slpc_set_min_freq(slpc, slpc->min_freq);
+
+ return err;
+}
+
static int max_granted_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps, u32 *max_act_freq)
{
struct intel_gt *gt = rps_to_gt(rps);
@@ -233,17 +319,23 @@ static int run_test(struct intel_gt *gt, int test_type)
err = max_granted_freq(slpc, rps, &max_act_freq);
break;
+
+ case SLPC_POWER:
+ err = slpc_power(gt, engine);
+ break;
}
- pr_info("Max actual frequency for %s was %d\n",
- engine->name, max_act_freq);
+ if (test_type != SLPC_POWER) {
+ pr_info("Max actual frequency for %s was %d\n",
+ engine->name, max_act_freq);
- /* Actual frequency should rise above min */
- if (max_act_freq <= slpc_min_freq) {
- pr_err("Actual freq did not rise above min\n");
- pr_err("Perf Limit Reasons: 0x%x\n",
- intel_uncore_read(gt->uncore, GT0_PERF_LIMIT_REASONS));
- err = -EINVAL;
+ /* Actual frequency should rise above min */
+ if (max_act_freq <= slpc_min_freq) {
+ pr_err("Actual freq did not rise above min\n");
+ pr_err("Perf Limit Reasons: 0x%x\n",
+ intel_uncore_read(gt->uncore, GT0_PERF_LIMIT_REASONS));
+ err = -EINVAL;
+ }
}
igt_spinner_end(&spin);
@@ -316,12 +408,29 @@ static int live_slpc_max_granted(void *arg)
return ret;
}
+static int live_slpc_power(void *arg)
+{
+ struct drm_i915_private *i915 = arg;
+ struct intel_gt *gt;
+ unsigned int i;
+ int ret;
+
+ for_each_gt(gt, i915, i) {
+ ret = run_test(gt, SLPC_POWER);
+ if (ret)
+ return ret;
+ }
+
+ return ret;
+}
+
int intel_slpc_live_selftests(struct drm_i915_private *i915)
{
static const struct i915_subtest tests[] = {
SUBTEST(live_slpc_vary_max),
SUBTEST(live_slpc_vary_min),
SUBTEST(live_slpc_max_granted),
+ SUBTEST(live_slpc_power),
};
struct intel_gt *gt;
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for Add SLPC selftest live_slpc_power (rev2)
2022-09-23 11:00 [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power Riana Tauro
` (2 preceding siblings ...)
2022-09-23 11:00 ` [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power Riana Tauro
@ 2022-09-23 13:44 ` Patchwork
2022-09-24 2:01 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
4 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-09-23 13:44 UTC (permalink / raw)
To: Riana Tauro; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 10135 bytes --]
== Series Details ==
Series: Add SLPC selftest live_slpc_power (rev2)
URL : https://patchwork.freedesktop.org/series/108900/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12174 -> Patchwork_108900v2
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/index.html
Participating hosts (42 -> 46)
------------------------------
Additional (5): fi-kbl-soraka fi-cml-u2 fi-bxt-dsi fi-icl-u2 fi-hsw-4770
Missing (1): fi-bdw-samus
Known issues
------------
Here are the changes found in Patchwork_108900v2 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_108900v2/fi-cml-u2/igt@gem_exec_fence@basic-busy@bcs0.html
* igt@gem_huc_copy@huc-copy:
- fi-icl-u2: NOTRUN -> [SKIP][2] ([i915#2190])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@gem_huc_copy@huc-copy.html
- fi-cml-u2: NOTRUN -> [SKIP][3] ([i915#2190])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@gem_huc_copy@huc-copy.html
- fi-bxt-dsi: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#2190])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-bxt-dsi/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@parallel-random-engines:
- fi-cml-u2: NOTRUN -> [SKIP][5] ([i915#4613]) +3 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@gem_lmem_swapping@parallel-random-engines.html
- fi-bxt-dsi: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#4613]) +3 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-bxt-dsi/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@gem_lmem_swapping@random-engines:
- fi-icl-u2: NOTRUN -> [SKIP][7] ([i915#4613]) +3 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@gem_lmem_swapping@random-engines.html
* igt@gem_softpin@allocator-basic-reserve:
- fi-hsw-4770: NOTRUN -> [SKIP][8] ([fdo#109271]) +9 similar issues
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-hsw-4770/igt@gem_softpin@allocator-basic-reserve.html
* igt@gem_tiled_blits@basic:
- fi-bxt-dsi: NOTRUN -> [SKIP][9] ([fdo#109271]) +12 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-bxt-dsi/igt@gem_tiled_blits@basic.html
* igt@i915_pm_backlight@basic-brightness:
- fi-hsw-4770: NOTRUN -> [SKIP][10] ([fdo#109271] / [i915#3012])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-hsw-4770/igt@i915_pm_backlight@basic-brightness.html
* igt@i915_selftest@live@gt_heartbeat:
- fi-bxt-dsi: NOTRUN -> [DMESG-FAIL][11] ([i915#5334])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-bxt-dsi/igt@i915_selftest@live@gt_heartbeat.html
* igt@kms_chamelium@dp-crc-fast:
- fi-hsw-4770: NOTRUN -> [SKIP][12] ([fdo#109271] / [fdo#111827]) +8 similar issues
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-hsw-4770/igt@kms_chamelium@dp-crc-fast.html
- fi-cml-u2: NOTRUN -> [SKIP][13] ([fdo#109284] / [fdo#111827]) +8 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@kms_chamelium@dp-crc-fast.html
* igt@kms_chamelium@hdmi-edid-read:
- fi-bxt-dsi: NOTRUN -> [SKIP][14] ([fdo#109271] / [fdo#111827]) +8 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-bxt-dsi/igt@kms_chamelium@hdmi-edid-read.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-icl-u2: NOTRUN -> [SKIP][15] ([fdo#111827]) +8 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor:
- fi-cml-u2: NOTRUN -> [SKIP][16] ([i915#4213])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html
- fi-icl-u2: NOTRUN -> [SKIP][17] ([i915#4103])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-icl-u2: NOTRUN -> [WARN][18] ([i915#6008])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/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][19] ([fdo#109285])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@kms_force_connector_basic@force-load-detect.html
- fi-icl-u2: NOTRUN -> [SKIP][20] ([fdo#109285])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_frontbuffer_tracking@basic:
- fi-cml-u2: NOTRUN -> [DMESG-WARN][21] ([i915#402])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html
* igt@kms_psr@sprite_plane_onoff:
- fi-hsw-4770: NOTRUN -> [SKIP][22] ([fdo#109271] / [i915#1072]) +3 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-hsw-4770/igt@kms_psr@sprite_plane_onoff.html
* igt@kms_setmode@basic-clone-single-crtc:
- fi-icl-u2: NOTRUN -> [SKIP][23] ([i915#3555])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@kms_setmode@basic-clone-single-crtc.html
- fi-cml-u2: NOTRUN -> [SKIP][24] ([i915#3555])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@kms_setmode@basic-clone-single-crtc.html
* igt@prime_vgem@basic-userptr:
- fi-icl-u2: NOTRUN -> [SKIP][25] ([fdo#109295] / [i915#3301])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-icl-u2/igt@prime_vgem@basic-userptr.html
- fi-cml-u2: NOTRUN -> [SKIP][26] ([fdo#109295] / [i915#3301])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-cml-u2/igt@prime_vgem@basic-userptr.html
* igt@runner@aborted:
- fi-kbl-soraka: NOTRUN -> [FAIL][27] ([i915#6641])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/fi-kbl-soraka/igt@runner@aborted.html
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s3@smem:
- {bat-rplp-1}: [DMESG-WARN][28] ([i915#2867]) -> [PASS][29]
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/bat-rplp-1/igt@gem_exec_suspend@basic-s3@smem.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/bat-rplp-1/igt@gem_exec_suspend@basic-s3@smem.html
- {bat-adlm-1}: [DMESG-WARN][30] ([i915#2867]) -> [PASS][31]
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/bat-adlm-1/igt@gem_exec_suspend@basic-s3@smem.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/bat-adlm-1/igt@gem_exec_suspend@basic-s3@smem.html
* igt@i915_selftest@live@hugepages:
- {bat-rpls-1}: [DMESG-WARN][32] ([i915#5278]) -> [PASS][33]
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/bat-rpls-1/igt@i915_selftest@live@hugepages.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/bat-rpls-1/igt@i915_selftest@live@hugepages.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#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#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
[i915#1208]: https://gitlab.freedesktop.org/drm/intel/issues/1208
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2867]: https://gitlab.freedesktop.org/drm/intel/issues/2867
[i915#3012]: https://gitlab.freedesktop.org/drm/intel/issues/3012
[i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
[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#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#5278]: https://gitlab.freedesktop.org/drm/intel/issues/5278
[i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
[i915#6008]: https://gitlab.freedesktop.org/drm/intel/issues/6008
[i915#6641]: https://gitlab.freedesktop.org/drm/intel/issues/6641
[i915#6818]: https://gitlab.freedesktop.org/drm/intel/issues/6818
Build changes
-------------
* Linux: CI_DRM_12174 -> Patchwork_108900v2
CI-20190529: 20190529
CI_DRM_12174: 3a5c0f391d00f80d1aa4d766511186412b338b74 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6662: dcb1d7a8822e62935f4fe3f2e6a04caaee669369 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_108900v2: 3a5c0f391d00f80d1aa4d766511186412b338b74 @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
3602b674f65b drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
a18a5f718629 drm/i915/selftests: Add helper function measure_power
e910e0659a45 drm/i915/guc/slpc: Run SLPC selftests on all tiles
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/index.html
[-- Attachment #2: Type: text/html, Size: 12657 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for Add SLPC selftest live_slpc_power (rev2)
2022-09-23 11:00 [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power Riana Tauro
` (3 preceding siblings ...)
2022-09-23 13:44 ` [Intel-gfx] ✓ Fi.CI.BAT: success for Add SLPC selftest live_slpc_power (rev2) Patchwork
@ 2022-09-24 2:01 ` Patchwork
2022-09-29 7:16 ` Gupta, Anshuman
4 siblings, 1 reply; 13+ messages in thread
From: Patchwork @ 2022-09-24 2:01 UTC (permalink / raw)
To: Riana Tauro; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 17269 bytes --]
== Series Details ==
Series: Add SLPC selftest live_slpc_power (rev2)
URL : https://patchwork.freedesktop.org/series/108900/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_12174_full -> Patchwork_108900v2_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (10 -> 9)
------------------------------
Missing (1): shard-tglu
Known issues
------------
Here are the changes found in Patchwork_108900v2_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_balancer@parallel-out-fence:
- shard-iclb: [PASS][1] -> [SKIP][2] ([i915#4525]) +2 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb4/igt@gem_exec_balancer@parallel-out-fence.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb5/igt@gem_exec_balancer@parallel-out-fence.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-tglb: [PASS][3] -> [FAIL][4] ([i915#2842])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gem_exec_fair@basic-pace-solo@rcs0:
- shard-apl: [PASS][5] -> [FAIL][6] ([i915#2842])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl6/igt@gem_exec_fair@basic-pace-solo@rcs0.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@gem_exec_fair@basic-pace-solo@rcs0.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-glk: [PASS][7] -> [FAIL][8] ([i915#2842])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk5/igt@gem_exec_fair@basic-throttle@rcs0.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk8/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gen9_exec_parse@allowed-single:
- shard-glk: [PASS][9] -> [DMESG-WARN][10] ([i915#5566] / [i915#716])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk2/igt@gen9_exec_parse@allowed-single.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk9/igt@gen9_exec_parse@allowed-single.html
* igt@i915_pm_dc@dc6-psr:
- shard-iclb: [PASS][11] -> [FAIL][12] ([i915#3989] / [i915#454])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb5/igt@i915_pm_dc@dc6-psr.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb3/igt@i915_pm_dc@dc6-psr.html
* igt@i915_suspend@sysfs-reader:
- shard-apl: NOTRUN -> [DMESG-WARN][13] ([i915#180])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@i915_suspend@sysfs-reader.html
* igt@kms_big_fb@linear-32bpp-rotate-270:
- shard-apl: NOTRUN -> [SKIP][14] ([fdo#109271]) +43 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_big_fb@linear-32bpp-rotate-270.html
* igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc:
- shard-apl: NOTRUN -> [SKIP][15] ([fdo#109271] / [i915#3886]) +2 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_color_chamelium@ctm-red-to-blue:
- shard-apl: NOTRUN -> [SKIP][16] ([fdo#109271] / [fdo#111827]) +1 similar issue
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_color_chamelium@ctm-red-to-blue.html
* igt@kms_flip@flip-vs-suspend-interruptible@a-dp1:
- shard-apl: [PASS][17] -> [DMESG-WARN][18] ([i915#180])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl2/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html
* igt@kms_flip@flip-vs-suspend@a-dp1:
- shard-apl: [PASS][19] -> [DMESG-WARN][20] ([i915#180] / [i915#1982])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl7/igt@kms_flip@flip-vs-suspend@a-dp1.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl8/igt@kms_flip@flip-vs-suspend@a-dp1.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-downscaling@pipe-a-default-mode:
- shard-iclb: NOTRUN -> [SKIP][21] ([i915#2672] / [i915#3555])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-downscaling@pipe-a-default-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-valid-mode:
- shard-iclb: NOTRUN -> [SKIP][22] ([i915#2587] / [i915#2672]) +2 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb4/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-upscaling@pipe-a-default-mode:
- shard-iclb: NOTRUN -> [SKIP][23] ([i915#2672]) +2 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-upscaling@pipe-a-default-mode.html
* igt@kms_frontbuffer_tracking@fbc-1p-shrfb-fliptrack-mmap-gtt:
- shard-iclb: [PASS][24] -> [FAIL][25] ([i915#1888] / [i915#2546])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-shrfb-fliptrack-mmap-gtt.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-1p-shrfb-fliptrack-mmap-gtt.html
* igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1:
- shard-iclb: [PASS][26] -> [SKIP][27] ([i915#5235]) +2 similar issues
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb8/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html
* igt@kms_psr2_sf@overlay-plane-move-continuous-sf:
- shard-apl: NOTRUN -> [SKIP][28] ([fdo#109271] / [i915#658]) +2 similar issues
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_psr2_sf@overlay-plane-move-continuous-sf.html
* igt@kms_psr2_su@frontbuffer-xrgb8888:
- shard-iclb: [PASS][29] -> [SKIP][30] ([fdo#109642] / [fdo#111068] / [i915#658])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr2_su@frontbuffer-xrgb8888.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr2_su@frontbuffer-xrgb8888.html
* igt@kms_psr@psr2_primary_blt:
- shard-iclb: [PASS][31] -> [SKIP][32] ([fdo#109441])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr@psr2_primary_blt.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr@psr2_primary_blt.html
* igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
- shard-tglb: [PASS][33] -> [SKIP][34] ([i915#5519])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-tglb6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-tglb6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html
* igt@sysfs_clients@split-10:
- shard-apl: NOTRUN -> [SKIP][35] ([fdo#109271] / [i915#2994])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@sysfs_clients@split-10.html
#### Possible fixes ####
* igt@feature_discovery@psr2:
- shard-iclb: [SKIP][36] ([i915#658]) -> [PASS][37]
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb5/igt@feature_discovery@psr2.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@feature_discovery@psr2.html
* igt@gem_exec_balancer@parallel-bb-first:
- shard-iclb: [SKIP][38] ([i915#4525]) -> [PASS][39]
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb6/igt@gem_exec_balancer@parallel-bb-first.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb1/igt@gem_exec_balancer@parallel-bb-first.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-glk: [FAIL][40] ([i915#2842]) -> [PASS][41]
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk7/igt@gem_exec_fair@basic-none-share@rcs0.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk6/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none-solo@rcs0:
- shard-apl: [FAIL][42] ([i915#2842]) -> [PASS][43] +1 similar issue
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl1/igt@gem_exec_fair@basic-none-solo@rcs0.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl2/igt@gem_exec_fair@basic-none-solo@rcs0.html
* igt@gem_exec_fair@basic-pace@rcs0:
- shard-iclb: [FAIL][44] ([i915#2842]) -> [PASS][45]
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb1/igt@gem_exec_fair@basic-pace@rcs0.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb7/igt@gem_exec_fair@basic-pace@rcs0.html
* igt@gem_huc_copy@huc-copy:
- shard-tglb: [SKIP][46] ([i915#2190]) -> [PASS][47]
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-tglb7/igt@gem_huc_copy@huc-copy.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-tglb2/igt@gem_huc_copy@huc-copy.html
* igt@i915_pm_rps@engine-order:
- shard-apl: [FAIL][48] ([i915#6537]) -> [PASS][49]
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl3/igt@i915_pm_rps@engine-order.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl7/igt@i915_pm_rps@engine-order.html
* igt@kms_flip@flip-vs-suspend-interruptible@b-dp1:
- shard-apl: [DMESG-WARN][50] ([i915#180]) -> [PASS][51]
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl2/igt@kms_flip@flip-vs-suspend-interruptible@b-dp1.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible@b-dp1.html
* igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1:
- shard-iclb: [SKIP][52] ([i915#5235]) -> [PASS][53] +2 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1.html
* igt@kms_psr@psr2_sprite_blt:
- shard-iclb: [SKIP][54] ([fdo#109441]) -> [PASS][55] +1 similar issue
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb8/igt@kms_psr@psr2_sprite_blt.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html
* igt@kms_vblank@pipe-a-accuracy-idle:
- shard-apl: [DMESG-WARN][56] ([i915#62]) -> [PASS][57] +41 similar issues
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl8/igt@kms_vblank@pipe-a-accuracy-idle.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl2/igt@kms_vblank@pipe-a-accuracy-idle.html
* igt@perf@polling-parameterized:
- shard-glk: [FAIL][58] ([i915#5639]) -> [PASS][59]
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk5/igt@perf@polling-parameterized.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk9/igt@perf@polling-parameterized.html
* igt@perf_pmu@module-unload:
- shard-apl: [INCOMPLETE][60] ([i915#1982]) -> [PASS][61]
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl8/igt@perf_pmu@module-unload.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl2/igt@perf_pmu@module-unload.html
#### Warnings ####
* igt@gem_exec_balancer@parallel-ordering:
- shard-iclb: [SKIP][62] ([i915#4525]) -> [FAIL][63] ([i915#6117])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb3/igt@gem_exec_balancer@parallel-ordering.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb4/igt@gem_exec_balancer@parallel-ordering.html
* igt@i915_pm_rc6_residency@rc6-idle@rcs0:
- shard-iclb: [FAIL][64] ([i915#2684]) -> [WARN][65] ([i915#2684])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb6/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb8/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html
* igt@kms_psr2_sf@cursor-plane-move-continuous-sf:
- shard-iclb: [SKIP][66] ([i915#2920]) -> [SKIP][67] ([i915#658])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr2_sf@cursor-plane-move-continuous-sf.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr2_sf@cursor-plane-move-continuous-sf.html
* igt@kms_psr2_sf@cursor-plane-update-sf:
- shard-iclb: [SKIP][68] ([i915#2920]) -> [SKIP][69] ([fdo#111068] / [i915#658])
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr2_sf@cursor-plane-update-sf.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr2_sf@cursor-plane-update-sf.html
* igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf:
- shard-iclb: [SKIP][70] ([i915#658]) -> [SKIP][71] ([i915#2920])
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb5/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html
[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#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#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2546]: https://gitlab.freedesktop.org/drm/intel/issues/2546
[i915#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920
[i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
[i915#3989]: https://gitlab.freedesktop.org/drm/intel/issues/3989
[i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
[i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
[i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
[i915#5519]: https://gitlab.freedesktop.org/drm/intel/issues/5519
[i915#5566]: https://gitlab.freedesktop.org/drm/intel/issues/5566
[i915#5639]: https://gitlab.freedesktop.org/drm/intel/issues/5639
[i915#6117]: https://gitlab.freedesktop.org/drm/intel/issues/6117
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#6537]: https://gitlab.freedesktop.org/drm/intel/issues/6537
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#716]: https://gitlab.freedesktop.org/drm/intel/issues/716
Build changes
-------------
* Linux: CI_DRM_12174 -> Patchwork_108900v2
CI-20190529: 20190529
CI_DRM_12174: 3a5c0f391d00f80d1aa4d766511186412b338b74 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6662: dcb1d7a8822e62935f4fe3f2e6a04caaee669369 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_108900v2: 3a5c0f391d00f80d1aa4d766511186412b338b74 @ 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_108900v2/index.html
[-- Attachment #2: Type: text/html, Size: 20709 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [PATCH 2/3] drm/i915/selftests: Add helper function measure_power
2022-09-23 11:00 ` [Intel-gfx] [PATCH 2/3] drm/i915/selftests: Add helper function measure_power Riana Tauro
@ 2022-09-26 15:57 ` Belgaumkar, Vinay
0 siblings, 0 replies; 13+ messages in thread
From: Belgaumkar, Vinay @ 2022-09-26 15:57 UTC (permalink / raw)
To: Riana Tauro, intel-gfx
On 9/23/2022 4:00 AM, Riana Tauro wrote:
> move the power measurement and the triangle filter
> to a different function. No functional changes.
>
> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
LGTM,
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> ---
> drivers/gpu/drm/i915/gt/selftest_rps.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/selftest_rps.c b/drivers/gpu/drm/i915/gt/selftest_rps.c
> index cfb4708dd62e..99a372486fb7 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_rps.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_rps.c
> @@ -1107,21 +1107,27 @@ static u64 __measure_power(int duration_ms)
> return div64_u64(1000 * 1000 * dE, dt);
> }
>
> -static u64 measure_power_at(struct intel_rps *rps, int *freq)
> +static u64 measure_power(struct intel_rps *rps, int *freq)
> {
> u64 x[5];
> int i;
>
> - *freq = rps_set_check(rps, *freq);
> for (i = 0; i < 5; i++)
> x[i] = __measure_power(5);
> - *freq = (*freq + read_cagf(rps)) / 2;
> +
> + *freq = (*freq + intel_rps_read_actual_frequency(rps)) / 2;
>
> /* A simple triangle filter for better result stability */
> sort(x, 5, sizeof(*x), cmp_u64, NULL);
> return div_u64(x[1] + 2 * x[2] + x[3], 4);
> }
>
> +static u64 measure_power_at(struct intel_rps *rps, int *freq)
> +{
> + *freq = rps_set_check(rps, *freq);
> + return measure_power(rps, freq);
> +}
> +
> int live_rps_power(void *arg)
> {
> struct intel_gt *gt = arg;
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles
2022-09-23 11:00 ` [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles Riana Tauro
@ 2022-09-26 16:02 ` Belgaumkar, Vinay
0 siblings, 0 replies; 13+ messages in thread
From: Belgaumkar, Vinay @ 2022-09-26 16:02 UTC (permalink / raw)
To: Riana Tauro, intel-gfx
On 9/23/2022 4:00 AM, Riana Tauro wrote:
> Run slpc selftests on all tiles
>
> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
LGTM,
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> ---
> drivers/gpu/drm/i915/gt/selftest_slpc.c | 45 ++++++++++++++++++++-----
> 1 file changed, 37 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> index f8a1d27df272..928f74718881 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> @@ -270,26 +270,50 @@ static int run_test(struct intel_gt *gt, int test_type)
> static int live_slpc_vary_min(void *arg)
> {
> struct drm_i915_private *i915 = arg;
> - struct intel_gt *gt = to_gt(i915);
> + struct intel_gt *gt;
> + unsigned int i;
> + int ret;
> +
> + for_each_gt(gt, i915, i) {
> + ret = run_test(gt, VARY_MIN);
> + if (ret)
> + return ret;
> + }
>
> - return run_test(gt, VARY_MIN);
> + return ret;
> }
>
> static int live_slpc_vary_max(void *arg)
> {
> struct drm_i915_private *i915 = arg;
> - struct intel_gt *gt = to_gt(i915);
> + struct intel_gt *gt;
> + unsigned int i;
> + int ret;
> +
> + for_each_gt(gt, i915, i) {
> + ret = run_test(gt, VARY_MAX);
> + if (ret)
> + return ret;
> + }
>
> - return run_test(gt, VARY_MAX);
> + return ret;
> }
>
> /* check if pcode can grant RP0 */
> static int live_slpc_max_granted(void *arg)
> {
> struct drm_i915_private *i915 = arg;
> - struct intel_gt *gt = to_gt(i915);
> + struct intel_gt *gt;
> + unsigned int i;
> + int ret;
> +
> + for_each_gt(gt, i915, i) {
> + ret = run_test(gt, MAX_GRANTED);
> + if (ret)
> + return ret;
> + }
>
> - return run_test(gt, MAX_GRANTED);
> + return ret;
> }
>
> int intel_slpc_live_selftests(struct drm_i915_private *i915)
> @@ -300,8 +324,13 @@ int intel_slpc_live_selftests(struct drm_i915_private *i915)
> SUBTEST(live_slpc_max_granted),
> };
>
> - if (intel_gt_is_wedged(to_gt(i915)))
> - return 0;
> + struct intel_gt *gt;
> + unsigned int i;
> +
> + for_each_gt(gt, i915, i) {
> + if (intel_gt_is_wedged(gt))
> + return 0;
> + }
>
> return i915_live_subtests(tests, i915);
> }
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
2022-09-23 11:00 ` [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power Riana Tauro
@ 2022-09-26 16:04 ` Belgaumkar, Vinay
2022-09-27 11:12 ` Gupta, Anshuman
0 siblings, 1 reply; 13+ messages in thread
From: Belgaumkar, Vinay @ 2022-09-26 16:04 UTC (permalink / raw)
To: Riana Tauro, intel-gfx
On 9/23/2022 4:00 AM, Riana Tauro wrote:
> A fundamental assumption is that at lower frequencies,
> not only do we run slower, but we save power compared to
> higher frequencies.
> live_slpc_power checks if running at low frequency saves power
>
> v2: re-use code to measure power
> fixed cosmetic review comments (Vinay)
>
> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
LGTM,
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> ---
> drivers/gpu/drm/i915/gt/selftest_slpc.c | 127 ++++++++++++++++++++++--
> 1 file changed, 118 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> index 928f74718881..4c6e9257e593 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> @@ -11,7 +11,8 @@
> enum test_type {
> VARY_MIN,
> VARY_MAX,
> - MAX_GRANTED
> + MAX_GRANTED,
> + SLPC_POWER,
> };
>
> static int slpc_set_min_freq(struct intel_guc_slpc *slpc, u32 freq)
> @@ -41,6 +42,39 @@ static int slpc_set_max_freq(struct intel_guc_slpc *slpc, u32 freq)
> return ret;
> }
>
> +static int slpc_set_freq(struct intel_gt *gt, u32 freq)
> +{
> + int err;
> + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
> +
> + err = slpc_set_max_freq(slpc, freq);
> + if (err) {
> + pr_err("Unable to update max freq");
> + return err;
> + }
> +
> + err = slpc_set_min_freq(slpc, freq);
> + if (err) {
> + pr_err("Unable to update min freq");
> + return err;
> + }
> +
> + return err;
> +}
> +
> +static u64 measure_power_at_freq(struct intel_gt *gt, int *freq, u64 *power)
> +{
> + int err = 0;
> +
> + err = slpc_set_freq(gt, *freq);
> + if (err)
> + return err;
> + *freq = intel_rps_read_actual_frequency(>->rps);
> + *power = measure_power(>->rps, freq);
> +
> + return err;
> +}
> +
> static int vary_max_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
> u32 *max_act_freq)
> {
> @@ -113,6 +147,58 @@ static int vary_min_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
> return err;
> }
>
> +static int slpc_power(struct intel_gt *gt, struct intel_engine_cs *engine)
> +{
> + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
> + struct {
> + u64 power;
> + int freq;
> + } min, max;
> + int err = 0;
> +
> + /*
> + * Our fundamental assumption is that running at lower frequency
> + * actually saves power. Let's see if our RAPL measurement supports
> + * that theory.
> + */
> + if (!librapl_supported(gt->i915))
> + return 0;
> +
> + min.freq = slpc->min_freq;
> + err = measure_power_at_freq(gt, &min.freq, &min.power);
> +
> + if (err)
> + return err;
> +
> + max.freq = slpc->rp0_freq;
> + err = measure_power_at_freq(gt, &max.freq, &max.power);
> +
> + if (err)
> + return err;
> +
> + pr_info("%s: min:%llumW @ %uMHz, max:%llumW @ %uMHz\n",
> + engine->name,
> + min.power, min.freq,
> + max.power, max.freq);
> +
> + if (10 * min.freq >= 9 * max.freq) {
> + pr_notice("Could not control frequency, ran at [%uMHz, %uMhz]\n",
> + min.freq, max.freq);
> + }
> +
> + if (11 * min.power > 10 * max.power) {
> + pr_err("%s: did not conserve power when setting lower frequency!\n",
> + engine->name);
> + err = -EINVAL;
> + }
> +
> + /* Restore min/max frequencies */
> + slpc_set_max_freq(slpc, slpc->rp0_freq);
> + slpc_set_min_freq(slpc, slpc->min_freq);
> +
> + return err;
> +}
> +
> static int max_granted_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps, u32 *max_act_freq)
> {
> struct intel_gt *gt = rps_to_gt(rps);
> @@ -233,17 +319,23 @@ static int run_test(struct intel_gt *gt, int test_type)
>
> err = max_granted_freq(slpc, rps, &max_act_freq);
> break;
> +
> + case SLPC_POWER:
> + err = slpc_power(gt, engine);
> + break;
> }
>
> - pr_info("Max actual frequency for %s was %d\n",
> - engine->name, max_act_freq);
> + if (test_type != SLPC_POWER) {
> + pr_info("Max actual frequency for %s was %d\n",
> + engine->name, max_act_freq);
>
> - /* Actual frequency should rise above min */
> - if (max_act_freq <= slpc_min_freq) {
> - pr_err("Actual freq did not rise above min\n");
> - pr_err("Perf Limit Reasons: 0x%x\n",
> - intel_uncore_read(gt->uncore, GT0_PERF_LIMIT_REASONS));
> - err = -EINVAL;
> + /* Actual frequency should rise above min */
> + if (max_act_freq <= slpc_min_freq) {
> + pr_err("Actual freq did not rise above min\n");
> + pr_err("Perf Limit Reasons: 0x%x\n",
> + intel_uncore_read(gt->uncore, GT0_PERF_LIMIT_REASONS));
> + err = -EINVAL;
> + }
> }
>
> igt_spinner_end(&spin);
> @@ -316,12 +408,29 @@ static int live_slpc_max_granted(void *arg)
> return ret;
> }
>
> +static int live_slpc_power(void *arg)
> +{
> + struct drm_i915_private *i915 = arg;
> + struct intel_gt *gt;
> + unsigned int i;
> + int ret;
> +
> + for_each_gt(gt, i915, i) {
> + ret = run_test(gt, SLPC_POWER);
> + if (ret)
> + return ret;
> + }
> +
> + return ret;
> +}
> +
> int intel_slpc_live_selftests(struct drm_i915_private *i915)
> {
> static const struct i915_subtest tests[] = {
> SUBTEST(live_slpc_vary_max),
> SUBTEST(live_slpc_vary_min),
> SUBTEST(live_slpc_max_granted),
> + SUBTEST(live_slpc_power),
> };
>
> struct intel_gt *gt;
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
2022-09-26 16:04 ` Belgaumkar, Vinay
@ 2022-09-27 11:12 ` Gupta, Anshuman
2022-09-28 5:57 ` Tauro, Riana
0 siblings, 1 reply; 13+ messages in thread
From: Gupta, Anshuman @ 2022-09-27 11:12 UTC (permalink / raw)
To: Belgaumkar, Vinay, Tauro, Riana, intel-gfx
> -----Original Message-----
> From: Belgaumkar, Vinay <vinay.belgaumkar@intel.com>
> Sent: Monday, September 26, 2022 9:35 PM
> To: Tauro, Riana <riana.tauro@intel.com>; intel-gfx@lists.freedesktop.org
> Cc: Gupta, Anshuman <anshuman.gupta@intel.com>; Dixit, Ashutosh
> <ashutosh.dixit@intel.com>
> Subject: Re: [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
>
>
> On 9/23/2022 4:00 AM, Riana Tauro wrote:
> > A fundamental assumption is that at lower frequencies, not only do we
> > run slower, but we save power compared to higher frequencies.
> > live_slpc_power checks if running at low frequency saves power
> >
> > v2: re-use code to measure power
> > fixed cosmetic review comments (Vinay)
> >
> > Signed-off-by: Riana Tauro <riana.tauro@intel.com>
>
> LGTM,
>
> Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
>
> > ---
> > drivers/gpu/drm/i915/gt/selftest_slpc.c | 127 ++++++++++++++++++++++--
> > 1 file changed, 118 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c
> > b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> > index 928f74718881..4c6e9257e593 100644
> > --- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
> > +++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> > @@ -11,7 +11,8 @@
> > enum test_type {
> > VARY_MIN,
> > VARY_MAX,
> > - MAX_GRANTED
> > + MAX_GRANTED,
> > + SLPC_POWER,
> > };
> >
> > static int slpc_set_min_freq(struct intel_guc_slpc *slpc, u32 freq)
> > @@ -41,6 +42,39 @@ static int slpc_set_max_freq(struct intel_guc_slpc *slpc,
> u32 freq)
> > return ret;
> > }
> >
> > +static int slpc_set_freq(struct intel_gt *gt, u32 freq) {
> > + int err;
> > + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
> > +
> > + err = slpc_set_max_freq(slpc, freq);
> > + if (err) {
> > + pr_err("Unable to update max freq");
> > + return err;
> > + }
> > +
> > + err = slpc_set_min_freq(slpc, freq);
> > + if (err) {
> > + pr_err("Unable to update min freq");
> > + return err;
> > + }
> > +
> > + return err;
> > +}
> > +
> > +static u64 measure_power_at_freq(struct intel_gt *gt, int *freq, u64
> > +*power) {
> > + int err = 0;
> > +
> > + err = slpc_set_freq(gt, *freq);
> > + if (err)
> > + return err;
> > + *freq = intel_rps_read_actual_frequency(>->rps);
> > + *power = measure_power(>->rps, freq);
> > +
> > + return err;
> > +}
> > +
> > static int vary_max_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
> > u32 *max_act_freq)
> > {
> > @@ -113,6 +147,58 @@ static int vary_min_freq(struct intel_guc_slpc *slpc,
> struct intel_rps *rps,
> > return err;
> > }
> >
> > +static int slpc_power(struct intel_gt *gt, struct intel_engine_cs
> > +*engine) {
> > + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
> > + struct {
> > + u64 power;
> > + int freq;
> > + } min, max;
> > + int err = 0;
> > +
> > + /*
> > + * Our fundamental assumption is that running at lower frequency
> > + * actually saves power. Let's see if our RAPL measurement supports
> > + * that theory.
> > + */
> > + if (!librapl_supported(gt->i915))
> > + return 0;
This seems a wrong abstraction, this should a generic call should check both hwmon registration for dgfx and rapl for igfx.
Br,
Anshuman Gupta.
> > +
> > + min.freq = slpc->min_freq;
> > + err = measure_power_at_freq(gt, &min.freq, &min.power);
> > +
> > + if (err)
> > + return err;
> > +
> > + max.freq = slpc->rp0_freq;
> > + err = measure_power_at_freq(gt, &max.freq, &max.power);
> > +
> > + if (err)
> > + return err;
> > +
> > + pr_info("%s: min:%llumW @ %uMHz, max:%llumW @ %uMHz\n",
> > + engine->name,
> > + min.power, min.freq,
> > + max.power, max.freq);
> > +
> > + if (10 * min.freq >= 9 * max.freq) {
> > + pr_notice("Could not control frequency, ran at [%uMHz,
> %uMhz]\n",
> > + min.freq, max.freq);
> > + }
> > +
> > + if (11 * min.power > 10 * max.power) {
> > + pr_err("%s: did not conserve power when setting lower
> frequency!\n",
> > + engine->name);
> > + err = -EINVAL;
> > + }
> > +
> > + /* Restore min/max frequencies */
> > + slpc_set_max_freq(slpc, slpc->rp0_freq);
> > + slpc_set_min_freq(slpc, slpc->min_freq);
> > +
> > + return err;
> > +}
> > +
> > static int max_granted_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
> u32 *max_act_freq)
> > {
> > struct intel_gt *gt = rps_to_gt(rps); @@ -233,17 +319,23 @@ static
> > int run_test(struct intel_gt *gt, int test_type)
> >
> > err = max_granted_freq(slpc, rps, &max_act_freq);
> > break;
> > +
> > + case SLPC_POWER:
> > + err = slpc_power(gt, engine);
> > + break;
> > }
> >
> > - pr_info("Max actual frequency for %s was %d\n",
> > - engine->name, max_act_freq);
> > + if (test_type != SLPC_POWER) {
> > + pr_info("Max actual frequency for %s was %d\n",
> > + engine->name, max_act_freq);
> >
> > - /* Actual frequency should rise above min */
> > - if (max_act_freq <= slpc_min_freq) {
> > - pr_err("Actual freq did not rise above min\n");
> > - pr_err("Perf Limit Reasons: 0x%x\n",
> > - intel_uncore_read(gt->uncore,
> GT0_PERF_LIMIT_REASONS));
> > - err = -EINVAL;
> > + /* Actual frequency should rise above min */
> > + if (max_act_freq <= slpc_min_freq) {
> > + pr_err("Actual freq did not rise above min\n");
> > + pr_err("Perf Limit Reasons: 0x%x\n",
> > + intel_uncore_read(gt->uncore,
> GT0_PERF_LIMIT_REASONS));
> > + err = -EINVAL;
> > + }
> > }
> >
> > igt_spinner_end(&spin);
> > @@ -316,12 +408,29 @@ static int live_slpc_max_granted(void *arg)
> > return ret;
> > }
> >
> > +static int live_slpc_power(void *arg) {
> > + struct drm_i915_private *i915 = arg;
> > + struct intel_gt *gt;
> > + unsigned int i;
> > + int ret;
> > +
> > + for_each_gt(gt, i915, i) {
> > + ret = run_test(gt, SLPC_POWER);
> > + if (ret)
> > + return ret;
> > + }
> > +
> > + return ret;
> > +}
> > +
> > int intel_slpc_live_selftests(struct drm_i915_private *i915)
> > {
> > static const struct i915_subtest tests[] = {
> > SUBTEST(live_slpc_vary_max),
> > SUBTEST(live_slpc_vary_min),
> > SUBTEST(live_slpc_max_granted),
> > + SUBTEST(live_slpc_power),
> > };
> >
> > struct intel_gt *gt;
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
2022-09-27 11:12 ` Gupta, Anshuman
@ 2022-09-28 5:57 ` Tauro, Riana
2022-09-29 5:46 ` Gupta, Anshuman
0 siblings, 1 reply; 13+ messages in thread
From: Tauro, Riana @ 2022-09-28 5:57 UTC (permalink / raw)
To: Gupta, Anshuman, Belgaumkar, Vinay, intel-gfx
On 9/27/2022 4:42 PM, Gupta, Anshuman wrote:
>
>
>> -----Original Message-----
>> From: Belgaumkar, Vinay <vinay.belgaumkar@intel.com>
>> Sent: Monday, September 26, 2022 9:35 PM
>> To: Tauro, Riana <riana.tauro@intel.com>; intel-gfx@lists.freedesktop.org
>> Cc: Gupta, Anshuman <anshuman.gupta@intel.com>; Dixit, Ashutosh
>> <ashutosh.dixit@intel.com>
>> Subject: Re: [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
>>
>>
>> On 9/23/2022 4:00 AM, Riana Tauro wrote:
>>> A fundamental assumption is that at lower frequencies, not only do we
>>> run slower, but we save power compared to higher frequencies.
>>> live_slpc_power checks if running at low frequency saves power
>>>
>>> v2: re-use code to measure power
>>> fixed cosmetic review comments (Vinay)
>>>
>>> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
>>
>> LGTM,
>>
>> Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
>>
>>> ---
>>> drivers/gpu/drm/i915/gt/selftest_slpc.c | 127 ++++++++++++++++++++++--
>>> 1 file changed, 118 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c
>>> b/drivers/gpu/drm/i915/gt/selftest_slpc.c
>>> index 928f74718881..4c6e9257e593 100644
>>> --- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
>>> +++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
>>> @@ -11,7 +11,8 @@
>>> enum test_type {
>>> VARY_MIN,
>>> VARY_MAX,
>>> - MAX_GRANTED
>>> + MAX_GRANTED,
>>> + SLPC_POWER,
>>> };
>>>
>>> static int slpc_set_min_freq(struct intel_guc_slpc *slpc, u32 freq)
>>> @@ -41,6 +42,39 @@ static int slpc_set_max_freq(struct intel_guc_slpc *slpc,
>> u32 freq)
>>> return ret;
>>> }
>>>
>>> +static int slpc_set_freq(struct intel_gt *gt, u32 freq) {
>>> + int err;
>>> + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
>>> +
>>> + err = slpc_set_max_freq(slpc, freq);
>>> + if (err) {
>>> + pr_err("Unable to update max freq");
>>> + return err;
>>> + }
>>> +
>>> + err = slpc_set_min_freq(slpc, freq);
>>> + if (err) {
>>> + pr_err("Unable to update min freq");
>>> + return err;
>>> + }
>>> +
>>> + return err;
>>> +}
>>> +
>>> +static u64 measure_power_at_freq(struct intel_gt *gt, int *freq, u64
>>> +*power) {
>>> + int err = 0;
>>> +
>>> + err = slpc_set_freq(gt, *freq);
>>> + if (err)
>>> + return err;
>>> + *freq = intel_rps_read_actual_frequency(>->rps);
>>> + *power = measure_power(>->rps, freq);
>>> +
>>> + return err;
>>> +}
>>> +
>>> static int vary_max_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
>>> u32 *max_act_freq)
>>> {
>>> @@ -113,6 +147,58 @@ static int vary_min_freq(struct intel_guc_slpc *slpc,
>> struct intel_rps *rps,
>>> return err;
>>> }
>>>
>>> +static int slpc_power(struct intel_gt *gt, struct intel_engine_cs
>>> +*engine) {
>>> + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
>>> + struct {
>>> + u64 power;
>>> + int freq;
>>> + } min, max;
>>> + int err = 0;
>>> +
>>> + /*
>>> + * Our fundamental assumption is that running at lower frequency
>>> + * actually saves power. Let's see if our RAPL measurement supports
>>> + * that theory.
>>> + */
>>> + if (!librapl_supported(gt->i915))
>>> + return 0;
> This seems a wrong abstraction, this should a generic call should check both hwmon registration for dgfx and rapl for igfx.
> Br,
> Anshuman Gupta.
The current librapl_supported has only rapl related changes. The hwmon
energy is yet to be added.
Will change the name with the hwmon patch
Thanks
Riana Tauro
>>> +
>>> + min.freq = slpc->min_freq;
>>> + err = measure_power_at_freq(gt, &min.freq, &min.power);
>>> +
>>> + if (err)
>>> + return err;
>>> +
>>> + max.freq = slpc->rp0_freq;
>>> + err = measure_power_at_freq(gt, &max.freq, &max.power);
>>> +
>>> + if (err)
>>> + return err;
>>> +
>>> + pr_info("%s: min:%llumW @ %uMHz, max:%llumW @ %uMHz\n",
>>> + engine->name,
>>> + min.power, min.freq,
>>> + max.power, max.freq);
>>> +
>>> + if (10 * min.freq >= 9 * max.freq) {
>>> + pr_notice("Could not control frequency, ran at [%uMHz,
>> %uMhz]\n",
>>> + min.freq, max.freq);
>>> + }
>>> +
>>> + if (11 * min.power > 10 * max.power) {
>>> + pr_err("%s: did not conserve power when setting lower
>> frequency!\n",
>>> + engine->name);
>>> + err = -EINVAL;
>>> + }
>>> +
>>> + /* Restore min/max frequencies */
>>> + slpc_set_max_freq(slpc, slpc->rp0_freq);
>>> + slpc_set_min_freq(slpc, slpc->min_freq);
>>> +
>>> + return err;
>>> +}
>>> +
>>> static int max_granted_freq(struct intel_guc_slpc *slpc, struct intel_rps *rps,
>> u32 *max_act_freq)
>>> {
>>> struct intel_gt *gt = rps_to_gt(rps); @@ -233,17 +319,23 @@ static
>>> int run_test(struct intel_gt *gt, int test_type)
>>>
>>> err = max_granted_freq(slpc, rps, &max_act_freq);
>>> break;
>>> +
>>> + case SLPC_POWER:
>>> + err = slpc_power(gt, engine);
>>> + break;
>>> }
>>>
>>> - pr_info("Max actual frequency for %s was %d\n",
>>> - engine->name, max_act_freq);
>>> + if (test_type != SLPC_POWER) {
>>> + pr_info("Max actual frequency for %s was %d\n",
>>> + engine->name, max_act_freq);
>>>
>>> - /* Actual frequency should rise above min */
>>> - if (max_act_freq <= slpc_min_freq) {
>>> - pr_err("Actual freq did not rise above min\n");
>>> - pr_err("Perf Limit Reasons: 0x%x\n",
>>> - intel_uncore_read(gt->uncore,
>> GT0_PERF_LIMIT_REASONS));
>>> - err = -EINVAL;
>>> + /* Actual frequency should rise above min */
>>> + if (max_act_freq <= slpc_min_freq) {
>>> + pr_err("Actual freq did not rise above min\n");
>>> + pr_err("Perf Limit Reasons: 0x%x\n",
>>> + intel_uncore_read(gt->uncore,
>> GT0_PERF_LIMIT_REASONS));
>>> + err = -EINVAL;
>>> + }
>>> }
>>>
>>> igt_spinner_end(&spin);
>>> @@ -316,12 +408,29 @@ static int live_slpc_max_granted(void *arg)
>>> return ret;
>>> }
>>>
>>> +static int live_slpc_power(void *arg) {
>>> + struct drm_i915_private *i915 = arg;
>>> + struct intel_gt *gt;
>>> + unsigned int i;
>>> + int ret;
>>> +
>>> + for_each_gt(gt, i915, i) {
>>> + ret = run_test(gt, SLPC_POWER);
>>> + if (ret)
>>> + return ret;
>>> + }
>>> +
>>> + return ret;
>>> +}
>>> +
>>> int intel_slpc_live_selftests(struct drm_i915_private *i915)
>>> {
>>> static const struct i915_subtest tests[] = {
>>> SUBTEST(live_slpc_vary_max),
>>> SUBTEST(live_slpc_vary_min),
>>> SUBTEST(live_slpc_max_granted),
>>> + SUBTEST(live_slpc_power),
>>> };
>>>
>>> struct intel_gt *gt;
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power
2022-09-28 5:57 ` Tauro, Riana
@ 2022-09-29 5:46 ` Gupta, Anshuman
0 siblings, 0 replies; 13+ messages in thread
From: Gupta, Anshuman @ 2022-09-29 5:46 UTC (permalink / raw)
To: Tauro, Riana, Belgaumkar, Vinay, intel-gfx
> -----Original Message-----
> From: Tauro, Riana <riana.tauro@intel.com>
> Sent: Wednesday, September 28, 2022 11:27 AM
> To: Gupta, Anshuman <anshuman.gupta@intel.com>; Belgaumkar, Vinay
> <vinay.belgaumkar@intel.com>; intel-gfx@lists.freedesktop.org
> Cc: Dixit, Ashutosh <ashutosh.dixit@intel.com>; Nilawar, Badal
> <badal.nilawar@intel.com>
> Subject: Re: [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest
> live_slpc_power
>
>
>
> On 9/27/2022 4:42 PM, Gupta, Anshuman wrote:
> >
> >
> >> -----Original Message-----
> >> From: Belgaumkar, Vinay <vinay.belgaumkar@intel.com>
> >> Sent: Monday, September 26, 2022 9:35 PM
> >> To: Tauro, Riana <riana.tauro@intel.com>;
> >> intel-gfx@lists.freedesktop.org
> >> Cc: Gupta, Anshuman <anshuman.gupta@intel.com>; Dixit, Ashutosh
> >> <ashutosh.dixit@intel.com>
> >> Subject: Re: [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest
> >> live_slpc_power
> >>
> >>
> >> On 9/23/2022 4:00 AM, Riana Tauro wrote:
> >>> A fundamental assumption is that at lower frequencies, not only do
> >>> we run slower, but we save power compared to higher frequencies.
> >>> live_slpc_power checks if running at low frequency saves power
> >>>
> >>> v2: re-use code to measure power
> >>> fixed cosmetic review comments (Vinay)
> >>>
> >>> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
> >>
> >> LGTM,
> >>
> >> Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> >>
> >>> ---
> >>> drivers/gpu/drm/i915/gt/selftest_slpc.c | 127
> ++++++++++++++++++++++--
> >>> 1 file changed, 118 insertions(+), 9 deletions(-)
> >>>
> >>> diff --git a/drivers/gpu/drm/i915/gt/selftest_slpc.c
> >>> b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> >>> index 928f74718881..4c6e9257e593 100644
> >>> --- a/drivers/gpu/drm/i915/gt/selftest_slpc.c
> >>> +++ b/drivers/gpu/drm/i915/gt/selftest_slpc.c
> >>> @@ -11,7 +11,8 @@
> >>> enum test_type {
> >>> VARY_MIN,
> >>> VARY_MAX,
> >>> - MAX_GRANTED
> >>> + MAX_GRANTED,
> >>> + SLPC_POWER,
> >>> };
> >>>
> >>> static int slpc_set_min_freq(struct intel_guc_slpc *slpc, u32
> >>> freq) @@ -41,6 +42,39 @@ static int slpc_set_max_freq(struct
> >>> intel_guc_slpc *slpc,
> >> u32 freq)
> >>> return ret;
> >>> }
> >>>
> >>> +static int slpc_set_freq(struct intel_gt *gt, u32 freq) {
> >>> + int err;
> >>> + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
> >>> +
> >>> + err = slpc_set_max_freq(slpc, freq);
> >>> + if (err) {
> >>> + pr_err("Unable to update max freq");
> >>> + return err;
> >>> + }
> >>> +
> >>> + err = slpc_set_min_freq(slpc, freq);
> >>> + if (err) {
> >>> + pr_err("Unable to update min freq");
> >>> + return err;
> >>> + }
> >>> +
> >>> + return err;
> >>> +}
> >>> +
> >>> +static u64 measure_power_at_freq(struct intel_gt *gt, int *freq,
> >>> +u64
> >>> +*power) {
> >>> + int err = 0;
> >>> +
> >>> + err = slpc_set_freq(gt, *freq);
> >>> + if (err)
> >>> + return err;
> >>> + *freq = intel_rps_read_actual_frequency(>->rps);
> >>> + *power = measure_power(>->rps, freq);
> >>> +
> >>> + return err;
> >>> +}
> >>> +
> >>> static int vary_max_freq(struct intel_guc_slpc *slpc, struct intel_rps
> *rps,
> >>> u32 *max_act_freq)
> >>> {
> >>> @@ -113,6 +147,58 @@ static int vary_min_freq(struct intel_guc_slpc
> >>> *slpc,
> >> struct intel_rps *rps,
> >>> return err;
> >>> }
> >>>
> >>> +static int slpc_power(struct intel_gt *gt, struct intel_engine_cs
> >>> +*engine) {
> >>> + struct intel_guc_slpc *slpc = >->uc.guc.slpc;
> >>> + struct {
> >>> + u64 power;
> >>> + int freq;
> >>> + } min, max;
> >>> + int err = 0;
> >>> +
> >>> + /*
> >>> + * Our fundamental assumption is that running at lower frequency
> >>> + * actually saves power. Let's see if our RAPL measurement supports
> >>> + * that theory.
> >>> + */
> >>> + if (!librapl_supported(gt->i915))
> >>> + return 0;
> > This seems a wrong abstraction, this should a generic call should
> check both hwmon registration for dgfx and rapl for igfx.
> > Br,
> > Anshuman Gupta.
> The current librapl_supported has only rapl related changes. The hwmon
> energy is yet to be added.
>
> Will change the name with the hwmon patch
HWMON series is already reviewed and ready to merge just waiting for CI results.
I think we can merge this after hwmom.
Br,
Anshuman Gupta.
>
> Thanks
> Riana Tauro
>
> >>> +
> >>> + min.freq = slpc->min_freq;
> >>> + err = measure_power_at_freq(gt, &min.freq, &min.power);
> >>> +
> >>> + if (err)
> >>> + return err;
> >>> +
> >>> + max.freq = slpc->rp0_freq;
> >>> + err = measure_power_at_freq(gt, &max.freq, &max.power);
> >>> +
> >>> + if (err)
> >>> + return err;
> >>> +
> >>> + pr_info("%s: min:%llumW @ %uMHz, max:%llumW @ %uMHz\n",
> >>> + engine->name,
> >>> + min.power, min.freq,
> >>> + max.power, max.freq);
> >>> +
> >>> + if (10 * min.freq >= 9 * max.freq) {
> >>> + pr_notice("Could not control frequency, ran at [%uMHz,
> >> %uMhz]\n",
> >>> + min.freq, max.freq);
> >>> + }
> >>> +
> >>> + if (11 * min.power > 10 * max.power) {
> >>> + pr_err("%s: did not conserve power when setting lower
> >> frequency!\n",
> >>> + engine->name);
> >>> + err = -EINVAL;
> >>> + }
> >>> +
> >>> + /* Restore min/max frequencies */
> >>> + slpc_set_max_freq(slpc, slpc->rp0_freq);
> >>> + slpc_set_min_freq(slpc, slpc->min_freq);
> >>> +
> >>> + return err;
> >>> +}
> >>> +
> >>> static int max_granted_freq(struct intel_guc_slpc *slpc, struct
> >>> intel_rps *rps,
> >> u32 *max_act_freq)
> >>> {
> >>> struct intel_gt *gt = rps_to_gt(rps); @@ -233,17 +319,23 @@
> >>> static int run_test(struct intel_gt *gt, int test_type)
> >>>
> >>> err = max_granted_freq(slpc, rps, &max_act_freq);
> >>> break;
> >>> +
> >>> + case SLPC_POWER:
> >>> + err = slpc_power(gt, engine);
> >>> + break;
> >>> }
> >>>
> >>> - pr_info("Max actual frequency for %s was %d\n",
> >>> - engine->name, max_act_freq);
> >>> + if (test_type != SLPC_POWER) {
> >>> + pr_info("Max actual frequency for %s was %d\n",
> >>> + engine->name, max_act_freq);
> >>>
> >>> - /* Actual frequency should rise above min */
> >>> - if (max_act_freq <= slpc_min_freq) {
> >>> - pr_err("Actual freq did not rise above min\n");
> >>> - pr_err("Perf Limit Reasons: 0x%x\n",
> >>> - intel_uncore_read(gt->uncore,
> >> GT0_PERF_LIMIT_REASONS));
> >>> - err = -EINVAL;
> >>> + /* Actual frequency should rise above min */
> >>> + if (max_act_freq <= slpc_min_freq) {
> >>> + pr_err("Actual freq did not rise above
> min\n");
> >>> + pr_err("Perf Limit Reasons: 0x%x\n",
> >>> + intel_uncore_read(gt->uncore,
> >> GT0_PERF_LIMIT_REASONS));
> >>> + err = -EINVAL;
> >>> + }
> >>> }
> >>>
> >>> igt_spinner_end(&spin);
> >>> @@ -316,12 +408,29 @@ static int live_slpc_max_granted(void *arg)
> >>> return ret;
> >>> }
> >>>
> >>> +static int live_slpc_power(void *arg) {
> >>> + struct drm_i915_private *i915 = arg;
> >>> + struct intel_gt *gt;
> >>> + unsigned int i;
> >>> + int ret;
> >>> +
> >>> + for_each_gt(gt, i915, i) {
> >>> + ret = run_test(gt, SLPC_POWER);
> >>> + if (ret)
> >>> + return ret;
> >>> + }
> >>> +
> >>> + return ret;
> >>> +}
> >>> +
> >>> int intel_slpc_live_selftests(struct drm_i915_private *i915)
> >>> {
> >>> static const struct i915_subtest tests[] = {
> >>> SUBTEST(live_slpc_vary_max),
> >>> SUBTEST(live_slpc_vary_min),
> >>> SUBTEST(live_slpc_max_granted),
> >>> + SUBTEST(live_slpc_power),
> >>> };
> >>>
> >>> struct intel_gt *gt;
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] ✓ Fi.CI.IGT: success for Add SLPC selftest live_slpc_power (rev2)
2022-09-24 2:01 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
@ 2022-09-29 7:16 ` Gupta, Anshuman
0 siblings, 0 replies; 13+ messages in thread
From: Gupta, Anshuman @ 2022-09-29 7:16 UTC (permalink / raw)
To: intel-gfx, Patchwork, Riana Tauro
On 9/24/2022 7:31 AM, Patchwork wrote:
> *Patch Details*
> *Series:* Add SLPC selftest live_slpc_power (rev2)
> *URL:* https://patchwork.freedesktop.org/series/108900/
> <https://patchwork.freedesktop.org/series/108900/>
> *State:* success
> *Details:*
> https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/index.html
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/index.html>
Thanks for patch.
Pushed to drm-intel-gt-next.
Br,
Anshuman.
>
>
> CI Bug Log - changes from CI_DRM_12174_full -> Patchwork_108900v2_full
>
>
> Summary
>
> *SUCCESS*
>
> No regressions found.
>
>
> Participating hosts (10 -> 9)
>
> Missing (1): shard-tglu
>
>
> Known issues
>
> Here are the changes found in Patchwork_108900v2_full that come from
> known issues:
>
>
> IGT changes
>
>
> Issues hit
>
> *
>
> igt@gem_exec_balancer@parallel-out-fence:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb4/igt@gem_exec_balancer@parallel-out-fence.html> -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb5/igt@gem_exec_balancer@parallel-out-fence.html> (i915#4525 <https://gitlab.freedesktop.org/drm/intel/issues/4525>) +2 similar issues
> *
>
> igt@gem_exec_fair@basic-pace-share@rcs0:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html> -> FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>)
> *
>
> igt@gem_exec_fair@basic-pace-solo@rcs0:
>
> o shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl6/igt@gem_exec_fair@basic-pace-solo@rcs0.html> -> FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@gem_exec_fair@basic-pace-solo@rcs0.html> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>)
> *
>
> igt@gem_exec_fair@basic-throttle@rcs0:
>
> o shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk5/igt@gem_exec_fair@basic-throttle@rcs0.html> -> FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk8/igt@gem_exec_fair@basic-throttle@rcs0.html> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>)
> *
>
> igt@gen9_exec_parse@allowed-single:
>
> o shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk2/igt@gen9_exec_parse@allowed-single.html> -> DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk9/igt@gen9_exec_parse@allowed-single.html> (i915#5566 <https://gitlab.freedesktop.org/drm/intel/issues/5566> / i915#716 <https://gitlab.freedesktop.org/drm/intel/issues/716>)
> *
>
> igt@i915_pm_dc@dc6-psr:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb5/igt@i915_pm_dc@dc6-psr.html> -> FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb3/igt@i915_pm_dc@dc6-psr.html> (i915#3989 <https://gitlab.freedesktop.org/drm/intel/issues/3989> / i915#454 <https://gitlab.freedesktop.org/drm/intel/issues/454>)
> *
>
> igt@i915_suspend@sysfs-reader:
>
> o shard-apl: NOTRUN -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@i915_suspend@sysfs-reader.html> (i915#180 <https://gitlab.freedesktop.org/drm/intel/issues/180>)
> *
>
> igt@kms_big_fb@linear-32bpp-rotate-270:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_big_fb@linear-32bpp-rotate-270.html> (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +43 similar issues
> *
>
> igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_ccs@pipe-b-ccs-on-another-bo-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>) +2 similar issues
> *
>
> igt@kms_color_chamelium@ctm-red-to-blue:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_color_chamelium@ctm-red-to-blue.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_flip@flip-vs-suspend-interruptible@a-dp1:
>
> o shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl2/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html> -> DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html> (i915#180 <https://gitlab.freedesktop.org/drm/intel/issues/180>)
> *
>
> igt@kms_flip@flip-vs-suspend@a-dp1:
>
> o shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl7/igt@kms_flip@flip-vs-suspend@a-dp1.html> -> DMESG-WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl8/igt@kms_flip@flip-vs-suspend@a-dp1.html> (i915#180 <https://gitlab.freedesktop.org/drm/intel/issues/180> / i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>)
> *
>
> igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-downscaling@pipe-a-default-mode:
>
> o shard-iclb: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-downscaling@pipe-a-default-mode.html> (i915#2672 <https://gitlab.freedesktop.org/drm/intel/issues/2672> / i915#3555 <https://gitlab.freedesktop.org/drm/intel/issues/3555>)
> *
>
> igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-valid-mode:
>
> o shard-iclb: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb4/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@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>) +2 similar issues
> *
>
> igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-upscaling@pipe-a-default-mode:
>
> o shard-iclb: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-upscaling@pipe-a-default-mode.html> (i915#2672 <https://gitlab.freedesktop.org/drm/intel/issues/2672>) +2 similar issues
> *
>
> igt@kms_frontbuffer_tracking@fbc-1p-shrfb-fliptrack-mmap-gtt:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-shrfb-fliptrack-mmap-gtt.html> -> FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-1p-shrfb-fliptrack-mmap-gtt.html> (i915#1888 <https://gitlab.freedesktop.org/drm/intel/issues/1888> / i915#2546 <https://gitlab.freedesktop.org/drm/intel/issues/2546>)
> *
>
> igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb8/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-5@pipe-b-edp-1.html> -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/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>) +2 similar issues
> *
>
> igt@kms_psr2_sf@overlay-plane-move-continuous-sf:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/igt@kms_psr2_sf@overlay-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>) +2 similar issues
> *
>
> igt@kms_psr2_su@frontbuffer-xrgb8888:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr2_su@frontbuffer-xrgb8888.html> -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr2_su@frontbuffer-xrgb8888.html> (fdo#109642 <https://bugs.freedesktop.org/show_bug.cgi?id=109642> / fdo#111068 <https://bugs.freedesktop.org/show_bug.cgi?id=111068> / i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>)
> *
>
> igt@kms_psr@psr2_primary_blt:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr@psr2_primary_blt.html> -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr@psr2_primary_blt.html> (fdo#109441 <https://bugs.freedesktop.org/show_bug.cgi?id=109441>)
> *
>
> igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-tglb6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html> -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-tglb6/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html> (i915#5519 <https://gitlab.freedesktop.org/drm/intel/issues/5519>)
> *
>
> igt@sysfs_clients@split-10:
>
> o shard-apl: NOTRUN -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl1/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@feature_discovery@psr2:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb5/igt@feature_discovery@psr2.html> (i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@feature_discovery@psr2.html>
> *
>
> igt@gem_exec_balancer@parallel-bb-first:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb6/igt@gem_exec_balancer@parallel-bb-first.html> (i915#4525 <https://gitlab.freedesktop.org/drm/intel/issues/4525>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb1/igt@gem_exec_balancer@parallel-bb-first.html>
> *
>
> igt@gem_exec_fair@basic-none-share@rcs0:
>
> o shard-glk: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk7/igt@gem_exec_fair@basic-none-share@rcs0.html> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk6/igt@gem_exec_fair@basic-none-share@rcs0.html>
> *
>
> igt@gem_exec_fair@basic-none-solo@rcs0:
>
> o shard-apl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl1/igt@gem_exec_fair@basic-none-solo@rcs0.html> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl2/igt@gem_exec_fair@basic-none-solo@rcs0.html> +1 similar issue
> *
>
> igt@gem_exec_fair@basic-pace@rcs0:
>
> o shard-iclb: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb1/igt@gem_exec_fair@basic-pace@rcs0.html> (i915#2842 <https://gitlab.freedesktop.org/drm/intel/issues/2842>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb7/igt@gem_exec_fair@basic-pace@rcs0.html>
> *
>
> igt@gem_huc_copy@huc-copy:
>
> o shard-tglb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-tglb7/igt@gem_huc_copy@huc-copy.html> (i915#2190 <https://gitlab.freedesktop.org/drm/intel/issues/2190>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-tglb2/igt@gem_huc_copy@huc-copy.html>
> *
>
> igt@i915_pm_rps@engine-order:
>
> o shard-apl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl3/igt@i915_pm_rps@engine-order.html> (i915#6537 <https://gitlab.freedesktop.org/drm/intel/issues/6537>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl7/igt@i915_pm_rps@engine-order.html>
> *
>
> igt@kms_flip@flip-vs-suspend-interruptible@b-dp1:
>
> o shard-apl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl2/igt@kms_flip@flip-vs-suspend-interruptible@b-dp1.html> (i915#180 <https://gitlab.freedesktop.org/drm/intel/issues/180>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible@b-dp1.html>
> *
>
> igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-edp-1.html> (i915#5235 <https://gitlab.freedesktop.org/drm/intel/issues/5235>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_plane_scaling@planes-downscale-factor-0-5@pipe-a-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_12174/shard-iclb8/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_108900v2/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html> +1 similar issue
> *
>
> igt@kms_vblank@pipe-a-accuracy-idle:
>
> o shard-apl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl8/igt@kms_vblank@pipe-a-accuracy-idle.html> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl2/igt@kms_vblank@pipe-a-accuracy-idle.html> +41 similar issues
> *
>
> igt@perf@polling-parameterized:
>
> o shard-glk: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-glk5/igt@perf@polling-parameterized.html> (i915#5639 <https://gitlab.freedesktop.org/drm/intel/issues/5639>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-glk9/igt@perf@polling-parameterized.html>
> *
>
> igt@perf_pmu@module-unload:
>
> o shard-apl: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-apl8/igt@perf_pmu@module-unload.html> (i915#1982 <https://gitlab.freedesktop.org/drm/intel/issues/1982>) -> PASS <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-apl2/igt@perf_pmu@module-unload.html>
>
>
> Warnings
>
> *
>
> igt@gem_exec_balancer@parallel-ordering:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb3/igt@gem_exec_balancer@parallel-ordering.html> (i915#4525 <https://gitlab.freedesktop.org/drm/intel/issues/4525>) -> FAIL <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb4/igt@gem_exec_balancer@parallel-ordering.html> (i915#6117 <https://gitlab.freedesktop.org/drm/intel/issues/6117>)
> *
>
> igt@i915_pm_rc6_residency@rc6-idle@rcs0:
>
> o shard-iclb: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb6/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html> (i915#2684 <https://gitlab.freedesktop.org/drm/intel/issues/2684>) -> WARN <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb8/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html> (i915#2684 <https://gitlab.freedesktop.org/drm/intel/issues/2684>)
> *
>
> igt@kms_psr2_sf@cursor-plane-move-continuous-sf:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb2/igt@kms_psr2_sf@cursor-plane-move-continuous-sf.html> (i915#2920 <https://gitlab.freedesktop.org/drm/intel/issues/2920>) -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/igt@kms_psr2_sf@cursor-plane-move-continuous-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_12174/shard-iclb2/igt@kms_psr2_sf@cursor-plane-update-sf.html> (i915#2920 <https://gitlab.freedesktop.org/drm/intel/issues/2920>) -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb6/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>)
> *
>
> igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12174/shard-iclb5/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html> (i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>) -> SKIP <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_108900v2/shard-iclb2/igt@kms_psr2_sf@overlay-plane-move-continuous-exceed-sf.html> (i915#2920 <https://gitlab.freedesktop.org/drm/intel/issues/2920>)
>
>
> Build changes
>
> * Linux: CI_DRM_12174 -> Patchwork_108900v2
>
> CI-20190529: 20190529
> CI_DRM_12174: 3a5c0f391d00f80d1aa4d766511186412b338b74 @
> git://anongit.freedesktop.org/gfx-ci/linux
> IGT_6662: dcb1d7a8822e62935f4fe3f2e6a04caaee669369 @
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> Patchwork_108900v2: 3a5c0f391d00f80d1aa4d766511186412b338b74 @
> git://anongit.freedesktop.org/gfx-ci/linux
> piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @
> git://anongit.freedesktop.org/piglit
>
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2022-09-29 7:17 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-23 11:00 [Intel-gfx] [PATCH 0/3] Add SLPC selftest live_slpc_power Riana Tauro
2022-09-23 11:00 ` [Intel-gfx] [PATCH 1/3] drm/i915/guc/slpc: Run SLPC selftests on all tiles Riana Tauro
2022-09-26 16:02 ` Belgaumkar, Vinay
2022-09-23 11:00 ` [Intel-gfx] [PATCH 2/3] drm/i915/selftests: Add helper function measure_power Riana Tauro
2022-09-26 15:57 ` Belgaumkar, Vinay
2022-09-23 11:00 ` [Intel-gfx] [PATCH 3/3] drm/i915/guc/slpc: Add SLPC selftest live_slpc_power Riana Tauro
2022-09-26 16:04 ` Belgaumkar, Vinay
2022-09-27 11:12 ` Gupta, Anshuman
2022-09-28 5:57 ` Tauro, Riana
2022-09-29 5:46 ` Gupta, Anshuman
2022-09-23 13:44 ` [Intel-gfx] ✓ Fi.CI.BAT: success for Add SLPC selftest live_slpc_power (rev2) Patchwork
2022-09-24 2:01 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2022-09-29 7:16 ` Gupta, Anshuman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).