RFR: 8357184: Test vmTestbase/nsk/jdi/ExceptionEvent/_itself_/exevent008/TestDescription.java fails with unreported exception [v2]

Leonid Mesnik lmesnik at openjdk.org
Fri May 30 15:03:58 UTC 2025


On Fri, 30 May 2025 03:15:40 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> 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

Thanks for fixing, now it is clear that volatile is enough.

Marked as reviewed by lmesnik (Reviewer).

-------------

PR Review: https://git.openjdk.org/jdk/pull/25236#pullrequestreview-2881627854
PR Review: https://git.openjdk.org/jdk/pull/25236#pullrequestreview-2881628214


More information about the serviceability-dev mailing list