[igt-dev] [PATCH i-g-t v2] lib/igt_debugfs: Add IGT_CRC_SOURCE environmental variable

[Petri Latvala]

On Thu, Jun 09, 2022 at 10:05:14AM -0700, Jessica Zhang wrote:
> Adds support for the IGT_CRC_SOURCE environmental variable to make it
> easier for drivers to run tests using custom CRC sources.
> 
> Example usage:
> 	`IGT_CRC_SOURCE=intf ./kms_pipe_crc_basic`
> 
> If the IGT_CRC_SOURCE isn't set, use the original source that was passed
> in as a parameter to pipe_crc_new().
> 
> Signed-off-by: Jessica Zhang <quic_jesszhan at quicinc.com>
> ---
>  lib/igt_debugfs.c | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
> index 7e7ccf22c455..f9404acc3942 100644
> --- a/lib/igt_debugfs.c
> +++ b/lib/igt_debugfs.c
> @@ -719,9 +719,15 @@ pipe_crc_new(int fd, enum pipe pipe, const char *source, int flags)
>  	igt_pipe_crc_t *pipe_crc;
>  	char buf[128];
>  	int debugfs;
> +	char *env_source;

const char *


With that fixed,
Reviewed-by: Petri Latvala <petri.latvala at intel.com>


>  
>  	igt_assert(source);
>  
> +	env_source = getenv("IGT_CRC_SOURCE");
> +
> +	if (!env_source)
> +		env_source = source;
> +
>  	debugfs = igt_debugfs_dir(fd);
>  	igt_assert(debugfs != -1);
>  
> @@ -736,7 +742,7 @@ pipe_crc_new(int fd, enum pipe pipe, const char *source, int flags)
>  	pipe_crc->fd = fd;
>  	pipe_crc->dir = debugfs;
>  	pipe_crc->pipe = pipe;
> -	pipe_crc->source = strdup(source);
> +	pipe_crc->source = strdup(env_source);
>  	igt_assert(pipe_crc->source);
>  	pipe_crc->flags = flags;
>  
> -- 
> 2.31.0
>