(S) RFR: 8139300: Internal Error (vm/utilities/debug.cpp:399), # Error: ShouldNotReachHere()

[Kim Barrett]

On Nov 16, 2015, at 4:32 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> On 17/11/2015 6:34 AM, Daniel D. Daugherty wrote:
>> On 11/11/15 11:33 PM, David Holmes wrote:
>>> webrev: http://cr.openjdk.java.net/~dholmes/8139300/webrev/
>> 
>> src/share/vm/utilities/debug.cpp
>>     No comments.
>> 
>> test/runtime/ErrorHandling/SecondaryErrorTest.java
>>     No comments.
>> 
>> Thumbs up.
>> 
>> Since you've used the POSIX equivalent code:
>> 
>>     pthread_kill(pthread_self(), SIGFPE);
>> 
>> I'm presuming we have no plans to change this code back once
>> MacOS X fixes the bug in raise().
> 
> No plans. Also I'm not aware of OSX accepting this as a bug, so it may never get fixed.

Note that the Darwin/BSD documentation for raise is different from POSIX:

https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man3/raise.3.html

DESCRIPTION

     The raise() function sends the signal sig to the current process.