vmm: simplify EntryPoint

EntryPoint had an optional entry_addr, but there is no usage of this
struct that makes it necessary that the address is optional.

Remove the Option to avoid being able to express things that are not
useful.

Signed-off-by: Julian Stecklina <julian.stecklina@cyberus-technology.de>
This commit is contained in:
Julian Stecklina 2023-09-08 12:02:28 +02:00 committed by Rob Bradford
parent 83f3b34405
commit 0d9749282a
2 changed files with 6 additions and 10 deletions

View File

@ -60,7 +60,7 @@ pub const _NSIG: i32 = 65;
/// is to be used to configure the guest initial state. /// is to be used to configure the guest initial state.
pub struct EntryPoint { pub struct EntryPoint {
/// Address in guest memory where the guest must start execution /// Address in guest memory where the guest must start execution
pub entry_addr: Option<GuestAddress>, pub entry_addr: GuestAddress,
} }
const E820_RAM: u32 = 1; const E820_RAM: u32 = 1;
@ -811,12 +811,10 @@ pub fn configure_vcpu(
regs::setup_msrs(vcpu).map_err(Error::MsrsConfiguration)?; regs::setup_msrs(vcpu).map_err(Error::MsrsConfiguration)?;
if let Some((kernel_entry_point, guest_memory)) = boot_setup { if let Some((kernel_entry_point, guest_memory)) = boot_setup {
if let Some(entry_addr) = kernel_entry_point.entry_addr { regs::setup_regs(vcpu, kernel_entry_point.entry_addr.raw_value())
// Safe to unwrap because this method is called after the VM is configured .map_err(Error::RegsConfiguration)?;
regs::setup_regs(vcpu, entry_addr.raw_value()).map_err(Error::RegsConfiguration)?; regs::setup_fpu(vcpu).map_err(Error::FpuConfiguration)?;
regs::setup_fpu(vcpu).map_err(Error::FpuConfiguration)?; regs::setup_sregs(&guest_memory.memory(), vcpu).map_err(Error::SregsConfiguration)?;
regs::setup_sregs(&guest_memory.memory(), vcpu).map_err(Error::SregsConfiguration)?;
}
} }
interrupts::set_lint(vcpu).map_err(|e| Error::LocalIntConfiguration(e.into()))?; interrupts::set_lint(vcpu).map_err(|e| Error::LocalIntConfiguration(e.into()))?;
Ok(()) Ok(())

View File

@ -980,9 +980,7 @@ impl Vm {
if let PvhEntryPresent(entry_addr) = entry_addr.pvh_boot_cap { if let PvhEntryPresent(entry_addr) = entry_addr.pvh_boot_cap {
// Use the PVH kernel entry point to boot the guest // Use the PVH kernel entry point to boot the guest
info!("Kernel loaded: entry_addr = 0x{:x}", entry_addr.0); info!("Kernel loaded: entry_addr = 0x{:x}", entry_addr.0);
Ok(EntryPoint { Ok(EntryPoint { entry_addr })
entry_addr: Some(entry_addr),
})
} else { } else {
Err(Error::KernelMissingPvhHeader) Err(Error::KernelMissingPvhHeader)
} }