summary refs log tree commit diff
path: root/release/checks
diff options
context:
space:
mode:
authorAlyssa Ross <hi@alyssa.is>2023-11-26 22:34:43 +0100
committerAlyssa Ross <hi@alyssa.is>2023-12-05 14:46:50 +0100
commitc1237194b7f0e4c8354b71ffc8f37f567bda7b33 (patch)
treebc0384e7674fc94b1ff8232f6fcc61982b6e51c0 /release/checks
parent109d87a0804fec3f9e07642099c12067d7b06277 (diff)
downloadspectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.tar
spectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.tar.gz
spectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.tar.bz2
spectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.tar.lz
spectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.tar.xz
spectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.tar.zst
spectrum-c1237194b7f0e4c8354b71ffc8f37f567bda7b33.zip
Revert "vm: build command line into kernels"
This reverts commit 5f556f806a70f3787fe725254ccf3d245bd5bebc.

This turns out to not really be workable on aarch64 at the moment,
without giving things up — both crosvm and Cloud Hypervisor inject
earlycon parameters onto the command line, which overrides the
built-in command line.  We could use CONFIG_CMDLINE_FORCE, but then we
wouldn't know which serial console to use, because crosvm and Cloud
Hypervisor both use different ones.

There's discussion about making it possible to have the bootloader
command line append to the built-in command line, but it's moving
extremely slowly, so we can't count on it any time soon.

Link: https://lore.kernel.org/lkml/20231110013817.2378507-1-danielwa@cisco.com/
Signed-off-by: Alyssa Ross <hi@alyssa.is>
Diffstat (limited to 'release/checks')
-rw-r--r--release/checks/wayland/default.nix4
1 files changed, 2 insertions, 2 deletions
diff --git a/release/checks/wayland/default.nix b/release/checks/wayland/default.nix
index d58e3b8..671ebe3 100644
--- a/release/checks/wayland/default.nix
+++ b/release/checks/wayland/default.nix
@@ -19,13 +19,13 @@ nixosTest ({ lib, pkgs, ... }: {
     systemd.services.cloud-hypervisor = {
       after = [ "crosvm-gpu.service" "weston.service" ];
       requires = [ "crosvm-gpu.service" "weston.service" ];
-      serviceConfig.ExecStart = "${lib.getExe pkgs.cloud-hypervisor} --memory shared=on --disk path=${appvm}/img/appvm/blk/root.img,readonly=on --disk path=${run}/blk/run.img,readonly=on --gpu socket=/run/crosvm-gpu.sock --serial tty --console null --kernel ${appvm}/img/appvm/vmlinux";
+      serviceConfig.ExecStart = "${lib.getExe pkgs.cloud-hypervisor} --memory shared=on --disk path=${appvm}/img/appvm/blk/root.img,readonly=on --disk path=${run}/blk/run.img,readonly=on --cmdline \"console=ttyS0 root=PARTLABEL=root\" --gpu socket=/run/crosvm-gpu.sock --serial tty --console null --kernel ${appvm}/img/appvm/vmlinux";
     };
 
     systemd.services.crosvm = {
       after = [ "crosvm-gpu.service" "weston.service" ];
       requires = [ "crosvm-gpu.service" "weston.service" ];
-      serviceConfig.ExecStart = "${lib.getExe pkgs.crosvm} run -s /run/crosvm --disk ${appvm}/img/appvm/blk/root.img --disk ${run}/blk/run.img --vhost-user-gpu /run/crosvm-gpu.sock --serial type=stdout,hardware=virtio-console,stdin=true ${appvm}/img/appvm/vmlinux";
+      serviceConfig.ExecStart = "${lib.getExe pkgs.crosvm} run -s /run/crosvm --disk ${appvm}/img/appvm/blk/root.img --disk ${run}/blk/run.img -p \"console=ttyS0 root=PARTLABEL=root\" --vhost-user-gpu /run/crosvm-gpu.sock --serial type=stdout,hardware=virtio-console,stdin=true ${appvm}/img/appvm/vmlinux";
       serviceConfig.ExecStop = "${lib.getExe pkgs.crosvm} stop /run/crosvm";
     };