[PATCH i-g-t] tests/vkms: Delete VKMS test list

[Kamil Konieczny]

Hi Maíra,
On 2024-12-01 at 08:45:06 -0300, Maíra Canal wrote:
> First, this list is severely outdated. Therefore, it doesn't make sense
> to keep an outdated test list in the repository, especially considering
> that it is not being maintained.
> 
> Second, DRM CI now supports VKMS, and developers are maintaining an
> updated list of IGT tests that VKMS supports. Thus, keeping this list
> in IGT is redundant.
> 
> Signed-off-by: Maíra Canal <mcanal at igalia.com>
> ---
>  tests/vkms_ci/README        |  21 ----

Can you at least move this into tests/vkms_ci.README

and update it to current state, for example write where more info
could be found and where/how is it testing now? It is not strong
opinion so with or without this you could add

Acked-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

Btw do you use igt_runner for testing? I am asking this
because we develop a few new features, for example there
is idea of adding facts checks, either into igt_runner or
as an outside programm runned by igt_runner hooks.


Regards,
Kamil

>  tests/vkms_ci/vkms.testlist | 234 ------------------------------------
>  2 files changed, 255 deletions(-)
>  delete mode 100644 tests/vkms_ci/README
>  delete mode 100644 tests/vkms_ci/vkms.testlist
> 
> diff --git a/tests/vkms_ci/README b/tests/vkms_ci/README
> deleted file mode 100644
> index 74a59ae8f..000000000
> --- a/tests/vkms_ci/README
> +++ /dev/null
> @@ -1,21 +0,0 @@
> -This directory contains test lists to be used for vkms's DRM support.
> -The files are passed to igt_runner/piglit with the --test-list parameter
> -directly.
> -
> -The test lists are contained in the IGT repository for several
> -reasons:
> -
> -- The lists stay synchronized with the IGT codebase.
> -- Public availability. Kernel developers can see what tests are run,
> -  and can see what changes are done to the set, when, and why.
> -- Keep track of the features implemented by the vkms DRM driver.
> -
> -=============
> -vkms.testlist
> -=============
> -
> -This test list is meant as a general test suite for the vkms DRM driver,
> -gathering all must-pass tests for the driver.
> -
> -As more features are implemented to the vkms DRM driver, the correspondent
> -tests should be added to this list.
> diff --git a/tests/vkms_ci/vkms.testlist b/tests/vkms_ci/vkms.testlist
> deleted file mode 100644
> index c3333bcb0..000000000
> --- a/tests/vkms_ci/vkms.testlist
> +++ /dev/null
> @@ -1,234 +0,0 @@
> -igt at kms_atomic@atomic-invalid-params
> -igt at kms_atomic@crtc-invalid-params
> -igt at kms_atomic@plane-cursor-legacy
> -igt at kms_atomic@plane-immutable-zpos
> -igt at kms_atomic@plane-invalid-params
> -igt at kms_atomic@plane-primary-legacy
> -igt at kms_atomic@test-only
> -igt at kms_atomic_transition@modeset-transition
> -igt at kms_atomic_transition@modeset-transition-fencing
> -igt at kms_atomic_transition@modeset-transition-nonblocking
> -igt at kms_atomic_transition@modeset-transition-nonblocking-fencing
> -igt at kms_atomic_transition@plane-all-modeset-transition
> -igt at kms_atomic_transition@plane-all-transition
> -igt at kms_atomic_transition@plane-all-transition-nonblocking
> -igt at kms_atomic_transition@plane-primary-toggle-with-vblank-wait
> -igt at kms_atomic_transition@plane-toggle-modeset-transition
> -igt at kms_atomic_transition@plane-use-after-nonblocking-unbind
> -igt at kms_bw@linear-tiling-1-displays-1920x1080p
> -igt at kms_bw@linear-tiling-1-displays-2560x1440p
> -igt at kms_bw@linear-tiling-1-displays-3840x2160p
> -igt at kms_bw@linear-tiling-2-displays-1920x1080p
> -igt at kms_bw@linear-tiling-2-displays-2560x1440p
> -igt at kms_bw@linear-tiling-2-displays-3840x2160p
> -igt at kms_concurrent@pipe-a
> -igt at kms_cursor_crc@cursor-alpha-opaque
> -igt at kms_cursor_crc@cursor-alpha-transparent
> -igt at kms_cursor_crc@cursor-dpms
> -igt at kms_cursor_crc@cursor-offscreen-128x128
> -igt at kms_cursor_crc@cursor-offscreen-128x42
> -igt at kms_cursor_crc@cursor-offscreen-256x256
> -igt at kms_cursor_crc@cursor-offscreen-256x85
> -igt at kms_cursor_crc@cursor-offscreen-32x10
> -igt at kms_cursor_crc@cursor-offscreen-32x32
> -igt at kms_cursor_crc@cursor-offscreen-512x170
> -igt at kms_cursor_crc@cursor-offscreen-512x512
> -igt at kms_cursor_crc@cursor-offscreen-64x21
> -igt at kms_cursor_crc@cursor-offscreen-64x64
> -igt at kms_cursor_crc@cursor-onscreen-128x128
> -igt at kms_cursor_crc@cursor-onscreen-128x42
> -igt at kms_cursor_crc@cursor-onscreen-256x256
> -igt at kms_cursor_crc@cursor-onscreen-256x85
> -igt at kms_cursor_crc@cursor-onscreen-32x10
> -igt at kms_cursor_crc@cursor-onscreen-32x32
> -igt at kms_cursor_crc@cursor-onscreen-512x170
> -igt at kms_cursor_crc@cursor-onscreen-512x512
> -igt at kms_cursor_crc@cursor-onscreen-64x21
> -igt at kms_cursor_crc@cursor-onscreen-64x64
> -igt at kms_cursor_crc@cursor-random-128x128
> -igt at kms_cursor_crc@cursor-random-128x42
> -igt at kms_cursor_crc@cursor-random-256x256
> -igt at kms_cursor_crc@cursor-random-256x85
> -igt at kms_cursor_crc@cursor-random-32x10
> -igt at kms_cursor_crc@cursor-random-32x32
> -igt at kms_cursor_crc@cursor-random-512x170
> -igt at kms_cursor_crc@cursor-random-512x512
> -igt at kms_cursor_crc@cursor-random-64x21
> -igt at kms_cursor_crc@cursor-random-64x64
> -igt at kms_cursor_crc@cursor-rapid-movement-128x128
> -igt at kms_cursor_crc@cursor-rapid-movement-128x42
> -igt at kms_cursor_crc@cursor-rapid-movement-256x256
> -igt at kms_cursor_crc@cursor-rapid-movement-256x85
> -igt at kms_cursor_crc@cursor-rapid-movement-32x10
> -igt at kms_cursor_crc@cursor-rapid-movement-32x32
> -igt at kms_cursor_crc@cursor-rapid-movement-512x170
> -igt at kms_cursor_crc@cursor-rapid-movement-512x512
> -igt at kms_cursor_crc@cursor-rapid-movement-64x21
> -igt at kms_cursor_crc@cursor-rapid-movement-64x64
> -igt at kms_cursor_crc@cursor-size-change
> -igt at kms_cursor_crc@cursor-sliding-128x128
> -igt at kms_cursor_crc@cursor-sliding-128x42
> -igt at kms_cursor_crc@cursor-sliding-256x256
> -igt at kms_cursor_crc@cursor-sliding-256x85
> -igt at kms_cursor_crc@cursor-sliding-32x10
> -igt at kms_cursor_crc@cursor-sliding-32x32
> -igt at kms_cursor_crc@cursor-sliding-512x170
> -igt at kms_cursor_crc@cursor-sliding-512x512
> -igt at kms_cursor_crc@cursor-sliding-64x21
> -igt at kms_cursor_crc@cursor-sliding-64x64
> -igt at kms_cursor_crc@cursor-suspend
> -igt at kms_cursor_edge_walk@128x128-left-edge
> -igt at kms_cursor_edge_walk@128x128-right-edge
> -igt at kms_cursor_edge_walk@128x128-top-bottom
> -igt at kms_cursor_edge_walk@128x128-top-edge
> -igt at kms_cursor_edge_walk@256x256-left-edge
> -igt at kms_cursor_edge_walk@256x256-right-edge
> -igt at kms_cursor_edge_walk@256x256-top-bottom
> -igt at kms_cursor_edge_walk@256x256-top-edge
> -igt at kms_cursor_edge_walk@64x64-left-edge
> -igt at kms_cursor_edge_walk@64x64-right-edge
> -igt at kms_cursor_edge_walk@64x64-top-bottom
> -igt at kms_cursor_edge_walk@64x64-top-edge
> -igt at kms_flip@absolute-wf_vblank
> -igt at kms_flip@absolute-wf_vblank-interruptible
> -igt at kms_flip@basic-flip-vs-dpms
> -igt at kms_flip@basic-flip-vs-modeset
> -igt at kms_flip@basic-flip-vs-wf_vblank
> -igt at kms_flip@basic-plain-flip
> -igt at kms_flip@blocking-absolute-wf_vblank
> -igt at kms_flip@blocking-absolute-wf_vblank-interruptible
> -igt at kms_flip@blocking-wf_vblank
> -igt at kms_flip@busy-flip
> -igt at kms_flip@dpms-off-confusion
> -igt at kms_flip@dpms-off-confusion-interruptible
> -igt at kms_flip@dpms-vs-vblank-race
> -igt at kms_flip@dpms-vs-vblank-race-interruptible
> -igt at kms_flip@flip-vs-absolute-wf_vblank
> -igt at kms_flip@flip-vs-absolute-wf_vblank-interruptible
> -igt at kms_flip@flip-vs-blocking-wf-vblank
> -igt at kms_flip@flip-vs-dpms-off-vs-modeset
> -igt at kms_flip@flip-vs-dpms-off-vs-modeset-interruptible
> -igt at kms_flip@flip-vs-expired-vblank
> -igt at kms_flip@flip-vs-expired-vblank-interruptible
> -igt at kms_flip@flip-vs-panning
> -igt at kms_flip@flip-vs-panning-interruptible
> -igt at kms_flip@flip-vs-rmfb
> -igt at kms_flip@flip-vs-rmfb-interruptible
> -igt at kms_flip@flip-vs-suspend
> -igt at kms_flip@flip-vs-suspend-interruptible
> -igt at kms_flip@flip-vs-wf_vblank-interruptible
> -igt at kms_flip@modeset-vs-vblank-race
> -igt at kms_flip@modeset-vs-vblank-race-interruptible
> -igt at kms_flip@nonblocking-read
> -igt at kms_flip@nonexisting-fb
> -igt at kms_flip@nonexisting-fb-interruptible
> -igt at kms_flip@plain-flip-fb-recreate
> -igt at kms_flip@plain-flip-fb-recreate-interruptible
> -igt at kms_flip@plain-flip-interruptible
> -igt at kms_flip@plain-flip-ts-check
> -igt at kms_flip@plain-flip-ts-check-interruptible
> -igt at kms_flip@single-buffer-flip-vs-dpms-off-vs-modeset
> -igt at kms_flip@single-buffer-flip-vs-dpms-off-vs-modeset-interruptible
> -igt at kms_flip@wf_vblank-ts-check
> -igt at kms_flip@wf_vblank-ts-check-interruptible
> -igt at kms_flip_event_leak@basic
> -igt at kms_getfb@getfb-addfb-different-handles
> -igt at kms_getfb@getfb-handle-closed
> -igt at kms_getfb@getfb-handle-not-fb
> -igt at kms_getfb@getfb-handle-protection
> -igt at kms_getfb@getfb-handle-valid
> -igt at kms_getfb@getfb-handle-zero
> -igt at kms_getfb@getfb-repeated-different-handles
> -igt at kms_getfb@getfb2-handle-closed
> -igt at kms_getfb@getfb2-handle-not-fb
> -igt at kms_getfb@getfb2-handle-protection
> -igt at kms_getfb@getfb2-handle-zero
> -igt at kms_getfb@getfb2-into-addfb2
> -igt at kms_lease@cursor_implicit_plane
> -igt at kms_lease@empty_lease
> -igt at kms_lease@implicit-plane-lease
> -igt at kms_lease@invalid-create-leases
> -igt at kms_lease@lease-uevent
> -igt at kms_lease@lease_again
> -igt at kms_lease@lease_get
> -igt at kms_lease@lease_invalid_connector
> -igt at kms_lease@lease_invalid_crtc
> -igt at kms_lease@lease_invalid_plane
> -igt at kms_lease@lease_revoke
> -igt at kms_lease@lessee_list
> -igt at kms_lease@master-vs-lease
> -igt at kms_lease@multimaster-lease
> -igt at kms_lease@page_flip_implicit_plane
> -igt at kms_lease@possible-crtcs-filtering
> -igt at kms_lease@setcrtc_implicit_plane
> -igt at kms_lease@simple_lease
> -igt at kms_multipipe_modeset@basic-max-pipe-crc-check
> -igt at kms_pipe_crc_basic@bad-source
> -igt at kms_pipe_crc_basic@compare-crc-sanitycheck
> -igt at kms_pipe_crc_basic@disable-crc-after-crtc
> -igt at kms_pipe_crc_basic@nonblocking-crc
> -igt at kms_pipe_crc_basic@nonblocking-crc-frame-sequence
> -igt at kms_pipe_crc_basic@read-crc
> -igt at kms_pipe_crc_basic@read-crc-frame-sequence
> -igt at kms_pipe_crc_basic@suspend-read-crc
> -igt at kms_plane@plane-panning-bottom-right
> -igt at kms_plane@plane-panning-bottom-right-suspend
> -igt at kms_plane@plane-panning-top-left
> -igt at kms_plane@plane-position-covered
> -igt at kms_plane@plane-position-hole
> -igt at kms_plane@plane-position-hole-dpms
> -igt at kms_plane_cursor@overlay
> -igt at kms_plane_cursor@primary
> -igt at kms_plane_cursor@viewport
> -igt at kms_plane_lowres@tiling-none
> -igt at kms_plane_multiple@tiling-none
> -igt at kms_plane_scaling@invalid-parameters
> -igt at kms_prop_blob@basic
> -igt at kms_prop_blob@blob-multiple
> -igt at kms_prop_blob@blob-prop-core
> -igt at kms_prop_blob@blob-prop-lifetime
> -igt at kms_prop_blob@blob-prop-validate
> -igt at kms_prop_blob@invalid-get-prop
> -igt at kms_prop_blob@invalid-get-prop-any
> -igt at kms_prop_blob@invalid-set-prop
> -igt at kms_prop_blob@invalid-set-prop-any
> -igt at kms_properties@connector-properties-atomic
> -igt at kms_properties@connector-properties-legacy
> -igt at kms_properties@crtc-properties-atomic
> -igt at kms_properties@crtc-properties-legacy
> -igt at kms_properties@get_properties-sanity-atomic
> -igt at kms_properties@get_properties-sanity-non-atomic
> -igt at kms_properties@invalid-properties-atomic
> -igt at kms_properties@invalid-properties-legacy
> -igt at kms_properties@plane-properties-atomic
> -igt at kms_properties@plane-properties-legacy
> -igt at kms_rmfb@close-fd
> -igt at kms_rmfb@rmfb-ioctl
> -igt at kms_sequence@get-busy
> -igt at kms_sequence@get-forked
> -igt at kms_sequence@get-forked-busy
> -igt at kms_sequence@get-idle
> -igt at kms_sequence@queue-busy
> -igt at kms_sequence@queue-idle
> -igt at kms_universal_plane@disable-primary-vs-flip-pipe-a
> -igt at kms_universal_plane@universal-plane-pageflip-windowed-pipe-a
> -igt at kms_universal_plane@universal-plane-pipe-a-functional
> -igt at kms_vblank@crtc-id
> -igt at kms_vblank@invalid
> -igt at kms_vblank@pipe-a-accuracy-idle
> -igt at kms_vblank@pipe-a-query-busy
> -igt at kms_vblank@pipe-a-query-forked
> -igt at kms_vblank@pipe-a-query-forked-busy
> -igt at kms_vblank@pipe-a-query-idle
> -igt at kms_vblank@pipe-a-ts-continuation-dpms-suspend
> -igt at kms_vblank@pipe-a-ts-continuation-idle
> -igt at kms_vblank@pipe-a-ts-continuation-modeset
> -igt at kms_vblank@pipe-a-ts-continuation-suspend
> -igt at kms_vblank@pipe-a-wait-busy
> -igt at kms_vblank@pipe-a-wait-forked
> -igt at kms_vblank@pipe-a-wait-forked-busy
> -igt at kms_vblank@pipe-a-wait-idle
> -igt at kms_writeback@writeback-check-output
> -igt at kms_writeback@writeback-fb-id
> -igt at kms_writeback@writeback-invalid-parameters
> -igt at kms_writeback@writeback-pixel-formats
> -- 
> 2.39.5 (Apple Git-154)
>