Remove support for --daemonize
The 'daemonize' feature allowed a VM spawned by shell to be held by
VirtualizationService indefinitely, eg. if adb was disconnected.
The setup is not compatible with the new virtmgr/virtualizationservice
split and can be replaced with 'adb shell vm run ... &' anyway. All
tests and scripts have been migrated away from it, so we can remove it
now.
Bug: 265248057
Bug: 245727626
Test: atest -p packages/modules/Virtualization:avf-presubmit
Change-Id: Iba4fc40ffd9ddc59afaa535cf746872b1d4ba87a
diff --git a/vm/src/run.rs b/vm/src/run.rs
index b99328a..6c21dbc 100644
--- a/vm/src/run.rs
+++ b/vm/src/run.rs
@@ -49,7 +49,6 @@
storage_size: Option<u64>,
config_path: Option<String>,
payload_binary_name: Option<String>,
- daemonize: bool,
console_path: Option<&Path>,
log_path: Option<&Path>,
debug_level: DebugLevel,
@@ -145,7 +144,7 @@
numCpus: cpus.unwrap_or(1) as i32,
taskProfiles: task_profiles,
});
- run(service, &config, &payload_config_str, daemonize, console_path, log_path)
+ run(service, &config, &payload_config_str, console_path, log_path)
}
const EMPTY_PAYLOAD_APK: &str = "com.android.microdroid.empty_payload";
@@ -180,7 +179,6 @@
work_dir: Option<PathBuf>,
storage: Option<&Path>,
storage_size: Option<u64>,
- daemonize: bool,
console_path: Option<&Path>,
log_path: Option<&Path>,
debug_level: DebugLevel,
@@ -211,7 +209,6 @@
storage_size,
/* config_path= */ None,
Some(payload_binary_name.to_owned()),
- daemonize,
console_path,
log_path,
debug_level,
@@ -229,7 +226,6 @@
name: Option<String>,
service: &dyn IVirtualizationService,
config_path: &Path,
- daemonize: bool,
console_path: Option<&Path>,
log_path: Option<&Path>,
mem: Option<u32>,
@@ -255,7 +251,6 @@
service,
&VirtualMachineConfig::RawConfig(config),
&format!("{:?}", config_path),
- daemonize,
console_path,
log_path,
)
@@ -277,7 +272,6 @@
service: &dyn IVirtualizationService,
config: &VirtualMachineConfig,
payload_config: &str,
- daemonize: bool,
console_path: Option<&Path>,
log_path: Option<&Path>,
) -> Result<(), Error> {
@@ -286,8 +280,6 @@
File::create(console_path)
.with_context(|| format!("Failed to open console file {:?}", console_path))?,
)
- } else if daemonize {
- None
} else {
Some(duplicate_stdout()?)
};
@@ -296,8 +288,6 @@
File::create(log_path)
.with_context(|| format!("Failed to open log file {:?}", log_path))?,
)
- } else if daemonize {
- None
} else {
Some(duplicate_stdout()?)
};
@@ -314,17 +304,10 @@
state_to_str(vm.state()?)
);
- if daemonize {
- // Pass the VM reference back to VirtualizationService and have it hold it in the
- // background.
- service.debugHoldVmRef(&vm.vm).context("Failed to pass VM to VirtualizationService")?;
- } else {
- // Wait until the VM or VirtualizationService dies. If we just returned immediately then the
- // IVirtualMachine Binder object would be dropped and the VM would be killed.
- let death_reason = vm.wait_for_death();
- println!("VM ended: {:?}", death_reason);
- }
-
+ // Wait until the VM or VirtualizationService dies. If we just returned immediately then the
+ // IVirtualMachine Binder object would be dropped and the VM would be killed.
+ let death_reason = vm.wait_for_death();
+ println!("VM ended: {:?}", death_reason);
Ok(())
}