diff mbox

lib/drmtest: Fix filedesc leak on gem_quiescent_gpu

Message ID 1385450777-23450-1-git-send-email-mika.kuoppala@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mika Kuoppala Nov. 26, 2013, 7:26 a.m. UTC 
to not run out of filedescs as some tests call gem_quiescent_gpu
directly.

Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
---
 lib/drmtest.c |    1 +
 1 file changed, 1 insertion(+)

Comments

Daniel Vetter Dec. 4, 2013, 12:23 p.m. UTC | #1 
On Tue, Nov 26, 2013 at 09:26:17AM +0200, Mika Kuoppala wrote:
> to not run out of filedescs as some tests call gem_quiescent_gpu
> directly.
> 
> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>

Applied, thanks.
-Daniel

> ---
>  lib/drmtest.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/drmtest.c b/lib/drmtest.c
> index 15ed847..2e35644 100644
> --- a/lib/drmtest.c
> +++ b/lib/drmtest.c
> @@ -166,6 +166,7 @@ void gem_quiescent_gpu(int fd)
>  
>  	gem_sync(fd, handle);
>  	igt_drop_caches_set(DROP_RETIRE);
> +	gem_close(fd, handle);
>  }
>  
>  /**
> -- 
> 1.7.9.5
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox

Patch

diff --git a/lib/drmtest.c b/lib/drmtest.c
index 15ed847..2e35644 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -166,6 +166,7 @@  void gem_quiescent_gpu(int fd)
 
 	gem_sync(fd, handle);
 	igt_drop_caches_set(DROP_RETIRE);
+	gem_close(fd, handle);
 }
 
 /**