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