This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA/ppc-aix] fix thread support breakage (was: "Re: [PATCH 4/5] powerpc64-aix ptrace64 when defined.")
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: Raunaq 12 <raunaq12 at in dot ibm dot com>, GDB Patches <gdb-patches at sourceware dot org>, Mark Kettenis <mark dot kettenis at xs4all dot nl>, Ulrich Weigand <Ulrich dot Weigand at de dot ibm dot com>
- Date: Fri, 23 Aug 2013 03:22:49 -0400
- Subject: Re: [RFA/ppc-aix] fix thread support breakage (was: "Re: [PATCH 4/5] powerpc64-aix ptrace64 when defined.")
- References: <CAGWvnynVeLt6F-QREsP_CtQvwvtZgwqHxtcS6pn58LMf=AX=GA at mail dot gmail dot com> <OF91B6D42F dot B0CFCB9C-ON65257BC5 dot 004284AA-65257BC5 dot 0042F553 at in dot ibm dot com> <CAGWvnynzsNL--qPhHOhqZ912A4M9ZpXtT2JbjW4EHRodEXAF9Q at mail dot gmail dot com> <20130822233234 dot GA13292 at adacore dot com> <20130823004837 dot GC5221 at adacore dot com>
On Thu, Aug 22, 2013 at 8:48 PM, Joel Brobecker <brobecker@adacore.com> wrote:
> Hello again,
>
>> Actually, I just found that it breaks thread support on 32bit AIX,
>> at least. A simpler way to show the problem is:
>>
>> (gdb) x /x &__n_pthreads
>> 0xf06a8258 <__n_pthreads>: Cannot access memory at address 0xf06a8258
>>
>> Prior to the patch, we have:
>>
>> (gdb) x /x &__n_pthreads
>> 0xf06a8258 <__n_pthreads>: 0x00000003
>
> Attached is the patch I checked in locally at AdaCore:
>
> gdb/ChangeLog:
>
> * rs6000-nat.c (rs6000_ptrace32): Cast "addr" to "uintptr_t"
> instead of "long long" in call to ptrace64.
>
> Tested on ppc-aix in full 32bit mode (both GDB and inferior are
> 32bit). I think the other scenario would be 64bit GDB debugging
> a 32bit inferior, but I don't have a 64bit compiler to try it.
>
> I'd like to commit this patch, unless there are comments/suggestions.
LGTM.
Is there a testcase that can be added to the GDB testsuite to catch
this type of regression?
Thanks, David