RFR: 8298379: JFR: Some UNTIMED events only sets endTime [v3]

Erik Helin ehelin at openjdk.org
Thu Dec 8 13:41:19 UTC 2022


On Thu, 8 Dec 2022 13:39:44 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

>> Greetings,
>> 
>> It was pointed out that some UNTIMED events only call set_endtime(), and not the corresponding set_starttime(), for example some events in jfrPeriodic.cpp. UNTIMED events need to have both timestamps set, because the UNTIMED parameter denotes that the user takes on the responsibility for controlling time stamping.
>> 
>> This change rectifies a few places by adding the corresponding set_starttime() call.
>> 
>> Test: jdk_jfr
>> 
>> Thanks
>> Markus
>
> Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
> 
>   copyright

Looks good, thanks @mgronlun for fixing!

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

Marked as reviewed by ehelin (Reviewer).

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


More information about the hotspot-dev mailing list