[cmake-developers] [CMake 0016006]: Incorrect physical/logical core counts from kwsys on Windows

[Mantis Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://cmake.org/Bug/view.php?id=16006 
====================================================================== 
Reported By:                Nils Gladitz
Assigned To:                
====================================================================== 
Project:                    CMake
Issue ID:                   16006
Category:                   CMake
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
====================================================================== 
Date Submitted:             2016-03-04 03:24 EST
Last Modified:              2016-03-04 03:24 EST
====================================================================== 
Summary:                    Incorrect physical/logical core counts from kwsys on
Windows
Description: 
I am not entirely sure if this is the right place for kwsys issues?

On a Windows 7 64-bit system with two of these:
http://ark.intel.com/products/47922/Intel-Xeon-Processor-X5650-12M-Cache-2_66-GHz-6_40-GTs-Intel-QPI

Hyper-threading is enabled.

In cmsys::SystemInformation
  info.GetNumberOfLogicalCPU() returns 32 (expected is 24)
  info.GetNumberOfPhysicalCPU() returns 0 (expected is 2)

In CMake I exposed these values through 
NUMBER_OF_LOGICAL_CORES and NUMBER_OF_PHYSICAL_CORES in
cmake_host_system_information().

The Windows GetSystemInfo() filled SYSTEM_INFO structure reports 24 in
dwNumberOfProcessors.



Steps to Reproduce: 
cmake_host_system_information(RESULT PHYSICAL_CORES
    QUERY NUMBER_OF_PHYSICAL_CORES
)

cmake_host_system_information(RESULT LOGICAL_CORES
    QUERY NUMBER_OF_LOGICAL_CORES
)

message("[${PHYSICAL_CORES}:${LOGICAL_CORES}]")

====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2016-03-04 03:24 Nils Gladitz   New Issue                                    
======================================================================