From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10643 invoked by alias); 30 Jul 2018 19:04:54 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 10633 invoked by uid 89); 30 Jul 2018 19:04:53 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-24.9 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 spammy=hits, subdirectory, launched, HOME X-HELO: smtp.polymtl.ca Received: from smtp.polymtl.ca (HELO smtp.polymtl.ca) (132.207.4.11) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 30 Jul 2018 19:04:51 +0000 Received: from simark.ca (simark.ca [158.69.221.121]) (authenticated bits=0) by smtp.polymtl.ca (8.14.7/8.14.7) with ESMTP id w6UJ4jgk023403 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Mon, 30 Jul 2018 15:04:49 -0400 Received: by simark.ca (Postfix, from userid 112) id 1480B1EF29; Mon, 30 Jul 2018 15:04:45 -0400 (EDT) Received: from simark.ca (localhost [127.0.0.1]) by simark.ca (Postfix) with ESMTP id 4D4461E519; Mon, 30 Jul 2018 15:04:43 -0400 (EDT) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Mon, 30 Jul 2018 19:04:00 -0000 From: Simon Marchi To: Eli Zaretskii Cc: Simon Marchi , gdb-patches@sourceware.org Subject: Re: [PATCH v2 4/4] Add doc and news for DWARF index cache In-Reply-To: <83pnz9rrjo.fsf@gnu.org> References: <1532558824-829-1-git-send-email-simon.marchi@ericsson.com> <1532558824-829-5-git-send-email-simon.marchi@ericsson.com> <83pnz9rrjo.fsf@gnu.org> Message-ID: <77255cf544dd62fdd4878c03bbae5632@polymtl.ca> X-Sender: simon.marchi@polymtl.ca User-Agent: Roundcube Webmail/1.3.6 X-IsSubscribed: yes X-SW-Source: 2018-07/txt/msg00780.txt.bz2 On 2018-07-27 04:50, Eli Zaretskii wrote: >> From: Simon Marchi >> CC: Simon Marchi >> Date: Wed, 25 Jul 2018 18:47:04 -0400 >> >> This patch adds doc and news for the feature introduced by the >> previous >> patch. > > Thanks. > >> +* DWARF index cache: GDB can now automatically save indices DWARF >> symbols on > > "indices of DWARF symbols", I presume? Yes, thanks. >> diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo >> index b36a39b..9533c72 100644 >> --- a/gdb/doc/gdb.texinfo >> +++ b/gdb/doc/gdb.texinfo >> @@ -20095,6 +20095,42 @@ There are currently some limitation on >> indices. They only work when >> for DWARF debugging information, not stabs. And, they do not >> currently work for programs using Ada. >> >> +@subsection Automatic symbol index cache >> + >> +It is possible for GDB to automatically save a copy of this index in >> a cache > ^^^ > @value{GDBN} Fixed. >> +on disk and retrieve it from there when loading the same binary in >> the future. >> +This feature can be turned on with @command{set index-cache on}. The >> following > > @command is incorrect here, it's the markup for shell commands, like > 'ls'. What you want is @kbd. Ok, fixed. >> +@table @code >> + >> +@item set index-cache on >> +@itemx set index-cache off >> + >> +Enable or disable the use of the symbol index cache. > > There should be no empty line between @item and the following > description. Ok. >> + >> +@item set index-cache directory @var{directory} >> +@itemx show index-cache directory >> +Set/show the directory where index files will be saved. By default, >> the value >> +@code{$XDG_CACHE_HOME/gdb} is used if the @code{XDG_CACHE_HOME} >> environment >> +variable is defined. The value @code{$HOME/.cache/gdb} is used >> otherwise. > > Please don't use $FOO to mean an environment variable, that is a Unix > shell convention. I suggest to rephrase: > > By default, the index is cached in the @file{gdb} subdirectory of > the directory pointed to by the @env{XDG_CACHE_HOME} environment > variable, if it is defined, else in the @file{.cache/gdb} > subdirectory of your home directory. Ok, done. >> +@item set index-cache format @var{format} >> +@itemx show index-cache format >> +Set/show the format in which index files are saved. @var{format} can >> be either >> +@code{gdb} (the default) or @code{dwarf-5}. Note that @value{GDBN} >> is currently >> +only able to read back files in the @code{gdb} format from the cache, >> so >> +@code{dwarf-5} is not very useful. > > If 'dwarf-5' cannot be used, why are we documenting it, and why are we > documenting/implementing this command in the first place? Good point. It's on my road map to add support for reading back DWARF-5 files from the index, but since it's a bit more complex I'm keeping it for another patch. It's true that it doesn't really make sense to include that setting now, so I'll probably just remove it in the next version of the series. >> +@item show index-cache stats >> +Print the number of cache hits and misses for the index cache since >> the launch >> +of @value{GDBN}. > > This begs the question: for which index cache will this show the > statistics? For the one defined by the latest "set index-cache > directory" command? Ah, no it's dumber than that. It's just how many hits and misses there were since GDB was launched, regardless of the cache directory used. Perhaps it would be clearer to keep it short like: Print the number of cache hits and misses since the launch of @value{GDBN}. ? Thanks, Simon