[igt-dev] [PATCH i-g-t 14/20] tests/psr: Make testing with sink CRC non-default

Katarzyna Dec katarzyna.dec at intel.com
Fri Apr 13 11:22:50 UTC 2018


On Tue, Apr 10, 2018 at 07:37:26PM -0700, Dhinakaran Pandiyan wrote:
> Testing with CRC has always been unreliable. CRC also appears to be
> suspectible to signal integrity of the link between source and sink.
> But, most importantly, I see that reading sink CRC causes PSR exits
> rendering CRC based testing for PSR exits pointless. So hide this under
> a non-default --sink-crc command line option.
> 
> This makes all tests other than basic and no-drrs require
> interactive-debug, but that's okay since using sink CRC is misleading.
> 
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
 Reviewed-by: Katarzyna Dec <katarzyna.dec at intel.com>
> ---
>  tests/kms_psr_sink_crc.c | 12 +++++++++++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
> index bb92d672..c9197c9d 100644
> --- a/tests/kms_psr_sink_crc.c
> +++ b/tests/kms_psr_sink_crc.c
> @@ -75,6 +75,7 @@ typedef struct {
>  	drmModeModeInfo *mode;
>  	igt_output_t *output;
>  	bool with_psr_disabled;
> +	bool with_sink_crc;
>  } data_t;
>  
>  static void create_cursor_fb(data_t *data)
> @@ -273,6 +274,10 @@ static void run_test(data_t *data)
>  	char crc[CRC_LEN];
>  	const char *expected = "";
>  
> +	if (!igt_interactive_debug)
> +		igt_require_f(data->with_sink_crc,
> +			      "Enable sink CRC with --sink-crc\n");
> +
>  	/* Confirm that screen became Green */
>  	get_sink_crc(data, ref_crc);
>  	assert_or_manual(is_green(ref_crc), "screen GREEN");
> @@ -438,6 +443,9 @@ static int opt_handler(int opt, int opt_index, void *_data)
>  	case 'n':
>  		data->with_psr_disabled = true;
>  		break;
> +	case 'c':
> +		data->with_sink_crc = true;
> +		break;
>  	default:
>  		igt_assert(0);
>  	}
> @@ -448,9 +456,11 @@ static int opt_handler(int opt, int opt_index, void *_data)
>  int main(int argc, char *argv[])
>  {
>  	const char *help_str =
> -	       "  --no-psr\tRun test without PSR to check the CRC test logic.";
> +	       "  --no-psr\tRun test without PSR to check the CRC test logic.\n"	\
> +	       "  --sink-crc\tRun tests using sink CRC";
>  	static struct option long_options[] = {
>  		{"no-psr", 0, 0, 'n'},
> +		{"sink-crc", 0, 0, 'c'},
>  		{ 0, 0, 0, 0 }
>  	};
>  	data_t data = {};
> -- 
> 2.14.1
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev


More information about the igt-dev mailing list