RFR: 8308341: JNI_GetCreatedJavaVMs returns a partially initialized JVM [v2]
David Holmes
dholmes at openjdk.org
Mon May 29 03:13:30 UTC 2023
On Fri, 26 May 2023 17:50:58 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
>> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Enable the test on all Posix platforms
>
> Looks fine to me.
>
> I have one question though: why wouldn't it be enough to move `vm_created = 1` from where we had it before, down to where we now have `vm_created = COMPLETE` ?
>
> I'm not 100% sure why we need the intermediate step `IN_PROGRESS`
Thanks for looking at this @gerard-ziemski . The test now runs on all non-Windows platforms.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14139#issuecomment-1566437051
More information about the hotspot-dev
mailing list