On Fri, 11 Sep 2026 17:13:06 GMT, Andrei Pangin <[email protected]> wrote:

>> The hard-coded stack size is something we want to move away from and allow 
>> to be set per event type. We want to allow native frames for all event 
>> types, so our plan is to extend the stackTrace setting to allow for more 
>> stack-related things to be set.
>> 
>> A -XX:FlightRecorderOptions:native-stack-depth=x might be a better approach 
>> for now, where the default is 0, and not add an event setting at all.
>
> While fixing this, I found a pre-existing concurrency bug in the CPU-time 
> sampler.
> Please take a look at that fix too (which is a prerequisite for this PR).
> https://github.com/openjdk/jdk/pull/32842

Alright, I've made JfrCPUTimeSampleRequests variable-size. No extra space is 
used when native frames are not recorded.
Also, replaced per-event setting with a global FlightRecordedOption as 
suggested.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32524#discussion_r4139124675

Reply via email to