RFR: 8352738: Implement JEP 520: JFR Method Timing and Tracing [v8]

Aleksey Shipilev shade at openjdk.org
Wed May 28 17:23:02 UTC 2025


On Thu, 22 May 2025 19:56:52 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

>> src/jdk.jfr/share/classes/jdk/jfr/internal/tracing/PlatformTracer.java line 249:
>> 
>>> 247:     }
>>> 248: 
>>> 249:     private synchronized static void reset() {
>> 
>> Any sense to do  `synchronized` here, given that `timed{Methods,Classes}` are `ConcurrentHashMap`-s?
>
> It's only used by PlatformTracer::initialize(), so it can be removed.

Should it be removed in this PR then?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25306#discussion_r2112395673


More information about the hotspot-jfr-dev mailing list