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

[Jan Kratochvil <jan.kratochvil@redhat.com>]

On Mon, 15 Mar 2010 01:45:02 +0100, Joel Brobecker wrote:
> The comment should say "a few sections" as opposed to "a couple" (a
> couple is two, or three at most),

http://en.wiktionary.org/wiki/couple#Usage_notes
* In the U.S., "a couple of things" or people may be used to mean two of them,
  but it is also often used to mean any small number.
  	The farm is a couple of miles off the main highway [=a few miles away].

But I understand I can't argue about U.S. English. :-)


> and it should say that there is no
> other way to identify these section except by name.
> 
> Pre-approved with that comment update.

I hope it is +/- OK, checked-in.


Thanks,
Jan


http://sourceware.org/ml/gdb-cvs/2010-03/msg00138.html

--- src/gdb/ChangeLog	2010/03/15 03:48:46	1.11479
+++ src/gdb/ChangeLog	2010/03/15 09:31:30	1.11480
@@ -1,3 +1,8 @@
+2010-03-15  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+	* symfile.c (addr_info_make_relative): Ignore also missing ".dynbss"
+	and ".sdynbss".  Update the comment.
+
 2010-03-15  Jie Zhang  <jie@codesourcery.com>
 
 	* MAINTAINERS: Update my email address.
--- src/gdb/symfile.c	2010/03/10 18:20:07	1.277
+++ src/gdb/symfile.c	2010/03/15 09:31:34	1.278
@@ -576,16 +576,19 @@
 	  /* This section does not exist in ABFD, which is normally
 	     unexpected and we want to issue a warning.
 
-	     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.  */
+	     However, the ELF prelinker does create a few sections 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.  There is
+	     no other way to identify them than by their name as the sections
+	     created by prelink have no special flags.  */
 
 	  if (!(strcmp (sect_name, ".gnu.liblist") == 0
-		|| strcmp (sect_name, ".gnu.conflict") == 0))
+		|| strcmp (sect_name, ".gnu.conflict") == 0
+		|| strcmp (sect_name, ".dynbss") == 0
+		|| strcmp (sect_name, ".sdynbss") == 0))
 	    warning (_("section %s not found in %s"), sect_name,
 		     bfd_get_filename (abfd));