On Fri, 17 Feb 2023 05:17:59 GMT, David Holmes <dhol...@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 1915:
> 
>> 1913:             CurrentMhz != (int) pppi->CurrentMhz ||
>> 1914:             MhzLimit != (int) pppi->MhzLimit) {
>> 1915:           same_vals_for_all_cpus = false;
> 
> You could break once this is false.

Added a break.

> src/hotspot/os/windows/os_windows.cpp line 1924:
> 
>> 1922: 
>> 1923:     if (same_vals_for_all_cpus && MaxMhz != -1) {
>> 1924:       st->print_cr("ProcessorInformation for all %d processors :", 
>> proc_count);
> 
> Nit: Processor Information

I adjusted the output.

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

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

Reply via email to