-
-
Notifications
You must be signed in to change notification settings - Fork 33.3k
gh-138004: Encode thread names on Solaris-based platforms as ASCII and fix main/worker threading test #138017
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
Changes from 2 commits
d97417d
1db08a7
0606968
b556774
38a75d3
31731b1
83fe205
aadf7f3
612a0a4
1fa51f8
d24d0bb
95d289f
d7a47bf
1970a00
6395323
241e097
66c058b
f817412
3349464
ac3d3dd
1ab9ecd
d9ba9e8
d7025f5
3689044
246f880
407b1e9
dc43fc4
7aa256c
b4934a2
ae5ac4b
4fc368d
392aa5a
84daea4
d5e9399
105d4e6
c67e93d
508f24d
25085bf
cd26833
dc86a29
c9f554d
c0f672e
69796ff
15c8481
99f13ec
8fc613b
5e5effb
9690eee
338593c
e642264
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| ``` | ||
| Core and Builtins | ||
| ----------------- | ||
|
|
||
jadonduff marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| * _thread.set_name() now retries with an ASCII fallback if pthread_setname_np() rejects UTF-8 names on some platforms (e.g. OpenIndiana). | ||
| * test_threading.py was updated to skip non-ASCII thread name tests on platforms that do not support them. | ||
| ``` | ||
jadonduff marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2611,20 +2611,57 @@ _thread_set_name_impl(PyObject *module, PyObject *name_obj) | |
| #endif | ||
|
|
||
| const char *name = PyBytes_AS_STRING(name_encoded); | ||
| int rc; | ||
| #ifdef __APPLE__ | ||
| int rc = pthread_setname_np(name); | ||
| rc = pthread_setname_np(name); | ||
| #elif defined(__NetBSD__) | ||
| pthread_t thread = pthread_self(); | ||
| int rc = pthread_setname_np(thread, "%s", (void *)name); | ||
| rc = pthread_setname_np(thread, "%s", (void *)name); | ||
| #elif defined(HAVE_PTHREAD_SETNAME_NP) | ||
| pthread_t thread = pthread_self(); | ||
| int rc = pthread_setname_np(thread, name); | ||
| rc = pthread_setname_np(thread, name); | ||
| #else /* defined(HAVE_PTHREAD_SET_NAME_NP) */ | ||
| pthread_t thread = pthread_self(); | ||
| int rc = 0; /* pthread_set_name_np() returns void */ | ||
| rc = 0; /* pthread_set_name_np() returns void */ | ||
| pthread_set_name_np(thread, name); | ||
| #endif | ||
| Py_DECREF(name_encoded); | ||
|
|
||
| // Fallback: If EINVAL, try ASCII encoding with "replace" | ||
| if (rc == EINVAL) { | ||
| name_encoded = PyUnicode_AsEncodedString(name_obj, "ascii", "replace"); | ||
| if (name_encoded == NULL) { | ||
| return NULL; | ||
| } | ||
| #ifdef _PYTHREAD_NAME_MAXLEN | ||
| if (PyBytes_GET_SIZE(name_encoded) > _PYTHREAD_NAME_MAXLEN) { | ||
| PyObject *truncated; | ||
| truncated = PyBytes_FromStringAndSize(PyBytes_AS_STRING(name_encoded), | ||
| _PYTHREAD_NAME_MAXLEN); | ||
| if (truncated == NULL) { | ||
| Py_DECREF(name_encoded); | ||
| return NULL; | ||
| } | ||
| Py_SETREF(name_encoded, truncated); | ||
| } | ||
| #endif | ||
| name = PyBytes_AS_STRING(name_encoded); | ||
| #ifdef __APPLE__ | ||
| rc = pthread_setname_np(name); | ||
| #elif defined(__NetBSD__) | ||
| thread = pthread_self(); | ||
| rc = pthread_setname_np(thread, "%s", (void *)name); | ||
| #elif defined(HAVE_PTHREAD_SETNAME_NP) | ||
| thread = pthread_self(); | ||
| rc = pthread_setname_np(thread, name); | ||
| #else /* defined(HAVE_PTHREAD_SET_NAME_NP) */ | ||
| thread = pthread_self(); | ||
| rc = 0; | ||
| pthread_set_name_np(thread, name); | ||
| #endif | ||
| Py_DECREF(name_encoded); | ||
|
||
| } | ||
|
|
||
| if (rc) { | ||
| errno = rc; | ||
| return PyErr_SetFromErrno(PyExc_OSError); | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is OSError even raised here? The test failure was different -- that
work_namewas unexpectedly empty.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added:
Kept OSError fallback because it was shown in original issue, but can remove if needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was shown that
_thread.set_name()fails, but that OSError is not leaked from the Thread code.