RFR: 8342818: Implement CPU Time Profiling for JFR [v4]

Johannes Bechberger jbechberger at openjdk.org
Thu Nov 7 13:18:01 UTC 2024


On Thu, 31 Oct 2024 12:36:32 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

>> Johannes Bechberger has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove min_valid_free_size_bytes
>
> test/jdk/jdk/jfr/event/profiling/TestCPUTimeSampleThrottling.java line 65:
> 
>> 63:     private static void testThrottleSettingsPeriod() throws Exception {
>> 64:         float rate = countEvents(1000, "1ms").rate();
>> 65:         Asserts.assertTrue(rate > 950 && rate < 1050, "Expected around 1000 events per second, got " + rate);
> 
> Fails on linux-aarch64-debug builds:
> 
> ----------System.out:(0/0)----------
> ----------System.err:(14/1061)----------
> java.lang.RuntimeException: Expected around 1000 events per second, got 245.15503: expected true, was false
> 	at jdk.test.lib.Asserts.fail(Asserts.java:691)
> 	at jdk.test.lib.Asserts.assertTrue(Asserts.java:543)
> 	at jdk.jfr.event.profiling.TestCPUTimeEventThrottling.testThrottleSettingsPeriod(TestCPUTimeEventThrottling.java:65)
> 	at jdk.jfr.event.profiling.TestCPUTimeEventThrottling.main(TestCPUTimeEventThrottling.java:49)
> 	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
> 	at java.base/java.lang.reflect.Method.invoke(Method.java:573)
> 	at com.sun.javatest.regtest.agent.MainWrapper$MainTask.run(MainWrapper.java:138)
> 	at java.base/java.lang.Thread.run(Thread.java:1576)
> 
> JavaTest Message: Test threw exception: java.lang.RuntimeException: Expected around 1000 events per second, got 245.15503: expected true, was false
> JavaTest Message: shutting down test

This is interesting, it shouldn't happen (assuming that the system has at least one CPU that the CPU wasting code can run alone).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20752#discussion_r1832662175


More information about the hotspot-dev mailing list