[ruby-dev:28190] Re: thread based generator.rb

なかだです。

At Thu, 29 Dec 2005 19:37:22 +0900,
nobuyoshi nakada wrote in [ruby-dev:28173]:
> * eval.c (rb_gc_abort_threads), gc.c (gc_sweep): kill unmarked
>   threads.  [ruby-dev:28172]

このときに新しくmark_stackに積まれる可能性もありました。

「後でコミット」。


Index: gc.c
===================================================================
RCS file: /cvs/ruby/src/ruby/gc.c,v
retrieving revision 1.226
diff -U2 -p -r1.226 gc.c
--- gc.c	29 Dec 2005 17:10:29 -0000	1.226
+++ gc.c	30 Dec 2005 01:15:17 -0000
@@ -1018,6 +1018,4 @@ gc_sweep(void)
     }
 
-    rb_gc_abort_threads();
-
     freelist = 0;
     final_list = deferred_final_list;
@@ -1345,12 +1343,16 @@ garbage_collect(void)
 
     /* gc_mark objects whose marking are not completed*/
-    while (!MARK_STACK_EMPTY){
-	if (mark_stack_overflow){
-	    gc_mark_all();
-	}
-	else {
-	    gc_mark_rest();
+    do {
+	while (!MARK_STACK_EMPTY) {
+	    if (mark_stack_overflow){
+		gc_mark_all();
+	    }
+	    else {
+		gc_mark_rest();
+	    }
 	}
-    }
+	rb_gc_abort_threads();
+    } while (!MARK_STACK_EMPTY);
+
     gc_sweep();
 


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