[PATCH i-g-t v2] tests/kms_color: Disable CTM at the end of CTM tests
Kamil Konieczny
kamil.konieczny at linux.intel.com
Mon Feb 17 13:52:12 UTC 2025
Hi Jessica,
On 2025-02-14 at 15:59:01 -0800, Jessica Zhang wrote:
> Currently, test_pipe_ctm() enables CTM on the primary pipe but doesn't
> disable CTM after it finishes running the tests.
>
> This can cause resource-availability issues in drivers where CTM-related
> hardware resources are shared between pipes.
>
> To avoid this, explicitly disable CTM on the primary pipe at the end of
> test_pipe_ctm()
>
> Reviewed-by: Abhinav Kumar <quic_abhinavk at quicinc.com>
> Signed-off-by: Jessica Zhang <quic_jesszhan at quicinc.com>
Thx, applied.
Regards,
Kamil
> ---
> Changes in v2:
> - Add igt-dev to email list
> - Link to v1: https://lore.kernel.org/r/20250214-disable-cwb-v1-1-65ff578ef983@quicinc.com
> ---
> tests/kms_color.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/kms_color.c b/tests/kms_color.c
> index 4b71d3dd3..83e64229f 100644
> --- a/tests/kms_color.c
> +++ b/tests/kms_color.c
> @@ -599,6 +599,7 @@ static bool test_pipe_ctm(data_t *data,
> */
> ret &= igt_skip_crc_compare || igt_check_crc_equal(&crc_software, &crc_hardware);
>
> + disable_ctm(primary->pipe);
> igt_plane_set_fb(primary, NULL);
> igt_output_set_pipe(output, PIPE_NONE);
> igt_display_commit(&data->display);
>
> ---
> base-commit: 7187a77fa2c4f1e0ea355cbc719dcd1f270148a8
> change-id: 20250214-disable-cwb-72da185ca3c6
>
> Best regards,
> --
> Jessica Zhang <quic_jesszhan at quicinc.com>
>
More information about the igt-dev
mailing list