fix: drop i8042.nopnp argument from default kernel cmdline

Since Firecracker gained ACPI support, disabling the ACPI probing of the
keyboard device caused the CTRL+DEL+ALT functionality to no longer work
unless the user explicitly modified the default kernel command lines in
the PUT /machine-config endpoint.

Restore pre-ACPI behavior of not needing to do this by dropping this
parameter from the default set.

Signed-off-by: Patrick Roy <[email protected]>

Author: roypat

src/vmm/src/vmm_config/boot_source.rs

@@ -14,10 +14,9 @@ use serde::{Deserialize, Serialize};
 /// - `8250.nr_uarts=0` disable 8250 serial interface;
 /// - `i8042.noaux` do not probe the i8042 controller for an attached mouse (save boot time);
 /// - `i8042.nomux` do not probe i8042 for a multiplexing controller (save boot time);
-/// - `i8042.nopnp` do not use ACPIPnP to discover KBD/AUX controllers (save boot time);
 /// - `i8042.dumbkbd` do not attempt to control kbd state via the i8042 (save boot time).
-pub const DEFAULT_KERNEL_CMDLINE: &str = "reboot=k panic=1 pci=off nomodule 8250.nr_uarts=0 \
-                                          i8042.noaux i8042.nomux i8042.nopnp i8042.dumbkbd";
+pub const DEFAULT_KERNEL_CMDLINE: &str =
+    "reboot=k panic=1 pci=off nomodule 8250.nr_uarts=0 i8042.noaux i8042.nomux i8042.dumbkbd";
 
 /// Strongly typed data structure used to configure the boot source of the
 /// microvm.