Skip to content

[work in progress] Coroutines the Third #12168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 371 commits into
base: development
Choose a base branch
from

Conversation

Aidan63
Copy link
Contributor

@Aidan63 Aidan63 commented Apr 12, 2025

Third times the charm, right?

This is an evolution of #11554, the core state machine transformation has remained pretty much untouched from that branch, but the surrounding scaffolding code has been changed to more closely align with kotlin's coroutine implementation. This should hopefully address the issues surrounding continuation, scheduling, and a few others which were discovered in the other PR.

Changes

Coroutine functions are now transformed to accept a Continuation<Any> argument, return Any, and contain the state machine transformation. The continuation argument either holds the state of a paused execution of the coroutine function, or what will be invoked once the coroutine completes. The any return will either be the result of the coroutines execution, or a special token indicating that it's execution has been suspended.

@:coroutine function foo(i:Int):String {}

function foo(i:Int, _hx_continuation:ICoroutine<Any>):Any {}

In addition each coroutine function has a class generated for it. This class implements IContinuation<Any> and stores data related to the execution of a coroutine, this includes, the current state, the result or error, variables accessed across states, and a reference to the continuation which will be invoked once this coroutine completes, essentially creating a chain. The continuation interface contains a resume function which when called resumes execution of the coroutine and is responsible to scheduling the execution of the coroutine into an appropriate thread.

class HxCoro__Main_foo implements IContinuation<Any> {
	public var _hx_result : Any;

	public var _hx_error : Exception;

	public var _hx_state : Int;

	public var _hx_context : CoroutineContext;

	public var _hx_completion : IContinuation<Any>;

	public function resume(result:Any, error:Exception) {
		_hx_result = result;
		_hx_error  = error;

		_hx_context.scheduler.schedule(() -> {
			try {
				var result = Main.foo(0, this);
				if (result is Primitive) {
					return;
				} else {
					_hx_completion.resume(result, null);
				}
			} catch (exn:Exception) {
				_hx_completion.resume(null, exn);
			}
		});
	}
}

Coroutines can be started with the blocking function Coroutine.run, which blocks until a coroutine completes. Internally it has its own event loop which all coroutine continuations are scheduled onto.

A quick hello world example is provided below. It should work on all sys targets, but some targets might need their generators updated.

import haxe.coro.Coroutine;
import haxe.coro.Coroutine.delay;

@:coroutine function test() {
    fancyPrint('hello');

    delay(1000);

    fancyPrint('world');
}

@:coroutine function fancyPrint(text:String):Void {
    for (i in 0...text.length) {
        Sys.print(text.charAt(i));

        delay(100);
    }

    Sys.print('\n');
}

function main() {
    Coroutine.run(test);
}

Resources

I'm putting a few links below, specifically related to kotlin's coroutines, which helped the puzzle come together a bit for me.

https://kt.academy/article/cc-under-the-hood

That is a good overview of the underlying machinery of kotlin's coroutines. Jumping straight into decompiled kotlin is very daunting as there are many, many layers of abstraction due to optimisations, features, etc. That article cuts through all of it and gets right to the core of whats happening. I pretty much copied that for this coroutine attempt.

https://www.droidcon.com/2022/09/22/design-of-kotlin-coroutines/

This then goes one layer deper, it covers a lot of those layers you see in kotlin's implementation and explains what they're there for.

https://github.com/JetBrains/kotlin/blob/master/compiler/backend/src/org/jetbrains/kotlin/codegen/coroutines/coroutines-codegen.md

Finally, the mega document. Goes in depth into the jvm codegen of kotlin's coroutines.

@Simn
Copy link
Member

Simn commented Apr 14, 2025

I noticed that we were stuck on an outdated hxcpp branch. After updating this we now have green CI for cpp, which is a good baseline.

As the next step I'd like to get the JVM tests green. And afterwards we'll have to figure out how to make all this work on non-sys targets, because our coroutine execution currently relies on sys.thread.EventLoop.

@skial skial mentioned this pull request Apr 14, 2025
1 task
@Simn
Copy link
Member

Simn commented Apr 15, 2025

Update:

TestBasic
  testResumeWithError: FAILURE F
    line: 30, exception of type haxe.Exception not raised
  testSimple: FAILURE F
    line: 6, expected 42 but it is 330584

Maybe @yuxiaomao could take a look at that some time. We also don't seem to run these tests on CI yet because HL is green.

Aidan63 and others added 30 commits June 6, 2025 19:08
* add ILocalContext

* use for awaitingChildContinuation
* add Key.fromClass to get rid of explicit .key expressions

* we actually don't care about the class itself at all
* split up hx_tmp for result/error

* try to get error wrapping under control

* add surprisingly simple tests
* add IScheduleObject

* use for continuations

* use for async

* also use for CancellationContinuation

* implement suspendCancellable directly

closes #138

* avoid some double-scheduling

* use AtomicInt instead of simon-mutexes for RacingContinuation too

* factor out delayImpl so both delay and yield can call it

This avoids the double suspension call from yield to delay, which I noticed in the yield benchmarks.
# Conflicts:
#	src/typing/typerEntry.ml
* change isCancellationRequested to cancellationException

* bring back isCancellationRequested as a convenience method
* start moving some types to continuation api

* more

* more

* don't load exception twice

* be lazier

* be even lazier

* fix

* ignore that
* support element deletion in Page

* get cpp green even though this seems weird

* add some more tests

* fix isEmpty but find more problems

* fix isEmpty resetting

* don't blit with length 0

* use ArraySort for stability

* change API to something that might actually be sound

* test middle page deletion

* move tests to TestPagedDeque
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants