Conversation
thaJeztah
left a comment
There was a problem hiding this comment.
thx! left some nits for consistency 😅
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
| "SCMP_ARCH_PPC": "ppc", | ||
| "SCMP_ARCH_PPC64": "ppc64", | ||
| "SCMP_ARCH_PPC64LE": "ppc64le", | ||
| "SCMP_ARCH_RISCV64": "riscv64", |
There was a problem hiding this comment.
oh! think this one ended up in the wrong commit, or was it intentional to be part of the vendor update?
There was a problem hiding this comment.
Yes. Tied with the new version of libseccomp-golang (seccomp/libseccomp-golang@e947d82).
runc/libcontainer reads "SCMP_ARCH_RISCV64" from config.json and converts it to "riscv64" via runc/libcontainer/seccomp.ConvertStringToArch().
And then libseccomp-golang converts it back to "riscv64" -> ArchRISCV64 -> C_ARCH_RISCV64 -> SCMP_ARCH_RISCV64.
|
@AkihiroSuda I can test on my side too if you want, I have an HiFive Unmatched. Let me know. |
|
@crazy-max Thanks, testing on a real machine is highly appreciated 🙏 . |
Let me know if this is enough to test its behavior. |
|
@crazy-max Thanks, could you test |
|
Superceded by #3446 |
|
|
Thank you @crazy-max , it is working as expected |
The first commit bumps up libseccomp-golang (seccomp/libseccomp-golang@3879420...e214ef1) to support
SCMP_ARCH_RISCV64(seccomp/libseccomp-golang@e947d82).The second commit updates the build scripts to build
runc.riscv64.Tested on Ubuntu 22.04 (kernel 5.15) on
qemu-system-riscv64 -M virt -cpu rv64(QEMU 6.2, lima-vm/lima#827)