[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