Re: [PATCH i-g-t 0/7] add dsc-fallback test

["Joshi, Kunal1" <kunal1.joshi@intel.com>]

Hello,

On 13-02-2025 11:23, Nautiyal, Ankit K wrote:
>
> On 2/11/2025 11:35 PM, Kunal Joshi wrote:
>> Add test to check we fallback to dsc when current
>> link params cannot accomadate bw required by
>> the current mode.
>>
>> Kunal Joshi (7):
>>    tests/intel/kms_dp_linktrain_fallback: refactor find_mst_outputs
>>    tests/intel/kms_mst_helper: add helper for MST-related functions
>>    tests/intel/kms_dp_linktrain_fallback: refactor dp-fallback subtest
>>    tests/intel/kms_dp_linktrain_fallback: fix typo
>>    tests/intel/kms_dp_linktrain_fallback: add dsc-fallback test
>>    tests/intel/kms_dp_linktrain_fallback: disable ignore long hpd
>
> The series looks good to me.
>
> Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
>
Thank you Ankit for the feedback and the review.
Thanks Swati for getting styling right and typo's fixed.

Below failures are not related to the series and skips are expected.

**Xe.CI.BAT**

- igt@kms_dp_linktrain_fallback@dsc-fallback (NEW):
     - bat-lnl-1: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_12576/bat-lnl-1/igt@kms_dp_linktrain_fallback@dsc-fallback.html) +1 
other test skip
     - bat-bmg-1: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_12576/bat-bmg-1/igt@kms_dp_linktrain_fallback@dsc-fallback.html) +1 
other test skip
     - bat-dg2-oem2: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_12576/bat-dg2-oem2/igt@kms_dp_linktrain_fallback@dsc-fallback.html)


**Xe.CI.Full**

- igt@kms_dp_linktrain_fallback@dp-fallback:
     - 
shard-lnl: [SKIP](https://intel-gfx-ci.01.org/tree/intel-xe/IGT_8228/shard-lnl-3/igt@kms_dp_linktrain_fallback@dp-fallback.html) ([Intel 
XE#3070](https://gitlab.freedesktop.org/drm/xe/kernel/issues/3070)) 
-> [SKIP](https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_12576/shard-lnl-8/igt@kms_dp_linktrain_fallback@dp-fallback.html)


**i915.CI.BAT**

- igt@kms_dp_linktrain_fallback@dp-fallbck:

     - Statuses : 40 skip(s)
     - Exec time: [0.0] s
- igt@kms_dp_linktrain_fallback@dsc-fallback:

     - Statuses : 2 pass(s) 38 skip(s)
     - Exec time: [0.0, 1.40] s


**i915.CI.Full**

- igt@gem_mmap_offset@open-flood:

     - 
shard-rkl: [PASS](https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16110/shard-rkl-1/igt@gem_mmap_offset@open-flood.html) -> [DMESG-WARN](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-rkl-5/igt@gem_mmap_offset@open-flood.html)
- igt@i915_suspend@debugfs-reader:

     - shard-glk: NOTRUN 
-> [INCOMPLETE](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-glk1/igt@i915_suspend@debugfs-reader.html)
- igt@kms_dp_linktrain_fallback@dp-fallback:

     - shard-dg2: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-dg2-3/igt@kms_dp_linktrain_fallback@dp-fallback.html) +1 
other test skip
     - shard-rkl: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-rkl-8/igt@kms_dp_linktrain_fallback@dp-fallback.html)
     - shard-tglu-1: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-tglu-1/igt@kms_dp_linktrain_fallback@dp-fallback.html)
     - shard-dg1: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-dg1-17/igt@kms_dp_linktrain_fallback@dp-fallback.html) +1 
other test skip
     - shard-mtlp: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-mtlp-7/igt@kms_dp_linktrain_fallback@dp-fallback.html) +1 
other test skip
- igt@kms_dp_linktrain_fallback@dsc-fallback (NEW):

     - shard-tglu: NOTRUN 
-> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-tglu-9/igt@kms_dp_linktrain_fallback@dsc-fallback.html)
- igt@perf_pmu@rc6@runtime-pm-gt0:

     - 
shard-rkl: [PASS](https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16110/shard-rkl-1/igt@perf_pmu@rc6@runtime-pm-gt0.html) -> [SKIP](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-rkl-7/igt@perf_pmu@rc6@runtime-pm-gt0.html)
- igt@sysfs_heartbeat_interval@nopreempt@vcs0:

     - 
shard-mtlp: [PASS](https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16110/shard-mtlp-5/igt@sysfs_heartbeat_interval@nopreempt@vcs0.html) -> [DMESG-WARN](https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_12576/shard-mtlp-7/igt@sysfs_heartbeat_interval@nopreempt@vcs0.html)

Series is merged now.

Thanks and Regards
Kunal Joshi

>
>>    HAX: Do not merge
>>
>>   tests/intel-ci/fast-feedback.testlist    |   3 +-
>>   tests/intel-ci/xe-fast-feedback.testlist |   3 +-
>>   tests/intel/kms_dp_linktrain_fallback.c  | 356 +++++++++++++++++++----
>>   tests/intel/kms_mst_helper.c             |  48 +++
>>   tests/intel/kms_mst_helper.h             |  15 +
>>   tests/meson.build                        |   3 +
>>   6 files changed, 367 insertions(+), 61 deletions(-)
>>   create mode 100644 tests/intel/kms_mst_helper.c
>>   create mode 100644 tests/intel/kms_mst_helper.h
>>