From patchwork Tue Jan 21 15:15:42 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Philipp Stanner X-Patchwork-Id: 13946411 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 670CFC02182 for ; Tue, 21 Jan 2025 15:16:01 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id DC56F10E5E5; Tue, 21 Jan 2025 15:16:00 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="pk8xJK6G"; dkim-atps=neutral Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by gabe.freedesktop.org (Postfix) with ESMTPS id D259E10E5E5 for ; Tue, 21 Jan 2025 15:15:59 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 8087F5C06C5; Tue, 21 Jan 2025 15:15:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 388C0C4CEE3; Tue, 21 Jan 2025 15:15:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1737472558; bh=+nIjqYD/j2RTpJxB2Fsvg3Iv/g326EMXel5aJPmyIj4=; h=From:To:Cc:Subject:Date:From; b=pk8xJK6GzwaH1fN8KBMMPadPwh8ZBSvwvHqfmmAEeImMJbMyu2gNofem4rDklUdFG N3gVPIqdGutH1zAXAAjdaB7+mJYEvSnDjVmr+yncCXEX7zAYbhsufQfrasBVvr7Ar6 3MXegwxKdxsqFTan2egweANthAwMPQELa3Y8/MHEfjcyLi9kUzlgq036C0YLKgStqJ KTZBFgSFRB0l8VVj8lz358643DL0YrqFwWgFU3+iubJuURHkXKsZNOTyTX/4Qa8fAE 5tJs6FPq0/2cJQbtKi3aP8Rkx7SWtx+fCEHN6D9iBz0sdVO6XDq+srhSVkM5B5BZa2 sFj5G8yglZAFw== From: Philipp Stanner To: Matthew Brost , Danilo Krummrich , Philipp Stanner , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Sumit Semwal , =?utf-8?q?Christian_K=C3=B6nig?= Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linaro-mm-sig@lists.linaro.org, Philipp Stanner Subject: [PATCH v2 0/3] drm/sched: Documentation and refcount improvements Date: Tue, 21 Jan 2025 16:15:42 +0100 Message-ID: <20250121151544.44949-2-phasta@kernel.org> X-Mailer: git-send-email 2.47.1 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" Changes in v2: - Document what run_job() is allowed to return. (Tvrtko) - Delete confusing comment about putting the fence. (Danilo) - Apply Danilo's RB to patch 1. - Delete info about job recovery for entities in patch 3. (Danilo, me) - Set the term "ring" as fix term for both HW rings and FW rings. A ring shall always be the thingy on the CPU ;) (Danilo) - Many (all) other comments improvements in patch 3. (Danilo) This is as series succeeding my previous patch [1]. I recognized that we are still referring to a non-existing function and a deprecated one in the callback docu. We should probably also point out the important distinction between hardware and firmware schedulers more cleanly. Please give me feedback, especially on the RFC comments in patch3. (This series still fires docu-build-warnings. I want to gather feedback on the opion questions first and will solve them in v2.) Thank you, Philipp [1] https://lore.kernel.org/all/20241220124515.93169-2-phasta@kernel.org/ Philipp Stanner (3): drm/sched: Document run_job() refcount hazard drm/sched: Adjust outdated docu for run_job() drm/sched: Update timedout_job()'s documentation drivers/gpu/drm/scheduler/sched_main.c | 5 +- include/drm/gpu_scheduler.h | 106 ++++++++++++++++--------- 2 files changed, 71 insertions(+), 40 deletions(-)