[PATCH v2 i-g-t] tests/kms_debugfs: Use static var for display

Kamil Konieczny kamil.konieczny at linux.intel.com
Mon Jul 28 08:34:23 UTC 2025


Hi Peter,
On 2025-07-24 at 14:27:10 +0200, Peter Senna Tschudin wrote:
> Commit ebe65a923a2b ("tests: Add kms_debugfs") introduced a new test.
> Unfortunately, that commit used dynamic allocation for `display` and
> missed a call to free(display) during cleanup. Use a stack variable
> instead, eliminating the need for dynamic memory management.
> 
> Fixes: ebe65a923a2b ("tests: Add kms_debugfs")
> Cc: Karthik B S <karthik.b.s at intel.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Signed-off-by: Peter Senna Tschudin <peter.senna at linux.intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>


> ---
>  tests/kms_debugfs.c | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/tests/kms_debugfs.c b/tests/kms_debugfs.c
> index 75ef15353..570ad1c0c 100644
> --- a/tests/kms_debugfs.c
> +++ b/tests/kms_debugfs.c
> @@ -97,7 +97,7 @@ IGT_TEST_DESCRIPTION("Read entries from debugfs with display on/off.");
>  igt_main
>  {
>  	int debugfs = -1;
> -	igt_display_t *display;
> +	igt_display_t display;
>  	int fd = -1;
>  
>  	igt_fixture {
> @@ -107,28 +107,27 @@ igt_main
>  
>  		kmstest_set_vt_graphics_mode();
>  
> -		display = calloc(1, sizeof(*display));
> -		igt_display_require(display, fd);
> +		igt_display_require(&display, fd);
>  
>  		/* Make sure we have at least one output connected */
> -		igt_display_require_output(display);
> +		igt_display_require_output(&display);
>  	}
>  
>  	igt_subtest("display-off-read-all") {
> -		igt_display_all_off(display);
> +		igt_display_all_off(&display);
>  
>  		igt_dir_process_files_simple(debugfs);
>  	}
>  
>  	igt_subtest("display-on-read-all") {
>  		/* try to light all pipes */
> -		igt_display_all_on(display);
> +		igt_display_all_on(&display);
>  
>  		igt_dir_process_files_simple(debugfs);
>  	}
>  
>  	igt_fixture {
> -		igt_display_fini(display);
> +		igt_display_fini(&display);
>  		close(debugfs);
>  		drm_close_driver(fd);
>  	}
> -- 
> 2.43.0
> 


More information about the igt-dev mailing list