Skip to content

gh-116664: Make module state Py_SETREF's in _warnings thread-safe #116959

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.

Sign up for GitHub

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
erlend-aasland merged 13 commits into from
Mar 28, 2024
Merged

gh-116664: Make module state Py_SETREF's in _warnings thread-safe #116959

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
42e98b8
gh-116664: Make module state Py_SETREF's in _warnings thread-safe
erlend-aasland Mar 18, 2024
46dc0f4
Pull in main
erlend-aasland Mar 18, 2024
d222f7c
Protect warn_explicit() instead
erlend-aasland Mar 18, 2024
99e04de
Pull in main
erlend-aasland Mar 19, 2024
be08b22
Forgot one call
erlend-aasland Mar 19, 2024
3357659
Pull in main
erlend-aasland Mar 19, 2024
a87d84c
Use more granular critical sections
erlend-aasland Mar 19, 2024
30ee26d
Pull in main
erlend-aasland Mar 26, 2024
f9e549d
Put a mutex in _warnings_runtime_state
erlend-aasland Mar 27, 2024
4f014ab
Pull in main
erlend-aasland Mar 27, 2024
a6b127f
Pass a pointer
erlend-aasland Mar 27, 2024
d5d1ba3
Pull in main
erlend-aasland Mar 28, 2024
c6c1866
Address review: assert state and remove unneeded mutex initialization
erlend-aasland Mar 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions Python/_warnings.c
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
#include "Python.h"
#include "pycore_critical_section.h"// Py_BEGIN_CRITICAL_SECTION()
#include "pycore_interp.h" // PyInterpreterState.warnings
#include "pycore_long.h" // _PyLong_GetZero()
#include "pycore_pyerrors.h" // _PyErr_Occurred()
Expand Down Expand Up @@ -257,7 +258,9 @@ get_once_registry(PyInterpreterState *interp)
Py_DECREF(registry);
return NULL;
}
Py_BEGIN_CRITICAL_SECTION(registry);
Comment thread
erlend-aasland marked this conversation as resolved.
Outdated
Py_SETREF(st->once_registry, registry);
Py_END_CRITICAL_SECTION();
return registry;
}

Expand Down Expand Up @@ -288,7 +291,9 @@ get_default_action(PyInterpreterState *interp)
Py_DECREF(default_action);
return NULL;
}
Py_BEGIN_CRITICAL_SECTION(default_action);
Py_SETREF(st->default_action, default_action);
Py_END_CRITICAL_SECTION();
return default_action;
}

Expand All @@ -313,7 +318,9 @@ get_filter(PyInterpreterState *interp, PyObject *category,
return NULL;
}
else {
Py_BEGIN_CRITICAL_SECTION(warnings_filters);
Py_SETREF(st->filters, warnings_filters);
Py_END_CRITICAL_SECTION();
}

PyObject *filters = st->filters;
Expand Down