[PATCH i-g-t] tests/intel/kms_frontbuffer_tracking: Reduce test complexity for simulation

[Reddy Guddati, Santhosh]

Hi Pranay,

On 11-07-2025 13:34, Pranay Samala wrote:
> Reduce the number of iterations, draw_method combinations, and limit
> execution to a single pipe to enable this test to run on simulation.
> 
> Signed-off-by: Pranay Samala <pranay.samala at intel.com>
> ---
>   tests/intel/kms_frontbuffer_tracking.c | 12 ++++++++++++
>   1 file changed, 12 insertions(+)
> 
> diff --git a/tests/intel/kms_frontbuffer_tracking.c b/tests/intel/kms_frontbuffer_tracking.c
> index 66dcdc281..631b13aa1 100644
> --- a/tests/intel/kms_frontbuffer_tracking.c
> +++ b/tests/intel/kms_frontbuffer_tracking.c
> @@ -2908,6 +2908,8 @@ static void multidraw_subtest(const struct test_mode *t)
>   					assertions |= ASSERT_PSR_DISABLED;
>   
>   				do_assertions(assertions);
> +				if (igt_run_in_simulation())
> +					break;
IMHO, Since the igt_run_in_simulation() is being called at multiple 
places and if there is further optimisation needed for further tests, I 
would recommend to add a static global variable and initialise it in 
main, replace all instances with this variable.  This way we can avoid 
function call overhead as the value remains constant through out the test.

Regards,
Santhosh>   			}
>   
>   			fill_fb_region(target, m2, COLOR_PRIM_BG);
> @@ -2915,7 +2917,11 @@ static void multidraw_subtest(const struct test_mode *t)
>   
>   			update_wanted_crc(t, &blue_crcs[t->format].crc);
>   			do_assertions(ASSERT_NO_ACTION_CHANGE);
> +			if (igt_run_in_simulation())
> +				break;
>   		}
> +		if (igt_run_in_simulation())
> +			break;
>   	}
>   }
>   
> @@ -3275,6 +3281,8 @@ static void move_subtest(const struct test_mode *t)
>   		update_wanted_crc(t, &pattern->crcs[t->format][r]);
>   
>   		do_assertions(assertions);
> +		if (igt_run_in_simulation())
> +			break;
>   
>   		/* "Move" the last rect to the same position just to make sure
>   		 * this works too. */
> @@ -3332,6 +3340,8 @@ static void onoff_subtest(const struct test_mode *t)
>   		}
>   
>   		do_assertions(assertions);
> +		if (igt_run_in_simulation())
> +			break;
>   	}
>   }
>   
> @@ -4184,6 +4194,8 @@ igt_main_args("", long_options, help_str, opt_handler, NULL)
>   
>   					break; /* One output is enough. */
>   				}
> +				if (igt_run_in_simulation())
> +					break;
>   			}
>   		}
>