Skip to content

Commit 5a2fd2f

Browse files
committed
fix create_guest_memory on aarch64
- make default_guest_memory conditonal for x86_64 - update create_guest_memory to current signature - reorder aarch64 test case to mimic x86_64 test - fix dead code warning for Payload::None Fixes: #441 Signed-off-by: Pepper Gray <[email protected]>
1 parent 246b3f7 commit 5a2fd2f

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

src/vmm/src/builder.rs

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -504,6 +504,7 @@ enum Payload {
504504
KernelCopy,
505505
ExternalKernel(ExternalKernel),
506506
#[cfg(test)]
507+
#[allow(dead_code)]
507508
Empty,
508509
Firmware,
509510
#[cfg(feature = "tee")]
@@ -2167,14 +2168,16 @@ fn attach_snd_device(vmm: &mut Vmm, intc: IrqChip) -> std::result::Result<(), St
21672168
#[cfg(test)]
21682169
pub mod tests {
21692170
use super::*;
2170-
use crate::vmm_config::kernel_bundle::KernelBundle;
21712171

2172+
#[cfg(target_arch = "x86_64")]
21722173
fn default_guest_memory(
21732174
mem_size_mib: usize,
21742175
) -> std::result::Result<
21752176
(GuestMemoryMmap, ArchMemoryInfo, ShmManager, PayloadConfig),
21762177
StartMicrovmError,
21772178
> {
2179+
use crate::vmm_config::kernel_bundle::KernelBundle;
2180+
21782181
let mut vm_resources = VmResources::default();
21792182
vm_resources.kernel_bundle = Some(KernelBundle {
21802183
host_addr: 0x1000,
@@ -2221,9 +2224,6 @@ pub mod tests {
22212224
#[test]
22222225
#[cfg(all(target_arch = "aarch64", target_os = "linux"))]
22232226
fn test_create_vcpus_aarch64() {
2224-
let (guest_memory, _arch_memory_info) =
2225-
create_guest_memory(128, None, Payload::Empty).unwrap();
2226-
let vm = setup_vm(&guest_memory, false).unwrap();
22272227
let vcpu_count = 2;
22282228

22292229
let vcpu_config = VcpuConfig {
@@ -2232,6 +2232,12 @@ pub mod tests {
22322232
cpu_template: None,
22332233
};
22342234

2235+
let vm_resources = VmResources::default();
2236+
2237+
let (guest_memory, _arch_memory_info, _shm_manager, _payload_config) =
2238+
create_guest_memory(128, &vm_resources, &Payload::Empty).unwrap();
2239+
let vm = setup_vm(&guest_memory, false).unwrap();
2240+
22352241
// Dummy entry_addr, vcpus will not boot.
22362242
let entry_addr = GuestAddress(0);
22372243
let vcpu_vec = create_vcpus_aarch64(

0 commit comments

Comments
 (0)