MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010037CMakeCTestpublic2009-12-11 13:212016-06-10 14:31
ReporterDavid Cole 
Assigned ToKitware Robot 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionmoved 
PlatformOSOS Version
Product VersionCMake-2-8 
Target VersionFixed in Version 
Summary0010037: SystemInformation.cxx is missing implementations for many functions on Win64 builds
DescriptionSee the definition of USE_ASM_INSTRUCTIONS: it is always 0 on Win64 builds, and most of the functions that use "#if USE_ASM_INSTRUCTIONS" do not have any fallback code in the "#else" clause -- so none of the Retrieve* functions actually retrieve anything useful.

Please implement SystemInformation fully for Win64 builds...

Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0007866closed Julien Jomier 64 Bit platform is not reported correctly on Windows64 
related to 0011302closed David Cole Provide a ProcessorCount.cmake module to determine the number of physical cores 
related to 0010895closed  Processor detection is very x86-specific 
related to 0011329closed David Cole No system information on QNX 
related to 0010326closed David Cole Wrong architecture for CMAKE_SYSTEM_PROCESSOR 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-12-11 13:21David ColeNew Issue
2009-12-11 13:21David ColeRelationship addedrelated to 0007866
2009-12-11 13:26David ColeStatusnew => assigned
2009-12-11 13:26David ColeAssigned To => David Cole
2009-12-11 15:18David ColeNote Added: 0018824
2009-12-11 15:18David ColeNote Deleted: 0018824
2009-12-11 15:18David ColeNote Added: 0018825
2009-12-11 16:13David ColeNote Added: 0018826
2010-11-10 21:09David ColeTarget Version => CMake 2.8.4
2010-12-08 14:43Alex NeundorfNote Added: 0023811
2010-12-08 16:13David ColeRelationship addedrelated to 0010895
2010-12-15 07:37David ColeRelationship addedrelated to 0011329
2010-12-15 07:37David ColeRelationship addedrelated to 0010326
2010-12-15 07:38David ColeRelationship addedrelated to 0011302
2011-01-05 14:32David ColeNote Added: 0024431
2011-01-05 14:32David ColeTarget VersionCMake 2.8.4 =>
2011-02-09 14:26David ColeStatusassigned => backlog
2011-02-09 15:10David ColeAssigned ToDavid Cole =>
2016-06-10 14:27Kitware RobotNote Added: 0041634
2016-06-10 14:27Kitware RobotStatusbacklog => resolved
2016-06-10 14:27Kitware RobotResolutionopen => moved
2016-06-10 14:27Kitware RobotAssigned To => Kitware Robot
2016-06-10 14:31Kitware RobotStatusresolved => closed

Notes
(0018825)
David Cole   
2009-12-11 15:18   
Related commit from earlier today:

cvs -q commit -m "Add fallback for detecting CPU speed from Windows registry on
builds where USE_ASM_INSTRUCTIONS gets defined to 0. (Win64 builds.) Clean up methods to return
false when not implemented or when they fail to detect the things they're supposed to detect. This file
still needs a lot more work to have a fully working implementation on Win64 builds. Committing it now
because it's slightly better than it was, and there is a need for the CPU speed detection code in projects
that depend on kwsys. Will file a bug shortly to request that the missing implementations get filled in..."
SystemInformation.cxx

/cvsroot/CMake/CMake/Source/kwsys/SystemInformation.cxx,v <-- SystemInformation.cxx
new revision: 1.54; previous revision: 1.53
(0018826)
David Cole   
2009-12-11 16:13   
The memory reported is different, but still definitely wrong, though Julien told me he just backported your fix.

I have 8 GB physical, and a 2 GB (!) pagefile, on Vista 64.

My 64 bits build reports:
- 4028 MB total physical (call to GetTotalPhysicalMemory)
- 4095 MB total virtual (GetTotalVirtualMemory())

My 32 bits build reports:
- 4028 MB total physical (call to GetTotalPhysicalMemory)
- 2047 MB total virtual (GetTotalVirtualMemory())

In any case, the physical mem is wrong.


The max total virtual memory for a Win32 process is always 2G. Unless you link with /LARGEADDRESSAWARE and set your machine up specially, and then you can get up to 3G for 32-bit processes. So that's what GetTotalVirtualMemory is reporting.

Not sure why GetTotalPhysicalMemory is reporting incorrectly. I'll add a note to the bug to take a look on a machine with 8G of RAM...
(0023811)
Alex Neundorf   
2010-12-08 14:43   
Might this one be of use here ? http://www.open-mpi.org/projects/hwloc/ [^]

Alex
(0024431)
David Cole   
2011-01-05 14:32   
Unsetting the target version field. Not enough time left to address this issue before the 2.8.4 release. Deferred.
(0041634)
Kitware Robot   
2016-06-10 14:27   
Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current CMake Issues page linked in the banner at the top of this page.