Re: [igt-dev] [PATCH i-g-t 1/1] i915/gem_blits: Add check for hardware based FENCE support

From: "Dixit, Ashutosh" <ashutosh.dixit@intel.com>
To: Ayaz A Siddiqui <ayaz.siddiqui@intel.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t 1/1] i915/gem_blits: Add check for hardware based FENCE support
Date: Wed, 05 Feb 2020 22:48:11 -0800	[thread overview]
Message-ID: <8736boyzx0.wl-ashutosh.dixit@intel.com> (raw)
In-Reply-To: <20200205102136.25808-2-ayaz.siddiqui@intel.com>

On Wed, 05 Feb 2020 02:21:36 -0800, Ayaz A Siddiqui wrote:
>
> This test is verifying various tiled source and destination data
> combination for blitter using FENCE based tiling. Since blitter don't use
> FENCE after Gen4+ so this test can only be useful for Platform prior to Gen4.
>
> A check has been added to skip this test if fences are not available in
> platform.
>
> Signed-off-by: Ayaz A Siddiqui <ayaz.siddiqui@intel.com>
> ---
>  tests/i915/gem_blits.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/tests/i915/gem_blits.c b/tests/i915/gem_blits.c
> index f9cb12bb..cc24a6c8 100644
> --- a/tests/i915/gem_blits.c
> +++ b/tests/i915/gem_blits.c
> @@ -722,10 +722,12 @@ igt_main
>		device.fd = drm_open_driver_render(DRIVER_INTEL);
>		igt_require_gem(device.fd);
>		gem_require_blitter(device.fd);
> +		igt_require(gem_available_fences(device.fd) > 0);
>
>		device.pciid = intel_get_drm_devid(device.fd);

I don't think there is any need to skip all these tests. See:

https://patchwork.freedesktop.org/series/73074/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev