RFR: 8327799: JFR view: the "Park Until" field of jdk.ThreadPark is invalid if the parking method is not absolute

Long Yang lyang at openjdk.org
Tue Mar 12 06:31:18 UTC 2024


Hi all

Could I have a review of this patch for [JDK-8327799](https://bugs.openjdk.org/browse/JDK-8327799)?


UNSAFE.park(true, System.currentTimeMillis() + 1000);
UNSAFE.park(false, 2000L * 1000 * 1000);



jfr view jdk.ThreadPark test.jfr

                                                    Java Thread Park
Start Time Duration Event Thread Stack Trace             Class Parked On Park Timeout Park Until Address of Object Pa...
---------- -------- ------------ ----------------------- --------------- ------------ ---------- -----------------------
20:13:21     1.00 s main         jdk.internal.misc.Un... N/A                      N/A 20:13:22   0x00000000
20:13:22     2.00 s main         jdk.internal.misc.Un... N/A                   2.00 s 08:05:43   0x00000000


If the parking method is not absolute (the second event), the real value of "until" in JFR event is `Long.MIN_VALUE`, which will be convert back to `java.time.Instant.MIN`, but `JFR view` displays this value as '08:05:43' of my timezone. This is somewhat misleading, better to show as `N/A`, just like what `jfr print --events` does.

Testing:
test/jdk/jdk/jfr/tool/TestView.java
test/jdk/jdk/jfr/jcmd/TestJcmdView.java

All passed.

Thanks

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

Commit messages:
 - 8327799: JFR view: the "Park Until" field of jdk.ThreadPark is invalid if the parking method is not absolute

Changes: https://git.openjdk.org/jdk/pull/18214/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=18214&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8327799
  Stats: 3 lines in 1 file changed: 3 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/18214.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18214/head:pull/18214

PR: https://git.openjdk.org/jdk/pull/18214


More information about the hotspot-jfr-dev mailing list