diff options
author | Alyssa Ross <hi@alyssa.is> | 2023-11-26 22:34:43 +0100 |
---|---|---|
committer | Alyssa Ross <hi@alyssa.is> | 2023-12-05 14:46:50 +0100 |
commit | c1237194b7f0e4c8354b71ffc8f37f567bda7b33 (patch) | |
tree | bc0384e7674fc94b1ff8232f6fcc61982b6e51c0 /release/checks | |
parent | 109d87a0804fec3f9e07642099c12067d7b06277 (diff) | |
download | spectrum-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.nix | 4 |
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"; }; |