(playground) Aborts cause state loss.

[Jean-Luc-Picard-2021]

Hi,

Not sure whether this ticket is redundant to "(ciaowasm) make '$yield'/0
interact with JS async" what is seen in the commits?

There is some state loss in manual abort:

?- assertz(foo).
yes

?- foo.
yes

?- repeat, fail.
{ Execution aborted }

?- foo.
{ERROR: No handle found for thrown exception error(
existence_error(procedure,'user:foo'/0),'user:foo'/0)}
aborted

And in timeout:

?- assertz(foo).
yes

?- repeat, fail.
{ABORTED: Time limit exceeded.}

?- foo.
{ERROR: No handle found for thrown exception error(
existence_error(procedure,'user:foo'/0),'user:foo'/0)}
aborted

https://ciao-lang.org/playground/

Expectation would be rather that this doesn't happen.

[jfmc]

Thank you. This is a known limitation similar to blocking IO. Timeouts are hard without asyncify or shared array buffers because there is no way for the executing thread to know that it has been asked to abort. Asyncify should be able but we've not measured the performance impact yet. Right now aborts restart the engine.

[Jean-Luc-Picard-2021]

Works in Jan W prototype, but his prototype is still a little slow.

https://dev.swi-prolog.org/wasm/shell

?- assertz(foo).
true.

?- foo.
true.

?- repeat, fail.
ERROR: Execution Aborted

?- foo.
true.

The version I am reporting uses a setTimout() driven auto yielding loop.
Through setTimeout() you give the browser the chance to
work its task queues, it will do all kind of stuff, in the above

example the browser stays responsive during repeat, fail,
please try it, it does all that without a Worker. I guess this is
a nice first proof of concept and one of the goals of this work.

[Jean-Luc-Picard-2021]

You also see here that it doesn’t have any Workers, there
is no Threads tab in the debugger. But interesting find, SWIPL itself
got promisified, and the very first next() call goes through toplevel():

[jfmc]

Thanks. Sorry, I was confused with the long thread at SWI discourse. We'll try automatic yield but if there is some serious impact on performance, we will consider also other options (maybe optionally). For some workers losing the state is not an issue.

[Jean-Luc-Picard-2021]

Now Jan W prototype got faster, but I don't have yet a live link to share.

[jfmc]

Not fixed yet.