[PATCH i-g-t v4] tests/kms_content_protection: Add force HDCP 1.4 subtest

Kandpal, Suraj suraj.kandpal at intel.com
Fri Jul 11 05:51:58 UTC 2025



> Subject: [PATCH i-g-t v4] tests/kms_content_protection: Add force HDCP 1.4
> subtest
> 
> If a Panel supports both HDCP1.4 and HDCP2.2 versions, the kernel will
> always choose the HDCP2.2 protection path and if this fails, then
> HDCP1.4 will be tried.
> 
> The subtest uses debugfs support to force hdcp1.4 on the connector and
> verify content protection.
> 
> V2: Use hdcp14 debugfs for all the supported subtests (Suraj).
> 
> v3: Create new separate subtests for hdcp1.4 and hdcp2.2 for all the
> supported tests. (Suraj)
> 
> v4: Add -hdcp14 suffix for subtests only for cases explicitly forcing hdcp1.4,
> retain original subtest names (eg atomic, type1) for default behaviour. (Suraj)
> 
> Signed-off-by: Santhosh Reddy Guddati <santhosh.reddy.guddati at intel.com>
> ---
>  tests/kms_content_protection.c | 154 ++++++++++++++++++++++++++++++---
>  1 file changed, 142 insertions(+), 12 deletions(-)
> 
> diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> index 51fc1d3be..4fa5d2c20 100644
> --- a/tests/kms_content_protection.c
> +++ b/tests/kms_content_protection.c
> @@ -43,8 +43,8 @@
>  #include "igt_panel.h"
> 
>  /**
> - * SUBTEST: lic-type-0
> - * Description: Test for the integrity of link for type-0 content.
> + * SUBTEST: lic-type-0-hdcp14
> + * Description: Test for the integrity of link for type-0 content with HDCP1.4.
>   *

Why delete the lic-type-0
Lic-type-0 works with both HDCP 2.2 and HDCP 1.4 this would essentially remove testing for HDCP 2.2 type 0 content
All you need to do is add not subtraction required
So the test will be lic-type-0 (no changes here)
And a new test  lic-type-0-hdcp14 where force_hdcp is true
Same for all the tests below.

Regards,
Suraj Kandpal

>   * SUBTEST: lic-type-1
>   * Description: Test for the integrity of link for type-1 content.
> @@ -65,6 +65,11 @@
>   * Description: Test to detect the HDCP status change when we are reading
> the
>   *              uevent sent with the corresponding connector id and property id.
>   *
> + * SUBTEST: uevent-hdcp14
> + * Description: Test to detect the status change when we are reading the
> + *              uevent sent with the corresponding connector id and property id
> + *              with HDCP1.4 content protection.
> + *
>   * SUBTEST: %s
>   * Description: Test content protection with %arg[1]
>   *
> @@ -74,6 +79,10 @@
>   * @atomic-dpms:    DPMS ON/OFF during atomic modesetting.
>   * @legacy:         legacy style commit
>   * @type1:          content type 1 that can be handled only through HDCP2.2.
> + * @legacy-hdcp14:  Test HDCP1.4 content protection with legacy style
> commit.
> + * @atomic-hdcp14:  Test HDCP1.4 content protection with atomic
> modesetting.
> + * @atomic-dpms-hdcp14: Test HDCP1.4 content protection with atomic
> modesetting and DPMS.
> + *
>   */
> 
>  /**
> @@ -83,8 +92,10 @@
>   * arg[1]:
>   *
>   * @lic-type-0:   Type 0 with LIC
> + * @lic-type-0-hdcp14: Type 0 with LIC and HDCP1.4
>   * @lic-type-1:   Type 1 with LIC.
>   * @type-0:       Type 0
> + * @type-0-hdcp14: Type 0 with HDCP1.4
>   * @type-1:       Type 1
>   */
> 
> @@ -96,6 +107,7 @@ struct data {
>  	struct igt_fb red, green;
>  	unsigned int cp_tests;
>  	struct udev_monitor *uevent_monitor;
> +	bool is_force_hdcp14;
>  } data;
> 
>  /* Test flags */
> @@ -581,6 +593,51 @@ static bool output_hdcp_capable(igt_output_t
> *output, int content_type)
>  		return true;
>  }
> 
> +static void set_i915_force_hdcp14(igt_output_t *output) {
> +	int fd, ret;
> +	char buf[MAX_SINK_HDCP_CAP_BUF_LEN];
> +
> +	fd = igt_debugfs_connector_dir(data.drm_fd, output->name,
> O_RDONLY);
> +	igt_require_f(fd >= 0, "Cannot open %s debugfs\n", output->name);
> +
> +	ret = igt_debugfs_simple_read(fd, "i915_force_hdcp14", buf,
> sizeof(buf));
> +	if (ret <= 0) {
> +		igt_info("i915_force_hdcp14 not supported\n");
> +		close(fd);
> +		return;
> +	}
> +
> +	ret = igt_sysfs_write(fd, "i915_force_hdcp14", "1", 2);
> +	igt_require_f(ret > 0, "i915_force_hdcp14 is not enabled\n");
> +
> +	ret = igt_debugfs_simple_read(fd, "i915_force_hdcp14", buf,
> sizeof(buf));
> +	igt_assert_f(ret > 0 && strstr(buf, "yes"),
> +			 "i915_force_hdcp14 is not set to 'yes' on %s
> debugfs\n",
> +			 output->name);
> +
> +	close(fd);
> +}
> +
> +static void reset_i915_force_hdcp14(igt_output_t *output) {
> +	int fd, ret;
> +	char buf[MAX_SINK_HDCP_CAP_BUF_LEN];
> +
> +	fd = igt_debugfs_connector_dir(data.drm_fd, output->name,
> O_RDONLY);
> +	igt_require_f(fd >= 0, "Cannot open %s debugfs\n", output->name);
> +
> +	ret = igt_sysfs_write(fd, "i915_force_hdcp14", "0", 2);
> +	igt_require_f(ret > 0, "i915_force_hdcp14 is not disabled\n");
> +
> +	ret = igt_debugfs_simple_read(fd, "i915_force_hdcp14", buf,
> sizeof(buf));
> +	igt_assert_f(ret > 0 && strstr(buf, "no"),
> +			 "i915_force_hdcp14 is not set to 'no' on %s
> debugfs\n",
> +			 "i915_force_hdcp14");
> +
> +	close(fd);
> +}
> +
>  static void
>  test_fini(igt_output_t *output, enum igt_commit_style commit_style)  { @@ -
> 654,10 +711,15 @@ test_content_protection(enum igt_commit_style
> commit_style, int content_type)
>  					  output->name);
>  				continue;
>  			}
> +			if (data.is_force_hdcp14)
> +				set_i915_force_hdcp14(output);
> 
>  			igt_dynamic_f("pipe-%s-%s",
> kmstest_pipe_name(pipe), output->name)
>  				test_content_protection_on_output(output,
> pipe, commit_style, content_type);
> 
> +			if (data.is_force_hdcp14)
> +				reset_i915_force_hdcp14(output);
> +
>  			test_fini(output, commit_style);
>  			/*
>  			 * Testing a output with a pipe is enough for HDCP
> @@ -719,6 +781,11 @@ test_mst_cp_enable_with_retry(igt_output_t
> *hdcp_mst_output[], int valid_outputs
>  	int retry_orig = retries, count, i;
>  	bool ret;
> 
> +	if (data.is_force_hdcp14) {
> +		for (count = 0; count < valid_outputs; count++)
> +			set_i915_force_hdcp14(hdcp_mst_output[count]);
> +	}
> +
>  	do {
>  		if (retry_orig != retries)
>  			test_mst_cp_disable(hdcp_mst_output,
> COMMIT_ATOMIC, valid_outputs); @@ -752,6 +819,11 @@
> test_mst_cp_enable_with_retry(igt_output_t *hdcp_mst_output[], int
> valid_outputs
>  		igt_display_commit2(display, COMMIT_ATOMIC);
>  	} while (retries && !ret);
> 
> +	if (data.is_force_hdcp14) {
> +		for (count = 0; count < valid_outputs; count++)
> +			reset_i915_force_hdcp14(hdcp_mst_output[count]);
> +	}
> +
>  	igt_assert_f(ret, "Content Protection not enabled on MST outputs\n");
> }
> 
> @@ -878,54 +950,84 @@ static void create_fbs(void)
>  			    0.f, 1.f, 0.f, &data.green);
>  }
> 
> +
> +
>  static const struct {
>  	const char *desc;
>  	const char *name;
>  	unsigned int cp_tests;
>  	bool content_type;
> +	bool is_force_hdcp14;
>  } subtests[] = {
> +	{ .desc = "Test content protection with atomic modesetting with
> HDCP1.4.",
> +	  .name = "atomic-hdcp14",
> +	  .cp_tests = 0,
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
> +	},
>  	{ .desc = "Test content protection with atomic modesetting",
>  	  .name = "atomic",
>  	  .cp_tests = 0,
> -	  .content_type = HDCP_CONTENT_TYPE_0
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = false,
>  	},
> -	{ .desc = "Test content protection with DPMS ON/OFF during atomic
> modesetting.",
> +	{ .desc = "Test content protection with DPMS ON/OFF during "
> +		  "atomic modesetting with HDCP1.4.",
> +	  .name = "atomic-dpms-hdcp14",
> +	  .cp_tests = CP_DPMS,
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
> +	},
> +	{ .desc = "Test content protection with DPMS ON/OFF during atomic
> +modesetting",
>  	  .name = "atomic-dpms",
>  	  .cp_tests = CP_DPMS,
> -	  .content_type = HDCP_CONTENT_TYPE_0
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = false,
>  	},
>  	{ .desc = "Test for the integrity of link with type 0 content.",
> -	  .name = "lic-type-0",
> +	  .name = "lic-type-0-hdcp14",
>  	  .cp_tests = CP_LIC,
>  	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
>  	},
>  	{ .desc = "Test for the integrity of link with type 1 content",
>  	  .name = "lic-type-1",
>  	  .cp_tests = CP_LIC,
>  	  .content_type = HDCP_CONTENT_TYPE_1,
> +	  .is_force_hdcp14 = false,
>  	},
>  	{ .desc = "Test content protection with content type 1 "
>  		  "that can be handled only through HDCP2.2.",
>  	  .name = "type1",
>  	  .cp_tests = 0,
>  	  .content_type = HDCP_CONTENT_TYPE_1,
> +	  .is_force_hdcp14 = false,
>  	},
>  	{ .desc = "Test the teardown and rebuild of the interface between "
>  		  "Intel and mei hdcp.",
>  	  .name = "mei-interface",
>  	  .cp_tests = CP_MEI_RELOAD,
>  	  .content_type = HDCP_CONTENT_TYPE_1,
> +	  .is_force_hdcp14 = false,
>  	},
>  	{ .desc = "Test the content type change when the content protection
> already enabled",
>  	  .name = "content-type-change",
>  	  .cp_tests = CP_TYPE_CHANGE,
>  	  .content_type = HDCP_CONTENT_TYPE_1,
>  	},
> -	{ .desc = "Test to detect the HDCP status change when we are reading
> the uevent "
> +	{.desc = "Test to detect the HDCP status change when we are reading
> the uevent "
>  		  "sent with the corresponding connector id and property id.",
>  	  .name = "uevent",
>  	  .cp_tests = CP_UEVENT,
>  	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = false,
> +	},
> +	{ .desc = "Test to detect the HDCP status change when we are reading
> the uevent "
> +		  "sent with the corresponding connector id and property id.",
> +	  .name = "uevent-hdcp14",
> +	  .cp_tests = CP_UEVENT,
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
>  	},
>  	/*
>  	 *  Testing the revocation check through SRM needs a HDCP sink with
> @@ -940,6 +1042,7 @@ static const struct {
>  	  .name = "srm",
>  	  .cp_tests = 0,
>  	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
>  	},
>  };
> 
> @@ -948,26 +1051,43 @@ static const struct {
>  	const char *name;
>  	unsigned int cp_tests;
>  	bool content_type;
> +	bool is_force_hdcp14;
>  } mst_subtests[] = {
> -	{ .desc = "Test Content protection(Type 0) over DP MST.",
> +	{ .desc = "Test Content protection(Type 0) over DP MST",
>  	  .name = "dp-mst-type-0",
>  	  .cp_tests = 0,
> -	  .content_type = HDCP_CONTENT_TYPE_0
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = false,
> +	},
> +	{ .desc = "Test Content protection(Type 0) over DP MST with
> HDCP1.4.",
> +	  .name = "dp-mst-type-0-hdcp14",
> +	  .cp_tests = 0,
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
>  	},
>  	{ .desc = "Test Content protection(Type 0) over DP MST with LIC.",
>  	  .name = "dp-mst-lic-type-0",
>  	  .cp_tests = CP_LIC,
> -	  .content_type = HDCP_CONTENT_TYPE_0
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = false,
> +	},
> +	{ .desc = "Test Content protection(Type 0) over DP MST with LIC.",
> +	  .name = "dp-mst-lic-type-0-hdcp14",
> +	  .cp_tests = CP_LIC,
> +	  .content_type = HDCP_CONTENT_TYPE_0,
> +	  .is_force_hdcp14 = true,
>  	},
>  	{ .desc = "Test Content protection(Type 1) over DP MST.",
>  	  .name = "dp-mst-type-1",
>  	  .cp_tests = 0,
>  	  .content_type = HDCP_CONTENT_TYPE_1,
> +	  .is_force_hdcp14 = false,
>  	},
>  	{ .desc = "Test Content protection(Type 1) over DP MST with LIC.",
>  	  .name = "dp-mst-lic-type-1",
>  	  .cp_tests = CP_LIC,
>  	  .content_type = HDCP_CONTENT_TYPE_1,
> +	  .is_force_hdcp14 = false,
>  	},
>  };
> 
> @@ -980,12 +1100,20 @@ igt_main
>  		create_fbs();
>  	}
> 
> -	igt_describe("Test content protection with legacy style commit.");
> -	igt_subtest_with_dynamic("legacy") {
> +	igt_describe("Test content protection with legacy style commit with
> HDCP1.4");
> +	igt_subtest_with_dynamic("legacy-hdcp14") {
>  		data.cp_tests = 0;
> +		data.is_force_hdcp14 = true;
>  		test_content_protection(COMMIT_LEGACY,
> HDCP_CONTENT_TYPE_0);
>  	}
> 
> +	igt_describe("Test content protection with legacy style commit with
> HDCP2.2");
> +	igt_subtest_with_dynamic("legacy") {
> +		data.cp_tests = 0;
> +		data.is_force_hdcp14 = false;
> +		test_content_protection(COMMIT_LEGACY,
> HDCP_CONTENT_TYPE_1);
> +	}
> +
>  	igt_subtest_group {
>  		igt_fixture
>  			igt_require(data.display.is_atomic);
> @@ -995,6 +1123,7 @@ igt_main
> 
>  			igt_subtest_with_dynamic(subtests[i].name) {
>  				data.cp_tests = subtests[i].cp_tests;
> +				data.is_force_hdcp14 =
> subtests[i].is_force_hdcp14;
> 
>  				if (!strcmp(subtests[i].name, "srm")) {
>  					bool ret;
> @@ -1018,6 +1147,7 @@ igt_main
> 
>  			igt_subtest(mst_subtests[i].name) {
>  				data.cp_tests = mst_subtests[i].cp_tests;
> +				data.is_force_hdcp14 =
> mst_subtests[i].is_force_hdcp14;
> 
> 	test_content_protection_mst(mst_subtests[i].content_type);
>  			}
>  		}
> --
> 2.34.1



More information about the igt-dev mailing list