RFR: JDK-8255711: Fix and unify hotspot signal handlers [v3]

Thomas Stuefe stuefe at openjdk.java.net
Thu Nov 5 07:04:56 UTC 2020


On Wed, 4 Nov 2020 23:28:48 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Feedback David
>
> src/hotspot/os/posix/signals_posix.cpp line 615:
> 
>> 613:           "\n#    /--------------------\\"
>> 614:           "\n#    |      %-7s       |"
>> 615:           "\n#    \\---\\ /--------------/"
> 
> Isn't the little robot supposed to say "segmentation fault" and would that be safer than calling get_signal_name in this context?  thanks for keeping the picture.

Thanks Coleen. Little robot now spells out the name of the signal (since "segmentation fault" is only correct for segv, and I wanted to see it for the other cases too. get_signal_name() is completely harmless, just uses a bit of stack buffer (and that only for the case of unknown signals which are printed numerically; I plan to change that and give us a simple version which just returns static strings).

-------------

PR: https://git.openjdk.java.net/jdk/pull/1034


More information about the hotspot-dev mailing list