Re: [PATCH 1/2] Put a NT_PSTATUS note on solaris cores.

Pedro Alves
2012-12-11T16:45:07Z

On 12/11/2012 03:16 PM, Mark Kettenis wrote:
>> Date: Tue, 11 Dec 2012 19:11:36 +0400
>> From: Joel Brobecker <brobecker@adacore.com>
>>
>>> You're doing the right test. The issue is that core_pid_to_str
>>> calls into the gdbarch_core_pid_to_str, and that isn't expecting
>>> to be passed a "process pid" instead of an "LWP pid".
>>>
>>> Could you try this one when you have a chance? This is rebased on
>>> top of current mainline, so it restores the elfcore_write_pstatus
>>> call that the Unixware-excise patch removed. I made sure it builds with
>>> --enable-targets=all.
>>
>> Nice detective work :)

Nah, it was obvious to me what the problem was. :-)

>>
>> (gdb) core core.26444
>> [New LWP 1]
>> Core was generated by `/[...]/simple_main'.
>> Program terminated with signal 5, Trace/breakpoint trap.
>> #0 simple_main () at simple_main.adb:4
>> 4 simple.test_simple;
>> (gdb) info inferiors
>> Num Description Executable
>> * 1 process 26444 /[...]/simple_main
>>
>> And with an older core file, since your patch handles it too:
>>
>> (gdb) info inferiors
>> Num Description Executable
>> * 1 <core> /[...]/simple_main
>>
>> I tested the patch against our testsuite again, no regression either.
>> So it looks all good to go!

Great!

>
> FWIIW, this is fine with me as well (both the Unixware removal and
> fixing the Solaris core dump stuff).
>

Thanks a lot guys. Everything's checked in now.

--
Pedro Alves