diff options
| author | Eric Snow <ericsnowcurrently@gmail.com> | 2023-01-30 19:07:48 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-01-30 19:07:48 (GMT) |
| commit | e11fc032a75d067d2167a21037722a770b9dfb51 (patch) | |
| tree | a9e1b00b661f5ccc131f62f161d2963737bcf03c /Include/cpython/pystate.h | |
| parent | ea232716d3de1675478db3a302629ba43194c967 (diff) | |
| download | cpython-e11fc032a75d067d2167a21037722a770b9dfb51.zip cpython-e11fc032a75d067d2167a21037722a770b9dfb51.tar.gz cpython-e11fc032a75d067d2167a21037722a770b9dfb51.tar.bz2 | |
gh-59956: Clarify Runtime State Status Expectations (gh-101308)
A PyThreadState can be in one of many states in its lifecycle, represented by some status value. Those statuses haven't been particularly clear, so we're addressing that here. Specifically:
* made the distinct lifecycle statuses clear on PyThreadState
* identified expectations of how various lifecycle-related functions relate to status
* noted the various places where those expectations don't match the actual behavior
At some point we'll need to address the mismatches.
(This change also includes some cleanup.)
https://github.com/python/cpython/issues/59956
Diffstat (limited to 'Include/cpython/pystate.h')
| -rw-r--r-- | Include/cpython/pystate.h | 27 |
1 files changed, 26 insertions, 1 deletions
diff --git a/Include/cpython/pystate.h b/Include/cpython/pystate.h index 3c1e70d..f5db52f 100644 --- a/Include/cpython/pystate.h +++ b/Include/cpython/pystate.h @@ -119,7 +119,30 @@ struct _ts { PyThreadState *next; PyInterpreterState *interp; - int _status; + struct { + /* Has been initialized to a safe state. + + In order to be effective, this must be set to 0 during or right + after allocation. */ + unsigned int initialized:1; + + /* Has been bound to an OS thread. */ + unsigned int bound:1; + /* Has been unbound from its OS thread. */ + unsigned int unbound:1; + /* Has been bound aa current for the GILState API. */ + unsigned int bound_gilstate:1; + /* Currently in use (maybe holds the GIL). */ + unsigned int active:1; + + /* various stages of finalization */ + unsigned int finalizing:1; + unsigned int cleared:1; + unsigned int finalized:1; + + /* padding to align to 4 bytes */ + unsigned int :24; + } _status; int py_recursion_remaining; int py_recursion_limit; @@ -245,6 +268,8 @@ struct _ts { // Alias for backward compatibility with Python 3.8 #define _PyInterpreterState_Get PyInterpreterState_Get +/* An alias for the internal _PyThreadState_New(), + kept for stable ABI compatibility. */ PyAPI_FUNC(PyThreadState *) _PyThreadState_Prealloc(PyInterpreterState *); /* Similar to PyThreadState_Get(), but don't issue a fatal error |