Re: [patch] Fix false warning: section .dynbss not found in ...

[Joel Brobecker <brobecker@adacore.com>]

> found two new sections by prelink:
> 	Reading symbols from /usr/bin/gdb...Reading symbols from /usr/lib/debug/usr/bin/gdb.debug...
> 	warning: section .dynbss not found in /usr/lib/debug/usr/bin/gdb.debug

:-) I was afraid that we'd find some of other sections that
needed to be excluded... :-/

> 2010-03-14  Jan Kratochvil  <jan.kratochvil@redhat.com>
> 
> 	* symfile.c (addr_info_make_relative): Ignore also missing ".dynbss"
> 	and ".sdynbss".  Simplify the comment.

OK, at least for now, but just a nit:

>  	     However, the ELF prelinker does create a couple of sections
> -	     (".gnu.liblist" and ".gnu.conflict") which are marked in the main
> -	     executable as loadable (they are loaded in memory from the
> -	     DYNAMIC segment) and yet are not present in separate debug info
> -	     files.  This is fine, and should not cause a warning.  Shared
> -	     libraries contain just the section ".gnu.liblist" but it is not
> -	     marked as loadable there.  */
> +	     which are marked in the main executable as loadable (they are
> +	     loaded in memory from the DYNAMIC segment) and yet are not present
> +	     in separate debug info files.  This is fine, and should not cause
> +	     a warning.  Shared libraries contain just the section
> +	     ".gnu.liblist" but it is not marked as loadable there.  */

The comment should say "a few sections" as opposed to "a couple" (a
couple is two, or three at most), and it should say that there is no
other way to identify these section except by name.

Pre-approved with that comment update.

-- 
Joel