Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Mar 9, 2020

python-gdb.py now checks for "take_gil" function name to check if a
frame tries to acquire the GIL, instead of checking for
"pthread_cond_timedwait" which is specific to Linux and can be a
diferent condition than the GIL.

https://bugs.python.org/issue36184

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @vstinner for commit 395956cb33bf711ec9d945c3325be67a21043d23 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Mar 9, 2020
@vstinner
Copy link
Member Author

vstinner commented Mar 9, 2020

Let me give a try to buildbot jobs since Travis CI doesn't test gdb if I recall correctly, and I would be more confident if I see a FreeBSD job passing here (even if I tested manually my fix with success, see https://bugs.python.org/issue36184#msg363723 ).

@vstinner
Copy link
Member Author

vstinner commented Mar 9, 2020

buildbot/AMD64 FreeBSD Non-Debug PR — Build done.

Sadly, gdb is not installed on this worker:

https://buildbot.python.org/all/#/builders/291/builds/30

0:11:31 load avg: 2.54 [297/420/1] test_gdb skipped
test_gdb skipped -- Couldn't find gdb on the path

python-gdb.py now checks for "take_gil" function name to check if a
frame tries to acquire the GIL, instead of checking for
"pthread_cond_timedwait" which is specific to Linux and can be a
different condition than the GIL.
@vstinner vstinner merged commit 6d0ee60 into python:master Mar 9, 2020
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8.
🐍🍒⛏🤖

@vstinner vstinner deleted the gdb_take_gil branch March 9, 2020 18:35
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 9, 2020
python-gdb.py now checks for "take_gil" function name to check if a
frame tries to acquire the GIL, instead of checking for
"pthread_cond_timedwait" which is specific to Linux and can be a
different condition than the GIL.
(cherry picked from commit 6d0ee60)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-bot
Copy link

GH-18881 is a backport of this pull request to the 3.8 branch.

@bedevere-bot
Copy link

GH-18882 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 9, 2020
python-gdb.py now checks for "take_gil" function name to check if a
frame tries to acquire the GIL, instead of checking for
"pthread_cond_timedwait" which is specific to Linux and can be a
different condition than the GIL.
(cherry picked from commit 6d0ee60)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington added a commit that referenced this pull request Mar 9, 2020
python-gdb.py now checks for "take_gil" function name to check if a
frame tries to acquire the GIL, instead of checking for
"pthread_cond_timedwait" which is specific to Linux and can be a
different condition than the GIL.
(cherry picked from commit 6d0ee60)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington added a commit that referenced this pull request Mar 9, 2020
python-gdb.py now checks for "take_gil" function name to check if a
frame tries to acquire the GIL, instead of checking for
"pthread_cond_timedwait" which is specific to Linux and can be a
different condition than the GIL.
(cherry picked from commit 6d0ee60)

Co-authored-by: Victor Stinner <vstinner@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants