From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27709 invoked by alias); 5 Aug 2011 02:13:21 -0000 Received: (qmail 27638 invoked by uid 22791); 5 Aug 2011 02:13:20 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 05 Aug 2011 02:13:06 +0000 Received: (qmail 3287 invoked from network); 5 Aug 2011 02:13:03 -0000 Received: from unknown (HELO ?192.168.0.101?) (yao@127.0.0.2) by mail.codesourcery.com with ESMTPA; 5 Aug 2011 02:13:03 -0000 Message-ID: <4E3B51A8.2070801@codesourcery.com> Date: Fri, 05 Aug 2011 02:13:00 -0000 From: Yao Qi User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.18) Gecko/20110617 Lightning/1.0b2 Thunderbird/3.1.11 MIME-Version: 1.0 To: Pedro Alves CC: gdb-patches@sourceware.org, "Joseph S. Myers" Subject: Re: [RFA 5/8] New port: TI C6x: gdb port References: <4E2638A6.1070406@codesourcery.com> <201108041333.42945.pedro@codesourcery.com> <201108041845.35876.pedro@codesourcery.com> In-Reply-To: <201108041845.35876.pedro@codesourcery.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes 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: 2011-08/txt/msg00089.txt.bz2 On 08/05/2011 01:45 AM, Pedro Alves wrote: > On Thursday 04 August 2011 16:11:35, Joseph S. Myers wrote: >> On Thu, 4 Aug 2011, Pedro Alves wrote: >> >>> Seems like there are a lot of non-core registers here. Now that >>> the gdbserver patch has been updated to send in standard tic6x xml >>> register description features, all registers the target includes in the >>> description that are not part of the defined features should not need >>> to be hardcoded in gdb, since the xml description itself will describe >>> their sizes and names. See arm-tdep.c:arm_register_name for example. >>> More comments on this further down. >> >> Won't GDB need a list of how the register names correspond to DWARF >> register numbers (which are allocated for all these registers) somewhere? > > AFAICS, gdbserver only exposes A0-A15, CSR, PC, A16-A31 > B16-B31, TSR, ILC, RILC, so I assumed all other registers would be > system dependent non general purpose control registers, but it's > probably then that they are only not "user space" registers, and will > be acessible by bare metal probes/stubs? Linux programs won't access > those, right? I now browsed the ABI's dwarf section and I got that > impression. These registers you mentioned are exposed because they are accessible via ptrace. In bare mental probe/stub, there are more registers can be accessed than ptrace. > > The default is to assume 1-1 mapping between gdb numbers and dwarf > numbers, so it's easier to ensure the numbers are the same. > I guess that clears up my comment to the register numbers. > As for names, types and sizes, if the target doesn't include those > registers in the description, then it's as if they don't exist. A > description that reports those extra registers should include those > properties (name, bitsize, type, regnum) per register, so gdb would still > get them from the description. I can't tell if it would make sense > to add a new feature or features for those extra registers though. > The current features of tic6x is sufficient, because they are all registers that ptrace can access so far. It makes no sense to list some registers but we can't access. Bare mental probes can access more control registers, so we may need new feature(s), but I don't think this patch series should cover this. -- Yao (齐尧)