Re: [RFA 6/8] New port: TI C6x: gdbserver

[Yao Qi <yao at codesourcery dot com>]

On 07/21/2011 09:59 PM, Joseph S. Myers wrote:
> In this patch, and also in the main GDB port, I think you've used the 
> wrong set of register names.  Specifically, it appears you used a list 
> provided by TI on 22 September 2010 as an early draft of the list of DWARF 
> register numbers for the ABI, which had a typo I pointed out the next day 
> (NPR should be NRP) and was missing ECR (later inserted in the middle of 
> the list) and various undocumented control registers (later added at the 
> end).  You should check the list carefully against ABI version 0.9.  In 

In gdb side, yes, a typo should be fixed and four new control registers
should be added.

> addition, the XML files should reflect the different sets of registers on 
> each processor variant (for details of what processors have what 
> undocumented control registers, see the details TI provided on 29 October 
> 2010 - again, with a subsequent correction; the documented registers are 
> covered in the ISA manuals or in 
> include/opcode/tic6x-control-registers.h).
> 

In gdbserver side, we don't have to reflect the different control
registers on each C6x variant, because they are not accessible from
current ptrace (ptrace only access TSR, RILC, ILC, PC, and CSR).  XML
files should be revised to reflect the different set ptrace-accessible
registers on C6x variant.  I'll take care of this.

-- 
Yao (éå)