On Fri, 7 Apr 2023 02:27:59 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> Alex Menkov has updated the pull request incrementally with one additional >> commit since the last revision: >> >> trailing spaces > > test/hotspot/jtreg/serviceability/jvmti/vthread/FollowReferences/libVThreadStackRefTest.cpp > line 179: > >> 177: } >> 178: >> 179: static volatile bool timeToExit = false; > > It is not enough to make variable volatile in c++. You need to make it atomic > or use monitors to correctly synchronize. fixed ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13254#discussion_r1160432556
