[PATCH i-g-t v4] tests/kms_content_protection: Add force HDCP 1.4 subtest
Reddy Guddati, Santhosh
santhosh.reddy.guddati at intel.com
Fri Jul 11 06:18:48 UTC 2025
On 11-07-2025 11:21, Kandpal, Suraj wrote:
>
>
>> 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.
>
Thanks Suraj for pointing out this. I someone have missed this only
subtest "lic-type-0" , others is taken care. I will update the patch to
include this subtest.
> 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