[igt-dev] [PATCH i-g-t 0/4] Fix bad documentation validation logic

Mauro Carvalho Chehab mauro.chehab at linux.intel.com
Mon Oct 9 10:01:37 UTC 2023


From: Mauro Carvalho Chehab <mchehab at kernel.org>

There's currently a bug at the logic which checks for IGT documentation:
it is not reporting all cases of undocumented tests.

Patch 1 on this series fix it, and uses a simpler logic to check for
differences between documented and existing tests.

After the fix, there will be 6 tests out of sync:

$ ./scripts/igt_doc.py --config tests/intel/xe_*.json --check
Warning: Documented igt at xe_sysfs_scheduler@job_timeout_ms-nonprivileged-user doesn't exist on source files
Warning: Documented igt at xe_sysfs_scheduler@preempt_timeout_us-nonprivileged-user doesn't exist on source files
Warning: Documented igt at xe_sysfs_scheduler@timeslice_duration_us-nonprivileged-user doesn't exist on source files

$ ./scripts/igt_doc.py --config tests/intel/kms_*.json --check
Warning: Documented igt at kms_big_fb@linear-addfb-size-offset-overflow doesn't exist on source files
Warning: Documented igt at kms_big_fb@linear-addfb-size-overflow doesn't exist on source files
Warning: Documented igt at kms_frontbuffer_tracking@fbc-slowdraw doesn't exist on source files

The remaining patches on this series fix them.

Mauro Carvalho Chehab (4):
  scripts/test_list.py: fix and simplify missing doc check
  intel/xe_sysfs_scheduler.c: remove nonpriviledged-user tests
  intel/kms_big_fb.c: remove linear-addfb-size*-overflow docs
  intel/kms_frontbuffer_tracking.c: remove fbc-slowdraw test

 scripts/test_list.py                   | 34 +++-----------------------
 tests/intel/kms_big_fb.c               | 16 ------------
 tests/intel/kms_frontbuffer_tracking.c |  7 ------
 tests/intel/xe_sysfs_scheduler.c       |  4 ---
 4 files changed, 4 insertions(+), 57 deletions(-)

-- 
2.41.0



More information about the igt-dev mailing list