TOTAL_PHYMEM test failing on OSX 10.5 32-bit

[giampaolo]

From g.rodola on November 11, 2010 19:58:31

======================================================================
FAIL: test_TOTAL_PHYMEM (_osx.OSXSpecificTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/tester/psutil/test/_osx.py", line 42, in test_TOTAL_PHYMEM
    self.assertEqual(sysctl_hwphymem, psutil.TOTAL_PHYMEM)
AssertionError: 2147483648L != 15032385536L

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

Original issue: http://code.google.com/p/psutil/issues/detail?id=133

[giampaolo]

From g.rodola on November 11, 2010 11:21:24

Confirmed also on OSX 10.6 64-bit.

[giampaolo]

From [email protected] on November 11, 2010 12:06:30

Possibly useful reference: 
http://stackoverflow.com/questions/583736/determine-physical-mem-size-programmatically-on-osx
 I suggest we try using an int64_t as shown there rather than uint64_t we are 
currently using and see if that makes any difference, otherwise the code looks 
identical to ours.

[giampaolo]

From [email protected] on November 13, 2010 11:51:30

I'd like to see all the sysctl mem values: 

  hw.physmem = 2147483648
  hw.usermem = 1593614336
  hw.memsize = 2147483648

We use HW_MEMSIZE in psutil, and it looks like your test case is using 
HW_PHYSMEM to compare against, which may be why the test fails. In my case 
they're the same value, but they might not be on a 64bit system (HW_MEMSIZE 
returns a 64bit counter instead of a 32bit). I suspect the issue is a problem 
with the test case rather than psutil.

[giampaolo]

From [email protected] on November 13, 2010 11:53:32

r818 committed to have the OS X test case for TOTAL_PHYSMEM use hw.memsize 
instead, if you get a chance to run this again on 64bit OS X systems 10.5 and 
10.6 I'd be curious to see the results.

[giampaolo]

From g.rodola on February 28, 2011 13:58:03

Status: FixedInSVN
Labels: -OpSys-All Milestone-0.2.1

[giampaolo]

From g.rodola on March 20, 2011 14:55:37

Status: Fixed

[giampaolo]

From g.rodola on March 02, 2013 03:56:40

Updated csets after the SVN -> Mercurial migration: r818 == revision 
183549e73634