[PATCH i-g-t v2 01/10] tests/intel/xe_drm_fdinfo: Rename basic to basic-memory test

[Lucas De Marchi]

On Tue, Jul 02, 2024 at 05:25:23PM GMT, Umesh Nerlige Ramappa wrote:
>Existing basic test is checking memory info in fdinfo. Rename
>accordingly.
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
>---
> tests/intel/xe_drm_fdinfo.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>diff --git a/tests/intel/xe_drm_fdinfo.c b/tests/intel/xe_drm_fdinfo.c
>index 2ceafba24..f0aa23e2f 100644
>--- a/tests/intel/xe_drm_fdinfo.c
>+++ b/tests/intel/xe_drm_fdinfo.c

I was also expecting a change to the testlist... but it seems we don't
really have this there.

https://intel-gfx-ci.01.org/tree/intel-xe/index.html?testfilter=xe_drm_fdinfo
returns an empty result so it seems we are not even running these tests?

Cc'ing some people. For the changes here:

Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>

thanks
Lucas De Marchi

>@@ -22,8 +22,8 @@
>  * Feature: SMI, core
>  * Test category: SysMan
>  *
>- * SUBTEST: basic
>- * Description: Check if basic fdinfo content is present
>+ * SUBTEST: basic-memory
>+ * Description: Check if basic fdinfo content is present for memory
>  *
>  * SUBTEST: drm-total-resident
>  * Description: Create and compare total and resident memory consumption by client
>@@ -259,7 +259,7 @@ static void test_total_resident(int xe)
> 	xe_vm_destroy(xe, vm);
> }
>
>-static void basic(int xe)
>+static void basic_memory(int xe)
> {
> 	struct drm_xe_mem_region *memregion;
> 	uint64_t memreg = all_memory_regions(xe), region;
>@@ -299,9 +299,9 @@ igt_main
> 		igt_require(igt_parse_drm_fdinfo(xe, &info, NULL, 0, NULL, 0));
> 	}
>
>-	igt_describe("Check if basic fdinfo content is present");
>-	igt_subtest("basic")
>-		basic(xe);
>+	igt_describe("Check if basic fdinfo content is present for memory");
>+	igt_subtest("basic-memory")
>+		basic_memory(xe);
>
> 	igt_describe("Create and compare total and resident memory consumption by client");
> 	igt_subtest("drm-total-resident")
>-- 
>2.38.1
>