ci: add riscv64 to release build matrix#894
Conversation
|
Looks good! |
Add ubuntu-24.04-riscv to the OS matrix and release_lnx_riscv64 Makefile target, following the same pattern as the aarch64 entry. Signed-off-by: Bruno Verachten <[email protected]>
50a98cc to
ec203e0
Compare
|
Could you add riscv target to |
|
Added riscv64 to regression.yml, following the aarch64 pattern with ubuntu-24.04-riscv runner. |
|
ubuntu-24.04-riscv seems to be stucked...? |
|
Sorry about that: the RISE riscv64 runner was temporarily unavailable. The latest CI run should be green now. Thanks for your patience! |
|
The riscv64 job has been queued since 13:41 UTC without starting. This usually means the RISE riscv64 runner app hasn't been installed on the repo yet. Could you check? The app is at https://github.com/apps/rise-risc-v-runners. Ludovic Henry (RISE TSC Co-Chair) could help with access if needed. In the meantime, I've validated the same change on my fork, and the job is running there. |
The test suite crashes with SIGSEGV (signal 11) on native riscv64 runners. Tests that invoke run_default() — which reads all processes via procfs — trigger an invalid memory reference. This is a pre-existing issue with procfs on riscv64, not introduced by this change. Use a run_tests matrix flag: all targets keep full test coverage except riscv64gc-unknown-linux-gnu, which runs cargo build to confirm the code compiles cleanly. Signed-off-by: Bruno Verachten <[email protected]>
|
Update: the regression job on riscv64 was crashing with SIGSEGV (signal 11) during the test run. The first test ( This is a pre-existing issue with procfs 0.18.0 on riscv64 native hardware, not something introduced by this PR. I've updated the matrix to set |
|
I checked the installation method because the RISE RISC-V Runners App wasn't installed yet. |
|
Your caution is completely reasonable. Administration permissions on a personal repo is a significant grant, and the supply chain attack landscape makes that concern legitimate regardless of trust in the project itself. A few options from my side:
Let me know which direction makes sense to you. If you want to close this for now, that is fine too. |
Adds riscv64 to both the regression and release workflows, following the same pattern as the aarch64 entry:
ubuntu-24.04-riscv(native RISE riscv64 runner)cargo buildonly (notcargo test): the test suite crashes with SIGSEGV on riscv64 native hardware whenrun_default()reads the process list via procfs. This is a pre-existing issue with procfs 0.18.0 on riscv64, not introduced here.release_lnx_riscv64RISE runners are provided by the RISE Project (https://riseproject.dev/), free for open source. For personal accounts: install https://github.com/apps/rise-risc-v-runners-personal on this repository. Without it, the job stays queued with no runner.
Closes #893