On Tue, 6 Dec 2022 15:31:22 GMT, Erik Gahlin <[email protected]> wrote:
> Could I have review of infrastructure that makes it easier to write periodic > events with the same timestamp. > > Testing: jdk/jdk/jfr + tier1 + tier2 > > Thanks > Erik This seems okay. A couple of comments below. Thanks. src/jdk.jfr/share/classes/jdk/jfr/internal/RequestEngine.java line 211: > 209: long last = lastTimeMillis; > 210: // Bug 9000556 - current time millis has rather lame resolution > 211: // The use of os::elapsed_counter() is deliberate here, we don't While you are here can you delete this comment block, or suitably modify it so that it makes sense for the code that follows. Thanks. ------------- PR: https://git.openjdk.org/jdk/pull/11541
