RFR: JDK-8301661: Enhance os::pd_print_cpu_info on macOS and Windows [v5]

Matthias Baesken mbaesken at openjdk.org
Mon Feb 20 08:06:30 UTC 2023


On Fri, 17 Feb 2023 05:21:41 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Do not print idle state infos on Windows, print same MHz values for all proces only once
>
> src/hotspot/os/windows/os_windows.cpp line 1901:
> 
>> 1899:   size_t sz_check = sizeof(PROCESSOR_POWER_INFORMATION) * (size_t)proc_count;
>> 1900:   NTSTATUS status = ::CallNtPowerInformation(ProcessorInformation, NULL, 0, buf, (ULONG) buflen);
>> 1901:   int MaxMhz = -1, CurrentMhz = -1, MhzLimit = -1;
> 
> Nit: variable names should not start with Capital letters

I adjusted the variable names.

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

PR: https://git.openjdk.org/jdk/pull/12403



More information about the build-dev mailing list