From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) by sourceware.org (Postfix) with ESMTPS id D2B0F3871F82 for ; Tue, 6 Dec 2022 15:43:00 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org D2B0F3871F82 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gnu.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gnu.org Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p2a5y-0003nj-16; Tue, 06 Dec 2022 10:42:58 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=yAkxPpsNNAUWqbg/z6cgnsiDmXjUHLxIj3eOJQ+K5OY=; b=foKTGA5uYv0H QtGy0pqx32PSgPzWoVLiKX6o9o1lWn4LYEqk8xRK0ZOjyCsVWxoufiuFxZfYQOdS0kY2/n9aEf5rA wbnxDhIAtgXn9j4ALxbES+W4oFv8Ciy6Idd5XuD/M9974DACL2R1TPXJRva8Grk0EtfvXovcj9TCU 8OlSnDen3Cgco2SUhBUY15RWVis5DUlbHT2D82bE08cBGb+/lR1ZQV/uNdLdl7QOINxHzlP29J742 rTDgUbLhYQnvrt2ATzsPqr3C/WfmTZO2UG5q+w6MJnpdOCAzgD4fhKjbW9N3gID4gZDPlcmzUYwII 75RB+kdTmOY6qcLCW8rn0w==; Received: from [87.69.77.57] (helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1p2a5x-00042L-0h; Tue, 06 Dec 2022 10:42:57 -0500 Date: Tue, 06 Dec 2022 17:42:41 +0200 Message-Id: <835yeo7d3i.fsf@gnu.org> From: Eli Zaretskii To: Simon Marchi Cc: gdb-patches@sourceware.org, simon.marchi@efficios.com, zoran.zaric@amd.com, laurent.morichetti@amd.com, Tony.Tye@amd.com, lancelot.six@amd.com, pedro@palves.net In-Reply-To: <20221206135729.3937767-13-simon.marchi@efficios.com> (message from Simon Marchi via Gdb-patches on Tue, 6 Dec 2022 08:57:29 -0500) Subject: Re: [PATCH 12/12] gdb: initial support for ROCm platform (AMDGPU) debugging References: <20221206135729.3937767-1-simon.marchi@efficios.com> <20221206135729.3937767-13-simon.marchi@efficios.com> X-Spam-Status: No, score=1.9 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_BARRACUDACENTRAL,SPF_HELO_PASS,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Level: * X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: > Cc: Simon Marchi , > Zoran Zaric , > Laurent Morichetti , > Tony Tye , Lancelot SIX , > Pedro Alves > Date: Tue, 6 Dec 2022 08:57:29 -0500 > From: Simon Marchi via Gdb-patches > > --- a/gdb/doc/gdb.texinfo > +++ b/gdb/doc/gdb.texinfo > @@ -7021,6 +7021,8 @@ signal happened. @value{GDBN} alerts you to the context switch with a > message such as @samp{[Switching to Thread @var{n}]} to identify the > thread. > > +@node set scheduler-locking This @node is without any @chapter/@section, and does not appear in any @menu. That doesn't look right; did you actually succeed in building the manual with these changes? > +The following @acronym{AMD GPU} architectures are supported: > + > +@table @emph > + > +@item @samp{gfx900} > +AMD Vega 10 devices, displayed as @samp{vega10} by @value{GDBN}. > + > +@item @samp{gfx906} > +AMD Vega 7nm devices, displayed as @samp{vega20} by @value{GDBN}. Do we really need this long list of architectures in the GDB manual? It sounds like an ad for AMD... > +@smallexample > +hipcc -O0 -g --offload-arch=gfx900 --offload-arch=gfx906 bit_extract.cpp -o bit_extract > +@end smallexample This and other @smallexamples in the patch have too long lines; please break them into two or more, to avoid problems with the printed format of the manual. > +@subsubsection @acronym{AMD GPU} Wavefronts I think a @cindex about wavefronts would be useful here. > +@item file_path > +The file's path specified as a URI encoded UTF-8 string. In URI > +encoding, every character that is not: > + > +@itemize > +@item In the @samp{a-z}, @samp{A-Z}, @samp{0-9} ranges > +@item @samp{/}, @samp{_}, @samp{.}, @samp{~} or @samp{-} > +@end itemize > + > +is encoded as two uppercase hexadecimal digits proceeded by @samp{%}. You want @noindent before the last line. > +Directories in the path are separated by @samp{/}. GNU coding standards frown on using "path" for anything that is not PATH-style lists of directories. Please use "file name" instead. > +The @acronym{AMD GPU} entities have the following target identifier formats: > + > +@table @var @var is wrong here, since the @item text is not a meta-syntactic variable: it doesn't stand for something else. I'd use @asis instead. > +Make sure the container user is a member of the @var{render} group for > +Ubuntu 20.04 onward and the @var{video} group for all other ??? Why are we talking about specific Linux distros in the manual? Finally: maybe it's just me, but isn't this documentation way too detailed? It weighs in at 800 lines, and includes many details that seem to be more related to AMD, GPU, and HIP than to GDB. Would it be reasonable to make this section shorter by omitting too low-level and unrelated details? Thanks.