On Tue, Oct 24, 2017 at 07:24:58PM +0100, Chris Wilson wrote:
> Since the headless subtest wants to compare execution latency of a
> headless mode vs a single head, it needs to be able to do a modeset and
> ergo it requires DRM_MASTER.
> 
> Signed-off-by: Chris Wilson <[email protected]>

Reviewed-by: Michał Winiarski <[email protected]>

-Michał

> ---
>  tests/gem_exec_nop.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/gem_exec_nop.c b/tests/gem_exec_nop.c
> index ce3a8ef7..c9280795 100644
> --- a/tests/gem_exec_nop.c
> +++ b/tests/gem_exec_nop.c
> @@ -711,8 +711,11 @@ igt_main
>       }
>  
>  #if !defined(ANDROID) || ANDROID_HAS_CAIRO
> -     igt_subtest("headless")
> +     igt_subtest("headless") {
> +             /* Requires master for changing display modes */
> +             igt_require(drmSetMaster(device) == 0);
>               headless(device, handle);
> +     }
>  #endif
>  
>       igt_fixture {
> -- 
> 2.15.0.rc2
> 
> _______________________________________________
> Intel-gfx mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to