[PATCH v3 i-g-t] drm-uapi/xe: remove persistence queue

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Feb 7 16:09:23 UTC 2024


Hi Tejas,
On 2024-02-07 at 16:05:41 +0530, Tejas Upadhyay wrote:
> KMD has removed support setting persistence queue property
> https://patchwork.freedesktop.org/series/129311/ here, to
> have parity remove test as well
> 
> V3:
>   - Update commit message - Kamil
>   - dont change drm-uapi/xe_drm.h manually - Lucas
> V2(Priyanka):
>   - Leave hole in uapi define to have parity with KMD
> 
> Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
> ---
>  tests/intel/xe_exec_queue_property.c | 8 --------

Now you are only removing subtest, no drm-uapi/xe_drm.h
changes, so this should be:

[PATCH i-g-t] tests/intel/xe_exec_queue_property: remove persistence subtest

Regards,
Kamil

>  1 file changed, 8 deletions(-)
> 
> diff --git a/tests/intel/xe_exec_queue_property.c b/tests/intel/xe_exec_queue_property.c
> index 53e08fb0e..4fdd4ceae 100644
> --- a/tests/intel/xe_exec_queue_property.c
> +++ b/tests/intel/xe_exec_queue_property.c
> @@ -11,8 +11,6 @@
>   * Test category: functionality test
>   * SUBTEST: priority-set-property
>   * Description: tests basic priority property by setting invalid values and positive values.
> - * SUBTEST: persistence-set-property
> - * Description: tests basic persistence property by setting positive values
>   * SUBTEST: %s-property-min-max
>   * Description: Test to check if %s arg[1] schedule parameter checks for min max values.
>   *
> @@ -214,12 +212,6 @@ igt_main
>  		igt_waitchildren();
>  	}
>  
> -	igt_subtest("persistence-set-property") {
> -		/* Tests persistence property by setting positive values. */
> -		test_set_property(xe, DRM_XE_EXEC_QUEUE_SET_PROPERTY_PERSISTENCE, 1, 0);
> -
> -	}
> -
>  	igt_subtest_group {
>  		igt_fixture {
>  			sys_fd = igt_sysfs_open(xe);
> -- 
> 2.25.1
> 


More information about the igt-dev mailing list