diff --git a/hypervisor/src/kvm/aarch64/gic/dist_regs.rs b/hypervisor/src/kvm/aarch64/gic/dist_regs.rs index 8c4809476..9135ad003 100644 --- a/hypervisor/src/kvm/aarch64/gic/dist_regs.rs +++ b/hypervisor/src/kvm/aarch64/gic/dist_regs.rs @@ -103,9 +103,8 @@ fn dist_attr_get(gic: &DeviceFd, offset: u32) -> Result { flags: 0, }; - // get_device_attr should be marked as unsafe, and will be in future. // SAFETY: gic_dist_attr.addr is safe to write to. - gic.get_device_attr(&mut gic_dist_attr).map_err(|e| { + unsafe { gic.get_device_attr(&mut gic_dist_attr) }.map_err(|e| { Error::GetDeviceAttribute(HypervisorDeviceError::GetDeviceAttribute(e.into())) })?; @@ -131,9 +130,8 @@ fn get_interrupts_num(gic: &DeviceFd) -> Result { addr: &mut num_irq as *mut u32 as u64, flags: 0, }; - // get_device_attr should be marked as unsafe, and will be in future. // SAFETY: nr_irqs_attr.addr is safe to write to. - gic.get_device_attr(&mut nr_irqs_attr).map_err(|e| { + unsafe { gic.get_device_attr(&mut nr_irqs_attr) }.map_err(|e| { Error::GetDeviceAttribute(HypervisorDeviceError::GetDeviceAttribute(e.into())) })?; Ok(num_irq) diff --git a/hypervisor/src/kvm/aarch64/gic/icc_regs.rs b/hypervisor/src/kvm/aarch64/gic/icc_regs.rs index 24fba1727..f99358184 100644 --- a/hypervisor/src/kvm/aarch64/gic/icc_regs.rs +++ b/hypervisor/src/kvm/aarch64/gic/icc_regs.rs @@ -105,9 +105,8 @@ fn icc_attr_get(gic: &DeviceFd, offset: u64, typer: u64) -> Result { flags: 0, }; - // get_device_attr should be marked as unsafe, and will be in future. // SAFETY: gic_icc_attr.addr is safe to write to. - gic.get_device_attr(&mut gic_icc_attr).map_err(|e| { + unsafe { gic.get_device_attr(&mut gic_icc_attr) }.map_err(|e| { Error::GetDeviceAttribute(HypervisorDeviceError::GetDeviceAttribute(e.into())) })?; diff --git a/hypervisor/src/kvm/aarch64/gic/mod.rs b/hypervisor/src/kvm/aarch64/gic/mod.rs index eead0da7d..9b8b8668e 100644 --- a/hypervisor/src/kvm/aarch64/gic/mod.rs +++ b/hypervisor/src/kvm/aarch64/gic/mod.rs @@ -49,13 +49,10 @@ fn gicv3_its_attr_get(its_device: &DeviceFd, group: u32, attr: u32) -> Result Result { flags: 0, }; - // get_device_attr should be marked as unsafe, and will be in future. // SAFETY: gic_redist_attr.addr is safe to write to. - gic.get_device_attr(&mut gic_redist_attr).map_err(|e| { + unsafe { gic.get_device_attr(&mut gic_redist_attr) }.map_err(|e| { Error::GetDeviceAttribute(HypervisorDeviceError::GetDeviceAttribute(e.into())) })?;