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