RFR: 8351187: Add JFR monitor notification event [v5]
Aleksey Shipilev
shade at openjdk.org
Tue Mar 18 07:20:27 UTC 2025
On Tue, 18 Mar 2025 03:32:43 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains eight commits:
>>
>> - Merge branch 'master' into JDK-8351187-jfr-monitor-notify
>> - Only emit event when notification happened
>> - Merge branch 'master' into JDK-8351187-jfr-monitor-notify
>> - Rewrite test to RecordingStream
>> - Drop threshold to 0ms
>> - Merge branch 'master' into JDK-8351187-jfr-monitor-notify
>> - Disable by default
>> - Fix
>
> src/hotspot/share/runtime/objectMonitor.cpp line 2005:
>
>> 2003: static void post_monitor_notify_event(EventJavaMonitorNotify* event,
>> 2004: ObjectMonitor* monitor,
>> 2005: int notified_count) {
>
> ```suggestion - indent is off
> static void post_monitor_notify_event(EventJavaMonitorNotify* event,
> ObjectMonitor* monitor,
> int notified_count) {
Right. Should be fixed in new commit.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23901#discussion_r2000353198
More information about the hotspot-dev
mailing list