[igt-dev] [Intel-gfx] [PATCH igt] igt/perf_pmu: Most-busy requires at least one busy engine

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Tue Mar 27 08:54:35 UTC 2018


On 27/03/2018 09:37, Chris Wilson wrote:
> The test is whether with all but one engine busy we record the correct
> load on each engine. If we only have one engine, this test degenerates
> into all-idle/all-busy, so we can skip to avoid crashing on the
> assumption that we have a busy spinner.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> ---
>   tests/perf_pmu.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
> index f27b7ec7..b59af818 100644
> --- a/tests/perf_pmu.c
> +++ b/tests/perf_pmu.c
> @@ -513,6 +513,7 @@ most_busy_check_all(int gem_fd, const struct intel_execution_engine2 *e,
>   		val[i++] = I915_PMU_ENGINE_BUSY(e_->class, e_->instance);
>   	}
>   	igt_assert(i == num_engines);
> +	igt_require(spin); /* at least one busy engine */

Or igt_require(num_engines > 1) higher up. But same effect:

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

Regards,

Tvrtko

>   
>   	fd[0] = -1;
>   	for (i = 0; i < num_engines; i++)
> 



More information about the igt-dev mailing list