RFR(S): 8247967: SparkExamples24H.java SIGSEGV in various places
markus.gronlund at oracle.com
Tue Jun 23 22:56:00 UTC 2020
Will remove the DEBUG_ONLY construct.
From: David Holmes
Sent: den 24 juni 2020 00:32
To: Markus Gronlund <markus.gronlund at oracle.com>; hotspot-runtime-dev at openjdk.java.net
Subject: Re:  RFR(S): 8247967: SparkExamples24H.java SIGSEGV in various places
On 24/06/2020 12:32 am, Markus Gronlund wrote:
> Please review the following change set to correct the improper thread transition introduced as part of JDK-8245113.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8247967
> Webrev: http://cr.openjdk.java.net/~mgronlun/8247967/webrev00/
> Testing: jdk_jfr
> Thanks to David Holmes for pinpointing this embarrassing issue.
I'm glad to see a simple use of the existing transition helper was possible here. The fix looks good.
One minor note:
444 // can safepoint here
445 ThreadInVMfromNative transition((JavaThread*)thread);
ThreadInVMfromNative will also assert that the thread is actually in native.
More information about the hotspot-runtime-dev