runtime-rs: fix ci complains

vfio commits introduce quite a lot change in runtime-rs, this commit is
for all the changes related to ci, including compilation errors and so on.

Signed-off-by: Chao Wu <chaowu@linux.alibaba.com>

src/dragonball/src/api/v1/vmm_action.rs

@@ -38,7 +38,7 @@ pub use crate::device_manager::fs_dev_mgr::{
 #[cfg(feature = "virtio-mem")]
 pub use crate::device_manager::mem_dev_mgr::{MemDeviceConfigInfo, MemDeviceError};
 #[cfg(feature = "host-device")]
-use crate::device_manager::vfio_dev_mgr::{HostDeviceConfig, VfioDeviceError, VfioDeviceHostInfo};
+use crate::device_manager::vfio_dev_mgr::{HostDeviceConfig, VfioDeviceError};
 #[cfg(feature = "vhost-net")]
 pub use crate::device_manager::vhost_net_dev_mgr::{
     VhostNetDeviceConfigInfo, VhostNetDeviceError, VhostNetDeviceMgr,
@@ -871,7 +871,7 @@ impl VmmService {
             .vfio_manager
             .lock()
             .unwrap()
-            .insert_device(&mut ctx, config.into())
+            .insert_device(&mut ctx, config)
             .map_err(VmmActionError::HostDeviceConfig)?;
         Ok(VmmData::Empty)
     }

src/dragonball/src/dbs_device/src/resources.rs

@@ -403,7 +403,7 @@ pub(crate) mod tests {
         resource.append(entry.clone());
         assert_eq!(entry, resource[6]);
 
-        let entry = Resource::MacAddresss(MAC_ADDRESS.to_string());
+        let entry = Resource::MacAddress(MAC_ADDRESS.to_string());
         resource.append(entry.clone());
         assert_eq!(entry, resource[7]);
 

src/dragonball/src/dbs_pci/src/vfio.rs

@@ -229,7 +229,7 @@ impl Interrupt {
 
     fn get_irq_pin(&self) -> u32 {
         if let Some(legacy_irq) = self.legacy_irq {
-            (PciInterruptPin::IntA as u32) << 8 | self.legacy_irq.unwrap()
+            (PciInterruptPin::IntA as u32) << 8 | legacy_irq
         } else {
             0
         }

src/dragonball/src/device_manager/vfio_dev_mgr/mod.rs

@@ -28,7 +28,7 @@ use dbs_pci::ECAM_SPACE_LENGTH;
 use dbs_pci::{VfioPciDevice, VENDOR_NVIDIA};
 use dbs_upcall::{DevMgrResponse, UpcallClientResponse};
 use kvm_ioctls::{DeviceFd, VmFd};
-use log::error;
+use log::{debug, error};
 use serde_derive::{Deserialize, Serialize};
 use vfio_ioctls::{VfioContainer, VfioDevice};
 use vm_memory::{
@@ -116,6 +116,10 @@ pub enum VfioDeviceError {
     #[error("failure while allocate device resource: {0:?}")]
     AllocateDeviceResource(#[source] ResourceError),
 
+    /// Failed to free device resource
+    #[error("failure while freeing device resource: {0:?}")]
+    FreeDeviceResource(#[source] ResourceError),
+
     /// Vfio container not found
     #[error("vfio container not found")]
     VfioContainerNotFound,
@@ -364,8 +368,10 @@ impl VfioDeviceMgr {
                             error!("send upcall result failed, due to {:?}!", e);
                         }
                     }
-                    #[cfg(test)]
-                    UpcallClientResponse::FakeResponse => {}
+                    #[allow(unreachable_patterns)]
+                    _ => {
+                        debug!("this arm should only be triggered under test");
+                    }
                 }));
             ctx.remove_hotplug_pci_device(dev, callback)
                 .map_err(VfioDeviceError::VfioDeviceMgr)?
@@ -678,7 +684,9 @@ impl VfioDeviceMgr {
             resources
         };
 
-        ctx.res_manager.free_device_resources(&filtered_resources);
+        ctx.res_manager
+            .free_device_resources(&filtered_resources)
+            .map_err(VfioDeviceError::FreeDeviceResource)?;
 
         vfio_pci_device
             .clear_device()

src/dragonball/src/test_utils.rs

@@ -39,6 +39,7 @@ pub mod tests {
                 sockets: 1,
             },
             vpmu_feature: 0,
+            pci_hotplug_enabled: false,
         };
         vm.set_vm_config(vm_config);
         vm.init_guest_memory().unwrap();

src/dragonball/src/vcpu/vcpu_manager.rs

@@ -1133,6 +1133,7 @@ mod tests {
                 sockets: 1,
             },
             vpmu_feature: 0,
+            pci_hotplug_enabled: false,
         };
         vm.set_vm_config(vm_config);
         vm.init_guest_memory().unwrap();
@@ -1181,6 +1182,7 @@ mod tests {
                 sockets: 1,
             },
             vpmu_feature: 0,
+            pci_hotplug_enabled: false,
         };
         vm.set_vm_config(vm_config.clone());
         vm.init_guest_memory().unwrap();

src/dragonball/src/vm/mod.rs

@@ -186,6 +186,7 @@ pub struct Vm {
     shared_info: Arc<RwLock<InstanceInfo>>,
 
     address_space: AddressSpaceMgr,
+    /// device manager for Dragonball
     pub device_manager: DeviceManager,
     dmesg_fifo: Option<Box<dyn io::Write + Send>>,
     kernel_config: Option<KernelConfigInfo>,

src/libs/kata-types/src/annotations/mod.rs

@@ -705,7 +705,7 @@ impl Annotation {
                     }
                     // Hypervisor Memory related annotations
                     KATA_ANNO_CFG_HYPERVISOR_DEFAULT_MEMORY => {
-                        match byte_unit::Byte::parse_str(value,true) {
+                        match byte_unit::Byte::parse_str(value, true) {
                             Ok(mem_bytes) => {
                                 let memory_size = mem_bytes
                                     .get_adjusted_unit(byte_unit::Unit::MiB)

src/runtime-rs/crates/hypervisor/Cargo.toml

@@ -35,7 +35,7 @@ kata-types = { path = "../../../libs/kata-types" }
 logging = { path = "../../../libs/logging" }
 shim-interface = { path = "../../../libs/shim-interface" }
 
-dragonball = { path = "../../../dragonball", features = ["atomic-guest-memory", "virtio-vsock", "hotplug", "virtio-blk", "virtio-net", "virtio-fs", "vhost-net", "dbs-upcall","virtio-mem", "virtio-balloon", "vhost-user-net"] }
+dragonball = { path = "../../../dragonball", features = ["atomic-guest-memory", "virtio-vsock", "hotplug", "virtio-blk", "virtio-net", "virtio-fs", "vhost-net", "dbs-upcall", "virtio-mem", "virtio-balloon", "vhost-user-net", "host-device"] }
 
 ch-config = { path = "ch-config", optional = true }
 tests_utils = { path = "../../tests/utils" }