From patchwork Fri Feb 7 12:36:50 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Philipp Stanner X-Patchwork-Id: 863520 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1944733EC; Fri, 7 Feb 2025 12:37:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738931828; cv=none; b=fr1i4pVbfpzmuY4vdwXxEHxNeDDtAU0WHZZdZ/kiKgMe+9OOBXDUzLkgS4QIqv+k+0E01BTfbKAP+ePOts0sk3fhUlM8kx5Q6Vczn7wJhLj6YC6kYb62IA+ARYvdOJui1xkd0Brc/u9/0KM/d7bytzGjgitvH1xuSWZ2QtZDdKg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738931828; c=relaxed/simple; bh=2bKyeP9zqVLSt/2JmFzcqHN0fT/i7oHIwjweK2UhX+8=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=JJMSmPVPa29v3k2OKdbT9/EVODjXXjn8PD//2e5j1mt5hjj9XnfBbhRkJDThKEtAwC3B+p+9+nyKM/GIGrJPvbWrWfiJ9F5Z9BDjyMMf4o2vlLFUGh2L86KlRZMT9/w7RJCZ4/D4CqUNs53XzfWDH69rEpO2N5lFC4AYkaMPSXU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=WXxaq+Zh; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="WXxaq+Zh" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5BAF0C4CED1; Fri, 7 Feb 2025 12:37:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738931827; bh=2bKyeP9zqVLSt/2JmFzcqHN0fT/i7oHIwjweK2UhX+8=; h=From:To:Cc:Subject:Date:From; b=WXxaq+ZhGmi7hvXDV36OvJO4i4ekNHYRQg7jepLn9lJqijd3q6UJAiumv4qIHbkdY wAsQ5w90FjWC/mx7uQTvlzAZjzUJAPOvLU6tRYj2nGzKvA8CFf7owLZJ84oYxyM8AF 0O+oMnD0fY9q3MnuCHnXFQMRrAFi3dSkQ8t/XOQT8cAJjdNse+YZnla98xSdA6UBPU pA93nCG05ZPiUKdFmIVNzPb7NnaaJNGAGozHNKkcpquyZZaDV4523IrJUpsdCo+nXH 3qHtJ9wLCdnTXfnWUoUcL730shutKuGnX4VFBFCLXRYm2osrlXeFNjLGNxZErQm96J gi9+kDSM6YVLg== From: Philipp Stanner To: Matthew Brost , Danilo Krummrich , Philipp Stanner , =?utf-8?q?Christian_K=C3=B6nig?= , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Sumit Semwal Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linaro-mm-sig@lists.linaro.org Subject: [PATCH v3 0/3] drm/sched: Documentation and refcount improvements Date: Fri, 7 Feb 2025 13:36:50 +0100 Message-ID: <20250207123652.27677-2-phasta@kernel.org> X-Mailer: git-send-email 2.47.1 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Changes in v3: - timedout_job(): various docu wording improvements. (Danilo) - Use the term "ring" consistently. (Danilo) - Add fully fledged docu for enum drm_gpu_sched_stat. (Danilo) 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 | 109 ++++++++++++++++--------- 2 files changed, 74 insertions(+), 40 deletions(-)