Re: [PATCH v2] lib/igt_kms.c: remove tests dependency on VT /dev/tty0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 24, 2015 at 12:29:22PM -0800, Marc Herbert wrote:
> Required to run on any recent, freon-based and X11-free ChromeOS release.
> 
> Signed-off-by: Marc Herbert <marc.herbert@xxxxxxxxx>
> ---
> 
> Changes from v1:
> - igt_debug() instead of igt_warn()
> - return KD_GRAPHICS instead of -1UL
> - print previous mode in debug statements. Among others this help a tiny
>   bit with the now confusing debug output ("cannot change" immediately
>   followed by a misleading "mode changed").

Thanks for the patch, merged to igt.
-Daniel

> 
> Note: a naive "mv /dev/tty0 /dev/tty0.orig" is typically enough to test
> this patch.
> 
>  lib/igt_kms.c | 22 ++++++++++++++++++----
>  1 file changed, 18 insertions(+), 4 deletions(-)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index d0c3690..9c131f0 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -299,12 +299,26 @@ int kmstest_get_pipe_from_crtc_id(int fd, int crtc_id)
>  	return pfci.pipe;
>  }
>  
> +/*
> + * Returns: the previous mode, or KD_GRAPHICS if no /dev/tty0 was
> + * found and nothing was done.
> + */
>  static signed long set_vt_mode(unsigned long mode)
>  {
>  	int fd;
>  	unsigned long prev_mode;
> +	static const char TTY0[] = "/dev/tty0";
> +
> +	if (access(TTY0, F_OK)) {
> +		/* errno message should be "No such file". Do not
> +		   hardcode but ask strerror() in the very unlikely
> +		   case something else happened. */
> +		igt_debug("VT: %s: %s, cannot change its mode\n",
> +			  TTY0, strerror(errno));
> +		return KD_GRAPHICS;
> +	}
>  
> -	fd = open("/dev/tty0", O_RDONLY);
> +	fd = open(TTY0, O_RDONLY);
>  	if (fd < 0)
>  		return -errno;
>  
> @@ -336,10 +350,10 @@ void kmstest_restore_vt_mode(void)
>  
>  	if (orig_vt_mode != -1UL) {
>  		ret = set_vt_mode(orig_vt_mode);
> -		orig_vt_mode = -1UL;
>  
>  		igt_assert(ret >= 0);
> -		igt_debug("VT: original mode restored\n");
> +		igt_debug("VT: original mode 0x%lx restored\n", orig_vt_mode);
> +		orig_vt_mode = -1UL;
>  	}
>  }
>  
> @@ -366,7 +380,7 @@ void kmstest_set_vt_graphics_mode(void)
>  	igt_assert(ret >= 0);
>  	orig_vt_mode = ret;
>  
> -	igt_debug("VT: graphics mode set\n");
> +	igt_debug("VT: graphics mode set (mode was 0x%lx)\n", ret);
>  }
>  
>  
> -- 
> 1.9.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux