> Based on the log output, the test failure seems to be the result of two > threads trying to set and access variables that are not volatile, so I've > made them volatile. Since I can't reproduce the failure, I can't confirm that > it is now fixed, but the lack of volatile is clearly a bug so should be done > anyway. Tested by running the test locally to make sure nothing breaks.
Chris Plummer has updated the pull request incrementally with one additional commit since the last revision: cleaned up event counters ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25236/files - new: https://git.openjdk.org/jdk/pull/25236/files/ef35ffa9..1304cd72 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25236&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25236&range=00-01 Stats: 9 lines in 1 file changed: 0 ins; 2 del; 7 mod Patch: https://git.openjdk.org/jdk/pull/25236.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/25236/head:pull/25236 PR: https://git.openjdk.org/jdk/pull/25236
