[ruby-dev:26329] Re: rb_gc_mark_threads spin

なかだです。

At Fri, 10 Jun 2005 14:54:47 +0900,
Tanaka Akira wrote in [ruby-dev:26323]:
> > うーん、backtraceは取れますか。
> 
> とりあえず 5つとれました。

スレッドリストから外されたrb_thread_tがGCされてしまっているのか
も知れません。

> あと、とりあえず無限ループの検査を書いてみました。無限ループより core
> dump のほうがいいひと (って私以外にいるかどうかは怪しいところですが...)
> はどうぞ。

こういうのでテストしてました。手抜きですが。

+    n = 0;
+    FOREACH_THREAD(th) {
+	if (++n > 100000) rb_bug("thread list is too big or currupted");
+    }
+    END_FOREACH(th);


Index: eval.c
===================================================================
RCS file: /cvs/ruby/src/ruby/eval.c,v
retrieving revision 1.787
diff -U2 -p -u -r1.787 eval.c
--- eval.c	9 Jun 2005 11:17:52 -0000	1.787
+++ eval.c	10 Jun 2005 06:59:29 -0000
@@ -10647,4 +10647,5 @@ rb_thread_schedule()
     rb_thread_t th;
     rb_thread_t curr;
+    rb_thread_t orig_thread;
     int found = 0;
 
@@ -10670,8 +10671,10 @@ rb_thread_schedule()
     next = 0;
     curr = curr_thread;		/* starting thread */
+    orig_thread = curr;
 
     while (curr->status == THREAD_KILLED) {
 	curr = curr->prev;
     }
+    curr_thread = curr;
 
     rb_fd_init(&readfds);
@@ -10884,10 +10887,10 @@ rb_thread_schedule()
     }
     next->wait_for = 0;
-    if (next->status == THREAD_RUNNABLE && next == curr_thread) {
+    if (next->status == THREAD_RUNNABLE && next == orig_thread) {
 	return;
     }
 
     /* context switch */
-    if (curr == curr_thread) {
+    if (curr == orig_thread) {
 	if (THREAD_SAVE_CONTEXT(curr)) {
 	    return;


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦