[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