[igt-dev] [PATCH i-g-t 2/2] tests/kms_psr2_su: Print debugfs when skipping test

Petri Latvala petri.latvala at intel.com
Mon Jun 1 08:18:26 UTC 2020


On Fri, May 29, 2020 at 06:16:13PM -0700, José Roberto de Souza wrote:
> This tests is being sporadically skipped in CI as it is not due
> "PSR sink not reliable: yes" lets print the i915_edp_psr_status to
> find out the reason.
> This can be reverted afterwards.
> 
> Reference: https://gitlab.freedesktop.org/drm/intel/-/issues/1911
> Cc: Gwan-gyeong Mun <gwan-gyeong.mun at intel.com>
> Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
> ---
>  lib/igt_psr.c       | 13 +++++++++++++
>  lib/igt_psr.h       |  1 +
>  tests/kms_psr2_su.c |  2 ++
>  3 files changed, 16 insertions(+)
> 
> diff --git a/lib/igt_psr.c b/lib/igt_psr.c
> index 956f6219c..8c2f4ce6c 100644
> --- a/lib/igt_psr.c
> +++ b/lib/igt_psr.c
> @@ -229,3 +229,16 @@ bool psr2_wait_su(int debugfs_fd, uint16_t *num_su_blocks)
>  {
>  	return igt_wait(psr2_read_last_num_su_blocks_val(debugfs_fd, num_su_blocks), 40, 1);
>  }
> +
> +void psr_print_debugfs(int debugfs_fd)
> +{
> +	char buf[PSR_STATUS_MAX_LEN];
> +	int ret;
> +
> +	ret = igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf,
> +				      sizeof(buf));
> +	if (ret < 0)
> +		return;
> +
> +	igt_debug("%s", buf);

Does the buf always have a newline here? Even when the file has
exactly PSR_STATUS_MAX_LEN bytes?


-- 
Petri Latvala


More information about the igt-dev mailing list