From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4413210E83C for ; Thu, 25 May 2023 09:05:59 +0000 (UTC) Received: from linux.intel.com (maurocar-mobl2.ger.corp.intel.com [10.252.20.31]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by linux.intel.com (Postfix) with ESMTPS id EC652580AA7 for ; Thu, 25 May 2023 02:05:40 -0700 (PDT) Received: from maurocar by linux.intel.com with local (Exim 4.96) (envelope-from ) id 1q26ug-00BiHZ-2w for igt-dev@lists.freedesktop.org; Thu, 25 May 2023 11:05:38 +0200 From: Mauro Carvalho Chehab To: igt-dev@lists.freedesktop.org Date: Thu, 25 May 2023 11:05:24 +0200 Message-Id: <20230525090535.2791739-1-mauro.chehab@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t v4 00/11] Improve description of tests used by i915 validation List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: From: Mauro Carvalho Chehab 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