RFR: JDK-8297184: Test runtime/ErrorHandling/TestSigInfoInHsErrFile.java is failing [v3]

[Coleen Phillimore]

On Thu, 17 Nov 2022 13:14:04 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> https://bugs.openjdk.org/browse/JDK-8296906 added a new test that artificially crashes the VM with a known signal/code/crashaddress and checks that we see those informations precisely in the hs-err file. It did this for SIGSEGV and SIGFPE.
>> 
>> SIGFPE, however, is problematic. We fall back to pthread_kill() if we cannot trigger a real FPE fault. In that case the code will be SI_KILL and the sending frame will be pthread_kill(). 
>> 
>> Since we cannot guarantee that a real signal is sent, I opted for just removing the part of the test that tests FPE. The other part, testing SIGSEGV, is enough to cover what the test should cover.
>> 
>> *update*
>> 
>> I also added a fix for the second error mode we see, where we don't crash with SIGSEGV SEGV_MAPERR but SIGSEGV ACCERR.
>
> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
> 
>   removed carefully chosen comment since appearantly it was not careful enough :(

Looks good.  I think you can check this in since the carefully choosen comment is removed.  Edit: I didn't see Dan's comment.  Wait for his OK.

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

Marked as reviewed by coleenp (Reviewer).

PR: https://git.openjdk.org/jdk/pull/11202