From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout-p-201.mailbox.org (mout-p-201.mailbox.org [80.241.56.171]) (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 A439427183F; Wed, 12 Feb 2025 12:48:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=80.241.56.171 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739364515; cv=none; b=aqLsTOc5jlCTf8236aouWhuZQj8QFFjN7W5D2kvT0SwAH7sbVZwz9uOP6bduYRZw9WSXMiDlc4rT5fKi/3yfWdPDFNxWikNjhSrOXm3XO3heZyaSPViJbpZt5DaqK+8xb6IPSkegPwtqazELhN03FENCWO36rX69fTmBFVqkYjA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739364515; c=relaxed/simple; bh=EDhW1xwyL7isb9C2EmKd1PTwPrB+uBSZBAvwvbLxlsU=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=arm3UrtdgEMUT2TFdTvDG1ZjwYbNgGUjzXldhxxMLGWId3sSb/xbSsxVm07h/B46K/AqRQU8sfMhU+3IxekS11a/gpGsCqj47pYOjHq7SPmlzZ2UFrYQBdU45UiA99p/ad41A1wCTdm2L7DNRM1DMMM24dUBS1EVWvL9ADY5oqs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mailbox.org; spf=pass smtp.mailfrom=mailbox.org; dkim=pass (2048-bit key) header.d=mailbox.org header.i=@mailbox.org header.b=j5pBgvZC; arc=none smtp.client-ip=80.241.56.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mailbox.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=mailbox.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=mailbox.org header.i=@mailbox.org header.b="j5pBgvZC" Received: from smtp1.mailbox.org (smtp1.mailbox.org [IPv6:2001:67c:2050:b231:465::1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-201.mailbox.org (Postfix) with ESMTPS id 4YtJ5T3Wp4z9sS2; Wed, 12 Feb 2025 13:48:29 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mailbox.org; s=mail20150812; t=1739364509; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=EDhW1xwyL7isb9C2EmKd1PTwPrB+uBSZBAvwvbLxlsU=; b=j5pBgvZCJxV57HJTeE5y2LPp56nXHIwxZvfgLlKWmkys1ydTVzWJfzON5zgpGilZdlArp3 rM3s/1WXQZAXcYlZftjqV2NmRMyPXh5OYim9lkb7Rhv9458jFeas8V/s7iemMO2jdZLyBD iZwg4EoadKFcSWldNTTmJPYn8A3kv7CIcgg8onJ8o/yGo1k2WivmsqZa7tJt757sCLfhQ7 T+tGLnR3bPcwtPVD+mCVbNPaMYtQ0+lFxpVbZcp6/VZNTXa2UqZEjdblIvom6SG88kdB9d 8sgUYqjBQjLJogg9rQSJP5u7/aVvZKNudWR0SqneomklpOur0COijAj1kSwbcQ== Message-ID: <25ea7d23e7b778a0c2372cc2bbc87fc2b3742dca.camel@mailbox.org> Subject: Re: [PATCH v3 0/3] drm/sched: Documentation and refcount improvements From: Philipp Stanner Reply-To: phasta@kernel.org To: Philipp Stanner , Matthew Brost , Danilo Krummrich , Christian =?ISO-8859-1?Q?K=F6nig?= , 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 Date: Wed, 12 Feb 2025 13:48:25 +0100 In-Reply-To: <20250207123652.27677-2-phasta@kernel.org> References: <20250207123652.27677-2-phasta@kernel.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MBO-RS-META: hthkmhf8qheg5okcazx6mxy5af9jigei X-MBO-RS-ID: ed3e673e6865dc3df69 On Fri, 2025-02-07 at 13:36 +0100, Philipp Stanner wrote: > Changes in v3: > =C2=A0 - timedout_job(): various docu wording improvements. (Danilo) > =C2=A0 - Use the term "ring" consistently. (Danilo) > =C2=A0 - Add fully fledged docu for enum drm_gpu_sched_stat. (Danilo) >=20 > Changes in v2: > =C2=A0 - Document what run_job() is allowed to return. (Tvrtko) > =C2=A0 - Delete confusing comment about putting the fence. (Danilo) > =C2=A0 - Apply Danilo's RB to patch 1. > =C2=A0 - Delete info about job recovery for entities in patch 3. (Danilo, > me) > =C2=A0 - Set the term "ring" as fix term for both HW rings and FW rings. = A > =C2=A0=C2=A0=C2=A0 ring shall always be the thingy on the CPU ;) (Danilo) > =C2=A0 - Many (all) other comments improvements in patch 3. (Danilo) >=20 > This is as series succeeding my previous patch [1]. >=20 > 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. >=20 > Please give me feedback, especially on the RFC comments in patch3. >=20 > (This series still fires docu-build-warnings. I want to gather > feedback > on the opion questions first and will solve them in v2.) >=20 > Thank you, > Philipp >=20 > [1] > https://lore.kernel.org/all/20241220124515.93169-2-phasta@kernel.org/ >=20 > Philipp Stanner (3): > =C2=A0 drm/sched: Document run_job() refcount hazard > =C2=A0 drm/sched: Adjust outdated docu for run_job() > =C2=A0 drm/sched: Update timedout_job()'s documentation I would like to pull this in soonish, since I think pushback is unlikely? RBs for the last two patches by someone would be nice, though. P. >=20 > =C2=A0drivers/gpu/drm/scheduler/sched_main.c |=C2=A0=C2=A0 5 +- > =C2=A0include/drm/gpu_scheduler.h=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0 | 109 ++++++++++++++++------- > -- > =C2=A02 files changed, 74 insertions(+), 40 deletions(-) >=20