From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6540 invoked by alias); 15 Mar 2010 00:45:10 -0000 Received: (qmail 6531 invoked by uid 22791); 15 Mar 2010 00:45:09 -0000 X-SWARE-Spam-Status: No, hits=-2.5 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 15 Mar 2010 00:45:06 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id A2A562BAB6D; Sun, 14 Mar 2010 20:45:04 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id cMImrbfw3Hu2; Sun, 14 Mar 2010 20:45:04 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 627D12BAB64; Sun, 14 Mar 2010 20:45:04 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id A7797F5917; Sun, 14 Mar 2010 17:45:02 -0700 (PDT) Date: Mon, 15 Mar 2010 00:45:00 -0000 From: Joel Brobecker To: Jan Kratochvil Cc: gdb-patches@sourceware.org Subject: Re: [patch] Fix false warning: section .dynbss not found in ... Message-ID: <20100315004502.GE3045@adacore.com> References: <20100314213746.GA32276@host0.dyn.jankratochvil.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100314213746.GA32276@host0.dyn.jankratochvil.net> User-Agent: Mutt/1.5.20 (2009-06-14) 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 X-SW-Source: 2010-03/txt/msg00517.txt.bz2 > 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 > > * 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