Re: [PATCH v3] gdbserver: uclinux: pull in ptrace offset definitions from asm/ptrace.h

[Pedro Alves <palves at redhat dot com>]

On 04/26/2012 07:27 PM, Mike Frysinger wrote:

> On Thursday 26 April 2012 14:14:59 Pedro Alves wrote:
>> On 04/26/2012 07:12 PM, Mike Frysinger wrote:
>>> On Thursday 26 April 2012 14:03:00 Will Deacon wrote:
>>>>> +/* PTRACE_TEXT_ADDR and friends.  */
>>>>> +#include <asm/ptrace.h>
>>>
>>> check for asm/ptrace.h in configure.ac and then just depend on
>>> HAVE_ASM_PTRACE_H ?  although this is fine too since uClibc is
>>> realistically the only C library being used with no-mmu Linux systems.
>>
>> Was there a time Linux didn't have this header?  When we include it GDB, we
>> include it unconditionally (though we don't include it in all ports), and
>> in current mainline, it seems that all 27 ports under arch/ have it.
> 
> it's not so much a matter of whether it's provided, but whether it can be 
> safely included.  ia64 has a history of being a pita to include their ptrace 
> headers, but maybe that doesn't matter to gdbserver.


There are ia64 boxes on the gcc compile farm.  Let me give that a try.

> might be worth seeing if we can do a release with an unconditional include and 
> see if anyone complains.


-- 
Pedro Alves