RFR: 8309296: jdk/jfr/event/runtime/TestAgentEvent.java fails due to "missing" dynamic JavaAgent

[Markus Grönlund]

On Tue, 20 Jun 2023 23:38:04 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> Could I have a review of a test fix.
> 
> Problem is that events use another clock source than Recording::getStartTime(). 
> 
> I added -XX:+UnlockExperimentalVMOptions -XX:-UseFastUnorderedTimeStamps in case the Attach thread and the periodic task thread executes on different cores and there is drift. The flags may, or may not, also fix the time source issue. Regardless, I think it's prudent to do time comparison against an event, not the recording timestamps, because they are different things conceptually and there are tweaks to the recording timestamps (to make them work well with Instant.now(), not the event clock source)
> 
> Testing: 100 times jdk/jdk/jfr/event/runtime/TestAgentEvent.java on Mac, Windows and Linux.
> 
> Thanks
> Erik

Marked as reviewed by mgronlun (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/14572#pullrequestreview-1490438508