blob: 77710c3f2fba05d54f2aaf0db36f8c094e536e1d [file] [log] [blame]
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001// Copyright 2021, The Android Open Source Project
2//
3// Licensed under the Apache License, Version 2.0 (the "License");
4// you may not use this file except in compliance with the License.
5// You may obtain a copy of the License at
6//
7// http://www.apache.org/licenses/LICENSE-2.0
8//
9// Unless required by applicable law or agreed to in writing, software
10// distributed under the License is distributed on an "AS IS" BASIS,
11// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12// See the License for the specific language governing permissions and
13// limitations under the License.
14
15//! Functions for running instances of `crosvm`.
16
Seungjae Yoo3271f172024-06-11 10:14:07 +090017use crate::aidl::{remove_temporary_files, Cid, GLOBAL_SERVICE, VirtualMachineCallbacks};
Seungjae Yoob2db3d22023-04-06 18:05:43 +090018use crate::atom::{get_num_cpus, write_vm_exited_stats_sync};
Jaewan Kim61f86142023-03-28 15:12:52 +090019use crate::debug_config::DebugConfig;
Seungjae Yoo6d265d92022-11-15 10:51:33 +090020use anyhow::{anyhow, bail, Context, Error, Result};
Seungjae Yoo3271f172024-06-11 10:14:07 +090021use binder::ParcelFileDescriptor;
Andrew Walbran02b8ec02021-06-22 13:07:02 +000022use command_fds::CommandFdExt;
Seungjae Yoo6d265d92022-11-15 10:51:33 +090023use libc::{sysconf, _SC_CLK_TCK};
Seungjae Yoo663d98d2025-02-17 15:23:39 +090024use log::{debug, error, info, warn};
Jiyong Parkdcf17412022-02-08 15:07:23 +090025use semver::{Version, VersionReq};
Jiyong Park08eee7b2022-11-11 15:01:26 +090026use nix::{fcntl::OFlag, unistd::pipe2, unistd::Uid, unistd::User};
Jiyong Park2d736562022-10-24 22:40:12 +090027use regex::{Captures, Regex};
Keir Fraserf25cb922022-11-23 14:26:00 +000028use rustutils::system_properties;
Andrew Walbrandae07162021-03-12 17:05:20 +000029use shared_child::SharedChild;
Jiyong Parke6ed0f92022-06-22 00:13:00 +090030use std::borrow::Cow;
Seungjae Yoo6d265d92022-11-15 10:51:33 +090031use std::cmp::max;
Frederick Maylef375ef12024-10-21 16:07:21 -070032use std::ffi::CString;
Jiyong Park08eee7b2022-11-11 15:01:26 +090033use std::fmt;
David Brazdil4b4c5102022-12-19 22:56:20 +000034use std::fs::{read_to_string, File};
Andrew Walbranb27681f2022-02-23 15:11:52 +000035use std::io::{self, Read};
Andrew Walbranf8d94112021-09-07 11:45:36 +000036use std::mem;
Nikita Ioffe5776f082023-02-10 21:38:26 +000037use std::num::{NonZeroU16, NonZeroU32};
Jiyong Park69001a22024-08-16 20:45:34 +090038use std::os::unix::io::{AsRawFd, OwnedFd};
Jiyong Parkf50ec332024-10-23 17:54:03 +090039use std::os::unix::process::CommandExt;
Seungjae Yoo93430e82022-12-05 16:37:42 +090040use std::os::unix::process::ExitStatusExt;
Jiyong Park1612b902022-08-22 14:47:39 +090041use std::path::{Path, PathBuf};
Andrew Walbranc92d35f2022-01-12 12:45:19 +000042use std::process::{Command, ExitStatus};
Andrew Walbran9c03a3a2024-09-03 12:12:59 +010043use std::sync::{Arc, Condvar, Mutex, LazyLock};
Seungjae Yoo2e7beea2022-08-24 16:09:12 +090044use std::time::{Duration, SystemTime};
David Brazdilf18e1162022-12-18 18:27:23 +000045use std::thread::{self, JoinHandle};
David Brazdil49f96f52022-12-16 21:29:13 +000046use android_system_virtualizationcommon::aidl::android::system::virtualizationcommon::DeathReason::DeathReason;
Jaewan Kim84b91212023-02-28 00:11:57 +090047use android_system_virtualizationservice::aidl::android::system::virtualizationservice::{
Jeongik Cha1df1c032024-04-03 16:03:12 +090048 VirtualMachineAppConfig::DebugLevel::DebugLevel,
Mu-Le Leeb2d5a312024-06-05 10:52:47 +080049 AudioConfig::AudioConfig as AudioConfigParcelable,
Elie Kheirallahb4b2f242025-01-23 03:38:07 +000050 CpuOptions::CpuOptions,
51 CpuOptions::CpuTopology::CpuTopology,
Jeongik Cha1df1c032024-04-03 16:03:12 +090052 DisplayConfig::DisplayConfig as DisplayConfigParcelable,
Jason Macnakea7b0fa2024-05-08 17:02:44 -070053 GpuConfig::GpuConfig as GpuConfigParcelable,
Elie Kheirallah29d72912024-08-07 20:17:26 +000054 UsbConfig::UsbConfig as UsbConfigParcelable,
Jaewan Kim84b91212023-02-28 00:11:57 +090055};
David Brazdil528e0472022-10-10 15:06:02 +010056use android_system_virtualizationservice_internal::aidl::android::system::virtualizationservice_internal::IGlobalVmContext::IGlobalVmContext;
Jakob Vukalovicd42aa2c2023-11-09 16:04:00 +000057use android_system_virtualizationservice_internal::aidl::android::system::virtualizationservice_internal::IBoundDevice::IBoundDevice;
Alan Stokes0e82b502022-08-08 14:44:48 +010058use binder::Strong;
Inseob Kimc7d28c72021-10-25 14:28:10 +000059use android_system_virtualmachineservice::aidl::android::system::virtualmachineservice::IVirtualMachineService::IVirtualMachineService;
Jiyong Park1612b902022-08-22 14:47:39 +090060use tombstoned_client::{TombstonedConnection, DebuggerdDumpType};
David Brazdil73988ea2022-11-11 15:10:32 +000061use rpcbinder::RpcServer;
Andrew Walbrand6dce6f2021-03-05 16:39:08 +000062
63const CROSVM_PATH: &str = "/apex/com.android.virt/bin/crosvm";
64
Jiyong Parkdcf17412022-02-08 15:07:23 +090065/// Version of the platform that crosvm currently implements. The format follows SemVer. This
66/// should be updated when there is a platform change in the crosvm side. Having this value here is
67/// fine because virtualizationservice and crosvm are supposed to be updated together in the virt
68/// APEX.
69const CROSVM_PLATFORM_VERSION: &str = "1.0.0";
70
Andrew Walbrand15c5632022-02-03 13:38:31 +000071/// The exit status which crosvm returns when it has an error starting a VM.
Alan Stokes7c459e82022-12-06 12:21:49 +000072const CROSVM_START_ERROR_STATUS: i32 = 1;
Andrew Walbranc92d35f2022-01-12 12:45:19 +000073/// The exit status which crosvm returns when a VM requests a reboot.
74const CROSVM_REBOOT_STATUS: i32 = 32;
Andrew Walbrand15c5632022-02-03 13:38:31 +000075/// The exit status which crosvm returns when it crashes due to an error.
76const CROSVM_CRASH_STATUS: i32 = 33;
Sebastian Ene23167d82022-10-07 14:09:53 +000077/// The exit status which crosvm returns when vcpu is stalled.
78const CROSVM_WATCHDOG_REBOOT_STATUS: i32 = 36;
Jiyong Park3115cdb2023-02-24 18:04:44 +090079/// The size of memory (in MiB) reserved for ramdump
80const RAMDUMP_RESERVED_MIB: u32 = 17;
Andrew Walbranc92d35f2022-01-12 12:45:19 +000081
Seungjae Yoo6d265d92022-11-15 10:51:33 +090082const MILLIS_PER_SEC: i64 = 1000;
83
Jaewan Kim46422812022-11-25 10:59:39 +090084const SYSPROP_CUSTOM_PVMFW_PATH: &str = "hypervisor.pvmfw.path";
85
Yi-Yo Chiang8dd32552024-05-22 19:38:16 +080086/// Serial device for VM console input.
87/// Hypervisor (virtio-console)
88const CONSOLE_HVC0: &str = "hvc0";
89/// Serial (emulated uart)
90const CONSOLE_TTYS0: &str = "ttyS0";
91
Andrew Walbran9c03a3a2024-09-03 12:12:59 +010092/// If the VM doesn't move to the Started state within this amount time, a hang-up error is
93/// triggered.
94static BOOT_HANGUP_TIMEOUT: LazyLock<Duration> = LazyLock::new(|| {
95 if nested_virt::is_nested_virtualization().unwrap() {
Jiyong Parke6ed0f92022-06-22 00:13:00 +090096 // Nested virtualization is slow, so we need a longer timeout.
Inseob Kime557cec2023-01-26 22:49:58 +090097 Duration::from_secs(300)
Jiyong Parke6ed0f92022-06-22 00:13:00 +090098 } else {
Inseob Kime557cec2023-01-26 22:49:58 +090099 Duration::from_secs(30)
Andrew Walbran9c03a3a2024-09-03 12:12:59 +0100100 }
101});
Jiyong Parke6ed0f92022-06-22 00:13:00 +0900102
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000103/// Configuration for a VM to run with crosvm.
104#[derive(Debug)]
Andrew Walbrand3a84182021-09-07 14:48:52 +0000105pub struct CrosvmConfig {
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000106 pub cid: Cid,
Seungjae Yoo62085c02022-08-12 04:44:52 +0000107 pub name: String,
Andrew Walbrand3a84182021-09-07 14:48:52 +0000108 pub bootloader: Option<File>,
109 pub kernel: Option<File>,
110 pub initrd: Option<File>,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000111 pub disks: Vec<DiskFile>,
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700112 pub shared_paths: Vec<SharedPathConfig>,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000113 pub params: Option<String>,
Andrew Walbranf8650422021-06-09 15:54:09 +0000114 pub protected: bool,
Jaewan Kim61f86142023-03-28 15:12:52 +0900115 pub debug_config: DebugConfig,
Frederick Mayle68d06e72024-07-12 17:18:11 -0700116 pub memory_mib: NonZeroU32,
Frederick Mayle384f5b52024-12-06 16:23:23 -0800117 pub swiotlb_mib: Option<NonZeroU32>,
Elie Kheirallahb4b2f242025-01-23 03:38:07 +0000118 pub cpus: CpuOptions,
Jiyong Parke6fb1672023-06-26 16:45:55 +0900119 pub console_out_fd: Option<File>,
120 pub console_in_fd: Option<File>,
Andrew Walbrand3a84182021-09-07 14:48:52 +0000121 pub log_fd: Option<File>,
Jiyong Parke558ab12022-07-07 20:18:55 +0900122 pub ramdump: Option<File>,
Andrew Walbrand3a84182021-09-07 14:48:52 +0000123 pub indirect_files: Vec<File>,
Jiyong Parkdcf17412022-02-08 15:07:23 +0900124 pub platform_version: VersionReq,
Jiyong Parke6ed0f92022-06-22 00:13:00 +0900125 pub detect_hangup: bool,
Nikita Ioffe5776f082023-02-10 21:38:26 +0000126 pub gdb_port: Option<NonZeroU16>,
Inseob Kim7307a892023-09-14 13:37:58 +0900127 pub vfio_devices: Vec<VfioDevice>,
David Brazdil2dfefd12023-11-17 14:07:36 +0000128 pub dtbo: Option<File>,
Jaewan Kimf43372b2024-12-13 18:33:58 +0900129 pub device_tree_overlays: Vec<File>,
Jeongik Cha1df1c032024-04-03 16:03:12 +0900130 pub display_config: Option<DisplayConfig>,
Jeongik Chac181be72024-03-27 00:18:30 +0900131 pub input_device_options: Vec<InputDeviceOption>,
Vincent Donnefort538a2c62024-03-20 16:01:10 +0000132 pub hugepages: bool,
Seungjae Yoo986d7a42024-05-28 14:06:58 +0900133 pub tap: Option<File>,
Yi-Yo Chiang8dd32552024-05-22 19:38:16 +0800134 pub console_input_device: Option<String>,
David Dai23cff712024-06-13 19:23:45 +0000135 pub boost_uclamp: bool,
Jason Macnakea7b0fa2024-05-08 17:02:44 -0700136 pub gpu_config: Option<GpuConfig>,
Mu-Le Leeb2d5a312024-06-05 10:52:47 +0800137 pub audio_config: Option<AudioConfig>,
Frederick Maylecc4e2d72024-12-06 16:54:40 -0800138 pub balloon: bool,
Elie Kheirallah29d72912024-08-07 20:17:26 +0000139 pub usb_config: UsbConfig,
Elie Kheirallah5c807a22024-09-23 20:40:42 +0000140 pub dump_dt_fd: Option<File>,
Frederick Mayle196ca2b2024-12-18 15:27:43 -0800141 pub enable_hypervisor_specific_auth_method: bool,
142 pub instance_id: [u8; 64],
Frederick Mayle31cc5b22024-12-19 14:42:43 -0800143 // (memfd, guest address, size)
144 pub custom_memory_backing_files: Vec<(OwnedFd, u64, u64)>,
Mu-Le Leeb2d5a312024-06-05 10:52:47 +0800145}
146
147#[derive(Debug)]
148pub struct AudioConfig {
149 pub use_microphone: bool,
150 pub use_speaker: bool,
151}
152
153impl AudioConfig {
154 pub fn new(raw_config: &AudioConfigParcelable) -> Self {
155 AudioConfig { use_microphone: raw_config.useMicrophone, use_speaker: raw_config.useSpeaker }
156 }
Jeongik Cha1df1c032024-04-03 16:03:12 +0900157}
158
159#[derive(Debug)]
Elie Kheirallah29d72912024-08-07 20:17:26 +0000160pub struct UsbConfig {
161 pub controller: bool,
162}
163
164impl UsbConfig {
165 pub fn new(raw_config: &UsbConfigParcelable) -> Result<UsbConfig> {
166 Ok(UsbConfig { controller: raw_config.controller })
167 }
168}
169
170#[derive(Debug)]
Jeongik Cha1df1c032024-04-03 16:03:12 +0900171pub struct DisplayConfig {
172 pub width: NonZeroU32,
173 pub height: NonZeroU32,
174 pub horizontal_dpi: NonZeroU32,
175 pub vertical_dpi: NonZeroU32,
176 pub refresh_rate: NonZeroU32,
177}
178
179impl DisplayConfig {
180 pub fn new(raw_config: &DisplayConfigParcelable) -> Result<DisplayConfig> {
181 let width = try_into_non_zero_u32(raw_config.width)?;
182 let height = try_into_non_zero_u32(raw_config.height)?;
183 let horizontal_dpi = try_into_non_zero_u32(raw_config.horizontalDpi)?;
184 let vertical_dpi = try_into_non_zero_u32(raw_config.verticalDpi)?;
185 let refresh_rate = try_into_non_zero_u32(raw_config.refreshRate)?;
186 Ok(DisplayConfig { width, height, horizontal_dpi, vertical_dpi, refresh_rate })
187 }
188}
189
Jason Macnakea7b0fa2024-05-08 17:02:44 -0700190#[derive(Debug)]
191pub struct GpuConfig {
192 pub backend: Option<String>,
193 pub context_types: Option<Vec<String>>,
194 pub pci_address: Option<String>,
195 pub renderer_features: Option<String>,
196 pub renderer_use_egl: Option<bool>,
197 pub renderer_use_gles: Option<bool>,
198 pub renderer_use_glx: Option<bool>,
199 pub renderer_use_surfaceless: Option<bool>,
200 pub renderer_use_vulkan: Option<bool>,
201}
202
203impl GpuConfig {
204 pub fn new(raw_config: &GpuConfigParcelable) -> Result<GpuConfig> {
205 Ok(GpuConfig {
206 backend: raw_config.backend.clone(),
207 context_types: raw_config.contextTypes.clone().map(|context_types| {
208 context_types.iter().filter_map(|context_type| context_type.clone()).collect()
209 }),
210 pci_address: raw_config.pciAddress.clone(),
211 renderer_features: raw_config.rendererFeatures.clone(),
212 renderer_use_egl: Some(raw_config.rendererUseEgl),
213 renderer_use_gles: Some(raw_config.rendererUseGles),
214 renderer_use_glx: Some(raw_config.rendererUseGlx),
215 renderer_use_surfaceless: Some(raw_config.rendererUseSurfaceless),
216 renderer_use_vulkan: Some(raw_config.rendererUseVulkan),
217 })
218 }
219}
220
Jeongik Cha1df1c032024-04-03 16:03:12 +0900221fn try_into_non_zero_u32(value: i32) -> Result<NonZeroU32> {
222 let u32_value = value.try_into()?;
223 NonZeroU32::new(u32_value).ok_or(anyhow!("value should be greater than 0"))
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000224}
225
226/// A disk image to pass to crosvm for a VM.
227#[derive(Debug)]
228pub struct DiskFile {
229 pub image: File,
230 pub writable: bool,
231}
232
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700233/// Shared path between host and guest VM.
234#[derive(Debug)]
235pub struct SharedPathConfig {
236 pub path: String,
237 pub host_uid: i32,
238 pub host_gid: i32,
239 pub guest_uid: i32,
240 pub guest_gid: i32,
241 pub mask: i32,
242 pub tag: String,
243 pub socket_path: String,
Akilesh Kailashc9aa0682024-11-25 10:27:24 -0800244 pub socket_fd: Option<File>,
245 pub app_domain: bool,
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700246}
247
Jeongik Chac181be72024-03-27 00:18:30 +0900248/// virtio-input device configuration from `external/crosvm/src/crosvm/config.rs`
249#[derive(Debug)]
250#[allow(dead_code)]
251pub enum InputDeviceOption {
252 EvDev(File),
253 SingleTouch { file: File, width: u32, height: u32, name: Option<String> },
Jeongik Cha985b6402024-04-15 18:13:00 +0900254 Keyboard(File),
Jeongik Cha343894e2024-05-17 20:39:31 +0900255 Mouse(File),
Jiyong Park6c1b9f02024-07-05 16:21:10 +0900256 Switches(File),
Jeongik Cha112a2682024-07-09 12:28:45 +0900257 MultiTouchTrackpad { file: File, width: u32, height: u32, name: Option<String> },
Jeongik Cha10494912024-07-16 11:19:50 +0900258 MultiTouch { file: File, width: u32, height: u32, name: Option<String> },
Jeongik Chac181be72024-03-27 00:18:30 +0900259}
260
Jakob Vukalovicd42aa2c2023-11-09 16:04:00 +0000261type VfioDevice = Strong<dyn IBoundDevice>;
Inseob Kim7307a892023-09-14 13:37:58 +0900262
Andrew Walbran6b650662021-09-07 13:13:23 +0000263/// The lifecycle state which the payload in the VM has reported itself to be in.
264///
265/// Note that the order of enum variants is significant; only forward transitions are allowed by
266/// [`VmInstance::update_payload_state`].
267#[derive(Copy, Clone, Debug, Eq, Ord, PartialEq, PartialOrd)]
268pub enum PayloadState {
269 Starting,
270 Started,
271 Ready,
272 Finished,
Jiyong Parka4eebde2022-07-12 18:01:12 +0900273 Hangup, // Hasn't reached to Ready before timeout expires
Andrew Walbran6b650662021-09-07 13:13:23 +0000274}
275
Andrew Walbranf8d94112021-09-07 11:45:36 +0000276/// The current state of the VM itself.
277#[derive(Debug)]
278pub enum VmState {
279 /// The VM has not yet tried to start.
280 NotStarted {
281 ///The configuration needed to start the VM, if it has not yet been started.
Inseob Kim7307a892023-09-14 13:37:58 +0900282 config: Box<CrosvmConfig>,
Andrew Walbranf8d94112021-09-07 11:45:36 +0000283 },
284 /// The VM has been started.
285 Running {
286 /// The crosvm child process.
287 child: Arc<SharedChild>,
David Brazdilf18e1162022-12-18 18:27:23 +0000288 /// The thread waiting for crosvm to finish.
289 monitor_vm_exit_thread: Option<JoinHandle<()>>,
Andrew Walbranf8d94112021-09-07 11:45:36 +0000290 },
291 /// The VM died or was killed.
292 Dead,
293 /// The VM failed to start.
294 Failed,
295}
296
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900297/// RSS values of VM and CrosVM process itself.
298#[derive(Copy, Clone, Debug, Default)]
299pub struct Rss {
300 pub vm: i64,
301 pub crosvm: i64,
302}
303
304/// Metrics regarding the VM.
305#[derive(Debug, Default)]
306pub struct VmMetric {
307 /// Recorded timestamp when the VM is started.
308 pub start_timestamp: Option<SystemTime>,
Pierre-Clément Tosi44aba642023-09-15 11:44:34 +0100309 /// Update most recent guest_time periodically from /proc/[crosvm pid]/stat while VM is
310 /// running.
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900311 pub cpu_guest_time: Option<i64>,
312 /// Update maximum RSS values periodically from /proc/[crosvm pid]/smaps while VM is running.
313 pub rss: Option<Rss>,
314}
315
Andrew Walbranf8d94112021-09-07 11:45:36 +0000316impl VmState {
317 /// Tries to start the VM, if it is in the `NotStarted` state.
318 ///
319 /// Returns an error if the VM is in the wrong state, or fails to start.
320 fn start(&mut self, instance: Arc<VmInstance>) -> Result<(), Error> {
321 let state = mem::replace(self, VmState::Failed);
322 if let VmState::NotStarted { config } = state {
Inseob Kim7307a892023-09-14 13:37:58 +0900323 let config = *config;
Jiyong Parke6ed0f92022-06-22 00:13:00 +0900324 let detect_hangup = config.detect_hangup;
Andrew Walbranb27681f2022-02-23 15:11:52 +0000325 let (failure_pipe_read, failure_pipe_write) = create_pipe()?;
Inseob Kim6ef80972023-07-20 17:23:36 +0900326 let vfio_devices = config.vfio_devices.clone();
Seungjae Yoo3271f172024-06-11 10:14:07 +0900327 let tap =
328 if let Some(tap_file) = &config.tap { Some(tap_file.try_clone()?) } else { None };
Andrew Walbranb27681f2022-02-23 15:11:52 +0000329
Yuan Yao2b654002024-12-25 05:19:12 +0000330 let vhost_fs_devices = run_virtiofs(&config)?;
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700331
Andrew Walbranf8d94112021-09-07 11:45:36 +0000332 // If this fails and returns an error, `self` will be left in the `Failed` state.
Keir Fraser13a956a2022-07-14 14:20:46 +0000333 let child =
Keir Frasercdd4b112022-11-24 14:02:25 +0000334 Arc::new(run_vm(config, &instance.crosvm_control_socket_path, failure_pipe_write)?);
Andrew Walbranf8d94112021-09-07 11:45:36 +0000335
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900336 let instance_monitor_status = instance.clone();
337 let child_monitor_status = child.clone();
338 thread::spawn(move || {
339 instance_monitor_status.clone().monitor_vm_status(child_monitor_status);
340 });
341
Andrew Walbranf8d94112021-09-07 11:45:36 +0000342 let child_clone = child.clone();
Jiyong Parka4eebde2022-07-12 18:01:12 +0900343 let instance_clone = instance.clone();
David Brazdilf18e1162022-12-18 18:27:23 +0000344 let monitor_vm_exit_thread = Some(thread::spawn(move || {
Yuan Yao2b654002024-12-25 05:19:12 +0000345 instance_clone.monitor_vm_exit(
346 child_clone,
347 failure_pipe_read,
348 vfio_devices,
349 tap,
350 vhost_fs_devices,
351 );
David Brazdilf18e1162022-12-18 18:27:23 +0000352 }));
Andrew Walbranf8d94112021-09-07 11:45:36 +0000353
Jiyong Parka4eebde2022-07-12 18:01:12 +0900354 if detect_hangup {
355 let child_clone = child.clone();
356 thread::spawn(move || {
357 instance.monitor_payload_hangup(child_clone);
358 });
359 }
360
Andrew Walbranf8d94112021-09-07 11:45:36 +0000361 // If it started correctly, update the state.
David Brazdilf18e1162022-12-18 18:27:23 +0000362 *self = VmState::Running { child, monitor_vm_exit_thread };
Andrew Walbranf8d94112021-09-07 11:45:36 +0000363 Ok(())
364 } else {
365 *self = state;
366 bail!("VM already started or failed")
367 }
368 }
369}
370
David Brazdil8cf8f482022-11-23 14:21:26 +0000371/// Internal struct that holds the handles to globally unique resources of a VM.
372#[derive(Debug)]
373pub struct VmContext {
374 #[allow(dead_code)] // Keeps the global context alive
Yi-Yo Chiang2fbf0da2024-06-14 22:56:56 +0800375 pub(crate) global_context: Strong<dyn IGlobalVmContext>,
David Brazdil8cf8f482022-11-23 14:21:26 +0000376 #[allow(dead_code)] // Keeps the server alive
Jiyong Park690c1ca2024-11-28 18:06:50 +0900377 vm_server: Option<RpcServer>,
David Brazdil8cf8f482022-11-23 14:21:26 +0000378}
379
380impl VmContext {
381 /// Construct new VmContext.
Jiyong Park690c1ca2024-11-28 18:06:50 +0900382 pub fn new(
383 global_context: Strong<dyn IGlobalVmContext>,
384 vm_server: Option<RpcServer>,
385 ) -> VmContext {
David Brazdil8cf8f482022-11-23 14:21:26 +0000386 VmContext { global_context, vm_server }
387 }
388}
389
Andrew Walbranf8d94112021-09-07 11:45:36 +0000390/// Information about a particular instance of a VM which may be running.
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000391#[derive(Debug)]
392pub struct VmInstance {
Andrew Walbranf8d94112021-09-07 11:45:36 +0000393 /// The current state of the VM.
394 pub vm_state: Mutex<VmState>,
David Brazdil8cf8f482022-11-23 14:21:26 +0000395 /// Global resources allocated for this VM.
396 #[allow(dead_code)] // Keeps the context alive
Yi-Yo Chiang2fbf0da2024-06-14 22:56:56 +0800397 pub(crate) vm_context: VmContext,
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000398 /// The CID assigned to the VM for vsock communication.
399 pub cid: Cid,
Keir Frasercdd4b112022-11-24 14:02:25 +0000400 /// Path to crosvm control socket
401 crosvm_control_socket_path: PathBuf,
Seungjae Yoo62085c02022-08-12 04:44:52 +0000402 /// The name of the VM.
403 pub name: String,
Andrew Walbranf8650422021-06-09 15:54:09 +0000404 /// Whether the VM is a protected VM.
405 pub protected: bool,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000406 /// Directory of temporary files used by the VM while it is running.
407 pub temporary_directory: PathBuf,
Andrew Walbranf6a1eb92021-04-01 11:16:02 +0000408 /// The UID of the process which requested the VM.
409 pub requester_uid: u32,
Andrew Walbranf6a1eb92021-04-01 11:16:02 +0000410 /// The PID of the process which requested the VM. Note that this process may no longer exist
411 /// and the PID may have been reused for a different process, so this should not be trusted.
Andrew Walbran02034492021-04-13 15:05:07 +0000412 pub requester_debug_pid: i32,
Andrew Walbrandae07162021-03-12 17:05:20 +0000413 /// Callbacks to clients of the VM.
414 pub callbacks: VirtualMachineCallbacks,
Inseob Kimc7d28c72021-10-25 14:28:10 +0000415 /// VirtualMachineService binder object for the VM.
Stephen Hinesaf21e512024-07-30 15:43:39 -0700416 #[allow(dead_code)]
Inseob Kimc7d28c72021-10-25 14:28:10 +0000417 pub vm_service: Mutex<Option<Strong<dyn IVirtualMachineService>>>,
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900418 /// Recorded metrics of VM such as timestamp or cpu / memory usage.
419 pub vm_metric: Mutex<VmMetric>,
Andrew Walbran6b650662021-09-07 13:13:23 +0000420 /// The latest lifecycle state which the payload reported itself to be in.
421 payload_state: Mutex<PayloadState>,
Jiyong Parka4eebde2022-07-12 18:01:12 +0900422 /// Represents the condition that payload_state was updated
423 payload_state_updated: Condvar,
Jiyong Park08eee7b2022-11-11 15:01:26 +0900424 /// The human readable name of requester_uid
425 requester_uid_name: String,
426}
427
428impl fmt::Display for VmInstance {
429 fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
430 let adj = if self.protected { "Protected" } else { "Non-protected" };
431 write!(
432 f,
433 "{} virtual machine \"{}\" (owner: {}, cid: {})",
434 adj, self.name, self.requester_uid_name, self.cid
435 )
436 }
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000437}
438
439impl VmInstance {
Andrew Walbranf8d94112021-09-07 11:45:36 +0000440 /// Validates the given config and creates a new `VmInstance` but doesn't start running it.
441 pub fn new(
Andrew Walbrand3a84182021-09-07 14:48:52 +0000442 config: CrosvmConfig,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000443 temporary_directory: PathBuf,
Andrew Walbranf6a1eb92021-04-01 11:16:02 +0000444 requester_uid: u32,
Andrew Walbran02034492021-04-13 15:05:07 +0000445 requester_debug_pid: i32,
David Brazdil8cf8f482022-11-23 14:21:26 +0000446 vm_context: VmContext,
Andrew Walbranf8d94112021-09-07 11:45:36 +0000447 ) -> Result<VmInstance, Error> {
448 validate_config(&config)?;
Andrew Walbrand3a84182021-09-07 14:48:52 +0000449 let cid = config.cid;
Seungjae Yoo62085c02022-08-12 04:44:52 +0000450 let name = config.name.clone();
Andrew Walbrand3a84182021-09-07 14:48:52 +0000451 let protected = config.protected;
Jiyong Park08eee7b2022-11-11 15:01:26 +0900452 let requester_uid_name = User::from_uid(Uid::from_raw(requester_uid))
453 .ok()
454 .flatten()
455 .map_or_else(|| format!("{}", requester_uid), |u| u.name);
456 let instance = VmInstance {
Inseob Kim7307a892023-09-14 13:37:58 +0900457 vm_state: Mutex::new(VmState::NotStarted { config: Box::new(config) }),
David Brazdil528e0472022-10-10 15:06:02 +0100458 vm_context,
Andrew Walbrand3a84182021-09-07 14:48:52 +0000459 cid,
Keir Frasercdd4b112022-11-24 14:02:25 +0000460 crosvm_control_socket_path: temporary_directory.join("crosvm.sock"),
Seungjae Yoo62085c02022-08-12 04:44:52 +0000461 name,
Andrew Walbrand3a84182021-09-07 14:48:52 +0000462 protected,
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000463 temporary_directory,
Andrew Walbran02034492021-04-13 15:05:07 +0000464 requester_uid,
Andrew Walbran02034492021-04-13 15:05:07 +0000465 requester_debug_pid,
Andrew Walbranf8d94112021-09-07 11:45:36 +0000466 callbacks: Default::default(),
Inseob Kimc7d28c72021-10-25 14:28:10 +0000467 vm_service: Mutex::new(None),
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900468 vm_metric: Mutex::new(Default::default()),
Andrew Walbranf8d94112021-09-07 11:45:36 +0000469 payload_state: Mutex::new(PayloadState::Starting),
Jiyong Parka4eebde2022-07-12 18:01:12 +0900470 payload_state_updated: Condvar::new(),
Jiyong Park08eee7b2022-11-11 15:01:26 +0900471 requester_uid_name,
472 };
473 info!("{} created", &instance);
474 Ok(instance)
Andrew Walbrandae07162021-03-12 17:05:20 +0000475 }
476
Andrew Walbranf8d94112021-09-07 11:45:36 +0000477 /// Starts an instance of `crosvm` to manage the VM. The `crosvm` instance will be killed when
478 /// the `VmInstance` is dropped.
479 pub fn start(self: &Arc<Self>) -> Result<(), Error> {
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900480 let mut vm_metric = self.vm_metric.lock().unwrap();
481 vm_metric.start_timestamp = Some(SystemTime::now());
Jiyong Park08eee7b2022-11-11 15:01:26 +0900482 let ret = self.vm_state.lock().unwrap().start(self.clone());
483 if ret.is_ok() {
484 info!("{} started", &self);
485 }
486 ret.with_context(|| format!("{} failed to start", &self))
Andrew Walbranf8d94112021-09-07 11:45:36 +0000487 }
488
Jiyong Parka4eebde2022-07-12 18:01:12 +0900489 /// Monitors the exit of the VM (i.e. termination of the `child` process). When that happens,
490 /// handles the event by updating the state, noityfing the event to clients by calling
491 /// callbacks, and removing temporary files for the VM.
Inseob Kim6ef80972023-07-20 17:23:36 +0900492 fn monitor_vm_exit(
493 &self,
494 child: Arc<SharedChild>,
Frederick Mayle402fd2e2024-11-14 17:56:32 -0800495 failure_pipe_read: File,
Inseob Kim7307a892023-09-14 13:37:58 +0900496 vfio_devices: Vec<VfioDevice>,
Seungjae Yoo3271f172024-06-11 10:14:07 +0900497 tap: Option<File>,
Yuan Yao2b654002024-12-25 05:19:12 +0000498 vhost_user_devices: Vec<SharedChild>,
Inseob Kim6ef80972023-07-20 17:23:36 +0900499 ) {
Frederick Mayle402fd2e2024-11-14 17:56:32 -0800500 let failure_reason_thread = std::thread::spawn(move || {
501 // Read the pipe to see if any failure reason is written
502 let mut failure_reason = String::new();
503 // Arbitrary max size in case of misbehaving guest.
504 const MAX_SIZE: u64 = 50_000;
505 match failure_pipe_read.take(MAX_SIZE).read_to_string(&mut failure_reason) {
506 Err(e) => error!("Error reading VM failure reason from pipe: {}", e),
507 Ok(len) if len > 0 => error!("VM returned failure reason '{}'", &failure_reason),
508 _ => (),
509 };
510 failure_reason
511 });
512
Andrew Walbranc92d35f2022-01-12 12:45:19 +0000513 let result = child.wait();
514 match &result {
Jooyung Hanbfe086f2021-10-28 10:15:45 +0900515 Err(e) => error!("Error waiting for crosvm({}) instance to die: {}", child.id(), e),
Sebastian Ene23167d82022-10-07 14:09:53 +0000516 Ok(status) => {
517 info!("crosvm({}) exited with status {}", child.id(), status);
518 if let Some(exit_status_code) = status.code() {
519 if exit_status_code == CROSVM_WATCHDOG_REBOOT_STATUS {
520 info!("detected vcpu stall on crosvm");
521 }
522 }
523 }
Andrew Walbrandae07162021-03-12 17:05:20 +0000524 }
Andrew Walbranf8d94112021-09-07 11:45:36 +0000525
Yuan Yao2b654002024-12-25 05:19:12 +0000526 // In crosvm, when vhost_user frontend is dead, vhost_user backend device will detect and
527 // exit. We can safely wait() for vhost user device after waiting crosvm main
528 // process.
529 for device in vhost_user_devices {
530 match device.wait() {
531 Ok(status) => {
532 info!("Vhost user device({}) exited with status {}", device.id(), status);
533 if !status.success() {
534 if let Some(code) = status.code() {
535 // vhost_user backend device exit with error code
536 error!(
537 "vhost user device({}) exited with error code: {}",
538 device.id(),
539 code
540 );
541 } else {
542 // The spawned child process of vhost_user backend device is
543 // killed by signal
544 error!("vhost user device({}) killed by signal", device.id());
545 }
546 }
547 }
548 Err(e) => {
549 error!("Error waiting for vhost user device({}) to die: {}", device.id(), e);
550 }
551 }
552 }
553
Frederick Mayle402fd2e2024-11-14 17:56:32 -0800554 let failure_reason = failure_reason_thread.join().expect("failure_reason_thread panic'd");
555
Andrew Walbranf8d94112021-09-07 11:45:36 +0000556 let mut vm_state = self.vm_state.lock().unwrap();
557 *vm_state = VmState::Dead;
558 // Ensure that the mutex is released before calling the callbacks.
559 drop(vm_state);
Jiyong Park08eee7b2022-11-11 15:01:26 +0900560 info!("{} exited", &self);
Andrew Walbranf8d94112021-09-07 11:45:36 +0000561
Jiyong Parka4eebde2022-07-12 18:01:12 +0900562 // In case of hangup, the pipe doesn't give us any information because the hangup can't be
563 // detected on the VM side (otherwise, it isn't a hangup), but in the
564 // monitor_payload_hangup function below which updates the payload state to Hangup.
565 let failure_reason =
566 if failure_reason.is_empty() && self.payload_state() == PayloadState::Hangup {
567 Cow::from("HANGUP")
568 } else {
569 Cow::from(failure_reason)
570 };
571
Jiyong Parke558ab12022-07-07 20:18:55 +0900572 self.handle_ramdump().unwrap_or_else(|e| error!("Error handling ramdump: {}", e));
Seungjae Yoob4c07ba2022-08-12 04:44:52 +0000573
574 let death_reason = death_reason(&result, &failure_reason);
Seungjae Yoo93430e82022-12-05 16:37:42 +0900575 let exit_signal = exit_signal(&result);
576
Seungjae Yoob4c07ba2022-08-12 04:44:52 +0000577 self.callbacks.callback_on_died(self.cid, death_reason);
Seungjae Yoo2e7beea2022-08-24 16:09:12 +0900578
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900579 let vm_metric = self.vm_metric.lock().unwrap();
Seungjae Yoob2db3d22023-04-06 18:05:43 +0900580 write_vm_exited_stats_sync(
Seungjae Yoo93430e82022-12-05 16:37:42 +0900581 self.requester_uid as i32,
582 &self.name,
583 death_reason,
584 exit_signal,
Chris Wailes75269622022-12-05 23:01:44 -0800585 &vm_metric,
Seungjae Yoo93430e82022-12-05 16:37:42 +0900586 );
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +0000587
David Brazdil4b4c5102022-12-19 22:56:20 +0000588 // Delete temporary files. The folder itself is removed by VirtualizationServiceInternal.
589 remove_temporary_files(&self.temporary_directory).unwrap_or_else(|e| {
590 error!("Error removing temporary files from {:?}: {}", self.temporary_directory, e);
591 });
Inseob Kim6ef80972023-07-20 17:23:36 +0900592
Seungjae Yoo3271f172024-06-11 10:14:07 +0900593 if let Some(tap_file) = tap {
594 GLOBAL_SERVICE
595 .deleteTapInterface(&ParcelFileDescriptor::new(OwnedFd::from(tap_file)))
596 .unwrap_or_else(|e| {
597 error!("Error deleting TAP interface: {e:?}");
598 });
599 }
600
Jakob Vukalovicd42aa2c2023-11-09 16:04:00 +0000601 drop(vfio_devices); // Cleanup devices.
Andrew Walbrandae07162021-03-12 17:05:20 +0000602 }
603
Jiyong Parka4eebde2022-07-12 18:01:12 +0900604 /// Waits until payload is started, or timeout expires. When timeout occurs, kill
605 /// the VM to prevent indefinite hangup and update the payload_state accordingly.
606 fn monitor_payload_hangup(&self, child: Arc<SharedChild>) {
607 debug!("Starting to monitor hangup for Microdroid({})", child.id());
Jiyong Parkcf975612024-04-07 13:27:10 +0900608 let (state, result) = self
Jiyong Parka4eebde2022-07-12 18:01:12 +0900609 .payload_state_updated
610 .wait_timeout_while(self.payload_state.lock().unwrap(), *BOOT_HANGUP_TIMEOUT, |s| {
611 *s < PayloadState::Started
612 })
613 .unwrap();
Jiyong Parkcf975612024-04-07 13:27:10 +0900614 drop(state); // we are not interested in state
Jiyong Parka4eebde2022-07-12 18:01:12 +0900615 let child_still_running = child.try_wait().ok() == Some(None);
616 if result.timed_out() && child_still_running {
617 error!(
618 "Microdroid({}) failed to start payload within {} secs timeout. Shutting down.",
619 child.id(),
620 BOOT_HANGUP_TIMEOUT.as_secs()
621 );
622 self.update_payload_state(PayloadState::Hangup).unwrap();
623 if let Err(e) = self.kill() {
624 error!("Error stopping timed-out VM with CID {}: {:?}", child.id(), e);
625 }
626 }
627 }
628
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900629 fn monitor_vm_status(&self, child: Arc<SharedChild>) {
630 let pid = child.id();
Seungjae Yoo3211ab42025-02-13 13:40:14 +0900631 let mut metric_countdown = 0;
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900632
633 loop {
634 {
635 // Check VM state
636 let vm_state = &*self.vm_state.lock().unwrap();
637 if let VmState::Dead = vm_state {
638 break;
639 }
640
Seungjae Yoo3211ab42025-02-13 13:40:14 +0900641 if metric_countdown > 0 {
642 metric_countdown -= 1;
643 } else {
644 metric_countdown = 10;
645 let mut vm_metric = self.vm_metric.lock().unwrap();
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900646
Seungjae Yoo3211ab42025-02-13 13:40:14 +0900647 // Get CPU Information
648 match get_guest_time(pid) {
649 Ok(guest_time) => vm_metric.cpu_guest_time = Some(guest_time),
650 Err(e) => {
651 metric_countdown = 0;
Seungjae Yoo663d98d2025-02-17 15:23:39 +0900652 warn!("Failed to get guest CPU time: {}", e);
Jiyong Park85ea5382024-02-19 15:04:23 +0900653 }
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900654 }
Seungjae Yoo3211ab42025-02-13 13:40:14 +0900655
656 // Get Memory Information
657 match get_rss(pid) {
658 Ok(rss) => {
659 vm_metric.rss = match &vm_metric.rss {
660 Some(x) => Some(Rss::extract_max(x, &rss)),
661 None => Some(rss),
662 }
663 }
664 Err(e) => {
665 metric_countdown = 0;
Seungjae Yoo663d98d2025-02-17 15:23:39 +0900666 warn!("Failed to get guest RSS: {}", e);
Seungjae Yoo3211ab42025-02-13 13:40:14 +0900667 }
668 }
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900669 }
670 }
671
672 thread::sleep(Duration::from_secs(1));
673 }
674 }
675
Andrew Walbran6b650662021-09-07 13:13:23 +0000676 /// Returns the last reported state of the VM payload.
677 pub fn payload_state(&self) -> PayloadState {
678 *self.payload_state.lock().unwrap()
679 }
680
681 /// Updates the payload state to the given value, if it is a valid state transition.
682 pub fn update_payload_state(&self, new_state: PayloadState) -> Result<(), Error> {
683 let mut state_locked = self.payload_state.lock().unwrap();
684 // Only allow forward transitions, e.g. from starting to started or finished, not back in
685 // the other direction.
686 if new_state > *state_locked {
687 *state_locked = new_state;
Jiyong Parka4eebde2022-07-12 18:01:12 +0900688 self.payload_state_updated.notify_all();
Andrew Walbran6b650662021-09-07 13:13:23 +0000689 Ok(())
690 } else {
691 bail!("Invalid payload state transition from {:?} to {:?}", *state_locked, new_state)
692 }
693 }
694
Andrew Walbranf8d94112021-09-07 11:45:36 +0000695 /// Kills the crosvm instance, if it is running.
Inseob Kima446f802022-07-11 19:46:37 +0900696 pub fn kill(&self) -> Result<(), Error> {
David Brazdilf18e1162022-12-18 18:27:23 +0000697 let monitor_vm_exit_thread = {
698 let vm_state = &mut *self.vm_state.lock().unwrap();
699 if let VmState::Running { child, monitor_vm_exit_thread } = vm_state {
700 let id = child.id();
701 debug!("Killing crosvm({})", id);
702 // TODO: Talk to crosvm to shutdown cleanly.
703 child.kill().with_context(|| format!("Error killing crosvm({id}) instance"))?;
704 monitor_vm_exit_thread.take()
Inseob Kima446f802022-07-11 19:46:37 +0900705 } else {
David Brazdilf18e1162022-12-18 18:27:23 +0000706 bail!("VM is not running")
Andrew Walbranf8d94112021-09-07 11:45:36 +0000707 }
David Brazdilf18e1162022-12-18 18:27:23 +0000708 };
709
710 // Wait for monitor_vm_exit() to finish. Must release vm_state lock
711 // first, as monitor_vm_exit() takes it as well.
712 monitor_vm_exit_thread.map(JoinHandle::join);
713
David Brazdil5af55832023-03-08 13:08:56 +0000714 // Now that the VM has been killed, shut down the VirtualMachineService
715 // server to eagerly free up the server threads.
Jiyong Park690c1ca2024-11-28 18:06:50 +0900716 if let Some(vm_server) = &self.vm_context.vm_server {
717 vm_server.shutdown()?;
718 }
David Brazdil5af55832023-03-08 13:08:56 +0000719
David Brazdilf18e1162022-12-18 18:27:23 +0000720 Ok(())
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000721 }
Jiyong Parke558ab12022-07-07 20:18:55 +0900722
Frederick Maylef375ef12024-10-21 16:07:21 -0700723 /// Returns current virtio-balloon size.
Keir Fraser48221ba2024-07-12 14:05:02 +0000724 pub fn get_memory_balloon(&self) -> Result<u64, Error> {
Frederick Maylef375ef12024-10-21 16:07:21 -0700725 let socket_path_cstring = path_to_cstring(&self.crosvm_control_socket_path);
726 let mut balloon_actual = 0u64;
727 // SAFETY: Pointers are valid for the lifetime of the call. Null `stats` is valid.
728 let success = unsafe {
729 crosvm_control::crosvm_client_balloon_stats(
730 socket_path_cstring.as_ptr(),
731 /* stats= */ std::ptr::null_mut(),
732 &mut balloon_actual,
733 )
734 };
735 if !success {
736 bail!("Error requesting balloon stats");
737 }
738 Ok(balloon_actual)
Keir Fraser48221ba2024-07-12 14:05:02 +0000739 }
740
Frederick Maylef375ef12024-10-21 16:07:21 -0700741 /// Inflates the virtio-balloon by `num_bytes` to reclaim guest memory. Called in response to
742 /// memory-trimming notifications.
Keir Fraser48221ba2024-07-12 14:05:02 +0000743 pub fn set_memory_balloon(&self, num_bytes: u64) -> Result<(), Error> {
Frederick Maylef375ef12024-10-21 16:07:21 -0700744 let socket_path_cstring = path_to_cstring(&self.crosvm_control_socket_path);
745 // SAFETY: Pointer is valid for the lifetime of the call.
746 let success = unsafe {
747 crosvm_control::crosvm_client_balloon_vms(socket_path_cstring.as_ptr(), num_bytes)
748 };
749 if !success {
750 bail!("Error sending balloon adjustment");
Keir Frasercdd4b112022-11-24 14:02:25 +0000751 }
752 Ok(())
753 }
754
Alan Stokes3e98d292022-12-14 15:18:22 +0000755 /// Checks if ramdump has been created. If so, send it to tombstoned.
Jiyong Parke558ab12022-07-07 20:18:55 +0900756 fn handle_ramdump(&self) -> Result<(), Error> {
757 let ramdump_path = self.temporary_directory.join("ramdump");
Jiyong Park933d2b22023-02-28 11:41:19 +0900758 if !ramdump_path.as_path().try_exists()? {
759 return Ok(());
760 }
Jiyong Parke558ab12022-07-07 20:18:55 +0900761 if std::fs::metadata(&ramdump_path)?.len() > 0 {
Jiyong Park1612b902022-08-22 14:47:39 +0900762 Self::send_ramdump_to_tombstoned(&ramdump_path)?;
Jiyong Parke558ab12022-07-07 20:18:55 +0900763 }
764 Ok(())
765 }
Jiyong Park1612b902022-08-22 14:47:39 +0900766
767 fn send_ramdump_to_tombstoned(ramdump_path: &Path) -> Result<(), Error> {
768 let mut input = File::open(ramdump_path)
Alan Stokes3e98d292022-12-14 15:18:22 +0000769 .context(format!("Failed to open ramdump {:?} for reading", ramdump_path))?;
Jiyong Park1612b902022-08-22 14:47:39 +0900770
771 let pid = std::process::id() as i32;
772 let conn = TombstonedConnection::connect(pid, DebuggerdDumpType::Tombstone)
773 .context("Failed to connect to tombstoned")?;
774 let mut output = conn
775 .text_output
776 .as_ref()
777 .ok_or_else(|| anyhow!("Could not get file to write the tombstones on"))?;
778
779 std::io::copy(&mut input, &mut output).context("Failed to send ramdump to tombstoned")?;
780 info!("Ramdump {:?} sent to tombstoned", ramdump_path);
781
782 conn.notify_completion()?;
783 Ok(())
784 }
Jiyong Park23b67392024-07-04 13:51:42 +0900785
Frederick Maylef375ef12024-10-21 16:07:21 -0700786 /// Suspends the VM's vCPUs.
Jiyong Park23b67392024-07-04 13:51:42 +0900787 pub fn suspend(&self) -> Result<(), Error> {
Frederick Maylef375ef12024-10-21 16:07:21 -0700788 let socket_path_cstring = path_to_cstring(&self.crosvm_control_socket_path);
789 // SAFETY: Pointer is valid for the lifetime of the call.
790 let success =
791 unsafe { crosvm_control::crosvm_client_suspend_vm(socket_path_cstring.as_ptr()) };
792 if !success {
793 bail!("Failed to suspend VM");
Jiyong Park23b67392024-07-04 13:51:42 +0900794 }
Frederick Maylef375ef12024-10-21 16:07:21 -0700795 Ok(())
Jiyong Park23b67392024-07-04 13:51:42 +0900796 }
797
Frederick Maylef375ef12024-10-21 16:07:21 -0700798 /// Resumes the VM's vCPUs.
Jiyong Park23b67392024-07-04 13:51:42 +0900799 pub fn resume(&self) -> Result<(), Error> {
Frederick Maylef375ef12024-10-21 16:07:21 -0700800 let socket_path_cstring = path_to_cstring(&self.crosvm_control_socket_path);
801 // SAFETY: Pointer is valid for the lifetime of the call.
802 let success =
803 unsafe { crosvm_control::crosvm_client_resume_vm(socket_path_cstring.as_ptr()) };
804 if !success {
805 bail!("Failed to resume VM");
Jiyong Park23b67392024-07-04 13:51:42 +0900806 }
Frederick Maylef375ef12024-10-21 16:07:21 -0700807 Ok(())
Jiyong Park23b67392024-07-04 13:51:42 +0900808 }
Andrew Walbrand6dce6f2021-03-05 16:39:08 +0000809}
810
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900811impl Rss {
812 fn extract_max(x: &Rss, y: &Rss) -> Rss {
813 Rss { vm: max(x.vm, y.vm), crosvm: max(x.crosvm, y.crosvm) }
814 }
815}
816
David Daiaaecf652024-02-22 14:41:39 -0800817// Get Cpus_allowed mask
818fn check_if_all_cpus_allowed() -> Result<bool> {
819 let file = read_to_string("/proc/self/status")?;
820 let lines: Vec<_> = file.split('\n').collect();
821
822 for line in lines {
823 if line.contains("Cpus_allowed_list") {
824 let prop: Vec<_> = line.split_whitespace().collect();
825 if prop.len() != 2 {
826 return Ok(false);
827 }
828 let cpu_list: Vec<_> = prop[1].split('-').collect();
829 //Only contiguous Cpu list allowed
830 if cpu_list.len() != 2 {
831 return Ok(false);
832 }
833 if let Some(cpus) = get_num_cpus() {
834 let max_cpu = cpu_list[1].parse::<usize>()?;
835 if max_cpu == cpus - 1 {
836 return Ok(true);
837 } else {
838 return Ok(false);
839 }
840 }
841 }
842 }
843 Ok(false)
844}
845
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900846// Get guest time from /proc/[crosvm pid]/stat
847fn get_guest_time(pid: u32) -> Result<i64> {
848 let file = read_to_string(format!("/proc/{}/stat", pid))?;
849 let data_list: Vec<_> = file.split_whitespace().collect();
850
851 // Information about guest_time is at 43th place of the file split with the whitespace.
852 // Example of /proc/[pid]/stat :
853 // 6603 (kworker/104:1H-kblockd) I 2 0 0 0 -1 69238880 0 0 0 0 0 88 0 0 0 -20 1 0 1845 0 0
854 // 18446744073709551615 0 0 0 0 0 0 0 2147483647 0 0 0 0 17 104 0 0 0 0 0 0 0 0 0 0 0 0 0
855 if data_list.len() < 43 {
856 bail!("Failed to parse command result for getting guest time : {}", file);
857 }
858
859 let guest_time_ticks = data_list[42].parse::<i64>()?;
Seungjae Yoo663d98d2025-02-17 15:23:39 +0900860 if guest_time_ticks == 0 {
861 bail!("zero value is measured on elapsed CPU guest_time");
862 }
Andrew Walbranb58d1b42023-07-07 13:54:49 +0100863 // SAFETY: It just returns an integer about CPU tick information.
Charisee96113f32023-01-26 09:00:42 +0000864 let ticks_per_sec = unsafe { sysconf(_SC_CLK_TCK) };
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900865 Ok(guest_time_ticks * MILLIS_PER_SEC / ticks_per_sec)
866}
867
868// Get rss from /proc/[crosvm pid]/smaps
869fn get_rss(pid: u32) -> Result<Rss> {
870 let file = read_to_string(format!("/proc/{}/smaps", pid))?;
871 let lines: Vec<_> = file.split('\n').collect();
872
873 let mut rss_vm_total = 0i64;
874 let mut rss_crosvm_total = 0i64;
875 let mut is_vm = false;
876 for line in lines {
877 if line.contains("crosvm_guest") {
878 is_vm = true;
879 } else if line.contains("Rss:") {
880 let data_list: Vec<_> = line.split_whitespace().collect();
881 if data_list.len() < 2 {
882 bail!("Failed to parse command result for getting rss :\n{}", line);
883 }
884 let rss = data_list[1].parse::<i64>()?;
885
886 if is_vm {
887 rss_vm_total += rss;
888 is_vm = false;
889 }
890 rss_crosvm_total += rss;
891 }
892 }
Seungjae Yoo663d98d2025-02-17 15:23:39 +0900893 if rss_crosvm_total == 0 {
894 bail!("zero value is measured on RSS of crosvm");
895 }
896 if rss_vm_total == 0 {
897 bail!("zero value is measured on RSS of VM");
898 }
Seungjae Yoo6d265d92022-11-15 10:51:33 +0900899 Ok(Rss { vm: rss_vm_total, crosvm: rss_crosvm_total })
900}
901
Alan Stokes3ba10fd2022-10-06 15:46:51 +0100902fn death_reason(result: &Result<ExitStatus, io::Error>, mut failure_reason: &str) -> DeathReason {
Pierre-Clément Tosic0611e72023-09-15 11:41:07 +0100903 if let Some((reason, info)) = failure_reason.split_once('|') {
Alan Stokes3ba10fd2022-10-06 15:46:51 +0100904 // Separator indicates extra context information is present after the failure name.
Pierre-Clément Tosic0611e72023-09-15 11:41:07 +0100905 error!("Failure info: {info}");
906 failure_reason = reason;
Alan Stokes3ba10fd2022-10-06 15:46:51 +0100907 }
Andrew Walbranc92d35f2022-01-12 12:45:19 +0000908 if let Ok(status) = result {
Andrew Walbranb27681f2022-02-23 15:11:52 +0000909 match failure_reason {
910 "PVM_FIRMWARE_PUBLIC_KEY_MISMATCH" => {
911 return DeathReason::PVM_FIRMWARE_PUBLIC_KEY_MISMATCH
912 }
913 "PVM_FIRMWARE_INSTANCE_IMAGE_CHANGED" => {
914 return DeathReason::PVM_FIRMWARE_INSTANCE_IMAGE_CHANGED
915 }
Inseob Kim272f5722022-06-13 17:14:51 +0900916 "MICRODROID_FAILED_TO_CONNECT_TO_VIRTUALIZATION_SERVICE" => {
917 return DeathReason::MICRODROID_FAILED_TO_CONNECT_TO_VIRTUALIZATION_SERVICE
918 }
919 "MICRODROID_PAYLOAD_HAS_CHANGED" => return DeathReason::MICRODROID_PAYLOAD_HAS_CHANGED,
920 "MICRODROID_PAYLOAD_VERIFICATION_FAILED" => {
921 return DeathReason::MICRODROID_PAYLOAD_VERIFICATION_FAILED
922 }
923 "MICRODROID_INVALID_PAYLOAD_CONFIG" => {
924 return DeathReason::MICRODROID_INVALID_PAYLOAD_CONFIG
925 }
926 "MICRODROID_UNKNOWN_RUNTIME_ERROR" => {
927 return DeathReason::MICRODROID_UNKNOWN_RUNTIME_ERROR
928 }
Jiyong Parke6ed0f92022-06-22 00:13:00 +0900929 "HANGUP" => return DeathReason::HANGUP,
Andrew Walbranb27681f2022-02-23 15:11:52 +0000930 _ => {}
931 }
Andrew Walbranc92d35f2022-01-12 12:45:19 +0000932 match status.code() {
933 None => DeathReason::KILLED,
934 Some(0) => DeathReason::SHUTDOWN,
Alan Stokes7c459e82022-12-06 12:21:49 +0000935 Some(CROSVM_START_ERROR_STATUS) => DeathReason::START_FAILED,
Andrew Walbranc92d35f2022-01-12 12:45:19 +0000936 Some(CROSVM_REBOOT_STATUS) => DeathReason::REBOOT,
Andrew Walbrand15c5632022-02-03 13:38:31 +0000937 Some(CROSVM_CRASH_STATUS) => DeathReason::CRASH,
Sebastian Ene23167d82022-10-07 14:09:53 +0000938 Some(CROSVM_WATCHDOG_REBOOT_STATUS) => DeathReason::WATCHDOG_REBOOT,
Andrew Walbranc92d35f2022-01-12 12:45:19 +0000939 Some(_) => DeathReason::UNKNOWN,
940 }
941 } else {
942 DeathReason::INFRASTRUCTURE_ERROR
943 }
944}
945
Seungjae Yoo93430e82022-12-05 16:37:42 +0900946fn exit_signal(result: &Result<ExitStatus, io::Error>) -> Option<i32> {
947 match result {
948 Ok(status) => status.signal(),
949 Err(_) => None,
950 }
951}
952
Inseob Kim6ef80972023-07-20 17:23:36 +0900953const SYSFS_PLATFORM_DEVICES_PATH: &str = "/sys/devices/platform/";
954const VFIO_PLATFORM_DRIVER_PATH: &str = "/sys/bus/platform/drivers/vfio-platform";
955
Inseob Kim7307a892023-09-14 13:37:58 +0900956fn vfio_argument_for_platform_device(device: &VfioDevice) -> Result<String, Error> {
Inseob Kim6ef80972023-07-20 17:23:36 +0900957 // Check platform device exists
Jakob Vukalovicd42aa2c2023-11-09 16:04:00 +0000958 let path = Path::new(&device.getSysfsPath()?).canonicalize()?;
Inseob Kim6ef80972023-07-20 17:23:36 +0900959 if !path.starts_with(SYSFS_PLATFORM_DEVICES_PATH) {
960 bail!("{path:?} is not a platform device");
961 }
962
963 // Check platform device is bound to VFIO driver
964 let dev_driver_path = path.join("driver").canonicalize()?;
965 if dev_driver_path != Path::new(VFIO_PLATFORM_DRIVER_PATH) {
966 bail!("{path:?} is not bound to VFIO-platform driver");
967 }
968
969 if let Some(p) = path.to_str() {
Jakob Vukalovicd42aa2c2023-11-09 16:04:00 +0000970 Ok(format!("--vfio={p},iommu=pkvm-iommu,dt-symbol={0}", device.getDtboLabel()?))
Inseob Kim6ef80972023-07-20 17:23:36 +0900971 } else {
972 bail!("invalid path {path:?}");
973 }
974}
975
Yuan Yao2b654002024-12-25 05:19:12 +0000976fn run_virtiofs(config: &CrosvmConfig) -> io::Result<Vec<SharedChild>> {
977 let mut devices: Vec<SharedChild> = Vec::new();
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700978 for shared_path in &config.shared_paths {
Akilesh Kailashc9aa0682024-11-25 10:27:24 -0800979 if shared_path.app_domain {
980 continue;
981 }
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700982 let ugid_map_value = format!(
983 "{} {} {} {} {} /",
984 shared_path.guest_uid,
985 shared_path.guest_gid,
986 shared_path.host_uid,
987 shared_path.host_gid,
988 shared_path.mask,
989 );
990
Akilesh Kailash8790ea72024-10-28 21:34:39 -0700991 let cfg_arg = format!("ugid_map='{}'", ugid_map_value);
Akilesh Kailash05515dc2024-10-22 21:42:52 -0700992
993 let mut command = Command::new(CROSVM_PATH);
994 command
995 .arg("device")
996 .arg("fs")
997 .arg(format!("--socket={}", &shared_path.socket_path))
998 .arg(format!("--tag={}", &shared_path.tag))
999 .arg(format!("--shared-dir={}", &shared_path.path))
1000 .arg("--cfg")
1001 .arg(cfg_arg.as_str())
Akilesh Kailash6676ea82024-10-25 17:14:33 -07001002 .arg("--disable-sandbox")
1003 .arg("--skip-pivot-root=true");
Akilesh Kailash05515dc2024-10-22 21:42:52 -07001004
1005 print_crosvm_args(&command);
1006
1007 let result = SharedChild::spawn(&mut command)?;
1008 info!("Spawned virtiofs crosvm({})", result.id());
Yuan Yao2b654002024-12-25 05:19:12 +00001009 devices.push(result);
Akilesh Kailash05515dc2024-10-22 21:42:52 -07001010 }
1011
Yuan Yao2b654002024-12-25 05:19:12 +00001012 Ok(devices)
Akilesh Kailash05515dc2024-10-22 21:42:52 -07001013}
1014
Andrew Walbrand3a84182021-09-07 14:48:52 +00001015/// Starts an instance of `crosvm` to manage a new VM.
Keir Fraser13a956a2022-07-14 14:20:46 +00001016fn run_vm(
1017 config: CrosvmConfig,
Keir Frasercdd4b112022-11-24 14:02:25 +00001018 crosvm_control_socket_path: &Path,
Keir Fraser13a956a2022-07-14 14:20:46 +00001019 failure_pipe_write: File,
1020) -> Result<SharedChild, Error> {
Andrew Walbrand3a84182021-09-07 14:48:52 +00001021 validate_config(&config)?;
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001022
1023 let mut command = Command::new(CROSVM_PATH);
Jiyong Parkf50ec332024-10-23 17:54:03 +09001024
1025 let vm_name = "crosvm_".to_owned() + &config.name;
1026 command.arg0(vm_name.clone());
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001027 // TODO(qwandor): Remove --disable-sandbox.
Andrew Walbranc92d35f2022-01-12 12:45:19 +00001028 command
1029 .arg("--extended-status")
Jiyong Park6c60fea2022-10-24 16:10:01 +09001030 // Configure the logger for the crosvm process to silence logs from the disk crate which
1031 // don't provide much information to us (but do spamming us).
1032 .arg("--log-level")
Frederick Mayle138f53c2023-01-24 18:55:26 -08001033 .arg("info,disk=warn")
Andrew Walbranc92d35f2022-01-12 12:45:19 +00001034 .arg("run")
Jiyong Parkf50ec332024-10-23 17:54:03 +09001035 .arg("--name")
1036 .arg(vm_name)
Andrew Walbranc92d35f2022-01-12 12:45:19 +00001037 .arg("--disable-sandbox")
1038 .arg("--cid")
1039 .arg(config.cid.to_string());
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001040
Frederick Maylecc4e2d72024-12-06 16:54:40 -08001041 if system_properties::read_bool("hypervisor.memory_reclaim.supported", false)? && config.balloon
Jeongik Cha75057452024-09-03 11:26:04 +09001042 {
Keir Fraserf25cb922022-11-23 14:26:00 +00001043 command.arg("--balloon-page-reporting");
1044 } else {
1045 command.arg("--no-balloon");
1046 }
1047
Elie Kheirallah29d72912024-08-07 20:17:26 +00001048 if !config.usb_config.controller {
1049 command.arg("--no-usb");
1050 }
1051
Jiyong Parkae625b62024-06-13 21:47:52 +09001052 let mut memory_mib = config.memory_mib;
1053
Frederick Mayle196ca2b2024-12-18 15:27:43 -08001054 if config.enable_hypervisor_specific_auth_method && !config.protected {
1055 bail!("hypervisor specific auth method only supported for protected VMs");
1056 }
Andrew Walbranf8650422021-06-09 15:54:09 +00001057 if config.protected {
Frederick Mayle196ca2b2024-12-18 15:27:43 -08001058 if config.enable_hypervisor_specific_auth_method {
1059 if !hypervisor_props::is_gunyah()? {
1060 bail!("hypervisor specific auth method not supported for current hypervisor");
1061 }
1062 // "QCOM Trusted VM" compatibility mode.
1063 //
1064 // When this mode is enabled, two hypervisor specific IDs are expected to be packed
1065 // into the instance ID. We extract them here and pass along to crosvm so they can be
1066 // given to the hypervisor driver via an ioctl.
Saad Sattera3a098b2025-02-04 10:49:24 -08001067 let pas_id = u32::from_le_bytes(config.instance_id[60..64].try_into().unwrap());
1068 let vm_id = u16::from_le_bytes(config.instance_id[58..60].try_into().unwrap());
Frederick Mayle196ca2b2024-12-18 15:27:43 -08001069 command.arg("--hypervisor").arg(
1070 format!("gunyah[device=/dev/gunyah,qcom_trusted_vm_id={vm_id},qcom_trusted_vm_pas_id={pas_id}]"),
1071 );
1072 // Put the FDT close to the payload (default is end of RAM) to so that CMA can be used
1073 // without bloating memory usage.
1074 command.arg("--fdt-position").arg("after-payload");
1075 }
1076
Jaewan Kim46422812022-11-25 10:59:39 +09001077 match system_properties::read(SYSPROP_CUSTOM_PVMFW_PATH)? {
1078 Some(pvmfw_path) if !pvmfw_path.is_empty() => {
Nikita Ioffe5f8e75d2024-11-04 17:38:05 +00001079 if pvmfw_path == "none" {
1080 command.arg("--protected-vm-without-firmware")
1081 } else {
1082 command.arg("--protected-vm-with-firmware").arg(pvmfw_path)
1083 }
Jaewan Kim46422812022-11-25 10:59:39 +09001084 }
1085 _ => command.arg("--protected-vm"),
1086 };
Andrew Walbran0b5789f2022-02-04 13:57:57 +00001087
Frederick Mayle384f5b52024-12-06 16:23:23 -08001088 let swiotlb_size_mib = config.swiotlb_mib.map(u32::from).unwrap_or({
1089 // 3 virtio-console devices + vsock = 4.
1090 // TODO: Count more device types, like balloon, input, and sound.
1091 let virtio_pci_device_count = 4 + config.disks.len();
1092 // crosvm virtio queue has 256 entries, so 2 MiB per device (2 pages per entry) should
1093 // be enough.
1094 // NOTE: The above explanation isn't completely accurate, e.g., circa 2024q4, each
1095 // virtio-block has 16 queues with 256 entries each and each virito-console has 2
1096 // queues of 256 entries each. So, it is allocating less than 2 pages per entry, but
1097 // seems to work well enough in practice.
1098 2 * virtio_pci_device_count as u32
1099 });
Andrew Walbran0b5789f2022-02-04 13:57:57 +00001100 command.arg("--swiotlb").arg(swiotlb_size_mib.to_string());
Frederick Mayle58baa302023-02-14 19:06:37 -08001101
Frederick Maylea24bbe02024-07-12 16:50:48 -07001102 // b/346770542 for consistent "usable" memory across protected and non-protected VMs.
Frederick Mayle68d06e72024-07-12 17:18:11 -07001103 memory_mib = memory_mib.saturating_add(swiotlb_size_mib);
Jiyong Parkae625b62024-06-13 21:47:52 +09001104
Frederick Mayle58baa302023-02-14 19:06:37 -08001105 // Workaround to keep crash_dump from trying to read protected guest memory.
1106 // Context in b/238324526.
1107 command.arg("--unmap-guest-memory-on-fork");
Jiyong Park3115cdb2023-02-24 18:04:44 +09001108
Jiyong Parked180932023-02-24 19:55:41 +09001109 if config.ramdump.is_some() {
Jaewan Kim3124ef02023-03-23 19:25:20 +09001110 // Protected VM needs to reserve memory for ramdump here. Note that we reserve more
Jiyong Parked180932023-02-24 19:55:41 +09001111 // memory for the restricted dma pool.
1112 let ramdump_reserve = RAMDUMP_RESERVED_MIB + swiotlb_size_mib;
1113 command.arg("--params").arg(format!("crashkernel={ramdump_reserve}M"));
1114 }
Jaewan Kimd4dae652023-03-28 13:45:43 +09001115 } else if config.ramdump.is_some() {
1116 command.arg("--params").arg(format!("crashkernel={RAMDUMP_RESERVED_MIB}M"));
1117 }
Jaewan Kim61f86142023-03-28 15:12:52 +09001118 if config.debug_config.debug_level == DebugLevel::NONE
1119 && config.debug_config.should_prepare_console_output()
1120 {
Jaewan Kimd4dae652023-03-28 13:45:43 +09001121 // bootconfig.normal will be used, but we need log.
1122 command.arg("--params").arg("printk.devkmsg=on");
1123 command.arg("--params").arg("console=hvc0");
Andrew Walbranf8650422021-06-09 15:54:09 +00001124 }
1125
Frederick Mayle803aa8c2024-11-22 13:45:55 -08001126 // Move the PCI MMIO regions to near the end of the low-MMIO space.
1127 // This is done to accommodate a limitation in a partner's hypervisor.
1128 #[cfg(target_arch = "aarch64")]
1129 command
1130 .arg("--pci")
Prakruthi Deepak Heragu2a8802a2025-01-23 16:29:12 -08001131 .arg("mem=[start=0x2c000000,size=0x2000000],cam=[start=0x2e000000,size=0x1000000]");
Frederick Mayle803aa8c2024-11-22 13:45:55 -08001132
Frederick Mayle68d06e72024-07-12 17:18:11 -07001133 command.arg("--mem").arg(memory_mib.to_string());
Andrew Walbranb15cd6e2021-07-05 16:38:07 +00001134
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001135 fn cpu_arg_command(command: &mut Command, count: usize) {
Elie Kheirallah1944db32025-01-29 13:08:30 -08001136 #[cfg(target_arch = "aarch64")]
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001137 command.arg("--cpus").arg(count.to_string() + ",sve=[auto=true]");
Elie Kheirallah1944db32025-01-29 13:08:30 -08001138 #[cfg(not(target_arch = "aarch64"))]
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001139 command.arg("--cpus").arg(count.to_string());
Jiyong Park032615f2022-01-10 13:55:34 +09001140 }
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001141 match config.cpus.cpuTopology {
1142 CpuTopology::MatchHost(_) => {
1143 if cfg!(virt_cpufreq) && check_if_all_cpus_allowed()? {
1144 command.arg("--host-cpu-topology");
1145 #[cfg(target_arch = "aarch64")]
1146 {
David Daic97e82d2024-02-17 12:10:31 -08001147 command.arg("--virt-cpufreq");
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001148 command.arg("--cpus").arg("sve=[auto=true]");
David Daic97e82d2024-02-17 12:10:31 -08001149 }
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001150 } else {
1151 cpu_arg_command(
1152 &mut command,
1153 get_num_cpus()
1154 .context("Could not determine the number of CPUs in the system")?,
1155 )
David Daic97e82d2024-02-17 12:10:31 -08001156 }
Elie Kheirallahb4b2f242025-01-23 03:38:07 +00001157 }
1158 CpuTopology::CpuCount(count) => {
1159 cpu_arg_command(&mut command, count.try_into().context("invalid cpu count")?)
David Brazdil25e8c052023-02-17 12:53:01 +00001160 }
David Brazdil7d1e5ec2023-02-06 17:56:29 +00001161 }
1162
Nikita Ioffe5776f082023-02-10 21:38:26 +00001163 if let Some(gdb_port) = config.gdb_port {
1164 command.arg("--gdb").arg(gdb_port.to_string());
1165 }
1166
Jiyong Parkfa91d702021-10-18 23:51:39 +09001167 // Keep track of what file descriptors should be mapped to the crosvm process.
Jiyong Park69001a22024-08-16 20:45:34 +09001168 let mut preserved_fds = config.indirect_files.into_iter().map(|f| f.into()).collect();
Jiyong Parkfa91d702021-10-18 23:51:39 +09001169
Elie Kheirallah5c807a22024-09-23 20:40:42 +00001170 if let Some(dump_dt_fd) = config.dump_dt_fd {
1171 let dump_dt_fd = add_preserved_fd(&mut preserved_fds, dump_dt_fd);
1172 command.arg("--dump-device-tree-blob").arg(dump_dt_fd);
1173 }
1174
Jiyong Park747d6362021-10-19 17:12:52 +09001175 // Setup the serial devices.
1176 // 1. uart device: used as the output device by bootloaders and as early console by linux
Andrew Walbranb27681f2022-02-23 15:11:52 +00001177 // 2. uart device: used to report the reason for the VM failing.
1178 // 3. virtio-console device: used as the console device where kmsg is redirected to
Jiyong Park4afe2012022-07-08 05:38:49 +09001179 // 4. virtio-console device: used as the ramdump output
Andrew Walbranb27681f2022-02-23 15:11:52 +00001180 // 5. virtio-console device: used as the logcat output
Jiyong Park747d6362021-10-19 17:12:52 +09001181 //
Jiyong Parkb8182bb2021-10-26 22:53:08 +09001182 // When [console|log]_fd is not specified, the devices are attached to sink, which means what's
1183 // written there is discarded.
Jiyong Park69001a22024-08-16 20:45:34 +09001184 let console_out_arg = format_serial_out_arg(&mut preserved_fds, config.console_out_fd);
Jiyong Parke6fb1672023-06-26 16:45:55 +09001185 let console_in_arg = config
1186 .console_in_fd
Jiyong Parke6fb1672023-06-26 16:45:55 +09001187 .map(|fd| format!(",input={}", add_preserved_fd(&mut preserved_fds, fd)))
1188 .unwrap_or_default();
Jiyong Park69001a22024-08-16 20:45:34 +09001189 let log_arg = format_serial_out_arg(&mut preserved_fds, config.log_fd);
1190 let failure_serial_path = add_preserved_fd(&mut preserved_fds, failure_pipe_write);
1191 let ramdump_arg = format_serial_out_arg(&mut preserved_fds, config.ramdump);
Yi-Yo Chiang8dd32552024-05-22 19:38:16 +08001192 let console_input_device = config.console_input_device.as_deref().unwrap_or(CONSOLE_HVC0);
1193 match console_input_device {
1194 CONSOLE_HVC0 | CONSOLE_TTYS0 => {}
1195 _ => bail!("Unsupported serial device {console_input_device}"),
1196 };
Jiyong Parkfa91d702021-10-18 23:51:39 +09001197
Jiyong Park747d6362021-10-19 17:12:52 +09001198 // Warning: Adding more serial devices requires you to shift the PCI device ID of the boot
1199 // disks in bootconfig.x86_64. This is because x86 crosvm puts serial devices and the block
1200 // devices in the same PCI bus and serial devices comes before the block devices. Arm crosvm
1201 // doesn't have the issue.
Jiyong Parkfa91d702021-10-18 23:51:39 +09001202 // /dev/ttyS0
Yi-Yo Chiang8dd32552024-05-22 19:38:16 +08001203 command.arg(format!(
1204 "--serial={}{},hardware=serial,num=1",
1205 &console_out_arg,
1206 if console_input_device == CONSOLE_TTYS0 { &console_in_arg } else { "" }
1207 ));
Andrew Walbranb27681f2022-02-23 15:11:52 +00001208 // /dev/ttyS1
1209 command.arg(format!("--serial=type=file,path={},hardware=serial,num=2", &failure_serial_path));
Jiyong Parkfa91d702021-10-18 23:51:39 +09001210 // /dev/hvc0
Jiyong Parke6fb1672023-06-26 16:45:55 +09001211 command.arg(format!(
1212 "--serial={}{},hardware=virtio-console,num=1",
Yi-Yo Chiang8dd32552024-05-22 19:38:16 +08001213 &console_out_arg,
1214 if console_input_device == CONSOLE_HVC0 { &console_in_arg } else { "" }
Jiyong Parke6fb1672023-06-26 16:45:55 +09001215 ));
Jiyong Park4afe2012022-07-08 05:38:49 +09001216 // /dev/hvc1
1217 command.arg(format!("--serial={},hardware=virtio-console,num=2", &ramdump_arg));
Jiyong Parkae5a4ed2021-11-01 18:36:28 +09001218 // /dev/hvc2
1219 command.arg(format!("--serial={},hardware=virtio-console,num=3", &log_arg));
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001220
Jiyong Park69001a22024-08-16 20:45:34 +09001221 if let Some(bootloader) = config.bootloader {
Andrew Walbran02b8ec02021-06-22 13:07:02 +00001222 command.arg("--bios").arg(add_preserved_fd(&mut preserved_fds, bootloader));
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001223 }
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001224
Jiyong Park69001a22024-08-16 20:45:34 +09001225 if let Some(initrd) = config.initrd {
Andrew Walbran02b8ec02021-06-22 13:07:02 +00001226 command.arg("--initrd").arg(add_preserved_fd(&mut preserved_fds, initrd));
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001227 }
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001228
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001229 if let Some(params) = &config.params {
1230 command.arg("--params").arg(params);
1231 }
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001232
Jiyong Park69001a22024-08-16 20:45:34 +09001233 for disk in config.disks {
Frederick Mayleeae6d2d2024-09-12 11:05:36 -07001234 // Disk file locking is disabled because of missing SELinux policies.
Frederick Mayle1ff7b842024-07-12 16:20:09 -07001235 command.arg("--block").arg(format!(
Frederick Mayleeae6d2d2024-09-12 11:05:36 -07001236 "path={},ro={},lock=false",
Jiyong Park69001a22024-08-16 20:45:34 +09001237 add_preserved_fd(&mut preserved_fds, disk.image),
Frederick Mayle1ff7b842024-07-12 16:20:09 -07001238 !disk.writable,
1239 ));
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001240 }
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001241
Jiyong Park69001a22024-08-16 20:45:34 +09001242 if let Some(kernel) = config.kernel {
Andrew Walbran02b8ec02021-06-22 13:07:02 +00001243 command.arg(add_preserved_fd(&mut preserved_fds, kernel));
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001244 }
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001245
Elie Kheirallah35dbb8e2024-10-23 00:15:14 +00001246 #[cfg(target_arch = "aarch64")]
1247 command.arg("--no-pmu");
1248
Frederick Maylee0ed2352024-09-12 16:16:00 -07001249 let control_sock = create_crosvm_control_listener(crosvm_control_socket_path)
1250 .context("failed to create control listener")?;
Jiyong Parkcd0f74b2024-08-20 11:06:52 +09001251 command.arg("--socket").arg(add_preserved_fd(&mut preserved_fds, control_sock));
Keir Fraser13a956a2022-07-14 14:20:46 +00001252
Jaewan Kimf43372b2024-12-13 18:33:58 +09001253 config.device_tree_overlays.into_iter().for_each(|dt_overlay| {
1254 let arg = add_preserved_fd(&mut preserved_fds, dt_overlay);
1255 command.arg("--device-tree-overlay").arg(arg);
1256 });
Jeongik Cha798401c2024-04-11 21:54:49 +09001257
1258 if cfg!(paravirtualized_devices) {
Jason Macnakea7b0fa2024-05-08 17:02:44 -07001259 if let Some(gpu_config) = &config.gpu_config {
1260 let mut gpu_args = Vec::new();
1261 if let Some(backend) = &gpu_config.backend {
1262 gpu_args.push(format!("backend={}", backend));
1263 }
1264 if let Some(context_types) = &gpu_config.context_types {
1265 gpu_args.push(format!("context-types={}", context_types.join(":")));
1266 }
1267 if let Some(pci_address) = &gpu_config.pci_address {
1268 gpu_args.push(format!("pci-address={}", pci_address));
1269 }
1270 if let Some(renderer_features) = &gpu_config.renderer_features {
1271 gpu_args.push(format!("renderer-features={}", renderer_features));
1272 }
1273 if gpu_config.renderer_use_egl.unwrap_or(false) {
1274 gpu_args.push("egl=true".to_string());
1275 }
1276 if gpu_config.renderer_use_gles.unwrap_or(false) {
1277 gpu_args.push("gles=true".to_string());
1278 }
1279 if gpu_config.renderer_use_glx.unwrap_or(false) {
1280 gpu_args.push("glx=true".to_string());
1281 }
1282 if gpu_config.renderer_use_surfaceless.unwrap_or(false) {
1283 gpu_args.push("surfaceless=true".to_string());
1284 }
1285 if gpu_config.renderer_use_vulkan.unwrap_or(false) {
1286 gpu_args.push("vulkan=true".to_string());
1287 }
1288 command.arg(format!("--gpu={}", gpu_args.join(",")));
1289 }
Jeongik Cha798401c2024-04-11 21:54:49 +09001290 if let Some(display_config) = &config.display_config {
Jason Macnakea7b0fa2024-05-08 17:02:44 -07001291 command
1292 .arg(format!(
1293 "--gpu-display=mode=windowed[{},{}],dpi=[{},{}],refresh-rate={}",
1294 display_config.width,
1295 display_config.height,
1296 display_config.horizontal_dpi,
1297 display_config.vertical_dpi,
1298 display_config.refresh_rate
1299 ))
1300 .arg(format!("--android-display-service={}", config.name));
Jeongik Cha798401c2024-04-11 21:54:49 +09001301 }
Jeongik Cha1df1c032024-04-03 16:03:12 +09001302 }
Seungjae Yooec3bc522023-11-09 10:14:30 +09001303
Seungjae Yoo986d7a42024-05-28 14:06:58 +09001304 if cfg!(network) {
Jiyong Park69001a22024-08-16 20:45:34 +09001305 if let Some(tap) = config.tap {
Andreas Huber487810d2024-08-22 07:43:01 -07001306 add_preserved_fd(&mut preserved_fds, tap);
1307 let tap_fd = preserved_fds.last().unwrap().as_raw_fd();
1308 command.arg("--net").arg(format!("tap-fd={tap_fd}"));
Seungjae Yoo986d7a42024-05-28 14:06:58 +09001309 }
1310 }
1311
Jeongik Cha6be4f8b2024-04-16 01:13:13 +09001312 if cfg!(paravirtualized_devices) {
Jiyong Park69001a22024-08-16 20:45:34 +09001313 for input_device_option in config.input_device_options.into_iter() {
Jeongik Chac181be72024-03-27 00:18:30 +09001314 command.arg("--input");
1315 command.arg(match input_device_option {
1316 InputDeviceOption::EvDev(file) => {
1317 format!("evdev[path={}]", add_preserved_fd(&mut preserved_fds, file))
1318 }
Jeongik Cha985b6402024-04-15 18:13:00 +09001319 InputDeviceOption::Keyboard(file) => {
1320 format!("keyboard[path={}]", add_preserved_fd(&mut preserved_fds, file))
1321 }
Jeongik Cha343894e2024-05-17 20:39:31 +09001322 InputDeviceOption::Mouse(file) => {
1323 format!("mouse[path={}]", add_preserved_fd(&mut preserved_fds, file))
1324 }
Jeongik Chac181be72024-03-27 00:18:30 +09001325 InputDeviceOption::SingleTouch { file, width, height, name } => format!(
1326 "single-touch[path={},width={},height={}{}]",
1327 add_preserved_fd(&mut preserved_fds, file),
1328 width,
1329 height,
1330 name.as_ref().map_or("".into(), |n| format!(",name={}", n))
1331 ),
Jiyong Park6c1b9f02024-07-05 16:21:10 +09001332 InputDeviceOption::Switches(file) => {
1333 format!("switches[path={}]", add_preserved_fd(&mut preserved_fds, file))
1334 }
Jeongik Cha112a2682024-07-09 12:28:45 +09001335 InputDeviceOption::MultiTouchTrackpad { file, width, height, name } => format!(
1336 "multi-touch-trackpad[path={},width={},height={}{}]",
1337 add_preserved_fd(&mut preserved_fds, file),
1338 width,
1339 height,
1340 name.as_ref().map_or("".into(), |n| format!(",name={}", n))
1341 ),
Jeongik Cha10494912024-07-16 11:19:50 +09001342 InputDeviceOption::MultiTouch { file, width, height, name } => format!(
1343 "multi-touch[path={},width={},height={}{}]",
1344 add_preserved_fd(&mut preserved_fds, file),
1345 width,
1346 height,
1347 name.as_ref().map_or("".into(), |n| format!(",name={}", n))
1348 ),
Jeongik Chac181be72024-03-27 00:18:30 +09001349 });
1350 }
1351 }
Vincent Donnefort538a2c62024-03-20 16:01:10 +00001352
1353 if config.hugepages {
1354 command.arg("--hugepages");
1355 }
1356
David Dai23cff712024-06-13 19:23:45 +00001357 if config.boost_uclamp {
1358 command.arg("--boost-uclamp");
1359 }
1360
Jiyong Park69001a22024-08-16 20:45:34 +09001361 if !config.vfio_devices.is_empty() {
1362 if let Some(dtbo) = config.dtbo {
1363 command.arg(format!(
1364 "--device-tree-overlay={},filter",
1365 add_preserved_fd(&mut preserved_fds, dtbo)
1366 ));
1367 } else {
1368 bail!("VFIO devices assigned but no DTBO available");
1369 }
1370 };
1371 for device in config.vfio_devices {
1372 command.arg(vfio_argument_for_platform_device(&device)?);
1373 }
Inseob Kim6ef80972023-07-20 17:23:36 +09001374
Akilesh Kailash05515dc2024-10-22 21:42:52 -07001375 for shared_path in &config.shared_paths {
Akilesh Kailashc9aa0682024-11-25 10:27:24 -08001376 if shared_path.app_domain {
1377 if let Some(socket_fd) = &shared_path.socket_fd {
1378 let socket_path =
1379 add_preserved_fd(&mut preserved_fds, socket_fd.try_clone().unwrap());
1380 let raw_fd: i32 = socket_path.rsplit_once('/').unwrap().1.parse().unwrap();
1381 command
1382 .arg("--vhost-user-fs")
1383 .arg(format!("tag={},socket-fd={}", &shared_path.tag, raw_fd));
1384 }
1385 } else {
1386 if let Err(e) = wait_for_file(&shared_path.socket_path, 5) {
1387 bail!("Error waiting for file: {}", e);
1388 }
1389 command
1390 .arg("--vhost-user-fs")
1391 .arg(format!("{},tag={}", &shared_path.socket_path, &shared_path.tag));
Akilesh Kailash5fcb6342024-10-30 15:32:09 -07001392 }
Akilesh Kailash05515dc2024-10-22 21:42:52 -07001393 }
1394
Frederick Mayle31cc5b22024-12-19 14:42:43 -08001395 for (fd, addr, size) in config.custom_memory_backing_files {
1396 command.arg("--file-backed-mapping").arg(format!(
1397 "{},addr={addr:#0x},size={size:#0x},rw,ram",
1398 add_preserved_fd(&mut preserved_fds, fd)
1399 ));
1400 }
1401
Andrew Walbran02b8ec02021-06-22 13:07:02 +00001402 debug!("Preserving FDs {:?}", preserved_fds);
1403 command.preserved_fds(preserved_fds);
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001404
Mu-Le Lee134180a2024-04-16 15:13:07 +08001405 if cfg!(paravirtualized_devices) {
Mu-Le Leeb2d5a312024-06-05 10:52:47 +08001406 if let Some(audio_config) = &config.audio_config {
1407 command.arg("--virtio-snd").arg(format!(
1408 "backend=aaudio,num_input_devices={},num_output_devices={}",
1409 if audio_config.use_microphone { 1 } else { 0 },
1410 if audio_config.use_speaker { 1 } else { 0 }
1411 ));
Mu-Le Lee134180a2024-04-16 15:13:07 +08001412 }
1413 }
1414
Jiyong Park2d736562022-10-24 22:40:12 +09001415 print_crosvm_args(&command);
1416
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001417 let result = SharedChild::spawn(&mut command)?;
Jooyung Hanbfe086f2021-10-28 10:15:45 +09001418 debug!("Spawned crosvm({}).", result.id());
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001419 Ok(result)
1420}
1421
Akilesh Kailash5fcb6342024-10-30 15:32:09 -07001422fn wait_for_file(path: &str, timeout_secs: u64) -> Result<(), std::io::Error> {
1423 let start_time = std::time::Instant::now();
1424 let timeout = Duration::from_secs(timeout_secs);
1425
1426 while start_time.elapsed() < timeout {
1427 if std::fs::metadata(path).is_ok() {
1428 return Ok(()); // File exists
1429 }
1430 thread::sleep(Duration::from_millis(100));
1431 }
1432
1433 Err(std::io::Error::new(
1434 std::io::ErrorKind::NotFound,
1435 format!("File not found within {} seconds: {}", timeout_secs, path),
1436 ))
1437}
1438
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001439/// Ensure that the configuration has a valid combination of fields set, or return an error if not.
Andrew Walbranf5fbb7d2021-05-12 17:15:48 +00001440fn validate_config(config: &CrosvmConfig) -> Result<(), Error> {
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001441 if config.bootloader.is_none() && config.kernel.is_none() {
1442 bail!("VM must have either a bootloader or a kernel image.");
1443 }
1444 if config.bootloader.is_some() && (config.kernel.is_some() || config.initrd.is_some()) {
1445 bail!("Can't have both bootloader and kernel/initrd image.");
1446 }
Jiyong Parkdcf17412022-02-08 15:07:23 +09001447 let version = Version::parse(CROSVM_PLATFORM_VERSION).unwrap();
1448 if !config.platform_version.matches(&version) {
1449 bail!(
1450 "Incompatible platform version. The config is compatible with platform version(s) \
1451 {}, but the actual platform version is {}",
1452 config.platform_version,
1453 version
1454 );
1455 }
1456
Andrew Walbran3a5a9212021-05-04 17:09:08 +00001457 Ok(())
1458}
1459
Jiyong Park2d736562022-10-24 22:40:12 +09001460/// Print arguments of the crosvm command. In doing so, /proc/self/fd/XX is annotated with the
1461/// actual file path if the FD is backed by a regular file. If not, the /proc path is printed
1462/// unmodified.
1463fn print_crosvm_args(command: &Command) {
1464 let re = Regex::new(r"/proc/self/fd/[\d]+").unwrap();
1465 info!(
1466 "Running crosvm with args: {:?}",
1467 command
1468 .get_args()
1469 .map(|s| s.to_string_lossy())
1470 .map(|s| {
1471 re.replace_all(&s, |caps: &Captures| {
1472 let path = &caps[0];
1473 if let Ok(realpath) = std::fs::canonicalize(path) {
1474 format!("{} ({})", path, realpath.to_string_lossy())
1475 } else {
1476 path.to_owned()
1477 }
1478 })
1479 .into_owned()
1480 })
1481 .collect::<Vec<_>>()
1482 );
1483}
1484
Andrew Walbran02b8ec02021-06-22 13:07:02 +00001485/// Adds the file descriptor for `file` to `preserved_fds`, and returns a string of the form
1486/// "/proc/self/fd/N" where N is the file descriptor.
Jiyong Park69001a22024-08-16 20:45:34 +09001487fn add_preserved_fd<F: Into<OwnedFd>>(preserved_fds: &mut Vec<OwnedFd>, file: F) -> String {
1488 let fd = file.into();
1489 let raw_fd = fd.as_raw_fd();
Andrew Walbran02b8ec02021-06-22 13:07:02 +00001490 preserved_fds.push(fd);
Jiyong Park69001a22024-08-16 20:45:34 +09001491 format!("/proc/self/fd/{}", raw_fd)
Andrew Walbrand6dce6f2021-03-05 16:39:08 +00001492}
Andrew Walbranb27681f2022-02-23 15:11:52 +00001493
1494/// Adds the file descriptor for `file` (if any) to `preserved_fds`, and returns the appropriate
1495/// string for a crosvm `--serial` flag. If `file` is none, creates a dummy sink device.
Jiyong Park69001a22024-08-16 20:45:34 +09001496fn format_serial_out_arg(preserved_fds: &mut Vec<OwnedFd>, file: Option<File>) -> String {
Andrew Walbranb27681f2022-02-23 15:11:52 +00001497 if let Some(file) = file {
1498 format!("type=file,path={}", add_preserved_fd(preserved_fds, file))
1499 } else {
1500 "type=sink".to_string()
1501 }
1502}
1503
1504/// Creates a new pipe with the `O_CLOEXEC` flag set, and returns the read side and write side.
1505fn create_pipe() -> Result<(File, File), Error> {
Frederick Maylefbbcfcd2024-04-08 16:31:54 -07001506 let (read_fd, write_fd) = pipe2(OFlag::O_CLOEXEC)?;
1507 Ok((read_fd.into(), write_fd.into()))
Andrew Walbranb27681f2022-02-23 15:11:52 +00001508}
Frederick Maylee0ed2352024-09-12 16:16:00 -07001509
1510/// Creates and binds a unix seqpacket listening socket to be passed as crosvm's `--socket`
1511/// argument. See `UnixSeqpacketListener::bind` in crosvm's code for reference.
1512fn create_crosvm_control_listener(crosvm_control_socket_path: &Path) -> Result<OwnedFd> {
1513 use nix::sys::socket;
1514 let fd = socket::socket(
1515 socket::AddressFamily::Unix,
1516 socket::SockType::SeqPacket,
1517 socket::SockFlag::empty(),
1518 None,
1519 )
1520 .context("socket failed")?;
1521 socket::bind(fd.as_raw_fd(), &socket::UnixAddr::new(crosvm_control_socket_path)?)
1522 .context("bind failed")?;
1523 // The exact backlog size isn't imporant. crosvm uses 128 internally. We use 127 here
1524 // because of a `nix` bug.
1525 socket::listen(&fd, socket::Backlog::new(127).unwrap()).context("listen failed")?;
1526 Ok(fd)
1527}
Frederick Maylef375ef12024-10-21 16:07:21 -07001528
1529fn path_to_cstring(path: &Path) -> CString {
1530 if let Some(s) = path.to_str() {
1531 if let Ok(s) = CString::new(s) {
1532 return s;
1533 }
1534 }
1535 // The path contains invalid utf8 or a null, which should never happen.
1536 panic!("bad path: {path:?}");
1537}