[PATCH i-g-t v2 1/1] tests/kms_cursor_legacy: don't inhibit pipe CRC calculation

[Sharma, Swati2]

Hi Michal,

On 22-07-2025 06:18 pm, Michał Grzelak wrote:
> Running either flip-vs-cursor-busy-crc-(legacy|atomic) subtest from
> kms_cursor_legacy results in entering PSR1 after the test. Even with
> PSR2 enabled panel, it is impossible to switch to PSR2 via
> i915_edp_psr_debug. Kernel does so because it prevents inhibiting pipe
> CRC calculation which is allowed in PSR1 but prohibited in PSR2.
> Therefore subtests from kms_psr testing PSR2 feature (e.g. psr2-no-drrs,
> psr2-primary-blt etc.) are failing after running any of those
> kms_cursor_legacy subtests.
>
> Restore enabling Panel Replay Selective Update by stopping pipe CRC
> calculation after running the subtest and before the clean-up.
If there is any bug which this patch is fixing, please
add Closes tag.

Otherwise patch LGTM.
Reviewed-by: Swati Sharma <swati2.sharma at intel.com>
>
> Signed-off-by: Michał Grzelak <michal.grzelak at intel.com>
> ---
>   tests/kms_cursor_legacy.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
> index 9f6e644e1..ed1c77d55 100644
> --- a/tests/kms_cursor_legacy.c
> +++ b/tests/kms_cursor_legacy.c
> @@ -1748,13 +1748,14 @@ static void flip_vs_cursor_busy_crc(igt_display_t *display, bool atomic)
>   		igt_assert_crc_equal(&crcs[i], &test_crc);
>   	}
>   
> +	igt_pipe_crc_stop(pipe_crc);
> +
>   	/* Clean-up */
>   	igt_plane_set_fb(plane_primary, NULL);
>   	igt_plane_set_fb(cursor, NULL);
>   	igt_output_set_pipe(output, PIPE_NONE);
>   	igt_display_commit2(display, display->is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
>   
> -	igt_pipe_crc_stop(pipe_crc);
>   	igt_remove_fb(display->drm_fd, &fb_info[1]);
>   	igt_remove_fb(display->drm_fd, &fb_info[0]);
>   	igt_remove_fb(display->drm_fd, &cursor_fb);