Hi Simon and Ulrich,
Thank you for the feedback for this patch series. Please see my comments. Addressing the same I am sending a 4 patch v4 series.
>> On this page I see:
>>
>> For ptracex or ptrace64: If the debuggee is a 32-bit process, the
>> options that refer to GPRs or SPRs fail with errno = EIO
>>
>> And I see that in the patch, you call ptrace64 to read registers
>> (PTT_READ_GPRS) for 32 bit processes. How does that work?
>
> I think Simon is correct here. For GPRS access, I understand only
> the following options are supported:
>
> ptrace: 32- or 64-bit GDB, 32-bit inferior
> ptracex: 32-bit GDB, 64-bit inferior
> ptrace64: 64-bit GDB, 64-bit inferior
>
> As I understand, the current set of indirections is intended to
> cover that space - for a 32-bit inferior, you call ptrace, and
> for a 64-bit inferior you call either ptracex or ptrace64,
> depending on whether GDB is built as 64-bit.
>Ideally, we wouldn't just guess. Do you run the testsuite on AIX? If
>so, you should be able to run it in a mode that builds the test programs
>as 32-bit programs. For example:
> $ make check RUNTESTFLAGS='--target_board "unix/-m32"
Yes, I missed this in the documentation. Ulrich and
you are right. Appreciate the same for correcting me. I have addressed this and ran the test cases in 32 bit and 64-bit mode on AIX to make sure the changes are functionally correct.
>> Why do you think we can simplify this? I guess we could make
>> possibly make the assumption that GDB itself is always built
>> as 64-bit these days, but then we still have to select between
>> ptrace and ptrace64 at the least
…
>I think that requiring GDB to be built as 64-bit program would be
>reasonable (if so, you should add a static check in the source for that).
Yes, from GDB - 13 we are building GDB on AIX as 64-bit application only. We can adjust the code now with this information. I have done some clean up due to this.
Have a nice day ahead.
Thanks and regards,
Aditya.