Conversation
|
This is based off the memory reporting implementation in Firefox. |
|
It would appear AppVeyor is unhappy about the usage of stdint.h on Windows, I'm guessing Visual C++ for Python 9 does not include it, but it was in my path due to having a full-fledged modern msvc install. |
|
Python 2.7 still has to be compiled with Visual Studio 2008 which does not support stdint.h. |
|
Related: #752 |
psutil/_psutil_windows_uss.c
Outdated
There was a problem hiding this comment.
Indentation should be 4 spaces. Also, there's no need to put this in a separate file. You can use psutil/arch/windows/process_info.c instead.
psutil/_psutil_windows_uss.c
Outdated
There was a problem hiding this comment.
On a second thought I would prefer you define this in psutil/_psutil_windows.c as proc_memory_uss.
From the python file (psutil/_pswindows.py) after you call cext.psutil_proc_memory_info (or cext.psutil_proc_memory_info_2 you will call cext.proc_memory_uss to get USS stats separately.
|
@giampaolo The latest pushes should cover your code review comments. |
psutil/_psutil_windows.c
Outdated
There was a problem hiding this comment.
please rename this to py_result
|
@giampaolo ee6024f should address the final comments. |
3 participants
This adds a USS (unique set size, aka private working set) measurement to
memory_info_exon Windows.Performance before patch:
Performance after patch:
So slower, but not as bad as Linux and OSX.