public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Andrew Burgess <andrew.burgess@embecosm.com>
To: Tom de Vries <tdevries@suse.de>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH][gdb] Remove trailing "done" after "Reading symbols from" message
Date: Wed, 04 Mar 2020 14:08:00 -0000	[thread overview]
Message-ID: <20200304140757.GZ3317@embecosm.com> (raw)
In-Reply-To: <20200304132133.GA26507@delia>

* Tom de Vries <tdevries@suse.de> [2020-03-04 14:21:34 +0100]:

> Hi,
> 
> Using verbose, we get some detail on symbol loading:
> ...
> $ gdb a.out -iex "set verbose on"
> Reading symbols from a.out...
> Reading in symbols for /home/vries/hello.c...done.
> (gdb)
> ...
> 
> And using debug symtab-create, much more detail:
> ...
> $ gdb a.out -iex "set verbose on" -iex "set debug symtab-create 1"
> Reading symbols from a.out...
> Reading minimal symbols of objfile /data/gdb_versions/devel/lto/a.out ...
> Installing 30 minimal symbols of objfile /data/gdb_versions/devel/lto/a.out.
> Done reading minimal symbols.
> Creating one or more psymtabs for objfile /data/gdb_versions/devel/lto/a.out ...
> Created psymtab 0x35a3de0 for module ../sysdeps/x86_64/start.S.
> Created psymtab 0x353e4e0 for module init.c.
> Created psymtab 0x353e560 for module ../sysdeps/x86_64/crti.S.
> Created psymtab 0x353e5e0 for module /home/vries/hello.c.
> Created psymtab 0x35bd530 for module elf-init.c.
> Created psymtab 0x35bd5b0 for module ../sysdeps/x86_64/crtn.S.
> Reading in symbols for /home/vries/hello.c...Created compunit symtab 0x354bd20 for hello.c.
> done.
> (gdb)

I'm struggling to understand what problem you're trying to solve
here.  In general, if we switch on any of the 'set debug ...' flags
the output from GDB gets confused.  These flags aren't intended for
general use, and trying to "fix" GDB's output in the face of these
flags is going to be a huge task, for I'm not sure what benefit.

Other than the output looking a little neater, is there an actual
problem solved here?

Thanks,
Andrew


> ...
> 
> The "Created compunit symtab" message gets inbetween the "Reading in symbols"
> and the trailing "done.". [ Strictly speaking this is a regression since
> commit faa17681cc "Make gdb_flush also flush the wrap buffer", but the
> same problem happens when using -batch before this commit. ]
> 
> Fix this by removing the trailing "done." altogether, such that we get:
> ...
> Created psymtab 0x3590520 for module ../sysdeps/x86_64/crtn.S.
> Reading in symbols for /home/vries/hello.c...
> Created compunit symtab 0x359dd20 for hello.c.
> (gdb)
> ...
> 
> [ Alternatively, we could fix this emitting a "Done reading in symbols" line
> or some such, like is done for minimal symbols.  See above. ]
> 
> [ Note: Removing the trailing "done." for the "Reading symbols from" message
> was done in commit 3453e7e409 'Clean up "Reading symbols" output'. ]
> 
> Build and reg-tested on x86_64-linux.
> 
> OK for trunk?
> 
> Thanks,
> - Tom
> 
> [gdb] Remove trailing "done" after "Reading symbols from" message
> 
> gdb/ChangeLog:
> 
> 2020-03-04  Tom de Vries  <tdevries@suse.de>
> 
> 	* psymtab.c (psymtab_to_symtab): Don't print "done.".
> 
> ---
>  gdb/psymtab.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/gdb/psymtab.c b/gdb/psymtab.c
> index fd7fc8feff..7d33dea0d4 100644
> --- a/gdb/psymtab.c
> +++ b/gdb/psymtab.c
> @@ -760,16 +760,12 @@ psymtab_to_symtab (struct objfile *objfile, struct partial_symtab *pst)
>  
>        if (info_verbose)
>  	{
> -	  printf_filtered (_("Reading in symbols for %s..."),
> +	  printf_filtered (_("Reading in symbols for %s...\n"),
>  			   pst->filename);
>  	  gdb_flush (gdb_stdout);
>  	}
>  
>        pst->read_symtab (objfile);
> -
> -      /* Finish up the debug error message.  */
> -      if (info_verbose)
> -	printf_filtered (_("done.\n"));
>      }
>  
>    return pst->get_compunit_symtab ();

  reply	other threads:[~2020-03-04 14:08 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-04 13:21 Tom de Vries
2020-03-04 14:08 ` Andrew Burgess [this message]
2020-03-04 14:13   ` Tom de Vries
2020-03-06 11:02     ` Andrew Burgess
2020-03-06 11:56       ` Tom de Vries

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200304140757.GZ3317@embecosm.com \
    --to=andrew.burgess@embecosm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=tdevries@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).