Request for review: 6749267: Signal handler should save/restore errno

[Karen Kinnear]

Harold,

Thank you for doing this.

I think you also want to catch SR_handler - this is our suspend/resume handler. Today this is on
linux and bsd. An upcoming change will use that handler for more than just the VM Thread, and will support it on Solaris as well.
Copying Rickard/Staffan who will want to extend this change for the Solaris SR_handler when it arrives.


On Feb 18, 2013, at 9:07 AM, harold seigel wrote:

> Hi,
> 
> Please review the following change.
> 
> Summary: With this change, the actual signal handler saves the errno before calling the function that does most of the signal handling processing.  It then restores the value of errno when the signal handling processing function returns.
> 
> Open webrev: http://cr.openjdk.java.net/~hseigel/bug_6749267/ <http://cr.openjdk.java.net/%7Ehseigel/bug_6749267/>
> 
> Bug link at: https://jbs.oracle.com/bugs/browse/JDK-6749267
> 
> The changes were tested with JPRT, JCK Lang and VM tests, UTE tests, and with JPRT.  Also, I used the debugger to change the value of errno inside of the signal handling processing method and then verified that it got restored properly.
Thank you for reporting what tests you ran. UTE tests - could you be more specific, that is just the framework.

We have some existing signal tests in the vmsqe tests - see runtime/signal/*.
> 
> Thanks!  Harold

thanks,
Karen