From 25c82f2c1233c2526d4e2f747f010864e0575bac Mon Sep 17 00:00:00 2001 From: ko1 Date: Thu, 26 Oct 2017 14:38:22 +0000 Subject: make root fiber at switching. * cont.c (fiber_switch): make sure the root fiber object is available before the first switching. * test/ruby/test_fiber.rb: remove "skip". git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60453 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- cont.c | 8 +++++++- test/ruby/test_fiber.rb | 1 - 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/cont.c b/cont.c index d0f9f27166..bc0c1fa3f3 100644 --- a/cont.c +++ b/cont.c @@ -1521,7 +1521,10 @@ return_fiber(void) rb_fiber_t *prev = fib->prev; if (!prev) { - rb_fiber_t *root_fiber = GET_THREAD()->root_fiber; + rb_thread_t *th = GET_THREAD(); + rb_fiber_t *root_fiber = th->root_fiber; + + VM_ASSERT(root_fiber != NULL); if (root_fiber == fib) { rb_raise(rb_eFiberError, "can't yield from root fiber"); @@ -1623,6 +1626,9 @@ fiber_switch(rb_fiber_t *fib, int argc, const VALUE *argv, int is_resume) rb_context_t *cont = &fib->cont; rb_thread_t *th = GET_THREAD(); + /* make sure the root_fiber object is available */ + if (th->root_fiber == NULL) root_fiber_alloc(th); + if (th->ec->fiber == fib) { /* ignore fiber context switch * because destination fiber is same as current fiber diff --git a/test/ruby/test_fiber.rb b/test/ruby/test_fiber.rb index 7670e1e1e0..92c9961259 100644 --- a/test/ruby/test_fiber.rb +++ b/test/ruby/test_fiber.rb @@ -184,7 +184,6 @@ class TestFiber < Test::Unit::TestCase end def test_fiber_transfer_segv - skip assert_normal_exit %q{ require 'fiber' f2 = nil -- cgit v1.2.3