[igt-dev] [RFC 2/3] tests/intel/xe*.c: drop Run type field
Kamil Konieczny
kamil.konieczny at linux.intel.com
Wed Sep 6 09:05:00 UTC 2023
Hi Mauro,
On 2023-09-05 at 18:48:29 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
>
> Currently, the information should be on both places, but that
---------------------------------------------------- ^
s/, but/ but/
imho add here also what information you are describing, like:
Currently, the information for BAT or FULL run should
be on both places, testlists and test source but that
> makes harder to maintain. In practice, only the testlists are
> updated for intel-ci.
>
> So, drop the explicit field data from the source files,
- ^^^ ----------------------- ^^^^
s/So, drop/Dropped/
s/data/"Run type"/
> relying on intel-ci testlists and main blacklist.
------------------------------------------ ^
s/blacklist/blocklist/
This is related to BAT versus FULL runs, no blocklist there,
so maybe cut this sentence at "and main ..."?
s/ and main blacklist././
>
Add here list of Cc. This looks good so with above corrections
you can add my Acked-by.
Regards,
Kamil
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>
> ---
> tests/intel/xe_ccs.c | 1 -
> tests/intel/xe_compute.c | 1 -
> tests/intel/xe_create.c | 2 --
> tests/intel/xe_debugfs.c | 1 -
> tests/intel/xe_dma_buf_sync.c | 4 ----
> tests/intel/xe_evict.c | 1 -
> tests/intel/xe_exec_balancer.c | 11 -----------
> tests/intel/xe_exec_basic.c | 6 ------
> tests/intel/xe_exec_compute_mode.c | 4 ----
> tests/intel/xe_exec_fault_mode.c | 8 --------
> tests/intel/xe_exec_reset.c | 23 -----------------------
> tests/intel/xe_exec_store.c | 2 --
> tests/intel/xe_exec_threads.c | 2 --
> tests/intel/xe_exercise_blt.c | 1 -
> tests/intel/xe_gpgpu_fill.c | 1 -
> tests/intel/xe_guc_pc.c | 9 ---------
> tests/intel/xe_huc_copy.c | 1 -
> tests/intel/xe_intel_bb.c | 13 -------------
> tests/intel/xe_live_ktest.c | 1 -
> tests/intel/xe_media_fill.c | 1 -
> tests/intel/xe_mmap.c | 1 -
> tests/intel/xe_mmio.c | 1 -
> tests/intel/xe_module_load.c | 6 ------
> tests/intel/xe_noexec_ping_pong.c | 1 -
> tests/intel/xe_pm.c | 2 --
> tests/intel/xe_pm_residency.c | 3 ---
> tests/intel/xe_prime_self_import.c | 8 --------
> tests/intel/xe_query.c | 1 -
> tests/intel/xe_spin_batch.c | 4 ----
> tests/intel/xe_sysfs_defaults.c | 1 -
> tests/intel/xe_sysfs_scheduler.c | 1 -
> tests/intel/xe_vm.c | 23 -----------------------
> tests/intel/xe_waitfence.c | 1 -
> 33 files changed, 146 deletions(-)
>
> diff --git a/tests/intel/xe_ccs.c b/tests/intel/xe_ccs.c
> index a3c6c46d1b54..9653b054e0f8 100644
> --- a/tests/intel/xe_ccs.c
> +++ b/tests/intel/xe_ccs.c
> @@ -22,7 +22,6 @@
> * Sub-category: IGT Lib
> * Functionality: blitter
> * Description: Exercise gen12 blitter with and without flatccs compression on Xe
> - * Run type: FULL
> *
> * SUBTEST: block-copy-compressed
> * Description: Check block-copy flatccs compressed blit
> diff --git a/tests/intel/xe_compute.c b/tests/intel/xe_compute.c
> index 2cf536701a58..41db9ff2f17c 100644
> --- a/tests/intel/xe_compute.c
> +++ b/tests/intel/xe_compute.c
> @@ -8,7 +8,6 @@
> * Category: Hardware building block
> * Sub-category: compute
> * Test category: functionality test
> - * Run type: BAT
> */
>
> #include <string.h>
> diff --git a/tests/intel/xe_create.c b/tests/intel/xe_create.c
> index 20e7a7da8dc8..bc371ef5f44a 100644
> --- a/tests/intel/xe_create.c
> +++ b/tests/intel/xe_create.c
> @@ -9,7 +9,6 @@
> * Sub-category: uapi
> * Functionality: device
> * Test category: functionality test
> - * Run type: BAT
> */
>
> #include <string.h>
> @@ -125,7 +124,6 @@ static uint32_t __xe_exec_queue_create(int fd, uint32_t vm,
> /**
> * SUBTEST: create-execqueues-%s
> * Description: Check process ability of multiple exec_queues creation
> - * Run type: FULL
> *
> * arg[1]:
> *
> diff --git a/tests/intel/xe_debugfs.c b/tests/intel/xe_debugfs.c
> index e51d03661022..44b657e3dcf3 100644
> --- a/tests/intel/xe_debugfs.c
> +++ b/tests/intel/xe_debugfs.c
> @@ -9,7 +9,6 @@
> * Sub-category: debugfs
> * Functionality: debugfs
> * Test category: functionality test
> - * Run type: BAT
> * Description: Validate debugfs entries
> */
>
> diff --git a/tests/intel/xe_dma_buf_sync.c b/tests/intel/xe_dma_buf_sync.c
> index 29d675154e8b..35eec3dea21e 100644
> --- a/tests/intel/xe_dma_buf_sync.c
> +++ b/tests/intel/xe_dma_buf_sync.c
> @@ -72,19 +72,15 @@ static bool sync_file_busy(int sync_file)
> /**
> * SUBTEST: export-dma-buf-once
> * Description: Test exporting a sync file from a dma-buf
> - * Run type: BAT
> *
> * SUBTEST: export-dma-buf-once-read-sync
> * Description: Test export prime BO as sync file and verify business
> - * Run type: BAT
> *
> * SUBTEST: export-dma-buf-many
> * Description: Test exporting many sync files from a dma-buf
> - * Run type: FULL
> *
> * SUBTEST: export-dma-buf-many-read-sync
> * Description: Test export many prime BO as sync file and verify business
> - * Run type: FULL
> */
>
> static void
> diff --git a/tests/intel/xe_evict.c b/tests/intel/xe_evict.c
> index b8f186809a91..2c8506938cd1 100644
> --- a/tests/intel/xe_evict.c
> +++ b/tests/intel/xe_evict.c
> @@ -467,7 +467,6 @@ static uint64_t calc_bo_size(uint64_t vram_size, int mul, int div)
> /**
> * SUBTEST: evict-%s
> * Description: %arg[1] evict test.
> - * Run type: FULL
> *
> * arg[1]:
> *
> diff --git a/tests/intel/xe_exec_balancer.c b/tests/intel/xe_exec_balancer.c
> index f364a4b7ab84..1d4f9af7b04d 100644
> --- a/tests/intel/xe_exec_balancer.c
> +++ b/tests/intel/xe_exec_balancer.c
> @@ -30,7 +30,6 @@
> * Description:
> * Run a test to check if virtual exec_queues can be running on all instances
> * of a class simultaneously
> - * Run type: FULL
> */
> static void test_all_active(int fd, int gt, int class)
> {
> @@ -137,23 +136,18 @@ static void test_all_active(int fd, int gt, int class)
> /**
> * SUBTEST: once-%s
> * Description: Run %arg[1] test only once
> - * Run type: FULL
> *
> * SUBTEST: many-%s
> * Description: Run %arg[1] test many times
> - * Run type: FULL
> *
> * SUBTEST: many-execqueues-%s
> * Description: Run %arg[1] test on many exec_queues
> - * Run type: FULL
> *
> * SUBTEST: twice-%s
> * Description: Run %arg[1] test twice
> - * Run type: BAT
> *
> * SUBTEST: no-exec-%s
> * Description: Run no-exec %arg[1] test
> - * Run type: BAT
> *
> * arg[1]:
> *
> @@ -357,24 +351,19 @@ test_exec(int fd, int gt, int class, int n_exec_queues, int n_execs,
> * SUBTEST: once-cm-%s
> * Description: Run compute mode virtual exec_queue arg[1] test only once
> *
> - * Run type: FULL
> *
> * SUBTEST: twice-cm-%s
> * Description: Run compute mode virtual exec_queue arg[1] test twice
> - * Run type: BAT
> *
> * SUBTEST: many-cm-%s
> * Description: Run compute mode virtual exec_queue arg[1] test many times
> - * Run type: FULL
> *
> * SUBTEST: many-execqueues-cm-%s
> * Description: Run compute mode virtual exec_queue arg[1] test on many exec_queues
> - * Run type: FULL
> *
> *
> * SUBTEST: no-exec-cm-%s
> * Description: Run compute mode virtual exec_queue arg[1] no-exec test
> - * Run type: BAT
> *
> * arg[1]:
> *
> diff --git a/tests/intel/xe_exec_basic.c b/tests/intel/xe_exec_basic.c
> index 46b8709a76a8..e1df98be958b 100644
> --- a/tests/intel/xe_exec_basic.c
> +++ b/tests/intel/xe_exec_basic.c
> @@ -33,27 +33,21 @@
> /**
> * SUBTEST: once-%s
> * Description: Run %arg[1] test only once
> - * Run type: FULL
> *
> * SUBTEST: many-%s
> * Description: Run %arg[1] test many times
> - * Run type: FULL
> *
> * SUBTEST: many-execqueues-%s
> * Description: Run %arg[1] test on many exec_queues
> - * Run type: FULL
> *
> * SUBTEST: many-execqueues-many-vm-%s
> * Description: Run %arg[1] test on many exec_queues and many VMs
> - * Run type: FULL
> *
> * SUBTEST: twice-%s
> * Description: Run %arg[1] test twice
> - * Run type: BAT
> *
> * SUBTEST: no-exec-%s
> * Description: Run no-exec %arg[1] test
> - * Run type: BAT
> *
> * arg[1]:
> *
> diff --git a/tests/intel/xe_exec_compute_mode.c b/tests/intel/xe_exec_compute_mode.c
> index 679b84fa1fd0..8be2f2a86c35 100644
> --- a/tests/intel/xe_exec_compute_mode.c
> +++ b/tests/intel/xe_exec_compute_mode.c
> @@ -34,15 +34,12 @@
> /**
> * SUBTEST: twice-%s
> * Description: Run %arg[1] compute machine test twice
> - * Run type: BAT
> *
> * SUBTEST: once-%s
> * Description: Run %arg[1] compute machine test only once
> - * Run type: FULL
> *
> * SUBTEST: many-%s
> * Description: Run %arg[1] compute machine test many times
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -65,7 +62,6 @@
> *
> * SUBTEST: many-execqueues-%s
> * Description: Run %arg[1] compute machine test on many exec_queues
> - * Run type: FULL
> *
> * arg[1]:
> *
> diff --git a/tests/intel/xe_exec_fault_mode.c b/tests/intel/xe_exec_fault_mode.c
> index 85c010039d38..0a9883434483 100644
> --- a/tests/intel/xe_exec_fault_mode.c
> +++ b/tests/intel/xe_exec_fault_mode.c
> @@ -37,19 +37,15 @@
> /**
> * SUBTEST: once-%s
> * Description: Run %arg[1] fault mode test only once
> - * Run type: FULL
> *
> * SUBTEST: twice-%s
> * Description: Run %arg[1] fault mode test twice
> - * Run type: FULL
> *
> * SUBTEST: many-%s
> * Description: Run %arg[1] fault mode test many times
> - * Run type: FULL
> *
> * SUBTEST: many-execqueues-%s
> * Description: Run %arg[1] fault mode test on many exec_queues
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -324,20 +320,16 @@ test_exec(int fd, struct drm_xe_engine_class_instance *eci,
> /**
> * SUBTEST: atomic-once
> * Description: Run atomic fault mode test only once
> - * Run type: FULL
> *
> * SUBTEST: atomic-once-wait
> * Description: Run atomic wait fault mode test once
> - * Run type: FULL
> *
> * SUBTEST: atomic-many
> * Description: Run atomic fault mode test many times
> * Description: atomic many
> - * Run type: FULL
> *
> * SUBTEST: atomic-many-wait
> * Description: Run atomic wait fault mode test many times
> - * Run type: FULL
> *
> */
> static void
> diff --git a/tests/intel/xe_exec_reset.c b/tests/intel/xe_exec_reset.c
> index a2d33baf13cf..f12af4d9217e 100644
> --- a/tests/intel/xe_exec_reset.c
> +++ b/tests/intel/xe_exec_reset.c
> @@ -24,7 +24,6 @@
> /**
> * SUBTEST: spin
> * Description: test spin
> - * Run type: FULL
> */
> static void test_spin(int fd, struct drm_xe_engine_class_instance *eci)
> {
> @@ -105,35 +104,27 @@ static void test_spin(int fd, struct drm_xe_engine_class_instance *eci)
> /**
> * SUBTEST: %s-cancel
> * Description: Test %arg[1] cancel
> - * Run type: FULL
> *
> * SUBTEST: %s-execqueue-reset
> * Description: Test %arg[1] exec_queue reset
> - * Run type: FULL
> *
> * SUBTEST: %s-cat-error
> * Description: Test %arg[1] cat error
> - * Run type: FULL
> *
> * SUBTEST: %s-gt-reset
> * Description: Test %arg[1] GT reset
> - * Run type: FULL
> *
> * SUBTEST: virtual-close-fd-no-exec
> * Description: Test virtual close fd no-exec
> - * Run type: BAT
> *
> * SUBTEST: parallel-close-fd-no-exec
> * Description: Test parallel close fd no-exec
> - * Run type: FULL
> *
> * SUBTEST: %s-close-fd
> * Description: Test %arg[1] close fd
> - * Run type: FULL
> *
> * SUBTEST: %s-close-execqueues-close-fd
> * Description: Test %arg[1] close exec_queues close fd
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -316,31 +307,24 @@ test_balancer(int fd, int gt, int class, int n_exec_queues, int n_execs,
> /**
> * SUBTEST: cancel
> * Description: Test cancel
> - * Run type: FULL
> *
> * SUBTEST: execqueue-reset
> * Description: Test exec_queue reset
> - * Run type: FULL
> *
> * SUBTEST: cat-error
> * Description: Test cat error
> - * Run type: FULL
> *
> * SUBTEST: gt-reset
> * Description: Test GT reset
> - * Run type: FULL
> *
> * SUBTEST: close-fd-no-exec
> * Description: Test close fd no-exec
> - * Run type: BAT
> *
> * SUBTEST: close-fd
> * Description: Test close fd
> - * Run type: FULL
> *
> * SUBTEST: close-execqueues-close-fd
> * Description: Test close exec_queues close fd
> - * Run type: FULL
> */
>
> static void
> @@ -489,27 +473,21 @@ test_legacy_mode(int fd, struct drm_xe_engine_class_instance *eci,
> /**
> * SUBTEST: cm-execqueue-reset
> * Description: Test compute mode exec_queue reset
> - * Run type: FULL
> *
> * SUBTEST: cm-cat-error
> * Description: Test compute mode cat-error
> - * Run type: FULL
> *
> * SUBTEST: cm-gt-reset
> * Description: Test compute mode GT reset
> - * Run type: FULL
> *
> * SUBTEST: cm-close-fd-no-exec
> * Description: Test compute mode close fd no-exec
> - * Run type: BAT
> *
> * SUBTEST: cm-close-fd
> * Description: Test compute mode close fd
> - * Run type: FULL
> *
> * SUBTEST: cm-close-execqueues-close-fd
> * Description: Test compute mode close exec_queues close fd
> - * Run type: FULL
> */
>
> static void
> @@ -744,7 +722,6 @@ static void *gt_reset_thread(void *data)
> * SUBTEST: gt-reset-stress
> * Description: Stress GT reset
> * Test category: stress test
> - * Run type: stress
> *
> */
> static void
> diff --git a/tests/intel/xe_exec_store.c b/tests/intel/xe_exec_store.c
> index bab5ea5bca49..14f7c9becdb2 100644
> --- a/tests/intel/xe_exec_store.c
> +++ b/tests/intel/xe_exec_store.c
> @@ -51,7 +51,6 @@ static void store_dword_batch(struct data *data, uint64_t addr, int value)
> /**
> * SUBTEST: basic-store
> * Description: Basic test to verify store dword.
> - * Run type: BAT
> */
> static void store(int fd)
> {
> @@ -109,7 +108,6 @@ static void store(int fd)
> /**
> * SUBTEST: basic-all
> * Description: Test to verify store dword on all available engines.
> - * Run type: BAT
> */
> static void store_all(int fd, int gt, int class)
> {
> diff --git a/tests/intel/xe_exec_threads.c b/tests/intel/xe_exec_threads.c
> index e64c1639a0e7..ae4da0568a80 100644
> --- a/tests/intel/xe_exec_threads.c
> +++ b/tests/intel/xe_exec_threads.c
> @@ -790,7 +790,6 @@ static void *vm_async_ops_err_thread(void *data)
> /**
> * SUBTEST: threads-%s
> * Description: Run threads %arg[1] test with multi threads
> - * Run type: BAT
> *
> * arg[1]:
> *
> @@ -810,7 +809,6 @@ static void *vm_async_ops_err_thread(void *data)
> /**
> * SUBTEST: threads-%s
> * Description: Run threads %arg[1] test with multi threads
> - * Run type: FULL
> * arg[1]:
> * @basic:
> * basic
> diff --git a/tests/intel/xe_exercise_blt.c b/tests/intel/xe_exercise_blt.c
> index 34522c0f8e5f..24cbea1712e0 100644
> --- a/tests/intel/xe_exercise_blt.c
> +++ b/tests/intel/xe_exercise_blt.c
> @@ -18,7 +18,6 @@
> * Sub-category: IGT Lib
> * Functionality: blitter
> * Description: Exercise blitter commands on Xe
> - * Run type: FULL
> * Test category: functionality test
> *
> * SUBTEST: fast-copy
> diff --git a/tests/intel/xe_gpgpu_fill.c b/tests/intel/xe_gpgpu_fill.c
> index 4da01ee8c48e..eee77d2520c1 100644
> --- a/tests/intel/xe_gpgpu_fill.c
> +++ b/tests/intel/xe_gpgpu_fill.c
> @@ -80,7 +80,6 @@ static void buf_check(uint8_t *ptr, int x, int y, uint8_t color)
> /**
> * SUBTEST: basic
> * Description: run gpgpu fill
> - * Run type: FULL
> */
>
> static void gpgpu_fill(data_t *data, igt_fillfunc_t fill, uint32_t region)
> diff --git a/tests/intel/xe_guc_pc.c b/tests/intel/xe_guc_pc.c
> index 032816921f3d..a664f0cff5df 100644
> --- a/tests/intel/xe_guc_pc.c
> +++ b/tests/intel/xe_guc_pc.c
> @@ -174,7 +174,6 @@ static uint32_t get_freq(int fd, int gt_id, const char *freq_name)
> /**
> * SUBTEST: freq_basic_api
> * Description: Test basic get and set frequency API
> - * Run type: BAT
> */
>
> static void test_freq_basic_api(int fd, int gt_id)
> @@ -213,11 +212,9 @@ static void test_freq_basic_api(int fd, int gt_id)
> /**
> * SUBTEST: freq_fixed_idle
> * Description: Test fixed frequency request with exec_queue in idle state
> - * Run type: BAT
> *
> * SUBTEST: freq_fixed_exec
> * Description: Test fixed frequency request when exec_queue is doing some work
> - * Run type: FULL
> */
>
> static void test_freq_fixed(int fd, int gt_id, bool gt_idle)
> @@ -279,11 +276,9 @@ static void test_freq_fixed(int fd, int gt_id, bool gt_idle)
> /**
> * SUBTEST: freq_range_idle
> * Description: Test range frequency request with exec_queue in idle state
> - * Run type: BAT
> *
> * SUBTEST: freq_range_exec
> * Description: Test range frequency request when exec_queue is doing some work
> - * Run type: FULL
> */
>
> static void test_freq_range(int fd, int gt_id, bool gt_idle)
> @@ -314,7 +309,6 @@ static void test_freq_range(int fd, int gt_id, bool gt_idle)
> /**
> * SUBTEST: freq_low_max
> * Description: Test frequency request to minimal and maximum values
> - * Run type: FULL
> */
>
> static void test_freq_low_max(int fd, int gt_id)
> @@ -336,7 +330,6 @@ static void test_freq_low_max(int fd, int gt_id)
> /**
> * SUBTEST: freq_suspend
> * Description: Check frequency after returning from suspend
> - * Run type: FULL
> */
>
> static void test_suspend(int fd, int gt_id)
> @@ -358,11 +351,9 @@ static void test_suspend(int fd, int gt_id)
> /**
> * SUBTEST: freq_reset
> * Description: test frequency reset only once
> - * Run type: FULL
> *
> * SUBTEST: freq_reset_multiple
> * Description: test frequency reset multiple times
> - * Run type: FULL
> */
>
> static void test_reset(int fd, int gt_id, int cycles)
> diff --git a/tests/intel/xe_huc_copy.c b/tests/intel/xe_huc_copy.c
> index cbc30a8f67ab..c9891a72903a 100644
> --- a/tests/intel/xe_huc_copy.c
> +++ b/tests/intel/xe_huc_copy.c
> @@ -98,7 +98,6 @@ gen12_create_batch_huc_copy(uint32_t *batch,
>
> /**
> * SUBTEST: huc_copy
> - * Run type: BAT
> * Description:
> * Loads the HuC copy firmware to copy the content of
> * the source buffer to the destination buffer. *
> diff --git a/tests/intel/xe_intel_bb.c b/tests/intel/xe_intel_bb.c
> index a5531cf80f8e..ddfc707523fa 100644
> --- a/tests/intel/xe_intel_bb.c
> +++ b/tests/intel/xe_intel_bb.c
> @@ -117,7 +117,6 @@ static void print_buf(struct intel_buf *buf, const char *name)
> /**
> * SUBTEST: reset-bb
> * Description: check bb reset
> - * Run type: FULL
> */
> static void reset_bb(struct buf_ops *bops)
> {
> @@ -132,7 +131,6 @@ static void reset_bb(struct buf_ops *bops)
> /**
> * SUBTEST: purge-bb
> * Description: check bb reset == full (purge)
> - * Run type: FULL
> */
> static void purge_bb(struct buf_ops *bops)
> {
> @@ -164,7 +162,6 @@ static void purge_bb(struct buf_ops *bops)
> /**
> * SUBTEST: simple-%s
> * Description: Run simple bb xe %arg[1] test
> - * Run type: BAT
> *
> * arg[1]:
> *
> @@ -213,7 +210,6 @@ static void simple_bb(struct buf_ops *bops, bool new_context)
> /**
> * SUBTEST: bb-with-allocator
> * Description: check bb with passed allocator
> - * Run type: FULL
> */
> static void bb_with_allocator(struct buf_ops *bops)
> {
> @@ -245,7 +241,6 @@ static void bb_with_allocator(struct buf_ops *bops)
> /**
> * SUBTEST: lot-of-buffers
> * Description: check running bb with many buffers
> - * Run type: FULL
> */
> #define NUM_BUFS 500
> static void lot_of_buffers(struct buf_ops *bops)
> @@ -285,7 +280,6 @@ static void lot_of_buffers(struct buf_ops *bops)
> /**
> * SUBTEST: add-remove-objects
> * Description: check bb object manipulation (add + remove)
> - * Run type: FULL
> */
> static void add_remove_objects(struct buf_ops *bops)
> {
> @@ -328,7 +322,6 @@ static void add_remove_objects(struct buf_ops *bops)
> /**
> * SUBTEST: destroy-bb
> * Description: check bb destroy/create
> - * Run type: FULL
> */
> static void destroy_bb(struct buf_ops *bops)
> {
> @@ -387,7 +380,6 @@ static void destroy_bb(struct buf_ops *bops)
> /**
> * SUBTEST: create-in-region
> * Description: check size validation on available regions
> - * Run type: FULL
> */
> static void create_in_region(struct buf_ops *bops, uint64_t region)
> {
> @@ -435,7 +427,6 @@ static void __emit_blit(struct intel_bb *ibb,
> /**
> * SUBTEST: blit-%s
> * Description: Run blit on %arg[1] allocator
> - * Run type: BAT
> *
> * arg[1]:
> *
> @@ -761,7 +752,6 @@ static int __do_intel_bb_blit(struct buf_ops *bops, uint32_t tiling)
> /**
> * SUBTEST: intel-bb-blit-%s
> * Description: Run simple bb xe %arg[1] test
> - * Run type: BAT
> *
> * arg[1]:
> *
> @@ -786,7 +776,6 @@ static void do_intel_bb_blit(struct buf_ops *bops, int loops, uint32_t tiling)
> /**
> * SUBTEST: offset-control
> * Description: check offset is kept on default simple allocator
> - * Run type: FULL
> */
> static void offset_control(struct buf_ops *bops)
> {
> @@ -870,7 +859,6 @@ static void offset_control(struct buf_ops *bops)
> /**
> * SUBTEST: delta-check
> * Description: check delta is honoured in intel-bb pipelines
> - * Run type: FULL
> */
> #define DELTA_BUFFERS 3
> static void delta_check(struct buf_ops *bops)
> @@ -935,7 +923,6 @@ static void delta_check(struct buf_ops *bops)
> /**
> * SUBTEST: render
> * Description: check intel-bb render pipeline
> - * Run type: FULL
> */
> static int render(struct buf_ops *bops, uint32_t tiling,
> uint32_t width, uint32_t height)
> diff --git a/tests/intel/xe_live_ktest.c b/tests/intel/xe_live_ktest.c
> index 779ff7efd402..fe7b2e69f49b 100644
> --- a/tests/intel/xe_live_ktest.c
> +++ b/tests/intel/xe_live_ktest.c
> @@ -8,7 +8,6 @@
> * Sub-category: kunit
> * Functionality: kunit
> * Test category: functionality test
> - * Run type: BAT, kunit
> *
> * SUBTEST: bo
> * Functionality: bo
> diff --git a/tests/intel/xe_media_fill.c b/tests/intel/xe_media_fill.c
> index 4b2c235fad76..34a321d042a3 100644
> --- a/tests/intel/xe_media_fill.c
> +++ b/tests/intel/xe_media_fill.c
> @@ -33,7 +33,6 @@
> * Description: Basic test for the media_fill() function,
> * a very simple workload for the Media pipeline.
> * Feature: media
> - * Run type: FULL
> *
> * SUBTEST: media-fill
> */
> diff --git a/tests/intel/xe_mmap.c b/tests/intel/xe_mmap.c
> index 8a70a5c35fd6..71344ac818fc 100644
> --- a/tests/intel/xe_mmap.c
> +++ b/tests/intel/xe_mmap.c
> @@ -9,7 +9,6 @@
> * Sub-category: mmap
> * Functionality: mmap
> * Test category: functionality test
> - * Run type: BAT
> */
>
> #include "igt.h"
> diff --git a/tests/intel/xe_mmio.c b/tests/intel/xe_mmio.c
> index 02e79d7e4a72..285700219e6d 100644
> --- a/tests/intel/xe_mmio.c
> +++ b/tests/intel/xe_mmio.c
> @@ -9,7 +9,6 @@
> * Sub-category: mmio
> * Functionality: mmap
> * Test category: functionality test
> - * Run type: BAT
> */
>
> #include "igt.h"
> diff --git a/tests/intel/xe_module_load.c b/tests/intel/xe_module_load.c
> index 75ee948a79d7..080e12ec88bc 100644
> --- a/tests/intel/xe_module_load.c
> +++ b/tests/intel/xe_module_load.c
> @@ -100,27 +100,21 @@ static const char * const unwanted_drivers[] = {
> /**
> * SUBTEST: force-load
> * Description: Load the Xe driver passing ``force_probe=*`` parameter
> - * Run type: BAT, boot
> *
> * SUBTEST: load
> * Description: Load the Xe driver
> - * Run type: FULL
> *
> * SUBTEST: unload
> * Description: Unload the Xe driver
> - * Run type: FULL
> *
> * SUBTEST: reload
> * Description: Reload the Xe driver
> - * Run type: FULL
> *
> * SUBTEST: reload-no-display
> * Description: Reload the Xe driver passing ``enable_display=0`` parameter
> - * Run type: FULL
> *
> * SUBTEST: many-reload
> * Description: Reload the Xe driver many times
> - * Run type: FULL
> */
> igt_main
> {
> diff --git a/tests/intel/xe_noexec_ping_pong.c b/tests/intel/xe_noexec_ping_pong.c
> index cc61a77c1810..f1ae7b9633c3 100644
> --- a/tests/intel/xe_noexec_ping_pong.c
> +++ b/tests/intel/xe_noexec_ping_pong.c
> @@ -35,7 +35,6 @@
> *
> * The correct driver behaviour should be not to rebind anything unless
> * there is worked queued on one of the VM's compute exec_queues.
> - * Run type: FULL
> */
> static void test_ping_pong(int fd, struct drm_xe_engine_class_instance *eci)
> {
> diff --git a/tests/intel/xe_pm.c b/tests/intel/xe_pm.c
> index 2ec108d14b14..658f6bc98fe9 100644
> --- a/tests/intel/xe_pm.c
> +++ b/tests/intel/xe_pm.c
> @@ -8,7 +8,6 @@
> * Category: Software building block
> * Sub-category: power management
> * Test category: functionality test
> - * Run type: FULL
> */
>
> #include <limits.h>
> @@ -369,7 +368,6 @@ NULL));
> * Description:
> * Validate whether card is limited to d3hot while vram used
> * is greater than vram_d3cold_threshold.
> - * Run type: FULL
> */
> static void test_vram_d3cold_threshold(device_t device, int sysfs_fd)
> {
> diff --git a/tests/intel/xe_pm_residency.c b/tests/intel/xe_pm_residency.c
> index fd4876eaf478..f7de0bdbfe4f 100644
> --- a/tests/intel/xe_pm_residency.c
> +++ b/tests/intel/xe_pm_residency.c
> @@ -39,17 +39,14 @@ enum test_type {
> /**
> * SUBTEST: gt-c6-on-idle
> * Description: Validate GT C6 state on idle
> - * Run type: BAT
> *
> * SUBTEST: idle-residency
> * Description: basic residency test to validate idle residency
> * measured over a time interval is within the tolerance
> - * Run type: FULL
> *
> * SUBTEST: gt-c6-freeze
> * Description: Validate idle residency measured over suspend(s2idle)
> * is greater than suspend time or within tolerance
> - * Run type: FULL
> */
> IGT_TEST_DESCRIPTION("Tests for gtidle properties");
>
> diff --git a/tests/intel/xe_prime_self_import.c b/tests/intel/xe_prime_self_import.c
> index bfa0d5e34122..994fd99f5bf5 100644
> --- a/tests/intel/xe_prime_self_import.c
> +++ b/tests/intel/xe_prime_self_import.c
> @@ -93,7 +93,6 @@ check_bo(int fd1, uint32_t handle1, int fd2, uint32_t handle2)
> /**
> * SUBTEST: basic-with_fd_dup
> * Description: basic prime import/export with fd_dup
> - * Run type: BAT
> */
>
> static void test_with_fd_dup(void)
> @@ -127,7 +126,6 @@ static void test_with_fd_dup(void)
> /**
> * SUBTEST: basic-with_two_bos
> * Description: basic prime import/export with two BOs
> - * Run type: FULL
> */
>
> static void test_with_two_bos(void)
> @@ -166,7 +164,6 @@ static void test_with_two_bos(void)
> /**
> * SUBTEST: basic-with_one_bo_two_files
> * Description: basic prime import/export with one BO and two files
> - * Run type: FULL
> */
>
> static void test_with_one_bo_two_files(void)
> @@ -200,7 +197,6 @@ static void test_with_one_bo_two_files(void)
> /**
> * SUBTEST: basic-with_one_bo
> * Description: basic prime import/export with one BO
> - * Run type: BAT
> */
>
> static void test_with_one_bo(void)
> @@ -272,7 +268,6 @@ static void *thread_fn_reimport_vs_close(void *p)
> /**
> * SUBTEST: reimport-vs-gem_close-race
> * Description: Reimport versus gem_close race
> - * Run type: FULL
> */
>
> static void test_reimport_close_race(void)
> @@ -370,7 +365,6 @@ static void *thread_fn_export_vs_close(void *p)
> /**
> * SUBTEST: export-vs-gem_close-race
> * Description: Export versus gem_close race test
> - * Run type: FULL
> */
>
> static void test_export_close_race(void)
> @@ -425,7 +419,6 @@ static void test_export_close_race(void)
> /**
> * SUBTEST: basic-llseek-size
> * Description: basic BO llseek size test
> - * Run type: FULL
> */
>
> static void test_llseek_size(void)
> @@ -458,7 +451,6 @@ static void test_llseek_size(void)
> /**
> * SUBTEST: basic-llseek-bad
> * Description: basid bad BO llseek size test
> - * Run type: FULL
> */
>
> static void test_llseek_bad(void)
> diff --git a/tests/intel/xe_query.c b/tests/intel/xe_query.c
> index a4e40afdd487..1cc074255ff6 100644
> --- a/tests/intel/xe_query.c
> +++ b/tests/intel/xe_query.c
> @@ -9,7 +9,6 @@
> * Sub-category: ioctl
> * Functionality: device
> * Test category: functionality test
> - * Run type: BAT
> * Description: Acquire configuration data for xe device
> */
>
> diff --git a/tests/intel/xe_spin_batch.c b/tests/intel/xe_spin_batch.c
> index 26f9daf362e7..a8af388dde22 100644
> --- a/tests/intel/xe_spin_batch.c
> +++ b/tests/intel/xe_spin_batch.c
> @@ -17,7 +17,6 @@
> /**
> * SUBTEST: spin-basic
> * Description: Basic test to submit spin batch submissons on copy engine.
> - * Run type: FULL
> */
>
> static void spin_basic(int fd)
> @@ -35,7 +34,6 @@ static void spin_basic(int fd)
> /**
> * SUBTEST: spin-batch
> * Description: Create vm and engine of hwe class and run the spinner on it.
> - * Run type: FULL
> */
>
> static void spin(int fd, struct drm_xe_engine_class_instance *hwe)
> @@ -61,7 +59,6 @@ static void spin(int fd, struct drm_xe_engine_class_instance *hwe)
> /**
> * SUBTEST: spin-basic-all
> * Description: Basic test which validates the functionality of spinner on all hwe.
> - * Run type: FULL
> */
> static void spin_basic_all(int fd)
> {
> @@ -92,7 +89,6 @@ static void spin_basic_all(int fd)
> /**
> * SUBTEST: spin-all
> * Description: Spinner test to run on all the engines!
> - * Run type: FULL
> */
>
> static void spin_all(int fd, int gt, int class)
> diff --git a/tests/intel/xe_sysfs_defaults.c b/tests/intel/xe_sysfs_defaults.c
> index 0fc2918b621a..a1cbfb11d7dd 100644
> --- a/tests/intel/xe_sysfs_defaults.c
> +++ b/tests/intel/xe_sysfs_defaults.c
> @@ -7,7 +7,6 @@
> * TEST: xe sysfs defaults
> * Category: Infrastructure
> * Functionality: sysman defaults
> - * Run type: FULL
> * Sub-category: sysman
> * Test category: functionality test
> * SUBTEST: engine-defaults
> diff --git a/tests/intel/xe_sysfs_scheduler.c b/tests/intel/xe_sysfs_scheduler.c
> index 0315ce915db9..1cfaf208b43b 100644
> --- a/tests/intel/xe_sysfs_scheduler.c
> +++ b/tests/intel/xe_sysfs_scheduler.c
> @@ -8,7 +8,6 @@
> * Sub-category: sysman
> * Functionality: scheduler control interface
> * Test category: functionality test
> - * Run type: FULL
> *
> * SUBTEST: %s-invalid
> * Description: Test to check if %s arg[1] schedule parameter rejects any unrepresentable intervals.
> diff --git a/tests/intel/xe_vm.c b/tests/intel/xe_vm.c
> index e42c04e33c33..bd2466d84cf8 100644
> --- a/tests/intel/xe_vm.c
> +++ b/tests/intel/xe_vm.c
> @@ -84,7 +84,6 @@ write_dwords(int fd, uint32_t vm, int n_dwords, uint64_t *addrs)
> * SUBTEST: scratch
> * Description: Test scratch page creation and write
> * Functionality: scratch page
> - * Run type: FULL
> */
>
> static void
> @@ -206,7 +205,6 @@ uint64_t addrs_57b[] = {
> * SUBTEST: bind-once
> * Description: bind once on one BO
> * Functionality: bind BO
> - * Run type: FULL
> */
>
> static void
> @@ -223,7 +221,6 @@ test_bind_once(int fd)
> * SUBTEST: bind-one-bo-many-times
> * Description: bind many times on one BO
> * Functionality: bind BO
> - * Run type: FULL
> */
>
> static void
> @@ -243,7 +240,6 @@ test_bind_one_bo_many_times(int fd)
> * SUBTEST: bind-one-bo-many-times-many-vm
> * Description: Test bind many times and many VM on one BO
> * Functionality: bind BO
> - * Run type: FULL
> */
>
> static void
> @@ -261,7 +257,6 @@ test_bind_one_bo_many_times_many_vm(int fd)
> * SUBTEST: unbind-all-%d-vmas
> * Description: Test unbind all with %arg[1] VMAs
> * Functionality: unbind
> - * Run type: FULL
> *
> * arg[1].values: 2, 8
> */
> @@ -301,7 +296,6 @@ static void unbind_all(int fd, int n_vmas)
> * Verifies that mapping an invalid userptr returns -EFAULT,
> * and that it is correctly handled.
> * Functionality: userptr
> - * Run type: FULL
> */
> static void userptr_invalid(int fd)
> {
> @@ -337,12 +331,10 @@ struct vm_thread_data {
> * SUBTEST: vm-async-ops-err
> * Description: Test VM async ops error
> * Functionality: VM
> - * Run type: FULL
> *
> * SUBTEST: vm-async-ops-err-destroy
> * Description: Test VM async ops error destroy
> * Functionality: VM
> - * Run type: FULL
> */
>
> static void *vm_async_ops_err_thread(void *data)
> @@ -502,7 +494,6 @@ static void vm_async_ops_err(int fd, bool destroy)
> /**
> * SUBTEST: shared-%s-page
> * Description: Test shared arg[1] page
> - * Run type: BAT
> *
> * Functionality: %arg[1] page
> * arg[1].values: pte, pde, pde2, pde3
> @@ -690,12 +681,10 @@ shared_pte_page(int fd, struct drm_xe_engine_class_instance *eci, int n_bo,
> * SUBTEST: bind-execqueues-independent
> * Description: Test independent bind exec_queues
> * Functionality: bind exec_queues
> - * Run type: BAT
> *
> * SUBTEST: bind-execqueues-conflict
> * Description: Test conflict bind exec_queues
> * Functionality: bind exec_queues
> - * Run type: BAT
> */
>
> #define CONFLICT (0x1 << 0)
> @@ -864,22 +853,18 @@ test_bind_execqueues_independent(int fd, struct drm_xe_engine_class_instance *ec
> * SUBTEST: bind-array-twice
> * Description: Test bind array twice
> * Functionality: bind exec_queues
> - * Run type: FULL
> *
> * SUBTEST: bind-array-many
> * Description: Test bind array many times
> * Functionality: bind exec_queues
> - * Run type: FULL
> *
> * SUBTEST: bind-array-exec_queue-twice
> * Description: Test bind array exec_queue twice
> * Functionality: bind exec_queues
> - * Run type: FULL
> *
> * SUBTEST: bind-array-exec_queue-many
> * Description: Test bind array exec_queue many times
> * Functionality: bind exec_queues
> - * Run type: FULL
> */
> static void
> test_bind_array(int fd, struct drm_xe_engine_class_instance *eci, int n_execs,
> @@ -1005,7 +990,6 @@ test_bind_array(int fd, struct drm_xe_engine_class_instance *eci, int n_execs,
> * SUBTEST: %s-%ld
> * Description: Test %arg[1] with %arg[2] bind size
> * Functionality: bind
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -1023,7 +1007,6 @@ test_bind_array(int fd, struct drm_xe_engine_class_instance *eci, int n_execs,
> * SUBTEST: %s-%ld
> * Description: Test %arg[1] with %arg[2] bind size
> * Functionality: userptr bind
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -1042,7 +1025,6 @@ test_bind_array(int fd, struct drm_xe_engine_class_instance *eci, int n_execs,
> * SUBTEST: %s-%ld
> * Description: Test %arg[1] with %arg[2] bind size
> * Functionality: mixed bind
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -1057,7 +1039,6 @@ test_bind_array(int fd, struct drm_xe_engine_class_instance *eci, int n_execs,
> * SUBTEST: %s-%ld
> * Description: Test %arg[1] with %arg[2] bind size
> * Functionality: mixed bind
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -1309,14 +1290,12 @@ static void *hammer_thread(void *tdata)
> * @userptr-front: userptr front
> * @userptr-inval-end: userptr inval end
> * @userptr-inval-front: userptr inval front
> - * Run type: BAT
> */
>
> /**
> * SUBTEST: munmap-style-unbind-%s
> * Description: Test munmap style unbind with %arg[1]
> * Functionality: unbind
> - * Run type: FULL
> *
> * arg[1]:
> *
> @@ -1628,8 +1607,6 @@ try_again_after_invalidate:
> /**
> * SUBTEST: mmap-style-bind-%s
> * Description: Test mmap style unbind with %arg[1]
> - * Run type: FULL
> - * TODO: change ``'Run type' == FULL`` to a better category
> *
> * arg[1]:
> *
> diff --git a/tests/intel/xe_waitfence.c b/tests/intel/xe_waitfence.c
> index 4f2df2a9d878..34005fbeba81 100644
> --- a/tests/intel/xe_waitfence.c
> +++ b/tests/intel/xe_waitfence.c
> @@ -19,7 +19,6 @@
> * Sub-category: waitfence
> * Functionality: waitfence
> * Test category: functionality test
> - * Run type: BAT
> * Description: Test waitfences functionality
> */
>
> --
> 2.41.0
>
More information about the igt-dev
mailing list