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

Lucas De Marchi lucas.demarchi at intel.com
Wed Mar 13 19:22:31 UTC 2024


On Wed, Mar 13, 2024 at 02:30:06PM -0300, Gustavo Sousa wrote:
>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. :-)

add the comma while applying.

>
>With or without this nitpick,
>
>Reviewed-by: Gustavo Sousa <gustavo.sousa at intel.com>

thanks. I pushed this and the kernel patch.

Lucas De Marchi


More information about the igt-dev mailing list