External/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2024-07-02 16:02:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

vmm: Handle the missing VM error

Browse Source 
When trying to boot or shut a VM down, return an error if the VM was not
previously created.

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
This commit is contained in:
Samuel Ortiz 2019-09-30 16:24:15 +02:00
parent 7e0cb078ed
commit b70344158b
2 changed files with 26 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
vmm/src/api/mod.rs
View File

@ -58,6 +58,12 @@ pub enum ApiError {
/// The VM config is missing.
VmMissingConfig,
/// The VM is not booted.
VmNotBooted,
/// The VM is not created.
VmNotCreated,
/// The VM could not shutdown.
VmShutdown(VmError),

30
vmm/src/lib.rs
View File

@ -88,6 +88,9 @@ pub enum Error {
/// Cannot boot a VM
VmBoot(VmError),
/// The Vm is not created
VmNotCreated,
/// Cannot shut a VM down
VmShutdown(VmError),
@ -279,6 +282,8 @@ impl Vmm {
// Then we start the new VM.
if let Some(ref mut vm) = self.vm {
vm.boot().map_err(Error::VmBoot)?;
} else {
return Err(Error::VmNotCreated);
}
Ok(())
@ -394,28 +399,33 @@ impl Vmm {
}
// Now let's boot it.
if let Some(ref mut vm) = self.vm {
let response = match vm.boot() {
let response = if let Some(ref mut vm) = self.vm {
match vm.boot() {
Ok(_) => Ok(ApiResponsePayload::Empty),
Err(e) => Err(ApiError::VmBoot(e)),
};
}
} else {
Err(ApiError::VmNotCreated)
};
sender.send(response).map_err(Error::ApiResponseSend)?;
}
sender.send(response).map_err(Error::ApiResponseSend)?;
}
ApiRequest::VmShutdown(sender) => {
if let Some(ref mut vm) = self.vm {
let response = match vm.shutdown() {
let response = if let Some(ref mut vm) = self.vm {
match vm.shutdown() {
Ok(_) => Ok(ApiResponsePayload::Empty),
Err(e) => Err(ApiError::VmShutdown(e)),
};
}
} else {
Err(ApiError::VmNotBooted)
};
sender.send(response).map_err(Error::ApiResponseSend)?;
}
sender.send(response).map_err(Error::ApiResponseSend)?;
}
ApiRequest::VmReboot(sender) => {
let response = match self.vm_reboot() {
Ok(_) => Ok(ApiResponsePayload::Empty),
Err(Error::VmNotCreated) => Err(ApiError::VmNotBooted),
Err(_) => Err(ApiError::VmReboot),
};