RFR: 8358448: JFR: Incorrect time unit for MethodTiming event

Erik Gahlin egahlin at openjdk.org
Tue Jun 3 14:36:07 UTC 2025


Could I have a review of a PR that fixes a regression introduced with [JDK-8047756](https://bugs.openjdk.org/projects/JDK/issues/JDK-8047756)?

The default unit for `@Timespan` in Java is nanos, but it should be set to ticks. Prior to JDK-8047756, annotations were taken from native code, so ticks were used.

Testing: 
100 * test/jdk//jdk/jfr/event/tracing/TestMethodTimingEvent.java on Windows, Linux and MacOS 
test/jdk//jdk/jfr/*

Thanks
Erik

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

Commit messages:
 - Initial

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

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


More information about the hotspot-jfr-dev mailing list