[Intel-gfx] [PATCH] drm/i915/perf: ensure selftests select valid format

Chris Wilson chris at chris-wilson.co.uk
Tue Oct 29 15:41:18 UTC 2019


Quoting Lionel Landwerlin (2019-10-29 14:28:26)
> Gen12 only support a single report format :
> I915_OA_FORMAT_A32u40_A4u32_B8_C8
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Fixes: 00a7f0d7155c ("drm/i915/tgl: Add perf support on TGL")
> ---
>  drivers/gpu/drm/i915/selftests/i915_perf.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/selftests/i915_perf.c b/drivers/gpu/drm/i915/selftests/i915_perf.c
> index dc6d689e4251..aabd07f67e49 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_perf.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_perf.c
> @@ -23,7 +23,8 @@ test_stream(struct i915_perf *perf)
>                                                    I915_ENGINE_CLASS_RENDER,
>                                                    0),
>                 .sample_flags = SAMPLE_OA_REPORT,
> -               .oa_format = I915_OA_FORMAT_C4_B8,
> +               .oa_format = IS_GEN(perf->i915, 12) ?
> +               I915_OA_FORMAT_A32u40_A4u32_B8_C8 : I915_OA_FORMAT_C4_B8,

Matches gen12_oa_formats[], must be good :)
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris


More information about the Intel-gfx mailing list