[Intel-gfx] [PATCH v9 4/4] drm/i915/selftests: skip comparison of power for discrete graphics

[Dixit, Ashutosh]

On Tue, 04 Apr 2023 23:59:30 -0700, Riana Tauro wrote:
>

Hi Riana,

> Hwmon reads the energy/power consumed by discrete soc,
> i.e. energy/power includes the power drawn from non-gfx discrete
> components
>
> This test uses the power consumed by GT to validate RC6
> power consumption.
> Skip comparison of power for discrete graphics
>
> TODO : measure power of GT in discrete graphics and modify the
> condition
>
> v2: update commit message (Anshuman)
>
> Signed-off-by: Riana Tauro <riana.tauro at intel.com>
> Reviewed-by: Anshuman Gupta <anshuman.gupta at intel.com>
> ---
>  drivers/gpu/drm/i915/gt/selftest_rc6.c | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/selftest_rc6.c b/drivers/gpu/drm/i915/gt/selftest_rc6.c
> index 682f2fe67b3a..47165f490449 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_rc6.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_rc6.c
> @@ -107,7 +107,15 @@ int live_rc6_manual(void *arg)
>				      ktime_to_ns(dt));
>		pr_info("GPU consumed %llduW in RC0 and %llduW in RC6\n",
>			rc0_power, rc6_power);
> -		if (2 * rc6_power > rc0_power) {

So this condition is not being met for dGfx?

> +
> +		/*
> +		 * Condition valid for integrated graphics
> +		 * On discrete graphics, hwwmon reads the energy/power from
> +		 * discrete SOC that includes non-gfx components.

On dGfx, is this true even when we have per-gt level energy available? Or
only when we have device level energy but not per-gt level energy (when
total number of gt's is 1 and we only expose device level energy but not gt
level energy)?


> +		 * TODO : Measure power of GT for discrete graphics and
> +		 * modify the condition

If we are adding this TODO, how are we planning to do this?

> +		 */
> +		if (!IS_DGFX(gt->i915) && (2 * rc6_power > rc0_power)) {
>			pr_err("GPU leaked energy while in RC6!\n");
>			err = -EINVAL;
>			goto out_unlock;

Thanks.
--
Ashutosh