On Thu, 25 Sep 2025 09:13:58 GMT, Joachim Kern <[email protected]> wrote:
>> Suchismith Roy has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 12 commits: >> >> - Merge branch 'openjdk:master' into cpuprocessload >> - Thread safety,struct and perfInit() >> - Thread safety,struct and perfInit() >> - Update ProblemList.txt >> - Merge branch 'master' into cpuprocessload >> - Merge branch 'master' into cpuprocessload >> - Update UnixOperatingSystem.c >> - Merge branch 'openjdk:master' into cpuprocessload >> - cleanup >> - system cpu load >> - ... and 2 more: https://git.openjdk.org/jdk/compare/86d6a2e0...46dcf313 > > src/jdk.management/aix/native/libmanagement_ext/UnixOperatingSystem.c line > 134: > >> 132: cpu_load = MAX(cpu_load, 0.0); >> 133: cpu_load = MIN(cpu_load, 1.0); >> 134: cpu_load = (user_diff + sys_diff) / delta_time; > > Why is the sequence here different to the sequence in > Java_com_sun_management_internal_OperatingSystemImpl_getCpuLoad0 ? Thanks for catching this. Missed this detail for this method. Had kept the right logic in other one. It does not get caught in the tests either! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25332#discussion_r2379374478
