Re: [PATCH i-g-t 11/12] lib/intel_bufops: Don't disable x and y on platforms without swizzling

[Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>]

Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>

On 27.5.2024 10.33, Zbigniew Kempczyński wrote:
> Upcoming tiling detection tool wants to produce X and Y surfaces
> in software so we can't disable X and Y in supported tilings,
> otherwise there's no possibility to generate reference tiled surface.
> 
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> Cc: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
> ---
>   lib/intel_bufops.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/intel_bufops.c b/lib/intel_bufops.c
> index d0f6ab481f..1dfc12bf45 100644
> --- a/lib/intel_bufops.c
> +++ b/lib/intel_bufops.c
> @@ -1720,7 +1720,7 @@ static struct buf_ops *__buf_ops_create(int fd, bool check_idempotency)
>   		bool supported = probe_hw_tiling(bops, I915_TILING_X,
>   						 &swizzling_supported);
>   
> -		if (!swizzling_supported) {
> +		if (!swizzling_supported && bops->intel_gen < 12) {
>   			igt_debug("Swizzling for X is not supported\n");
>   			bops->supported_tiles &= ~TILE_X;
>   		}
> @@ -1738,7 +1738,7 @@ static struct buf_ops *__buf_ops_create(int fd, bool check_idempotency)
>   		bool supported = probe_hw_tiling(bops, I915_TILING_Y,
>   						 &swizzling_supported);
>   
> -		if (!swizzling_supported) {
> +		if (!swizzling_supported && bops->intel_gen < 12) {
>   			igt_debug("Swizzling for Y is not supported\n");
>   			bops->supported_tiles &= ~TILE_Y;
>   		}