[External] : Re: JFR Recordings Missing Periodic Events

[Erik Gahlin]

Hi Carter,

I have filed a bug report. I will look into your fix tomorrow.
https://bugs.openjdk.org/browse/JDK-8322142

Erik
________________________________
From: Carter Kozak <ckozak at ckozak.net>
Sent: Friday, December 15, 2023 12:13 AM
To: Erik Gahlin <erik.gahlin at oracle.com>; hotspot-jfr-dev at openjdk.java.net <hotspot-jfr-dev at openjdk.java.net>
Subject: [External] : Re: JFR Recordings Missing Periodic Events

Hi Erik,

Thanks for the quick reply!

I've simplified my reproducer based on your recommendations, however the result is the same. To clarify the issue I'm facing, I'm not worried about the EventStream/RecordingStream so much as the resulting JFR itself, which is missing periodic event data.
The CI output includes the updated logging, for example:
https://urldefense.com/v3/__https://github.com/carterkozak/periodic-event-repro/actions/runs/7215619705/job/19660234573__;!!ACWV5N9M2RV99hQ!OAej4SiBWTTz-IgdpnU3OtixXqEWHZoghi9QqV7eOVcz-5HOalkFt7SiI_beWUIMrSIYZgoxcDuen_1bww$

I've put together a quick draft based on the proposal in my last reply, I have confirmed that it resolves the failures:
https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/17114__;!!ACWV5N9M2RV99hQ!OAej4SiBWTTz-IgdpnU3OtixXqEWHZoghi9QqV7eOVcz-5HOalkFt7SiI_beWUIMrSIYZgoxcDvdxiNkkg$

Carter Kozak

On Thu, Dec 14, 2023, at 17:12, Erik Gahlin wrote:
> Hi Carter,
>
> t's not possible to guarantee that events will be available unless a RecordingStream is used, the period is everyChunk and the stop() method is used.
>
> That said, it looks strange. You could try running with -Xlog:jfr+system+parser=info,jfr+periodic=debug and only enable the ThreadStatistics event:
>
> recording.enable("jdk.JavaThreadStatistics").with("period", "1000 ms");
>
> It might make it easier to diagnose.
>
> Erik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-jfr-dev/attachments/20231214/f607a9f4/attachment-0001.htm>