View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0008496CMakeCTestpublic2009-02-10 18:022009-02-12 10:09
Reporteraugiedoggie 
Assigned To 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0008496: kwsys tries to read /proc/cpuinfo on Haiku OS
DescriptionWhile attempting to submit ctest results to the public dashboard I noticed that it was attempting to read /proc/cpuinfo to gather information. Haiku OS does not have a /proc filesystem, so I wrote the needed method for Source/kwsys/SystemInformation.cpp to gather CPU/Memory information.
TagsNo tags attached.
Attached Filesdiff file icon cmake_systeminformation_haiku.diff [^] (3,278 bytes) 2009-02-10 18:02 [Show Content]
diff file icon cmake_systeminformation_haiku2.diff [^] (3,763 bytes) 2009-02-11 21:11 [Show Content]

 Relationships

  Notes
(0014887)
augiedoggie (reporter)
2009-02-10 18:42
edited on: 2009-02-10 19:15

The attached patch is only partially working. No crashes, but, need to figure out why the vendor name has garbage at the end. Need to reduce the cpu frequency from Hz to MHz. Also, TotalPhysicalMemory isn't being reported properly in the log files. Have to check and see if it's supposed to be in bytes/KB/MB/etc...

(0014890)
Bill Hoffman (manager)
2009-02-10 19:31

strcpy(this->ChipID.Vendor,cpu_info.eax_0.vendor_id); Is eax_0.vendor_id perhaps not null terminated?
(0014907)
augiedoggie (reporter)
2009-02-11 21:11

Yup, sorry about that. The new patch should fix the problems. CPU speed is now in MHz, memory is calculated properly, and vendor_id is put into a temporary buffer with a NULL character appended.
(0014911)
Bill Hoffman (manager)
2009-02-12 10:09

$ cvs commit -m "BUG: 0008496 add support for system info on haiku" SystemInformation.cxx
Committer: Bill Hoffman <bill.hoffman@kitware.com>
Commit to directory "CMake/Source/kwsys" lies in external project KWSys
/cvsroot/CMake/CMake/Source/kwsys/SystemInformation.cxx,v <-- SystemInformation.cxx
new revision: 1.38; previous revision: 1.37

Thanks for the patch!

 Issue History
Date Modified Username Field Change
2009-02-10 18:02 augiedoggie New Issue
2009-02-10 18:02 augiedoggie File Added: cmake_systeminformation_haiku.diff
2009-02-10 18:42 augiedoggie Note Added: 0014887
2009-02-10 19:15 augiedoggie Note Edited: 0014887
2009-02-10 19:31 Bill Hoffman Note Added: 0014890
2009-02-11 21:11 augiedoggie Note Added: 0014907
2009-02-11 21:11 augiedoggie File Added: cmake_systeminformation_haiku2.diff
2009-02-12 10:09 Bill Hoffman Note Added: 0014911
2009-02-12 10:09 Bill Hoffman Status new => closed
2009-02-12 10:09 Bill Hoffman Resolution open => fixed


Copyright © 2000 - 2018 MantisBT Team