Message ID | 1483744529-22462-1-git-send-email-lgustavo@codesourcery.com |
---|---|
State | New, archived |
Headers |
Received: (qmail 89345 invoked by alias); 6 Jan 2017 23:15:48 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <gdb-patches.sourceware.org> List-Unsubscribe: <mailto:gdb-patches-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/gdb-patches/> List-Post: <mailto:gdb-patches@sourceware.org> List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 89329 invoked by uid 89); 6 Jan 2017 23:15:47 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL, BAYES_00, RCVD_IN_DNSWL_NONE, SPF_PASS, URIBL_RED autolearn=ham version=3.3.2 spammy=Hx-languages-length:2609 X-HELO: relay1.mentorg.com Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 06 Jan 2017 23:15:37 +0000 Received: from svr-orw-fem-05.mgc.mentorg.com ([147.34.97.43]) by relay1.mentorg.com with esmtp id 1cPdjT-000223-Ro from Luis_Gustavo@mentor.com for gdb-patches@sourceware.org; Fri, 06 Jan 2017 15:15:35 -0800 Received: from Opsys.world.mentorg.com (147.34.91.1) by svr-orw-fem-05.mgc.mentorg.com (147.34.97.43) with Microsoft SMTP Server id 14.3.224.2; Fri, 6 Jan 2017 15:15:35 -0800 From: Luis Machado <lgustavo@codesourcery.com> To: <gdb-patches@sourceware.org> Subject: [PATCH, v2] Improve load command's help text Date: Fri, 6 Jan 2017 17:15:29 -0600 Message-ID: <1483744529-22462-1-git-send-email-lgustavo@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain X-IsSubscribed: yes |
Commit Message
Luis Machado
Jan. 6, 2017, 11:15 p.m. UTC
Changes in v2: - Improved description of how FILE and OFFSET must be passed. - Complemented description in the GDB manual since it was missing the description for OFFSET. This fairly obvious patch adds usage text to the load command's help text. Originally it did not have usage and mentioned things like FILE and OFFSET without explaining how those should be passed in the command. gdb/ChangeLog: 2017-01-06 Luis Machado <lgustavo@codesourcery.com> * symfile (_initialize_symfile): Add usage text to the load command's help text. gdb/doc/ChangeLog: 2017-01-06 Luis Machado <lgustavo@codesourcery.com> * gdb.texinfo (_initialize_symfile): Document the optional offset argument. --- gdb/doc/gdb.texinfo | 8 ++++++-- gdb/symfile.c | 5 ++++- 2 files changed, 10 insertions(+), 3 deletions(-)
Comments
ping? I suppose this is in good shape now? On 01/06/2017 05:15 PM, Luis Machado wrote: > Changes in v2: > > - Improved description of how FILE and OFFSET must be passed. > - Complemented description in the GDB manual since it was missing the > description for OFFSET. > > This fairly obvious patch adds usage text to the load command's help text. > > Originally it did not have usage and mentioned things like FILE and OFFSET > without explaining how those should be passed in the command. > > gdb/ChangeLog: > > 2017-01-06 Luis Machado <lgustavo@codesourcery.com> > > * symfile (_initialize_symfile): Add usage text to the load command's > help text. > > gdb/doc/ChangeLog: > > 2017-01-06 Luis Machado <lgustavo@codesourcery.com> > > * gdb.texinfo (_initialize_symfile): Document the optional offset > argument. > --- > gdb/doc/gdb.texinfo | 8 ++++++-- > gdb/symfile.c | 5 ++++- > 2 files changed, 10 insertions(+), 3 deletions(-) > > diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo > index ccb95ad..d9c77f7 100644 > --- a/gdb/doc/gdb.texinfo > +++ b/gdb/doc/gdb.texinfo > @@ -19591,8 +19591,8 @@ Show the current status of displaying communications between > > @table @code > > -@kindex load @var{filename} > -@item load @var{filename} > +@kindex load @var{filename} @var{offset} > +@item load @var{filename} @var{offset} > @anchor{load} > Depending on what remote debugging facilities are configured into > @value{GDBN}, the @code{load} command may be available. Where it exists, it > @@ -19611,6 +19611,10 @@ link the program; for other formats, like a.out, the object file format > specifies a fixed address. > @c FIXME! This would be a good place for an xref to the GNU linker doc. > > +It is also possible to tell @value{GDBN} to load the symbol file at a specific > +offset described by the optional argument @var{offset}. When @var{offset} is > +provided, @var{filename} must also be provided. > + > Depending on the remote side capabilities, @value{GDBN} may be able to > load programs into flash memory. > > diff --git a/gdb/symfile.c b/gdb/symfile.c > index 61b9e5b..6ffc86b 100644 > --- a/gdb/symfile.c > +++ b/gdb/symfile.c > @@ -3952,7 +3952,10 @@ that lies within the boundaries of this symbol file in memory."), > c = add_cmd ("load", class_files, load_command, _("\ > Dynamically load FILE into the running program, and record its symbols\n\ > for access from GDB.\n\ > -A load OFFSET may also be given."), &cmdlist); > +An optional load OFFSET may also be given as a literal address.\n\ > +When OFFSET is provided, FILE must also be provided. FILE can be provided\n\ > +on its own.\n\ > +Usage: load [FILE] [OFFSET]"), &cmdlist); > set_cmd_completer (c, filename_completer); > > add_prefix_cmd ("overlay", class_support, overlay_command, >
ping? On 01/17/2017 07:08 AM, Luis Machado wrote: > ping? I suppose this is in good shape now? > > On 01/06/2017 05:15 PM, Luis Machado wrote: >> Changes in v2: >> >> - Improved description of how FILE and OFFSET must be passed. >> - Complemented description in the GDB manual since it was missing the >> description for OFFSET. >> >> This fairly obvious patch adds usage text to the load command's help >> text. >> >> Originally it did not have usage and mentioned things like FILE and >> OFFSET >> without explaining how those should be passed in the command. >> >> gdb/ChangeLog: >> >> 2017-01-06 Luis Machado <lgustavo@codesourcery.com> >> >> * symfile (_initialize_symfile): Add usage text to the load command's >> help text. >> >> gdb/doc/ChangeLog: >> >> 2017-01-06 Luis Machado <lgustavo@codesourcery.com> >> >> * gdb.texinfo (_initialize_symfile): Document the optional offset >> argument. >> --- >> gdb/doc/gdb.texinfo | 8 ++++++-- >> gdb/symfile.c | 5 ++++- >> 2 files changed, 10 insertions(+), 3 deletions(-) >> >> diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo >> index ccb95ad..d9c77f7 100644 >> --- a/gdb/doc/gdb.texinfo >> +++ b/gdb/doc/gdb.texinfo >> @@ -19591,8 +19591,8 @@ Show the current status of displaying >> communications between >> >> @table @code >> >> -@kindex load @var{filename} >> -@item load @var{filename} >> +@kindex load @var{filename} @var{offset} >> +@item load @var{filename} @var{offset} >> @anchor{load} >> Depending on what remote debugging facilities are configured into >> @value{GDBN}, the @code{load} command may be available. Where it >> exists, it >> @@ -19611,6 +19611,10 @@ link the program; for other formats, like >> a.out, the object file format >> specifies a fixed address. >> @c FIXME! This would be a good place for an xref to the GNU linker doc. >> >> +It is also possible to tell @value{GDBN} to load the symbol file at a >> specific >> +offset described by the optional argument @var{offset}. When >> @var{offset} is >> +provided, @var{filename} must also be provided. >> + >> Depending on the remote side capabilities, @value{GDBN} may be able to >> load programs into flash memory. >> >> diff --git a/gdb/symfile.c b/gdb/symfile.c >> index 61b9e5b..6ffc86b 100644 >> --- a/gdb/symfile.c >> +++ b/gdb/symfile.c >> @@ -3952,7 +3952,10 @@ that lies within the boundaries of this symbol >> file in memory."), >> c = add_cmd ("load", class_files, load_command, _("\ >> Dynamically load FILE into the running program, and record its >> symbols\n\ >> for access from GDB.\n\ >> -A load OFFSET may also be given."), &cmdlist); >> +An optional load OFFSET may also be given as a literal address.\n\ >> +When OFFSET is provided, FILE must also be provided. FILE can be >> provided\n\ >> +on its own.\n\ >> +Usage: load [FILE] [OFFSET]"), &cmdlist); >> set_cmd_completer (c, filename_completer); >> >> add_prefix_cmd ("overlay", class_support, overlay_command, >> > >
Ping? https://sourceware.org/ml/gdb-patches/2017-01/msg00116.html On 01/06/2017 05:15 PM, Luis Machado wrote: > Changes in v2: > > - Improved description of how FILE and OFFSET must be passed. > - Complemented description in the GDB manual since it was missing the > description for OFFSET. > > This fairly obvious patch adds usage text to the load command's help text. > > Originally it did not have usage and mentioned things like FILE and OFFSET > without explaining how those should be passed in the command. > > gdb/ChangeLog: > > 2017-01-06 Luis Machado <lgustavo@codesourcery.com> > > * symfile (_initialize_symfile): Add usage text to the load command's > help text. > > gdb/doc/ChangeLog: > > 2017-01-06 Luis Machado <lgustavo@codesourcery.com> > > * gdb.texinfo (_initialize_symfile): Document the optional offset > argument.
I suspect that the reason these weren't originally mentioned in the help was/is that the accepted arguments depend on target. Back when we had support for a bunch of different remote targets, we had more target_load implementations. Nowadays, there's much fewer targets, though still a couple load implemenations: $ grep "[.\->]to_load = " * remote.c: remote_ops.to_load = remote_load; remote-sim.c: gdbsim_ops.to_load = gdbsim_load; remote_load calls generic_load, which accepts the offset. gdbsim_load doesn't support an offset, but understands that what comes after the filename is an offset. (we could probably factor out more bits from those two implementations to load_command.) > -@kindex load @var{filename} > -@item load @var{filename} > +@kindex load @var{filename} @var{offset} > +@item load @var{filename} @var{offset} > @anchor{load} > Depending on what remote debugging facilities are configured into > @value{GDBN}, the @code{load} command may be available. Where it exists, it > @@ -19611,6 +19611,10 @@ link the program; for other formats, like a.out, the object file format > specifies a fixed address. > @c FIXME! This would be a good place for an xref to the GNU linker doc. > > +It is also possible to tell @value{GDBN} to load the symbol file at a specific executable file, not symbol file. > +offset described by the optional argument @var{offset}. When @var{offset} is > +provided, @var{filename} must also be provided. LGTM with that, but Eli should review this. Thanks, Pedro Alves
> From: Luis Machado <lgustavo@codesourcery.com> > Date: Tue, 7 Feb 2017 04:20:44 -0600 > > Ping? > > https://sourceware.org/ml/gdb-patches/2017-01/msg00116.html Sorry, the documentation changes are OK, except ... > > gdb/doc/ChangeLog: > > > > 2017-01-06 Luis Machado <lgustavo@codesourcery.com> > > > > * gdb.texinfo (_initialize_symfile): Document the optional offset > > argument. ... this: the thing in the parentheses should be the name of the node where you make the changes. Thanks.
On 02/07/2017 05:12 AM, Pedro Alves wrote: > I suspect that the reason these weren't originally mentioned > in the help was/is that the accepted arguments depend on target. > > Back when we had support for a bunch of different remote > targets, we had more target_load implementations. > Nowadays, there's much fewer targets, though still a couple > load implemenations: > > $ grep "[.\->]to_load = " * > remote.c: remote_ops.to_load = remote_load; > remote-sim.c: gdbsim_ops.to_load = gdbsim_load; > > remote_load calls generic_load, which accepts the offset. > gdbsim_load doesn't support an offset, but understands that > what comes after the filename is an offset. > > (we could probably factor out more bits from those > two implementations to load_command.) > >> -@kindex load @var{filename} >> -@item load @var{filename} >> +@kindex load @var{filename} @var{offset} >> +@item load @var{filename} @var{offset} >> @anchor{load} >> Depending on what remote debugging facilities are configured into >> @value{GDBN}, the @code{load} command may be available. Where it exists, it >> @@ -19611,6 +19611,10 @@ link the program; for other formats, like a.out, the object file format >> specifies a fixed address. >> @c FIXME! This would be a good place for an xref to the GNU linker doc. >> >> +It is also possible to tell @value{GDBN} to load the symbol file at a specific > > executable file, not symbol file. > >> +offset described by the optional argument @var{offset}. When @var{offset} is >> +provided, @var{filename} must also be provided. > > LGTM with that, but Eli should review this. > > Thanks, > Pedro Alves > > Thanks. I've addressed both the above and Eli's comments. Pushed as 5cf30ebf64d3c6da961094c615a94d2f1682a478.
> From: Luis Machado <lgustavo@codesourcery.com> > Date: Mon, 13 Feb 2017 07:32:09 -0600 > > > LGTM with that, but Eli should review this. LGTM, thanks.
diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index ccb95ad..d9c77f7 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -19591,8 +19591,8 @@ Show the current status of displaying communications between @table @code -@kindex load @var{filename} -@item load @var{filename} +@kindex load @var{filename} @var{offset} +@item load @var{filename} @var{offset} @anchor{load} Depending on what remote debugging facilities are configured into @value{GDBN}, the @code{load} command may be available. Where it exists, it @@ -19611,6 +19611,10 @@ link the program; for other formats, like a.out, the object file format specifies a fixed address. @c FIXME! This would be a good place for an xref to the GNU linker doc. +It is also possible to tell @value{GDBN} to load the symbol file at a specific +offset described by the optional argument @var{offset}. When @var{offset} is +provided, @var{filename} must also be provided. + Depending on the remote side capabilities, @value{GDBN} may be able to load programs into flash memory. diff --git a/gdb/symfile.c b/gdb/symfile.c index 61b9e5b..6ffc86b 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -3952,7 +3952,10 @@ that lies within the boundaries of this symbol file in memory."), c = add_cmd ("load", class_files, load_command, _("\ Dynamically load FILE into the running program, and record its symbols\n\ for access from GDB.\n\ -A load OFFSET may also be given."), &cmdlist); +An optional load OFFSET may also be given as a literal address.\n\ +When OFFSET is provided, FILE must also be provided. FILE can be provided\n\ +on its own.\n\ +Usage: load [FILE] [OFFSET]"), &cmdlist); set_cmd_completer (c, filename_completer); add_prefix_cmd ("overlay", class_support, overlay_command,