gh-150114: Log the memory usage in regrtest on macOS#150396
Conversation
|
cc @vstinner |
|
!buildbot macOS |
|
🤖 New build scheduled with the buildbot fleet by @corona10 for commit 7327561 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F150396%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
|
!buildbot macOS |
|
🤖 New build scheduled with the buildbot fleet by @corona10 for commit 03abe46 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F150396%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
vstinner
left a comment
There was a problem hiding this comment.
Recently, code to get the memory usage of the current process was removed from the GC: https://github.com/python/cpython/pull/149475/changes#diff-4722ae850a2da3e12f50815c0e3932627ebe67edeaa8b7f28a94f20c8698204dL2022. I used task_info(mach_task_self(), TASK_VM_INFO, ...) and gets vm_info.phys_footprint. In private, @corona10 told me that task_info() cannot be used to get the memory usage of other processes.
|
psutil also uses |
|
!buildbot macOS |
|
🤖 New build scheduled with the buildbot fleet by @corona10 for commit e3870ab 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F150396%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
vstinner
left a comment
There was a problem hiding this comment.
You should run make clinic, no?
| @@ -702,7 +715,22 @@ get_process_memory_usage(PyObject *self, PyObject *args) | |||
| if (!PyArg_ParseTuple(args, "i", &pid)) { | |||
| return NULL; | |||
| } | |||
There was a problem hiding this comment.
Please add an empty line for readability.
Uh oh!
There was an error while loading. Please reload this page.