Re: [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master

[Chris Wilson <chris@chris-wilson.co.uk>]

Quoting Katarzyna Dec (2018-05-14 11:19:30)
> Let's add additional information to igt_device_drop_master
> in case we try to run tests with e.g. x11 enabled.
> 
> Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  lib/igt_device.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_device.c b/lib/igt_device.c
> index 8a2dd0e3..96fde899 100644
> --- a/lib/igt_device.c
> +++ b/lib/igt_device.c
> @@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
>   */
>  void igt_device_drop_master(int fd)
>  {
> -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> +       if (__igt_device_drop_master(fd)) {
> +               igt_require_f(__igt_device_drop_master(fd) == 0,
> +                             "Can't drop DRM master, "
> +                             "please check if no other DRM client is running.\n");

How does one check? Could we not present that information to the user?  ;)

It should remain an assert here I think.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev