[igt-dev] [PATCH i-g-t v4] lib/igt_kms: Fix memory corruption
Vignesh Raman
vignesh.raman at collabora.com
Fri Nov 10 06:52:54 UTC 2023
Hi Bhanu,
On 10/11/23 11:36, Modem, Bhanuprakash wrote:
>> ---
>> lib/igt_kms.c | 6 +++++-
>> lib/igt_kms.h | 20 +++++++++++++++++++-
>> tests/intel-ci/blacklist.txt | 10 +++++++++-
>> tests/intel-ci/xe.blocklist.txt | 6 +++---
>> tests/intel/kms_ccs.c | 24 ++++++++++++++++++++++++
>> tests/kms_bw.c | 2 +-
>
> Probably, you need to split this patch into 3.
>
> 1- lib
> 2- tests
> 3- intel-ci
Sure, will split the changes into 3 commits.
>> @@ -2770,6 +2770,10 @@ void igt_display_require(igt_display_t
>> *display, int drm_fd)
>> }
>> #endif
>> + igt_assert_f(resources->count_crtcs <= IGT_MAX_PIPES,
> --------^
> Could be igt_require_f()?
>
> IMHO, we need to skip the test instead of fail. Since it is a basic
> requirement to fill display struct.
Okay, will skip the test. I was going through the failures mentioned in
https://patchwork.freedesktop.org/series/125599/
Are these known issues?
>> --- a/tests/intel-ci/blacklist.txt
>> +++ b/tests/intel-ci/blacklist.txt
>> @@ -117,12 +117,20 @@ igt at gem_ctx_isolation@.*-s4
>> ###############################################
>> igt at i915_suspend@shrink
>> ###############################################
>> -# pipe-e, pipe-f, pipe-g, pipe-h no current HW support
>> +# pipe-e, pipe-f, pipe-g, pipe-h, pipe-i, pipe-j, pipe-k, pipe-l,
>> pipe-m, pipe-n, pipe-o, pipe-p no current HW support
>> ###############################################
>> igt at .*@.*pipe-e($|-.*)
>> igt at .*@.*pipe-f($|-.*)
>> igt at .*@.*pipe-g($|-.*)
>> igt at .*@.*pipe-h($|-.*)
>> +igt at .*@.*pipe-i($|-.*)
>> +igt at .*@.*pipe-j($|-.*)
>> +igt at .*@.*pipe-k($|-.*)
>> +igt at .*@.*pipe-l($|-.*)
>> +igt at .*@.*pipe-m($|-.*)
>> +igt at .*@.*pipe-n($|-.*)
>> +igt at .*@.*pipe-o($|-.*)
>> +igt at .*@.*pipe-p($|-.*)
>
> Shall we simplify the regex as below?
>
> igt at .*@.*pipe-[e-pE-P]($|-.*)
Yes, this looks good.
>
>> ###############################################
>> # Temporary workarounds for CI-impacting bugs
>> diff --git a/tests/intel-ci/xe.blocklist.txt
>> b/tests/intel-ci/xe.blocklist.txt
>> index 135ab18a3..bec6ee23b 100644
>> --- a/tests/intel-ci/xe.blocklist.txt
>> +++ b/tests/intel-ci/xe.blocklist.txt
>> @@ -44,11 +44,11 @@ igt at syncobj_.*
>> igt at template
>> igt at tools_test
>> ##################################################################
>> -# KMS: Pipe E, F, G and H are not available on Intel hardware,
>> +# KMS: Pipe E, F, G, H, I, J, K, L, M, N, O and P are not available
>> on Intel hardware,
>> # hence can't use more than 4 displays.
>> ##################################################################
>> -igt at .*@.*pipe-(e|f|g|h).*
>> -igt at kms_bw@.*-(5|6|7|8)-displays-.*
>> +igt at .*@.*pipe-(e|f|g|h|i|j|k|l|m|n|o|p).*
>> +igt at kms_bw@.*-(5|6|7|8|9|10|11|12|13|14|15|16)-displays-.*
>
> igt at .*@.*pipe-[e-pE-P].*
> igt at kms_bw@.*-([5-9]|1[0-6])-displays-.*
>
> I used to validate my regex at https://regex101.com/
>
> You can get my R-b after addressing above comments.
Thank you.
Regards,
Vignesh
More information about the igt-dev
mailing list