Skip to content

Commit 43080c1

Browse files
committed
Fix various warnings
1 parent d9e26b8 commit 43080c1

File tree

9 files changed

+17
-16
lines changed

9 files changed

+17
-16
lines changed

mythril/src/kmain.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ fn default_vm(
4242
physdev::com::Uart8250::new(0x3f8)
4343
.expect("Failed to create UART"),
4444
),
45+
ps2_keyboard: None,
4546
}
4647
};
4748

mythril/src/virtdev/dma.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
use crate::error::Result;
2-
use crate::virtdev::{DeviceEvent, DeviceRegion, EmulatedDevice, Event, Port};
2+
use crate::virtdev::{DeviceRegion, EmulatedDevice, Event, Port};
33
use alloc::sync::Arc;
44
use alloc::vec::Vec;
55
use spin::RwLock;
@@ -43,7 +43,7 @@ impl EmulatedDevice for Dma8237 {
4343
]
4444
}
4545

46-
fn on_event(&mut self, event: Event) -> Result<()> {
46+
fn on_event(&mut self, _event: Event) -> Result<()> {
4747
Ok(())
4848
}
4949
}

mythril/src/virtdev/ignore.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
use crate::error::Result;
2-
use crate::virtdev::{DeviceEvent, DeviceRegion, EmulatedDevice, Event, Port};
2+
use crate::virtdev::{DeviceRegion, EmulatedDevice, Event};
33
use alloc::sync::Arc;
44
use alloc::vec::Vec;
55
use spin::RwLock;
@@ -33,7 +33,7 @@ impl EmulatedDevice for IgnoredDevice {
3333
]
3434
}
3535

36-
fn on_event(&mut self, event: Event) -> Result<()> {
36+
fn on_event(&mut self, _event: Event) -> Result<()> {
3737
Ok(())
3838
}
3939
}

mythril/src/virtdev/keyboard.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ impl EmulatedDevice for Keyboard8042 {
2626

2727
fn on_event(&mut self, event: Event) -> Result<()> {
2828
match event.kind {
29-
DeviceEvent::PortRead((port, mut val)) => {
29+
DeviceEvent::PortRead((_port, mut val)) => {
3030
//FIXME: For now just return 0xff for everything
3131
val.copy_from_u32(0xff);
3232
}

mythril/src/virtdev/lapic.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
use crate::error::Result;
22
use crate::memory::GuestPhysAddr;
3-
use crate::virtdev::{DeviceEvent, DeviceRegion, EmulatedDevice, Event, Port};
3+
use crate::virtdev::{DeviceRegion, EmulatedDevice, Event};
44
use alloc::sync::Arc;
55
use alloc::vec::Vec;
66
use spin::RwLock;
@@ -31,7 +31,7 @@ impl EmulatedDevice for LocalApic {
3131
]
3232
}
3333

34-
fn on_event(&mut self, event: Event) -> Result<()> {
34+
fn on_event(&mut self, _event: Event) -> Result<()> {
3535
Ok(())
3636
}
3737
}

mythril/src/virtdev/pos.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ impl EmulatedDevice for ProgrammableOptionSelect {
3232

3333
fn on_event(&mut self, event: Event) -> Result<()> {
3434
match event.kind {
35-
DeviceEvent::PortRead((port, mut val)) => {
35+
DeviceEvent::PortRead((_port, mut val)) => {
3636
val.copy_from_u32(0);
3737
}
3838
_ => (),

mythril/src/vm.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,8 @@ use crate::memory::{
55
Raw4kPage,
66
};
77
use crate::physdev;
8-
use crate::vcpu;
98
use crate::virtdev::{
10-
DeviceEvent, DeviceInteraction, DeviceMap, Event, MemReadRequest,
11-
MemWriteRequest, Port, PortReadRequest, PortWriteRequest,
12-
ResponseEventArray,
9+
DeviceEvent, DeviceInteraction, DeviceMap, Event, ResponseEventArray,
1310
};
1411
use alloc::boxed::Box;
1512
use alloc::collections::BTreeMap;
@@ -25,6 +22,9 @@ pub static mut VM_MAP: Option<BTreeMap<usize, Arc<RwLock<VirtualMachine>>>> =
2522
pub struct PhysicalDeviceConfig {
2623
/// The physical serial connection for this VM (if any).
2724
pub serial: Option<physdev::com::Uart8250>,
25+
26+
/// The physical ps2 keyboard connection for this VM (if any).
27+
pub ps2_keyboard: Option<physdev::keyboard::Ps2Controller>,
2828
}
2929

3030
/// A configuration for a `VirtualMachine`

mythril/src/vmcs.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -402,7 +402,7 @@ impl Vmcs {
402402

403403
pub struct ActiveVmcs {
404404
vmcs: Vmcs,
405-
vmx: vmx::Vmx,
405+
pub vmx: vmx::Vmx,
406406
}
407407

408408
impl ActiveVmcs {
@@ -549,13 +549,13 @@ impl fmt::Display for ActiveVmcs {
549549

550550
pub struct TemporaryActiveVmcs<'a> {
551551
vmcs: &'a mut Vmcs,
552-
_vmx: &'a mut vmx::Vmx,
552+
pub vmx: &'a mut vmx::Vmx,
553553
}
554554

555555
impl<'a> TemporaryActiveVmcs<'a> {
556556
fn new(vmcs: &'a mut Vmcs, vmx: &'a mut vmx::Vmx) -> Result<Self> {
557557
vmcs_activate(vmcs, vmx)?;
558-
Ok(Self { vmcs, _vmx: vmx })
558+
Ok(Self { vmcs, vmx: vmx })
559559
}
560560

561561
pub fn read_field(&mut self, field: VmcsField) -> Result<u64> {

mythril/src/vmx.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
use crate::error::{self, Error, Result};
2-
use crate::memory::Raw4kPage;
2+
use crate::memory::{GuestVirtAddr, Raw4kPage};
33
use alloc::boxed::Box;
44
use raw_cpuid::CpuId;
55
use x86::msr;

0 commit comments

Comments
 (0)