[PATCH i-g-t v2] tests/kms_plane:Add more logs to skips/failure

[B, Jeevan]

> -----Original Message-----
> From: Samala, Pranay <pranay.samala at intel.com>
> Sent: Thursday, November 28, 2024 12:06 PM
> To: igt-dev at lists.freedesktop.org
> Cc: B S, Karthik <karthik.b.s at intel.com>; B, Jeevan <jeevan.b at intel.com>;
> Lattannavar, Sameer <sameer.lattannavar at intel.com>; Samala, Pranay
> <pranay.samala at intel.com>
> Subject: [PATCH i-g-t v2] tests/kms_plane:Add more logs to skips/failure
> 
> Having some logs for test failures and skips would make debugging much easier.
> 
> v2: Remove the extra logging statements (Jeevan)
> 
> Signed-off-by: Pranay Samala <pranay.samala at intel.com>
LGTM
Reviewed-by: Jeevan B <jeevan.b at intel.com>

> ---
>  tests/kms_plane.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_plane.c b/tests/kms_plane.c index 77a669dc5..b9bf025bd
> 100644
> --- a/tests/kms_plane.c
> +++ b/tests/kms_plane.c
> @@ -293,8 +293,11 @@ test_plane_position_with_output(data_t *data,
>  	create_fb_for_mode(data, mode, &green, &rect, 1, &primary_fb);
>  	igt_plane_set_fb(primary, &primary_fb);
> 
> -	if (!igt_plane_has_format_mod(sprite, DRM_FORMAT_XRGB8888,
> DRM_FORMAT_MOD_LINEAR))
> +	if (!igt_plane_has_format_mod(sprite, DRM_FORMAT_XRGB8888,
> DRM_FORMAT_MOD_LINEAR)) {
> +		igt_info("plane-%d does not supports XRGB format and Linear
> modifier\n",
> +			 sprite->index);
>  		return;
> +	}
> 
>  	igt_create_color_fb(data->drm_fd,
>  			    64, 64, /* width, height */
> @@ -493,7 +496,7 @@ test_plane_panning(data_t *data, enum pipe pipe)
>  		mode_found = true;
>  		break;
>  	}
> -	igt_require(mode_found);
> +	igt_require_f(mode_found, "All connector modes are skipped due to low
> +memory\n");
> 
>  	if (data->flags & TEST_PANNING_TOP_LEFT)
>  		test_grab_crc(data, output, pipe, &red, data->flags, &ref_crc);
> @@ -1263,7 +1266,7 @@ static void test_planar_settings(data_t *data)
>  	 * If here is added non-intel tests below require will need to be
>  	 * changed to if(..)
>  	 */
> -	igt_require(data->display.is_atomic);
> +	igt_require_f(data->display.is_atomic, "Atomic mode-set not
> +supported\n");
>  	igt_require_intel(data->drm_fd);
>  	devid = intel_get_drm_devid(data->drm_fd);
>  	igt_require(intel_display_ver(devid) >= 9);
> --
> 2.34.1