[PATCH i-g-t 1/3] tests/intel/xe_drm_fdinfo: Add basic-engine-utilization test

[B, Jeevan]

Please ignore. Floated by mistake. 

> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of Jeevan B
> Sent: Wednesday, July 31, 2024 8:33 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>; Joshi, Kunal1
> <kunal1.joshi at intel.com>; Manna, Animesh <animesh.manna at intel.com>;
> Nerlige Ramappa, Umesh <umesh.nerlige.ramappa at intel.com>; De Marchi,
> Lucas <lucas.demarchi at intel.com>
> Subject: [PATCH i-g-t 1/3] tests/intel/xe_drm_fdinfo: Add basic-engine-utilization
> test
> 
> From: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
> 
> Add the basic test for engine utilization.
> 
> Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
> Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>
> Link: https://lore.kernel.org/r/20240703002532.3156277-3-
> umesh.nerlige.ramappa at intel.com
> ---
>  tests/intel/xe_drm_fdinfo.c | 33 ++++++++++++++++++++++++++++++---
>  1 file changed, 30 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/intel/xe_drm_fdinfo.c b/tests/intel/xe_drm_fdinfo.c index
> f0aa23e2f..2f9661a0b 100644
> --- a/tests/intel/xe_drm_fdinfo.c
> +++ b/tests/intel/xe_drm_fdinfo.c
> @@ -14,17 +14,20 @@
>  #include "xe/xe_spin.h"
>  /**
>   * TEST: xe drm fdinfo
> - * Description: Read and verify drm client memory consumption using fdinfo
> + * Description: Read and verify drm client memory consumption and
> + engine utilization using fdinfo
>   * Category: Core
>   * Mega feature: General Core features
>   * Sub-category: driver
> - * Functionality: Per client memory statistics
> + * Functionality: Per client memory and engine utilization statistics
>   * Feature: SMI, core
>   * Test category: SysMan
>   *
>   * SUBTEST: basic-memory
>   * Description: Check if basic fdinfo content is present for memory
>   *
> + * SUBTEST: basic-engine-utilization
> + * Description: Check if basic fdinfo content is present for engine
> + utilization
> + *
>   * SUBTEST: drm-total-resident
>   * Description: Create and compare total and resident memory consumption by
> client
>   *
> @@ -35,10 +38,18 @@
>   * Description: Create and compare active memory consumption by client
>   */
> 
> -IGT_TEST_DESCRIPTION("Read and verify drm client memory consumption
> using fdinfo");
> +IGT_TEST_DESCRIPTION("Read and verify drm client memory consumption and
> +engine utilization using fdinfo");
> 
>  #define BO_SIZE (65536)
> 
> +static const char *engine_map[] = {
> +	"rcs",
> +	"bcs",
> +	"vcs",
> +	"vecs",
> +	"ccs",
> +};
> +
>  /* Subtests */
>  static void test_active(int fd, struct drm_xe_engine *engine)  { @@ -287,6
> +298,18 @@ static void basic_memory(int xe)
>  	}
>  }
> 
> +static void basic_engine_utilization(int xe) {
> +	struct drm_client_fdinfo info = { };
> +	unsigned int ret;
> +
> +	ret = igt_parse_drm_fdinfo(xe, &info, engine_map,
> +				   ARRAY_SIZE(engine_map), NULL, 0);
> +	igt_assert_f(ret != 0, "failed with err:%d\n", errno);
> +	igt_assert(!strcmp(info.driver, "xe"));
> +	igt_require(info.num_engines);
> +}
> +
>  igt_main
>  {
>  	int xe;
> @@ -303,6 +326,10 @@ igt_main
>  	igt_subtest("basic-memory")
>  		basic_memory(xe);
> 
> +	igt_describe("Check if basic fdinfo content is present for engine
> utilization");
> +	igt_subtest("basic-engine-utilization")
> +		basic_engine_utilization(xe);
> +
>  	igt_describe("Create and compare total and resident memory
> consumption by client");
>  	igt_subtest("drm-total-resident")
>  		test_total_resident(xe);
> --
> 2.25.1