sightglass icon indicating copy to clipboard operation
sightglass copied to clipboard

benchmark: failed to rebuild spidermonkey benchmark

Open abrown opened this issue 2 years ago • 2 comments

The "rebuild benchmarks" CI workflow found an issue rebuilding the SpiderMonkey benchmark: https://github.com/bytecodealliance/sightglass/actions/runs/4453084060/jobs/7821309649#step:4:5101. This CI workflow only rebuilds a subset of the benchmarks each CI run (for time's sake) to check that they can be rebuilt deterministically, so the failure will only reappear occasionally. cc: @cfallin

abrown avatar Mar 20 '23 10:03 abrown

Hmm -- I took a look at this and was able to reproduce locally. I'm not sure where the mach bootstrap error is coming from though. The version of SpiderMonkey we have pinned is fairly outdated so it's possible that the toolchain it's trying to download no longer exists, and an update to latest SpiderMonkey would fix this. (I don't have the bandwidth to do this at the moment, but we can leave this open to track...)

cfallin avatar Mar 21 '23 19:03 cfallin

I was able to get around this by editing the Spidermonkey Dockerfile to clone the fork https://github.com/fastly/spidermonkey-wasi-embedding instead of https://github.com/tschneidereit/spidermonkey-wasi-embedding. ~~But for some reason the runtime.cpp gets stuck somewhere inside JS_Init() when I attempt to run the benchmark.~~

~~EDIT: I found out that the reason the benchmark got stuck was because for some reason __wasm_call_ctors never got invoked in the _start function. When I manually called it in the main function of runtime.cpp the benchmark ran successfully.~~

EDIT 2: Nevermid me, I had inserted a call to __wasm_call_ctors at a different place in the code, which I think causes wasi sdk version 14 to think that the different place was the entrypoint and not the main function. Using the fork I mentioned works for me at least.

william-stacken avatar Mar 22 '23 08:03 william-stacken