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