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
runtime-rs on arm64: Fixes unable to Boot Container Image using Cloud… #8422
runtime-rs on arm64: Fixes unable to Boot Container Image using Cloud… #8422
Conversation
Can one of the admins verify this patch? |
b20c44d
to
c9d07af
Compare
Hi @brianwang12 -, please remove the "changeId" line in your commit message. |
c9d07af
to
6bad5cd
Compare
6bad5cd
to
07f69b7
Compare
Done |
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.
Thanks @brianwang12.
lgtm
|
||
|
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.
Nit: Extra blank line not required.
#[cfg(target_arch = "aarch64")] | ||
let console_param_debug = KernelParams::from_string("console=ttyAMA0,115200n8"); | ||
|
||
|
||
let mut rootfs_param = KernelParams::new_rootfs_kernel_params(rootfs_driver, rootfs_type)?; | ||
|
||
let mut console_params = if enable_debug { | ||
if confidential_guest { |
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.
Not related to this PR, but would you be able to look at raising a PR to allow confidential_guest=true
on aarch64 with runtime-rs and CH?
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.
There is no confidential computing support in any of VMM on aarch64 for now. So, I think it's not the time to enable confidential guest.
/test |
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
07f69b7
to
9cfbfc1
Compare
/test |
In the case of Cloud Hypervisor running on arm64 architecture, only arm AMBA UART (pl011) is supported as the TTY. Consequently, when enabling Hypervisor debug mode, it's essential to configure the console as "ttyAMA0" rather than "ttyS0 Fixes: kata-containers#8381 Signed-off-by: briwan01 <brian.wang@arm.com>
9cfbfc1
to
20f75d9
Compare
/test |
/test-arm |
Thanks @brianwang12 |
This patch addresses the issue of being unable to boot container images using Cloud-Hypervisor on arm64
Fitxes: #8381