[igt-dev] [PATCH i-g-t 2/2] lib/i915/perf: fix subslice availability testing

Chris Wilson chris at chris-wilson.co.uk
Mon Feb 24 22:48:23 UTC 2020


Quoting Lionel Landwerlin (2020-02-24 15:03:58)
> Fixes the perf oa-formats tests on Gen9LP.
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Fixes: 53f8f541ca ("lib: Add i915_perf library")
> ---
>  lib/i915/perf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/i915/perf.c b/lib/i915/perf.c
> index 9977c745..5b5b7f90 100644
> --- a/lib/i915/perf.c
> +++ b/lib/i915/perf.c
> @@ -157,7 +157,7 @@ intel_perf_for_devinfo(uint32_t device_id,
>  
>                 perf->devinfo.slice_mask |= 1ULL << s;
>                 for (uint32_t ss = 0; ss < topology->max_subslices; ss++) {
> -                       if (!subslice_available(topology, 0, s))
> +                       if (!subslice_available(topology, s, ss))
>                                 continue;

Yes, that does seem to match the arguments better.
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris


More information about the igt-dev mailing list