[igt-dev] [PATCH i-g-t 1/1] Add check for HW tiling support in gem_set_tiling_vs_blt test

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Fri Jan 31 11:02:00 UTC 2020


On Fri, Jan 31, 2020 at 12:24:56PM +0530, Ayaz A Siddiqui wrote:
> This  test is currently assuming that platform always support HW tiling.
> A check has been added to skip the test if HW tiling is not supported
> by platform.
> 
> Signed-off-by: Ayaz A Siddiqui <ayaz.siddiqui at intel.com>
> ---
>  tests/i915/gem_set_tiling_vs_blt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/gem_set_tiling_vs_blt.c b/tests/i915/gem_set_tiling_vs_blt.c
> index c317d1ef..0cb3fdf1 100644
> --- a/tests/i915/gem_set_tiling_vs_blt.c
> +++ b/tests/i915/gem_set_tiling_vs_blt.c
> @@ -232,7 +232,7 @@ igt_main
>  		fd = drm_open_driver(DRIVER_INTEL);
>  		igt_require_gem(fd);
>  		gem_require_blitter(fd);
> -
> +		igt_require(gem_available_fences(fd) > 0);
>  		bufmgr = drm_intel_bufmgr_gem_init(fd, 4096);
>  		drm_intel_bufmgr_gem_enable_reuse(bufmgr);
>  		devid = intel_get_drm_devid(fd);

Small nit - we still have HW tiling, but fences are not available
on newer gens. 

Anyway,
Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com> 


More information about the igt-dev mailing list