From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26648 invoked by alias); 2 Nov 2010 18:53:55 -0000 Received: (qmail 26634 invoked by uid 22791); 2 Nov 2010 18:53:54 -0000 X-SWARE-Spam-Status: No, hits=-2.1 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; Tue, 02 Nov 2010 18:53:50 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 94A872BACB5; Tue, 2 Nov 2010 14:53:48 -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 DaRUQ6TRhxYx; Tue, 2 Nov 2010 14:53:48 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 2BEB42BACB4; Tue, 2 Nov 2010 14:53:48 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id 611D5F588F; Tue, 2 Nov 2010 11:53:41 -0700 (PDT) Date: Tue, 02 Nov 2010 18:53:00 -0000 From: Joel Brobecker To: Nathan Froyd Cc: gdb-patches@sourceware.org Subject: Re: [PATCH] fix debugging code compiled for newer PPC BookE processors Message-ID: <20101102185341.GG2492@adacore.com> References: <1288380041-22165-1-git-send-email-froydnj@codesourcery.com> <20101102182326.GD2492@adacore.com> <20101102184612.GD26083@nightcrawler> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101102184612.GD26083@nightcrawler> 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-11/txt/msg00042.txt.bz2 > I don't know exactly what the purpose is. The E500 ABI User's Guide > says simply: > > For the .PPC.EMB.apuinfo section, the name shall be ???APUinfo\0???, the > type shall be 2 (as type 1 is already reserved for ELF_NOTE_ABI), and > the data shall contain a series of words containing APU information, > one per word. The APU information contains two unsigned halfwords: the > upper half contains the unique APU identi???er, and the lower half > contains the revision of that APU. Interesting. Perhaps the purpose is for cases where all those separate sections can no longer be separate, but merged somehow somewhere. In that case, the type becomes important whereas, right now, it's only redundant with the section name... In any case, the patch looked fine to me. -- Joel