Skip to content

bpo-37737: mmap module, track anonymous paging case on macOS#15068

Closed
devnexen wants to merge 2 commits intopython:masterfrom
devnexen:mmap_module_id_anon_darwin
Closed

bpo-37737: mmap module, track anonymous paging case on macOS#15068
devnexen wants to merge 2 commits intopython:masterfrom
devnexen:mmap_module_id_anon_darwin

Conversation

@devnexen
Copy link
Copy Markdown
Contributor

@devnexen devnexen commented Aug 1, 2019

@devnexen devnexen changed the title mmap module, track anonymous paging case on macOS bpo-37737: mmap module, track anonymous paging case on macOS Aug 1, 2019
@devnexen
Copy link
Copy Markdown
Contributor Author

devnexen commented Aug 1, 2019

Allows output as :
image

@devnexen devnexen force-pushed the mmap_module_id_anon_darwin branch from 30dd5f5 to 1c8906f Compare August 1, 2019 13:49
@ronaldoussoren
Copy link
Copy Markdown
Contributor

What is the use case for this?

And why one memory tag with a specific value?

@devnexen
Copy link
Copy Markdown
Contributor Author

devnexen commented Aug 1, 2019

If it is you wish the python mmap module mapping standing out to monitor it more accurately.
Otherwise tag ID is allowed up to 255, guaranteed from 240 to be freely available at least.

@ronaldoussoren
Copy link
Copy Markdown
Contributor

I still don't understand what the use case is, why is it useful to track usage of mmap.mmap in this way?

I'd expect that tracking Python's memory allocator would be more useful to track memory usage patterns and debug production issues.

@devnexen
Copy link
Copy Markdown
Contributor Author

devnexen commented Aug 5, 2019

It is a more native mean ... when it occurs you monitor the whole system and allows to distinguish python from the rest.

@ronaldoussoren
Copy link
Copy Markdown
Contributor

I still don't think this is a useful feature. I'm therefore closing this PR (and the related issue).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants