Re: [Intel-gfx] [PATCH igt] igt/gem_ctx_isolation: Fix checking for context support

2018-02-22 Thread Mika Kuoppala
Chris Wilson  writes:

> Missed the new method for igt_require(gem_has_contexts()) in the rebase.
>
> Signed-off-by: Chris Wilson 
> ---
>  tests/gem_ctx_isolation.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/gem_ctx_isolation.c b/tests/gem_ctx_isolation.c
> index cb822c5b..d8109aa0 100644
> --- a/tests/gem_ctx_isolation.c
> +++ b/tests/gem_ctx_isolation.c
> @@ -630,8 +630,8 @@ igt_main
>  
>   fd = drm_open_driver(DRIVER_INTEL);
>   igt_require_gem(fd);
> + igt_require(gem_has_contexts(fd));
>  
> - gem_context_destroy(fd, gem_context_create(fd));

Seems that this was the last user of this pattern atleast.

Reviewed-by: Mika Kuoppala 
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] [PATCH igt] igt/gem_ctx_isolation: Fix checking for context support

2018-02-22 Thread Chris Wilson
Missed the new method for igt_require(gem_has_contexts()) in the rebase.

Signed-off-by: Chris Wilson 
---
 tests/gem_ctx_isolation.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/gem_ctx_isolation.c b/tests/gem_ctx_isolation.c
index cb822c5b..d8109aa0 100644
--- a/tests/gem_ctx_isolation.c
+++ b/tests/gem_ctx_isolation.c
@@ -630,8 +630,8 @@ igt_main
 
fd = drm_open_driver(DRIVER_INTEL);
igt_require_gem(fd);
+   igt_require(gem_has_contexts(fd));
 
-   gem_context_destroy(fd, gem_context_create(fd));
has_context_isolation = __has_context_isolation(fd);
igt_require(has_context_isolation);
 
-- 
2.16.1

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx