-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build runc.riscv64 #3463
Build runc.riscv64 #3463
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx! left some nits for consistency 😅
Signed-off-by: Akihiro Suda <[email protected]>
Signed-off-by: Akihiro Suda <[email protected]>
@@ -64,6 +64,7 @@ var archs = map[string]string{ | |||
"SCMP_ARCH_PPC": "ppc", | |||
"SCMP_ARCH_PPC64": "ppc64", | |||
"SCMP_ARCH_PPC64LE": "ppc64le", | |||
"SCMP_ARCH_RISCV64": "riscv64", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (left one question)
@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)