Wasmtime GitHub notifications bot (Sep 10 2021 at 09:46): Wasmtime GitHub notifications bot (Sep 10 2021 at 09:46):afonso360 commented on issue #3325:
CI Passes, but I think that's a false positive, it looks like QEMU is segfaulting.
Wasmtime GitHub notifications bot (Sep 10 2021 at 09:54):sparker-arm commented on issue #3325:
CI Passes, but I think that's a false positive, it looks like QEMU is segfaulting.
This isn't an issue with this patch though, I think you'll find the same thing happening if you look in any of the recent other patches too.
Wasmtime GitHub notifications bot (Sep 10 2021 at 09:55):sparker-arm edited a comment on issue #3325:
CI Passes, but I think that's a false positive, it looks like QEMU is segfaulting.
This isn't an issue with this patch though, I think you'll find the same thing happening if you look in any of the recent other patches too.
I only say recent, because it's only relatively recently that segfaulting was happening.
Wasmtime GitHub notifications bot (Sep 10 2021 at 09:55):sparker-arm edited a comment on issue #3325:
CI Passes, but I think that's a false positive, it looks like QEMU is segfaulting.
This isn't an issue with this patch though, I think you'll find the same thing happening if you look in any of the recent other patches too.
I only say recent, because it's only relatively recently that I was made aware that segfaulting was happening.
Wasmtime GitHub notifications bot (Sep 10 2021 at 17:59):alexcrichton commented on issue #3325:
Hm that does look a little worrisome, but is it something that's known to be a failure? Our test suite triggers lots of segfaults which end up being caught, so I'm not sure if QEMU is printing the segfault and then we catch it or what. Also QEMU doesn't entirely exit because the test suite keeps running, so if it all runs to completion I'm not sure what the segfault message is actually pointing to.
Wasmtime GitHub notifications bot (Sep 15 2021 at 12:18):akirilov-arm commented on issue #3325:
I realize that you started this patch with a narrow scope, but how about upgrading to QEMU 6.1.0 as well - should be a simple change? From an AArch64 perspective the biggest advantage is IMHO SVE2 support.
Wasmtime GitHub notifications bot (Sep 15 2021 at 12:55):afonso360 commented on issue #3325:
Yeah, sounds good.
It should also eventually allow us to add the s390x runner, since it looks like the issues with s390x are fixed in 6.1.0
Wasmtime GitHub notifications bot (Sep 15 2021 at 14:14):alexcrichton commented on issue #3325:
I double-checked the current builds we have and they don't have qemu printing out anything related to a segfault, which is somewhat worrisome. Would you be able to do a binary search on enabled features here to see which one is causing the segfault messages to appear? I still don't know how to interpret them because emulation continues to success but if we know which feature is causing the fault we may be able to disable it or debug further?
Wasmtime GitHub notifications bot (Sep 15 2021 at 14:21):afonso360 commented on issue #3325:
Are you sure? I'm seeing the segfaults on the latest PR we merged (here)
Wasmtime GitHub notifications bot (Sep 15 2021 at 14:29):afonso360 edited a comment on issue #3325:
Are you sure? I'm seeing the segfaults on the latest PR we merged (here).
Do we have a way to search the logs of previous builds to see when this first appeared? If it has been happening for a long time the logs might have been deleted though.
Wasmtime GitHub notifications bot (Sep 15 2021 at 14:30):alexcrichton commented on issue #3325:
Oops I think I must have checked the wrong logs, I see them now. shakes fist at GitHub Actions UI
Last updated: Nov 22 2024 at 17:03 UTC