ractor_sync.c: Optimize ractor_set_successor_once to be lock free

Author: byroot

ractor_sync.c

@@ -728,22 +728,8 @@ ractor_sync_init(rb_ractor_t *r)
 static rb_ractor_t *
 ractor_set_successor_once(rb_ractor_t *r, rb_ractor_t *cr)
 {
-    if (r->sync.successor == NULL) {
-        RACTOR_LOCK(r);
-        {
-            if (r->sync.successor != NULL) {
-                // already `value`ed
-            }
-            else {
-                r->sync.successor = cr;
-            }
-        }
-        RACTOR_UNLOCK(r);
-    }
-
-    VM_ASSERT(r->sync.successor != NULL);
-
-    return r->sync.successor;
+    rb_ractor_t *successor = ATOMIC_PTR_CAS(r->sync.successor, NULL, cr);
+    return successor == NULL ? cr : successor;
 }
 
 static VALUE ractor_reset_belonging(VALUE obj);