Stream: git-wasmtime

Topic: wasmtime / PR #13489 LTS: Fix build failure on s390x/musl

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 23:00):

WhyNotHugo opened PR #13489 from WhyNotHugo:release-36.0.0-s390x/musl to bytecodealliance:release-36.0.0:

Patch-Source: https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/83794

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 23:00):

WhyNotHugo requested dicej for a review on PR #13489.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 23:00):

WhyNotHugo requested wasmtime-core-reviewers for a review on PR #13489.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 23:50):

alexcrichton unassigned dicej from PR #13489 LTS: Fix build failure on s390x/musl.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 23:50):

alexcrichton requested alexcrichton for a review on PR #13489.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 23:52):

:memo: alexcrichton submitted PR review:

Thanks for the PR. To confirm, have you run this code and tested it? I ask because the fp in the glibc case does a dereference of the 15th register, but the code here does not do a dereference. That looks like a bug to me, but I wanted to ask. If you haven't tested this I'm personally a bit hesitant to land this because it's generally much more difficult to diagnose these sorts of issues after it lands.

Additionally, do you know the status of exposing the context structures in musl?

And, finally, can you send this PR to the main branch first, and then cherry-pick it for a backport once it's accepted on main? We don't have a ton of precedent here but I think it'd be best to land things on main first and then backport from there.

view this post on Zulip Wasmtime GitHub notifications bot (May 27 2026 at 04:03):

github-actions[bot] added the label wasmtime:api on PR #13489.

view this post on Zulip Wasmtime GitHub notifications bot (May 27 2026 at 14:12):

WhyNotHugo commented on PR #13489:

We built the downstream Alpine packages with this patch, and https://github.com/open-policy-agent/opa (a dependency of wasmtime) built fine with it and passes all tests.

I was also going to say that all tests passed with this patch, but noticed that tests were actually disabled for the wasmtime package. I enabled them and some tests actually fail even with this patch, so I'll inspect closer and report back.

view this post on Zulip Wasmtime GitHub notifications bot (May 27 2026 at 14:12):

WhyNotHugo converted PR #13489 LTS: Fix build failure on s390x/musl to a draft.

view this post on Zulip Wasmtime GitHub notifications bot (May 27 2026 at 14:28):

:memo: alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 27 2026 at 14:28):

:speech_balloon: alexcrichton created PR review comment:

Thanks for running tests! I think this may be at least one minor issue (it's different from the fp definition above which reads memory). I'm not sure if S390xRegs is the correct definition, as well.

Also, and I'm sorry for lots of comments, but mind renaming S390xRegs to ucontext_t? That matches the intention of the structure where it's matching the C definition of ucontext_t

Last updated: Jun 01 2026 at 09:49 UTC