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