RFR: 8358318: JFR: Tighten up PlatformTracer initialization
Aleksey Shipilev
shade at openjdk.org
Mon Jun 2 17:28:32 UTC 2025
On Mon, 2 Jun 2025 11:42:11 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:
> All event settings modifications happen with the "metadata" lock, so only one thread at a time can modify the state in MethodSetting. I think it's sufficient to remove volatile in that class. It's misleading.
See new commit. This drops `volatile`, looks generally cleaner, and avoids triggering analysis that complains about static field modifications from instance methods on top.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25584#issuecomment-2931689864
More information about the hotspot-jfr-dev
mailing list