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@xxxxxxxxxxxxxxx>
> Cc: arkadiusz.hiler@xxxxxxxxx
> ---
> 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.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
