From patchwork Mon Mar 10 16:15:10 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Tvrtko Ursulin X-Patchwork-Id: 14010389 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6FF21C282DE for ; Mon, 10 Mar 2025 16:15:29 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C316210E47C; Mon, 10 Mar 2025 16:15:28 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=igalia.com header.i=@igalia.com header.b="mh8UI97A"; dkim-atps=neutral Received: from fanzine2.igalia.com (fanzine.igalia.com [178.60.130.6]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7844B10E484 for ; Mon, 10 Mar 2025 16:15:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=Content-Transfer-Encoding:Content-Type:MIME-Version:Message-ID: Date:Subject:Cc:To:From:Sender:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: In-Reply-To:References:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=xSskNVptJbvMTkPzKho3ucI4cPchjASDk5rW61oE5gc=; b=mh8UI97A7814QVAgQOl8oWg8Xs VOz7wJJqagkxjlfl49p15LtEOGM4zEm2ZSXVJbZB7CkGIp4kaxi14v5AqxAy6TE+Fxjsf0/v7vpjh eITLv+K7AbSWCOjwxcRCZbJArtiS7g2XP8y83g0WocfFp3UPZKyIGaOmq2VoQgm85HT8oB7SxAz5W 2CT8Wvw/mFd3JA34Sle56OBhmvNgoTqY/+m5AysXTzb0cDsH/0VMEPet7VlzUdu6yTmY9o5T874Or 2ys7M8md+gla4D4cBvBrJJfoHlQJ0zqyue7tTN33bmtcAKKWfUp40etaZsM3fEPm+JwDQLYuN3EWu HhRrOt2g==; Received: from [90.241.98.187] (helo=localhost) by fanzine2.igalia.com with esmtpsa (Cipher TLS1.3:ECDHE_SECP256R1__RSA_PSS_RSAE_SHA256__AES_256_GCM:256) (Exim) id 1trfma-006YAW-Oj; Mon, 10 Mar 2025 17:15:18 +0100 From: Tvrtko Ursulin To: dri-devel@lists.freedesktop.org Cc: kernel-dev@igalia.com, Tvrtko Ursulin , =?utf-8?q?Christian_K=C3=B6nig?= , Danilo Krummrich , Matthew Brost , Philipp Stanner Subject: [PATCH v6 0/6] DRM scheduler kunit tests Date: Mon, 10 Mar 2025 16:15:10 +0000 Message-ID: <20250310161516.39942-1-tvrtko.ursulin@igalia.com> X-Mailer: git-send-email 2.48.0 MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" There has repeatedly been quite a bit of apprehension when any change to the DRM scheduler is proposed, with two main reasons being code base is considered fragile, not well understood and not very well documented, and secondly the lack of systematic testing outside the vendor specific tests suites and/or test farms. This series is an attempt to dislodge this status quo by adding some unit tests using the kunit framework. General approach is that there is a mock "hardware" backend which can be controlled from tests, which in turn allows exercising various scheduler code paths. Only some simple basic tests get added in the series and hopefully it is easy to understand what tests are doing. An obligatory "screenshot" for reference: [14:09:05] ============ drm_sched_basic_tests (3 subtests) ============ [14:09:06] [PASSED] drm_sched_basic_submit [14:09:06] ================== drm_sched_basic_test =================== [14:09:06] [PASSED] A queue of jobs in a single entity [14:09:06] [PASSED] A chain of dependent jobs across multiple entities [14:09:06] [PASSED] Multiple independent job queues [14:09:06] [PASSED] Multiple inter-dependent job queues [14:09:07] ============== [PASSED] drm_sched_basic_test =============== [14:09:07] [PASSED] drm_sched_basic_entity_cleanup [14:09:07] ============== [PASSED] drm_sched_basic_tests ============== [14:09:07] ======== drm_sched_basic_timeout_tests (1 subtest) ========= [14:09:08] [PASSED] drm_sched_basic_timeout [14:09:08] ========== [PASSED] drm_sched_basic_timeout_tests ========== [14:09:08] ======= drm_sched_basic_priority_tests (2 subtests) ======== [14:09:10] [PASSED] drm_sched_priorities [14:09:10] [PASSED] drm_sched_change_priority [14:09:10] ========= [PASSED] drm_sched_basic_priority_tests ========== [14:09:10] ====== drm_sched_basic_modify_sched_tests (1 subtest) ====== [14:09:11] [PASSED] drm_sched_test_modify_sched [14:09:11] ======= [PASSED] drm_sched_basic_modify_sched_tests ======== [14:09:11] ======== drm_sched_basic_credits_tests (1 subtest) ========= [14:09:12] [PASSED] drm_sched_test_credits [14:09:12] ========== [PASSED] drm_sched_basic_credits_tests ========== [14:09:12] ============================================================ [14:09:12] Testing complete. Ran 11 tests: passed: 11 [14:09:13] Elapsed time: 13.539s total, 0.001s configuring, 3.004s building, 10.462s running v2: * Parameterize a bunch of similar tests. * Improve test commentary. * Rename TDR test to timeout. (Christian) * Improve quality and consistency of naming. (Philipp) RFC v2 -> series v1: * Rebased for drm_sched_init changes. * Fixed modular build. * Added some comments. * Filename renames. (Philipp) v2: * Dealt with a bunch of checkpatch warnings. v3: * Some mock API renames, kerneldoc grammar fixes and indentation fixes. v4: * Fix use after free caused by relying on scheduler fence for querying status. * Kerneldoc fixes. v5: * Cleanup in-flight jobs on scheduler shutdown. * Change hang_limit to 1. v6: * Use KUNIT_ASSERT_TRUE/FALSE. * Fixed patch titles. * Added credit_limit test. * Added CONFIG_DRM_SCHED_KUNIT_TEST_ASPIRATIONAL. Cc: Christian König Cc: Danilo Krummrich Cc: Matthew Brost Cc: Philipp Stanner Tvrtko Ursulin (6): drm/sched: Add scheduler unit testing infrastructure and some basic tests drm/sched: Add aspirational unit test mode drm/sched: Add a simple timeout test drm/sched: Add basic priority tests drm/sched: Add a basic test for modifying entities scheduler list drm/sched: Add a basic test for checking credit limit drivers/gpu/drm/Kconfig.debug | 25 + drivers/gpu/drm/scheduler/.kunitconfig | 12 + drivers/gpu/drm/scheduler/Makefile | 2 + drivers/gpu/drm/scheduler/tests/Makefile | 7 + .../gpu/drm/scheduler/tests/mock_scheduler.c | 345 +++++++++++++ drivers/gpu/drm/scheduler/tests/sched_tests.h | 224 +++++++++ drivers/gpu/drm/scheduler/tests/tests_basic.c | 476 ++++++++++++++++++ 7 files changed, 1091 insertions(+) create mode 100644 drivers/gpu/drm/scheduler/.kunitconfig create mode 100644 drivers/gpu/drm/scheduler/tests/Makefile create mode 100644 drivers/gpu/drm/scheduler/tests/mock_scheduler.c create mode 100644 drivers/gpu/drm/scheduler/tests/sched_tests.h create mode 100644 drivers/gpu/drm/scheduler/tests/tests_basic.c