create: Use UEFI

This is required when enabling ACPI on aarch64, and probably a good idea in general. Signed-off-by: Alberto Faria <[email protected]>
containers · Apr 25, 2024 · 3a8c7b8 · 3a8c7b8
1 parent 102a425
commit 3a8c7b8
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/commands/create/domain.rs b/src/commands/create/domain.rs
@@ -61,12 +61,18 @@ fn generate(
         let memory = get_memory_size(spec).to_string();
         st(w, "memory", &[("unit", "b")], memory.as_str())?;
 
-        s(w, "os", &[], |w| {
+        s(w, "os", &[("firmware", "efi")], |w| {
             let attrs = match ["x86", "x86_64"].contains(&env::consts::ARCH) {
                 true => [("machine", "q35")].as_slice(),
                 false => [].as_slice(), // use libvirt's default
             };
-            st(w, "type", attrs, "hvm")
+            st(w, "type", attrs, "hvm")?;
+
+            s(w, "firmware", &[], |w| {
+                se(w, "feature", &[("enabled", "no"), ("name", "secure-boot")])
+            })?;
+
+            Ok(())
         })?;
 
         // fw_cfg requires ACPI