RFR: 8342818: Implement JEP 509: JFR CPU-Time Profiling [v10]

Erik Gahlin egahlin at openjdk.org
Tue Jun 3 12:19:04 UTC 2025


On Mon, 26 May 2025 15:42:57 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

>>> This is added automatically. If I add "(Experimental)" to the title, then I get "X (Experimental) (Experimental)"
>> 
>> Sweet.
>> 
>>> I'm unsure how to implement this using the SQL version that is used for the views
>> 
>> I will see if I can create an example with some other events that show the syntax, and then you can fill in the CPU-Time events.
>
>> I will see if I can create an example with some other events that show the syntax, and then you can fill in the CPU-Time events.
> 
> I have a Mac, so I could not try it with an actual recording, but something like this:
> 
> [application.cpu-time-statistics]
> label = "CPU Time Samples Statistics"
> form = "COLUMN 'Successful Samples', 'Failed Samples', 'Total Samples', 'Lost Samples'
>         SELECT COUNT(S.startTime), COUNT(F.startTime), Count(A.startTime), SUM(L.lostSamples)
>         FROM 
>           CPUTimeSample AS S,
>           CPUTimeSample AS F,
>           CPUTimeSample AS A,
>           CPUTimeSampleLoss AS L
>         WHERE
>           S.failed = 'false' AND
>           F.failed = 'true'"
> 
> 
>  I removed biased, because I wonder If we should have such a field? There can be many types of biases, and the implementation may change in the future.

> Hold on, shouldn't this really be "Lost"? @egahlin and @mgronlun need to chime in here.

Lost might be better.

I wonder if `<Field type="Thread" name="eventThread" label="Thread" />` is needed,  instead of thread = true?

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

PR Comment: https://git.openjdk.org/jdk/pull/25302#issuecomment-2934959447


More information about the serviceability-dev mailing list