diff mbox

[i-g-t] igt/gem_unfence_active_buffers: Check GEM before use

Message ID 20180712154003.12630-1-chris@chris-wilson.co.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Chris Wilson July 12, 2018, 3:40 p.m. UTC
As we want to make the buffers active on the GPU before removing their
fence, an operational GPU (not wedged!) is required.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/gem_unfence_active_buffers.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Ville Syrjälä July 12, 2018, 7:01 p.m. UTC | #1
On Thu, Jul 12, 2018 at 04:40:03PM +0100, Chris Wilson wrote:
> As we want to make the buffers active on the GPU before removing their
> fence, an operational GPU (not wedged!) is required.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> ---
>  tests/gem_unfence_active_buffers.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/gem_unfence_active_buffers.c b/tests/gem_unfence_active_buffers.c
> index 6df23cc53..b78fbafa7 100644
> --- a/tests/gem_unfence_active_buffers.c
> +++ b/tests/gem_unfence_active_buffers.c
> @@ -74,6 +74,7 @@ igt_simple_main
>  		data[i] = i;
>  
>  	fd = drm_open_driver(DRIVER_INTEL);
> +	igt_require_gem(fd);
>  
>  	bufmgr = drm_intel_bufmgr_gem_init(fd, 4096);
>  	drm_intel_bufmgr_gem_enable_reuse(bufmgr);
> -- 
> 2.18.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox

Patch

diff --git a/tests/gem_unfence_active_buffers.c b/tests/gem_unfence_active_buffers.c
index 6df23cc53..b78fbafa7 100644
--- a/tests/gem_unfence_active_buffers.c
+++ b/tests/gem_unfence_active_buffers.c
@@ -74,6 +74,7 @@  igt_simple_main
 		data[i] = i;
 
 	fd = drm_open_driver(DRIVER_INTEL);
+	igt_require_gem(fd);
 
 	bufmgr = drm_intel_bufmgr_gem_init(fd, 4096);
 	drm_intel_bufmgr_gem_enable_reuse(bufmgr);