* [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
* 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
* [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
* 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
* [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
* 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@lists.freedesktop.org > -----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@lists.freedesktop.org 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@lists.freedesktop.org > -----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
* [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] ✓ 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