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