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
Debian bigmem buildbot hanging in test_bigmem #58315
Comments
On the debian bigmem buildbot, test_bigmem hangs until it gets killed
This has happened on every run I've checked that hasn't crashed before bpo-5438 might be related. |
New changeset 4afcb25988c4 by Victor Stinner in branch 'default': |
I ran test_bigmem on my computer. The test pass but it is really slow. $ ./python -m test -v -M 8G test_bigmem
(...
Ran 164 tests in 3228.227s OK (skipped=44) One hour, whereas 44 tests are skipped. Skipped tests are the tests using the most quantity of memory, so I expect that they are even slower. Do you need these tests using so much memory (and are really slow)? |
Is this with or without the fix you just pushed? How much RAM does your
In general, I think it's important to exercise cases where 32-bit [Still, 48GB minimum? ೦_೦ ] |
My PC has 12 GB of RAM and no swap. I ran the test after my commit. I tried to enable swap but writing to the swap does completly crash my |
That explains. I ran it earlier (obviously), so it included a bunch of |
The buildbot still isn't happy:
Also, I ran the test on my own machine and noticed two tests that are
|
""" Current thread 0x00002ba57b2d4260: There's a problem with the _file_watchdog thread: """
static void
cancel_file_watchdog(void)
{
/* Notify cancellation */
PyThread_release_lock(watchdog.cancel_event);
/* Wait for thread to join */
PyThread_acquire_lock(watchdog.running, 1);
PyThread_release_lock(watchdog.running);
/* The main thread should always hold the cancel_event lock */
PyThread_acquire_lock(watchdog.cancel_event, 1);
}
""" The
""" AFAICT, this can't happen, because the write end of the pipe is not in Otherwise, there are two things I don't understand:
|
Scratch that. It does bring an improvement for a post facto analysis |
I think my original plan was to put it in non-blocking mode, but I must
I can't think of any drawback off the top of my head, so that sounds |
Here's a patch.
I'll try to write up a patch (I have only 1.2GB RAM, so it won't be easy |
New changeset 52dc4fcd0d6f by Charles-François Natali in branch 'default': |
test_bigmem now passes on the buildbot: |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: