]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
drm/xe/kunit: Drop xe_wa_test_exit
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Fri, 29 Aug 2025 17:19:21 +0000 (19:19 +0200)
committerMichal Wajdeczko <michal.wajdeczko@intel.com>
Fri, 5 Sep 2025 10:57:29 +0000 (12:57 +0200)
Remove xe_wa_test_exit() as it could crach the KUnit kernel in
case of hitting some asserts in xe_wa_test_init() as test->priv
could not be pointing to expected data.

 |    # xe_wa_gt: ASSERTION FAILED at drivers/gpu/drm/xe/tests/xe_wa_test.c:34
 |    Expected ret == 0, but
 |        ret == -19 (0xffffffffffffffed)
 |Bus error - the host /dev/shm or /tmp mount likely just ran out of space
 |Kernel panic - not syncing: Kernel mode signal 7

Note that there is no need to call drm_kunit_helper_free_device()
since our fake device allocated by drm_kunit_helper_alloc_device()
will be cleaned up automatically.

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20250829171922.572-7-michal.wajdeczko@intel.com
drivers/gpu/drm/xe/tests/xe_wa_test.c

index d67bfa3ec49413fe29d94241c272189b186b4d91..49d191043dfaafd65fc9de17491a25ca6c5637bf 100644 (file)
@@ -43,13 +43,6 @@ static int xe_wa_test_init(struct kunit *test)
        return 0;
 }
 
-static void xe_wa_test_exit(struct kunit *test)
-{
-       struct xe_device *xe = test->priv;
-
-       drm_kunit_helper_free_device(test, xe->drm.dev);
-}
-
 static void xe_wa_gt(struct kunit *test)
 {
        struct xe_device *xe = test->priv;
@@ -74,7 +67,6 @@ static struct kunit_case xe_wa_tests[] = {
 static struct kunit_suite xe_rtp_test_suite = {
        .name = "xe_wa",
        .init = xe_wa_test_init,
-       .exit = xe_wa_test_exit,
        .test_cases = xe_wa_tests,
 };