Skip to content
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

Use Go 1.21 WASI GOOS in examples #653

Open
leighmcculloch opened this issue Jul 3, 2023 · 2 comments
Open

Use Go 1.21 WASI GOOS in examples #653

leighmcculloch opened this issue Jul 3, 2023 · 2 comments

Comments

@leighmcculloch
Copy link
Contributor

leighmcculloch commented Jul 3, 2023

Go 1.21 will ship with a new GOARCH=wasm GOOS=wasip1 target. More details at https://tip.golang.org/doc/go1.21.

It should be possible to use that target instead of using tinygo.

@leighmcculloch
Copy link
Contributor Author

I guess it is not so straightforward.

A simple Go app built to wasip1 requires the following imports:

wasi_snapshot_preview1.sched_yield
wasi_snapshot_preview1.proc_exit
wasi_snapshot_preview1.args_get
wasi_snapshot_preview1.args_sizes_get
wasi_snapshot_preview1.clock_time_get
wasi_snapshot_preview1.environ_get
wasi_snapshot_preview1.environ_sizes_get
wasi_snapshot_preview1.fd_write
wasi_snapshot_preview1.random_get
wasi_snapshot_preview1.poll_oneoff

Also, when I stub them with the following changes:

diff --git a/runtimes/web/src/runtime.ts b/runtimes/web/src/runtime.ts
index bf83d76..01c81d5 100644
--- a/runtimes/web/src/runtime.ts
+++ b/runtimes/web/src/runtime.ts
@@ -159,8 +159,21 @@ export class Runtime {
             tracef: this.tracef.bind(this),
         };
 
+        const wasi_snapshot_preview1 = {
+            sched_yield: () => {},
+            proc_exit: () => {},
+            args_get: () => {},
+            args_sizes_get: () => {},
+            clock_time_get: () => {},
+            environ_get: () => {},
+            environ_sizes_get: () => {},
+            fd_write: () => {},
+            random_get: () => {},
+            poll_oneoff: () => {},
+        };
+
         await this.bluescreenOnError(async () => {
-            const module = await WebAssembly.instantiate(wasmBuffer, { env });
+            const module = await WebAssembly.instantiate(wasmBuffer, { env, wasi_snapshot_preview1 });
             this.wasm = module.instance;
 
             // Call the WASI _start/_initialize function (different from WASM-4's start callback!)

It still doesn't run, and produces this error in the browser:

Unhandled Promise Rejection: RangeError: Maximum call stack size exceeded.

@leighmcculloch
Copy link
Contributor Author

Also, it seems Go 1.21 wasip1 won't allow exporting functions to WASM:

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

No branches or pull requests

1 participant