-
-
Notifications
You must be signed in to change notification settings - Fork 33.5k
gh-141770: Annotate anonymous mmap usage in debug builds only #142079
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
cc @picnixz |
|
FYI, Fedora does not support yet even if the kernel version itself already support it. |
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
|
FWIW, it would be useful to have an option to enable this in release builds too. |
|
@SpecLad Thanks for the feedback, I will create a separate PR :) |
|
@SpecLad Just out of curiosity, passing the build flag through |
|
To be clear, by "option" I really mean a runtime option. 🙂 It would be pretty annoying to have to rebuild Python just for this. If you compare with other implementations, none of them require a compile-time option:
TBH, I don't see why Python shouldn't also permanently enable this, but I would settle for "off by default, enabled with environment variable or |
|
It's a bit annoying that this would only be supported on Linux. Do we have an option that isn't supported everywhere? |
|
@SpecLad Let’s move this forward conservatively for 3.15, and we can consider making it the default in the future. A runtime check would introduce an overall performance regression. If some distros want to use this feature, they will enable it through a build flag, and it always happens. |
Looks not and I belive that this is off-topic. :) |
|
My point was: I prefer not having an |


mmapin_PyMem_ArenaAllocfor debugging #141770