RFR: 8292559: Add test for -XX:+CheckJNICalls showing changed signal handlers
Thomas Stuefe
stuefe at openjdk.org
Thu Aug 18 13:04:06 UTC 2022
On Thu, 18 Aug 2022 12:48:30 GMT, Harold Seigel <hseigel at openjdk.org> wrote:
> Please review this small change to add a test that option -XX:+CheckJNICalls shows changed signal handlers. This new test is a copy of the recently deleted TestPosixSig.java with small changes. This version of the test only checks that handlers were changed for SIGFPE and SIGILL signals.
>
> The test was tested using Mach5 to run it on Linux, Mac OS, and Windows.
>
> Thanks, Harold
LGTM. Thanks for this test.
test/hotspot/jtreg/runtime/posixSig/libTestPsig.c line 38:
> 36:
> 37: printf( " HANDLER (1) " );
> 38: }
Nit: handler could be file scope static, and does not need extern "C"
-------------
Marked as reviewed by stuefe (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9919
More information about the build-dev
mailing list