RFR: 8335625: Update Javadoc for GetCpuLoad [v4]
duke
duke at openjdk.org
Tue Sep 10 13:43:09 UTC 2024
On Mon, 26 Aug 2024 13:54:36 GMT, Joakim Nordström <jnordstrom at openjdk.org> wrote:
>> Can I get a review of this documentation update to clarify the usage of GetCpuLoad (and inherently deprecated GetSystemCpuLoad) and GetProcessCpuLoad.
>>
>> Calling either of these methods in quick succession can lead to unrepresentative results due to too few data points.
>>
>> This behavior is easy to reproduce on at least Linux (Windows implementation enforces a 500 ticks duration); when calling GetCpuLoad repeatedly CPU load values of either 0, 0.5, or 1 will be returned.
>>
>> double cpuLoad1 = getCpuLoad();
>> double cpuLoad2 = getCpuLoad(); // not enough ticks has passed to give representative values
>>
>> Worth noting is that this holds true even if getSystemCpuLoad() is called.
>>
>> double cpuLoad1 = getCpuLoad();
>> double cpuLoad2 = getSystemCpuLoad(); // not enough ticks has passed to give representative values, since getSystemCpuLoad effectively calls getCpuLoad.
>
> Joakim Nordström has updated the pull request incrementally with one additional commit since the last revision:
>
> Removed implNote
@jaokim
Your change (at version ba7484e4d220c7e728e33f8c3f4490c555effdf7) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20546#issuecomment-2340827450
More information about the serviceability-dev
mailing list