Skip to content

bpo-38644: Add _PyEval_EvalFrame() with tstate #17131

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

Merged
merged 1 commit into from
Nov 14, 2019
Merged

bpo-38644: Add _PyEval_EvalFrame() with tstate #17131

merged 1 commit into from
Nov 14, 2019

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Nov 12, 2019

Add _PyEval_EvalFrame() static inline function to get eval_frame from
tstate->interp.

https://bugs.python.org/issue38644

Add _PyEval_EvalFrame() static inline function to get eval_frame from
tstate->interp.
@vstinner
Copy link
Member Author

cc @markshannon @ericsnowcurrently

@vstinner
Copy link
Member Author

"Pass the Python thread state to internal C functions" thread on python-dev list confirmed that tstate is a performance bottleneck to scale Python on N CPUs: https://mail.python.org/archives/list/[email protected]/thread/PQBGECVGVYFTVDLBYURLCXA3T7IPEHHO/

So I merged my PR.

@vstinner vstinner merged commit b9e6812 into python:master Nov 14, 2019
@vstinner vstinner deleted the evalframe_tstate branch November 14, 2019 11:20
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
Add _PyEval_EvalFrame() static inline function to get eval_frame from
tstate->interp.
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
Add _PyEval_EvalFrame() static inline function to get eval_frame from
tstate->interp.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants