Skip to content

bpo-36184: Port python-gdb.py to FreeBSD#18873

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:gdb_take_gil
Mar 9, 2020
Merged

bpo-36184: Port python-gdb.py to FreeBSD#18873
vstinner merged 1 commit into
python:masterfrom
vstinner:gdb_take_gil

Conversation

@vstinner

@vstinner vstinner commented Mar 9, 2020

Copy link
Copy Markdown
Member

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
Copy Markdown

🤖 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

vstinner commented Mar 9, 2020

Copy link
Copy Markdown
Member Author

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

vstinner commented Mar 9, 2020

Copy link
Copy Markdown
Member Author

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
Copy Markdown
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
Copy Markdown

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

@bedevere-bot

Copy link
Copy Markdown

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