Re: [PATCH i-g-t v1] tests/kms_plane_scaling: Use selective modes for scaling checks

["Sharma, Swati2" <swati2.sharma@intel.com>]

Hi Ramanaidu,

On 17-03-2026 12:22 am, Naladala Ramanaidu wrote:
> Update the scaling test to try only three representative modes:
> the first, one from the middle, and the last. This decreases test
> runtime while maintaining adequate coverage.
>
> Signed-off-by: Naladala Ramanaidu <ramanaidu.naladala@intel.com>
> ---
>   tests/kms_plane_scaling.c | 10 ++++++++--
>   1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c
> index ab619fb2b..33546d5bf 100644
> --- a/tests/kms_plane_scaling.c
> +++ b/tests/kms_plane_scaling.c
> @@ -584,9 +584,15 @@ check_scaling_pipe_plane_rot(data_t *d, igt_plane_t *plane,
>   	int commit_ret;
>   	int w, h;
>   	int width, height;
> +	uint32_t mod_count = output->config.connector->count_modes;
> +	uint32_t index[3] = {0, mod_count/2, mod_count - 1};
>   
> -	for_each_connector_mode(output) {

This change was introduced in

commit 27926dd5792a225f28eaf938134505fd44ba8872
Author: Naladala Ramanaidu <ramanaidu.naladala@intel.com>
Date:   Thu Jul 18 19:35:45 2024 +0530

     tests/kms_plane_scaling: Find display mode fitting in BW for rotations

where we were lowering modes to address BW issues, but here adhoc we are

choosing 3 modes..doesn't look right. May be we can skip 2 modes and 
check instead

of choosing first, mid and last.

++JP

> -		mode = &output->config.connector->modes[j__];
> +	for (int j = 0; j < 3; j++) {
> +
> +		if ((mod_count <= 2) && (j > 1))
> +			break;
> +
> +		mode = &output->config.connector->modes[index[j]];
>   		igt_output_override_mode(output, mode);
>   		igt_debug("Trying mode %dx%d\n",
>   			  mode->hdisplay, mode->vdisplay);