| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Annotation
Without annotation
I verified that this annotation actually works, and I believe it will be very helpful for debugging when needed. |
Sorry, something went wrong.
|
FYI, Fedora does not support yet even if the kernel version itself already support it. |
Sorry, something went wrong.
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. |
Sorry, something went wrong.
|
@SpecLad Thanks for the feedback, I will create a separate PR :) |
Sorry, something went wrong.
|
@SpecLad Just out of curiosity, passing the build flag through ./configure --enable-annotate-mmap would be fine to you? |
Sorry, something went wrong.
|
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 -X option". |
Sorry, something went wrong.
|
It's a bit annoying that this would only be supported on Linux. Do we have an option that isn't supported everywhere? |
Sorry, something went wrong.
Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Victor Stinner <vstinner@python.org>
| modules that are missing or packaged separately. | ||
| (Contributed by Stan Ulbrych and Petr Viktorin in :gh:`139707`.) | ||
|
|
||
| * Annotating anonymous mmap usage is now supported if Linux kernel supports ``PR_SET_VMA_ANON_NAME`` (Linux 5.17 or newer). |
There was a problem hiding this comment.
* Annotating anonymous mmap usage is now supported if Linux kernel
supports :manpage:`PR_SET_VMA_ANON_NAME <PR_SET_VMA(2const)>`
(Linux 5.17 or newer).I don't know under which manpage entry it is, but if you locally run Sphinx, you should be able to find the correct one. That way, users would have a link to click on. If there isn't a manpage, nevermind.
Sorry, something went wrong.
…e-141770.JURnvg.rst Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
…e-141770.JURnvg.rst Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Petr Viktorin <encukou@gmail.com>
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
Co-authored-by: Victor Stinner <vstinner@python.org>
⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️Hi! The buildbot AMD64 Ubuntu Shared 3.x (tier-1) has failed when building commit c4ccaf4. What do you need to do:
You can take a look at the buildbot page here: https://buildbot.python.org/#/builders/506/builds/11950 Failed tests:
Failed subtests:
Summary of the results of the build (if available): == Click to see traceback logsTraceback (most recent call last):
File "/srv/buildbot/buildarea/3.x.bolen-ubuntu/build/Lib/test/test_audit.py", line 332, in test_sys_remote_exec
self.fail(stderr)
~~~~~~~~~^^^^^^^^
AssertionError: Fatal Python error: _Py_CheckFunctionResult: a function returned a result with an exception set
Python runtime state: initialized
OSError: Cannot open ELF file '[anon:cpython:pymalloc]' for section 'PyRuntime' search: No such file or directory
|
Sorry, something went wrong.
|
Please can you check the buildbot failure? |
Sorry, something went wrong.
… used (pythongh-142079)" This reverts commit c4ccaf4.
|
I think it's a problem with the buildbot machine: it's timing out on other branches now, and the builds are taking nearly five hours, when they used to take around 50 minutes: |
Sorry, something went wrong.
|
I just sent an email to David Bolen (the buildbot owner) to ask him to have a look. Something seems to be wrong with this buildbot worker. |
Sorry, something went wrong.
|
This change introduced a regression: #142627. |
Sorry, something went wrong.
|
Note for myself: The feature is not available on Fedora, since Fedora builds the Linux kernel without the CONFIG_ANON_VMA_NAME option: $ grep CONFIG_ANON_VMA_NAME /boot/config-$(uname -r) # CONFIG_ANON_VMA_NAME is not set |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.