[igt-dev] [PATCH i-g-t v4 00/11] Improve description of tests used by i915 validation

Mauro Carvalho Chehab mauro.chehab at linux.intel.com
Thu May 25 09:05:24 UTC 2023


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

This patch series was entirely generated by a script which imports data
from an internal spreadsheet containing additional documentation from
IGT tests.

Once we get this added, it should be easy to edit each test file to
improve tests documentation.

---

v4:
- no changes on patches 1 to 7.
- added 4 patches at the end:
  tests: add documentation for dynamic subtests
  scripts/test_list.py: add dynamic subtest support
  i915_test_config.json: exclude gem_concurrent_blit.c
  testplan/meson.build: make it check for missing i915 documentation
- it now documents i915 dynamic subtests as well
- building IGT will now check if i915 documentation was also updated.

v3:
- add a typo fix as the first patch to avoid adding a different text
  after running my scripts;
- fixed a bug at the logic which simplifies the subtests by moving
  fields to TEST:
- did several fixes at the original spreadsheet, in order to place
  descriptions at the right place.

v2:
- Patches 1 to 4 unchanged. R-B added to them.
- Use commas instead of new lines for Feature set update;
- Don't add the same feature twice;
- Don't add functionality if identical to a comment substring.

Mauro Carvalho Chehab (11):
  gem_exec_fence: fix a typo
  tests: better document core_* tests used for i915 validation
  tests: better document the remaining core used by i915 validation
  i915/gem_exec_*: better document gem exec tests used for i915
    validation
  i915/gem_ctx_*: better document i915 context tests
  i915/perf_*: better document tests
  i915/: better document the remaining i915 tests
  tests: add documentation for dynamic subtests
  scripts/test_list.py: add dynamic subtest support
  i915_test_config.json: exclude gem_concurrent_blit.c
  testplan/meson.build: make it check for missing i915 documentation

 docs/testplan/meson.build                |   4 +-
 scripts/test_list.py                     |  10 +-
 tests/core_auth.c                        |   4 +
 tests/core_getclient.c                   |   4 +
 tests/core_getstats.c                    |   4 +
 tests/core_getversion.c                  |   4 +
 tests/core_hotunplug.c                   |   8 +
 tests/core_setmaster.c                   |   4 +
 tests/core_setmaster_vs_auth.c           |   4 +
 tests/debugfs_test.c                     |  23 +-
 tests/device_reset.c                     |   8 +-
 tests/dmabuf.c                           |  12 +
 tests/drm_mm.c                           | 120 ++++++
 tests/dumb_buffer.c                      |   4 +
 tests/i915/api_intel_allocator.c         |  40 +-
 tests/i915/api_intel_bb.c                | 150 ++++++-
 tests/i915/gem_barrier_race.c            |   1 +
 tests/i915/gem_basic.c                   |  13 +
 tests/i915/gem_blits.c                   |   4 +
 tests/i915/gem_busy.c                    |  30 +-
 tests/i915/gem_caching.c                 |   4 +
 tests/i915/gem_close.c                   |   4 +
 tests/i915/gem_close_race.c              |  10 +
 tests/i915/gem_create.c                  |  50 ++-
 tests/i915/gem_cs_tlb.c                  |   4 +
 tests/i915/gem_ctx_bad_destroy.c         |   4 +
 tests/i915/gem_ctx_create.c              |  22 +-
 tests/i915/gem_ctx_engines.c             |   4 +
 tests/i915/gem_ctx_exec.c                |  28 +-
 tests/i915/gem_ctx_freq.c                |   4 +
 tests/i915/gem_ctx_isolation.c           |  38 +-
 tests/i915/gem_ctx_param.c               |  76 ++++
 tests/i915/gem_ctx_persistence.c         | 112 ++++++
 tests/i915/gem_ctx_shared.c              |  37 +-
 tests/i915/gem_eio.c                     | 106 ++++-
 tests/i915/gem_exec_alignment.c          |   4 +
 tests/i915/gem_exec_async.c              |   4 +
 tests/i915/gem_exec_await.c              |   4 +
 tests/i915/gem_exec_balancer.c           | 100 +++++
 tests/i915/gem_exec_basic.c              |   8 +-
 tests/i915/gem_exec_big.c                |   4 +
 tests/i915/gem_exec_capture.c            |  12 +
 tests/i915/gem_exec_create.c             |  21 +-
 tests/i915/gem_exec_endless.c            |   4 +
 tests/i915/gem_exec_fence.c              | 124 +++++-
 tests/i915/gem_exec_gttfill.c            |  12 +
 tests/i915/gem_exec_nop.c                |  12 +
 tests/i915/gem_exec_parallel.c           |  26 +-
 tests/i915/gem_exec_params.c             |  76 ++++
 tests/i915/gem_exec_schedule.c           | 204 ++++++++++
 tests/i915/gem_exec_store.c              |   4 +
 tests/i915/gem_exec_suspend.c            |  20 +-
 tests/i915/gem_exec_whisper.c            |  90 +++++
 tests/i915/gem_exercise_blt.c            |   9 +-
 tests/i915/gem_flink_basic.c             |  32 +-
 tests/i915/gem_flink_race.c              |   4 +
 tests/i915/gem_gpgpu_fill.c              |   6 +-
 tests/i915/gem_linear_blits.c            |  16 +-
 tests/i915/gem_lmem_evict.c              |   1 +
 tests/i915/gem_lmem_swapping.c           |  80 +++-
 tests/i915/gem_madvise.c                 |   8 +
 tests/i915/gem_mmap_offset.c             |  54 ++-
 tests/i915/gem_pipe_control_store_loop.c |   4 +
 tests/i915/gem_ppgtt.c                   |   6 +
 tests/i915/gem_request_retire.c          |   4 +
 tests/i915/gem_ringfill.c                |  14 +-
 tests/i915/gem_shrink.c                  |   4 +
 tests/i915/gem_softpin.c                 | 105 ++++-
 tests/i915/gem_spin_batch.c              |  37 ++
 tests/i915/gem_sync.c                    |  24 +-
 tests/i915/gem_unref_active_buffers.c    |   4 +
 tests/i915/gem_userptr_blits.c           |  76 ++++
 tests/i915/gem_vm_create.c               |   4 +
 tests/i915/gem_wait.c                    |  52 ++-
 tests/i915/gem_workarounds.c             |  30 ++
 tests/i915/i915_getparams_basic.c        |   4 +
 tests/i915/i915_hangman.c                |  40 +-
 tests/i915/i915_hwmon.c                  |   1 +
 tests/i915/i915_module_load.c            |  12 +
 tests/i915/i915_pciid.c                  |   4 +
 tests/i915/i915_pm_rpm.c                 |   3 +
 tests/i915/i915_pm_rps.c                 |  10 +-
 tests/i915/i915_power.c                  |   1 +
 tests/i915/i915_query.c                  |  32 ++
 tests/i915/i915_selftest.c               | 485 +++++++++++++++++++++++
 tests/i915/i915_suspend.c                |  13 +-
 tests/i915/i915_test_config.json         |   2 +-
 tests/i915/perf.c                        |  96 +++--
 tests/i915/perf_pmu.c                    | 137 +++++--
 tests/i915/sysfs_defaults.c              |   4 +
 tests/i915/sysfs_heartbeat_interval.c    |  10 +
 tests/i915/sysfs_preempt_timeout.c       |   6 +
 tests/i915/sysfs_timeslice_duration.c    |   8 +
 tests/prime_busy.c                       |   4 +
 tests/prime_mmap.c                       |   4 +
 tests/prime_mmap_coherency.c             |   4 +
 tests/prime_self_import.c                |   4 +
 tests/prime_vgem.c                       |  37 +-
 tests/sw_sync.c                          |   4 +
 tests/syncobj_basic.c                    |   4 +
 tests/syncobj_timeline.c                 |   4 +
 tests/syncobj_wait.c                     |   4 +
 tests/vgem_basic.c                       |  14 +
 tests/vgem_slow.c                        |   4 +
 104 files changed, 3071 insertions(+), 141 deletions(-)

-- 
2.40.1



More information about the igt-dev mailing list