From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9110410E04E for ; Mon, 12 Jun 2023 13:15:05 +0000 (UTC) Date: Mon, 12 Jun 2023 15:15:00 +0200 From: Mauro Carvalho Chehab To: Dominik Karol Piatkowski Message-ID: <20230612151500.612491b1@maurocar-mobl2> In-Reply-To: <20230612124916.7372-9-dominik.karol.piatkowski@intel.com> References: <20230612124916.7372-1-dominik.karol.piatkowski@intel.com> <20230612124916.7372-9-dominik.karol.piatkowski@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [igt-dev] [PATCH i-g-t 8/9] KUnit: Change subtest name from all-tests to module name List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On Mon, 12 Jun 2023 14:49:15 +0200 Dominik Karol Piatkowski wrote: > Use KUnit module name instead of all-tests when no name is specified. >=20 > Signed-off-by: Dominik Karol Pi=C4=85tkowski > Cc: Janusz Krzysztofik > Cc: Mauro Carvalho Chehab Works for me. Reviewed-by: Mauro Carvalho Chehab > --- > lib/igt_kmod.c | 2 +- > tests/drm_mm.c | 42 +++++++++++++++++++++--------------------- > 2 files changed, 22 insertions(+), 22 deletions(-) >=20 > diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c > index a6e751482..1511bdef4 100644 > --- a/lib/igt_kmod.c > +++ b/lib/igt_kmod.c > @@ -855,7 +855,7 @@ void igt_kunit(const char *module_name, const char *n= ame, const char *opts) > * and for documentation. > */ > if (name =3D=3D NULL) > - name =3D "all-tests"; > + name =3D module_name; > =20 > igt_subtest_with_dynamic(name) > __igt_kunit(module_name, opts); > diff --git a/tests/drm_mm.c b/tests/drm_mm.c > index 089eae2b9..9a8b3f3fc 100644 > --- a/tests/drm_mm.c > +++ b/tests/drm_mm.c > @@ -29,123 +29,123 @@ > * Feature: mapping > * Run type: FULL > * > - * SUBTEST: all-tests > + * SUBTEST: drm_mm_test > * > - * SUBTEST: all-tests@align > + * SUBTEST: drm_mm_test@align > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@align32 > + * SUBTEST: drm_mm_test@align32 > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@align64 > + * SUBTEST: drm_mm_test@align64 > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@bottomup > + * SUBTEST: drm_mm_test@bottomup > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@color > + * SUBTEST: drm_mm_test@color > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@color_evict > + * SUBTEST: drm_mm_test@color_evict > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@color_evict_range > + * SUBTEST: drm_mm_test@color_evict_range > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@debug > + * SUBTEST: drm_mm_test@debug > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@evict > + * SUBTEST: drm_mm_test@evict > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@evict_range > + * SUBTEST: drm_mm_test@evict_range > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@frag > + * SUBTEST: drm_mm_test@frag > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@highest > + * SUBTEST: drm_mm_test@highest > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@init > + * SUBTEST: drm_mm_test@init > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@insert > + * SUBTEST: drm_mm_test@insert > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@insert_range > + * SUBTEST: drm_mm_test@insert_range > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@lowest > + * SUBTEST: drm_mm_test@lowest > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@replace > + * SUBTEST: drm_mm_test@replace > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@reserve > + * SUBTEST: drm_mm_test@reserve > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@sanitycheck > + * SUBTEST: drm_mm_test@sanitycheck > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt > * Test category: GEM_Legacy > * > - * SUBTEST: all-tests@topdown > + * SUBTEST: drm_mm_test@topdown > * Category: Infrastructure > * Description: drm_mm range manager SW validation > * Functionality: DRM memory mangemnt