[Intel-gfx] [PATCH i-g-t] igt/debugfs_test: Skip dummy reads for crtc-n/crc/data
Abdiel Janulgue
abdiel.janulgue at linux.intel.com
Thu Jul 6 07:41:38 UTC 2017
On 06.07.2017 09:05, Maarten Lankhorst wrote:
> Hey,
>
> Op 03-07-17 om 12:19 schreef Abdiel Janulgue:
>> When descending thru debugfs directory. Doing this tends to
>> takes forever. CRC pipe read tests for this are already
>> covered under kms_pipe_crc_basic.
>>
>> v2: Be more verbose that the check is meant for the crtc control data
>>
>> Signed-off-by: Abdiel Janulgue <abdiel.janulgue at linux.intel.com>
>> Cc: arkadiusz.hiler at intel.com
>> ---
>> tests/debugfs_test.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/tests/debugfs_test.c b/tests/debugfs_test.c
>> index 3f0eaae..66367d0 100644
>> --- a/tests/debugfs_test.c
>> +++ b/tests/debugfs_test.c
>> @@ -44,6 +44,8 @@ static void read_and_discard_sysfs_entries(int path_fd)
>> continue;
>> if (dirent->d_type == DT_DIR) {
>> int sub_fd = -1;
>> + if (strstr(dirent->d_name, "crtc-"))
>> + continue;
>> igt_assert((sub_fd =
>> openat(path_fd, dirent->d_name, O_RDONLY |
>> O_DIRECTORY)) > 0);
>
> Please revert this patch. We already have a specific check for allowing reading that file to fail, which is way more specific than this patch. That opening the file takes indefinitely is a bug in the kernel, not in the test.
Nice to know that this caught the kernel bug! Will revert that...
>
>
More information about the Intel-gfx
mailing list