2023-03-07 04:38:08

by Bagas Sanjaya

[permalink] [raw]
Subject: [PATCH] accel: Link to compute accelerator subsystem intro

Commit 2c204f3d53218d ("accel: add dedicated minor for accelerator
devices") adds link to accelerator nodes section of DRM internals doc
(Documentation/gpu/drm-internals.rst), but the target doesn't exist.
Instead, there is only an introduction doc for computer accelerator
subsytem.

Link to that doc until there is documentation of accelerator internals.

Fixes: 2c204f3d53218d ("accel: add dedicated minor for accelerator devices")
Signed-off-by: Bagas Sanjaya <[email protected]>
---

This comes from discussion on Dylan Le's attempted fix, where Jeffrey
Hugo suspected that the proper target should be accelerator subsystem
doc [2].

[1]: https://lore.kernel.org/linux-doc/7799513f0fb5aee27b64752c037980471be669c5.1677120686.git.self@dylanle.dev/
[2]: https://lore.kernel.org/linux-doc/CAOCk7NrAKZ2G-VbQW3MJUZmkKkO1Ccrs2wo12kg6QHysH-d=Yg@mail.gmail.com/

include/drm/drm_file.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/drm/drm_file.h b/include/drm/drm_file.h
index 0d1f853092ab8a..ecffe24e2b1b0a 100644
--- a/include/drm/drm_file.h
+++ b/include/drm/drm_file.h
@@ -408,7 +408,8 @@ static inline bool drm_is_render_client(const struct drm_file *file_priv)
* Returns true if this is an open file of the compute acceleration node, i.e.
* &drm_file.minor of @file_priv is a accel minor.
*
- * See also the :ref:`section on accel nodes <drm_accel_node>`.
+ * See also :doc:`Introduction to compute accelerators subsystem
+ * </accel/introduction>`.
*/
static inline bool drm_is_accel_client(const struct drm_file *file_priv)
{

base-commit: 5c8cf1664f288098a971a1d1e65716a2b6a279e1
--
An old man doll... just what I always wanted! - Clara



2023-03-07 16:07:07

by Jeffrey Hugo

[permalink] [raw]
Subject: Re: [PATCH] accel: Link to compute accelerator subsystem intro

On 3/6/2023 9:35 PM, Bagas Sanjaya wrote:
> Commit 2c204f3d53218d ("accel: add dedicated minor for accelerator
> devices") adds link to accelerator nodes section of DRM internals doc
> (Documentation/gpu/drm-internals.rst), but the target doesn't exist.
> Instead, there is only an introduction doc for computer accelerator
> subsytem.
>
> Link to that doc until there is documentation of accelerator internals.
>
> Fixes: 2c204f3d53218d ("accel: add dedicated minor for accelerator devices")
> Signed-off-by: Bagas Sanjaya <[email protected]>

Reviewed-by: Jeffrey Hugo <[email protected]>

2023-03-19 08:28:33

by Oded Gabbay

[permalink] [raw]
Subject: Re: [PATCH] accel: Link to compute accelerator subsystem intro

On Tue, Mar 7, 2023 at 6:04 PM Jeffrey Hugo <[email protected]> wrote:
>
> On 3/6/2023 9:35 PM, Bagas Sanjaya wrote:
> > Commit 2c204f3d53218d ("accel: add dedicated minor for accelerator
> > devices") adds link to accelerator nodes section of DRM internals doc
> > (Documentation/gpu/drm-internals.rst), but the target doesn't exist.
> > Instead, there is only an introduction doc for computer accelerator
> > subsytem.
> >
> > Link to that doc until there is documentation of accelerator internals.
> >
> > Fixes: 2c204f3d53218d ("accel: add dedicated minor for accelerator devices")
> > Signed-off-by: Bagas Sanjaya <[email protected]>
>
> Reviewed-by: Jeffrey Hugo <[email protected]>
Thanks.
Will upstream through my tree (habanalabs-next)
Oded