On Wed, 7 Dec 2022 05:15:54 GMT, David Holmes <[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
>
> 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.

Actually what is all this `lastTimeMillis` code for? And how does it relate to 
the passed in timestamp?

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

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

Reply via email to