diff options
| author | Koichi Sasada <[email protected]> | 2022-05-24 02:56:59 +0900 |
|---|---|---|
| committer | Koichi Sasada <[email protected]> | 2022-05-24 10:06:51 +0900 |
| commit | d9984f39d32f4cd692a35f4d803f7754ea262805 () | |
| 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 removes the support of `NON_SCALAR_THREAD_ID` and make the code simple.
Notes: Merged: https://.com/ruby/ruby/pull/5933
| -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); #else fprintf(stderr, "th: %p, native_id: %p\n", (void *)th, (void *)(uintptr_t)th->nt->thread_id); #endif |