diff options
author | Koichi Sasada <ko1@atdot.net> | 2022-05-24 02:56:59 +0900 |
---|---|---|
committer | Koichi Sasada <ko1@atdot.net> | 2022-05-24 10:06:51 +0900 |
commit | d9984f39d32f4cd692a35f4d803f7754ea262805 (patch) | |
tree | 935aa00da1be80acd99b25dfb7e5e845020ca70d /vm_dump.c | |
parent | 741ac503309f32b5c39073f46a205c99a31d4b0e (diff) |
remove `NON_SCALAR_THREAD_ID` support
`NON_SCALAR_THREAD_ID` shows `pthread_t` is non-scalar (non-pointer)
and only s390x is known platform. However, the supporting code is
very complex and it is only used for deubg print information.
So this patch removes the support of `NON_SCALAR_THREAD_ID`
and make the code simple.
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/5933
Diffstat (limited to 'vm_dump.c')
-rw-r--r-- | vm_dump.c | 8 |
1 files changed, 1 insertions, 7 deletions
@@ -1180,10 +1180,6 @@ rb_vm_bugreport(const void *ctx) } } -#ifdef NON_SCALAR_THREAD_ID -const char *ruby_fill_thread_id_string(rb_nativethread_id_t thid, rb_thread_id_string_t buf); -#endif - void rb_vmdebug_stack_dump_all_threads(void) { @@ -1193,9 +1189,7 @@ rb_vmdebug_stack_dump_all_threads(void) // TODO: now it only shows current ractor ccan_list_for_each(&r->threads.set, th, lt_node) { #ifdef NON_SCALAR_THREAD_ID - rb_thread_id_string_t buf; - ruby_fill_thread_id_string(th->nt->thread_id, buf); - fprintf(stderr, "th: %p, native_id: %s\n", th, buf); + fprintf(stderr, "th: %p, native_id: N/A\n", th); #else fprintf(stderr, "th: %p, native_id: %p\n", (void *)th, (void *)(uintptr_t)th->nt->thread_id); #endif |