[PATCH i-g-t] xe_live_ktest: Use xe_live_test kernel module

Gustavo Sousa gustavo.sousa at intel.com
Wed Mar 13 17:30:06 UTC 2024


Quoting Lucas De Marchi (2024-03-12 14:45:33-03:00)
>https://patchwork.freedesktop.org/patch/582554/?series=131032&rev=1
>groups all the live tests into a single kernel module. Use that
>module to run any of the tests triggered by igt.
>
>Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
>---
> tests/intel/xe_live_ktest.c | 17 ++++++-----------
> 1 file changed, 6 insertions(+), 11 deletions(-)
>
>diff --git a/tests/intel/xe_live_ktest.c b/tests/intel/xe_live_ktest.c
>index bb612bae3..08b80d4e0 100644
>--- a/tests/intel/xe_live_ktest.c
>+++ b/tests/intel/xe_live_ktest.c
>@@ -31,16 +31,11 @@
>  * Functionality: mocs
>  */
> 
>-struct kunit_tests {
>-        const char *kunit;
>-        const char *name;
>-};
>-
>-static const struct kunit_tests live_tests[] = {
>-        { "xe_bo_test",                "xe_bo" },
>-        { "xe_dma_buf_test",        "xe_dma_buf" },
>-        { "xe_migrate_test",        "xe_migrate" },
>-        { "xe_mocs_test",        "xe_mocs" },
>+static const char *live_tests[] = {
>+        "xe_bo",
>+        "xe_dma_buf",
>+        "xe_migrate",
>+        "xe_mocs"

I would keep the trailing comma. :-)

With or without this nitpick,

Reviewed-by: Gustavo Sousa <gustavo.sousa at intel.com>

> };
> 
> igt_main
>@@ -48,5 +43,5 @@ igt_main
>         int i;
> 
>         for (i = 0; i < ARRAY_SIZE(live_tests); i++)
>-                igt_kunit(live_tests[i].kunit, live_tests[i].name, NULL);
>+                igt_kunit("xe_live_test", live_tests[i], NULL);
> }
>-- 
>2.43.0
>


More information about the igt-dev mailing list