refactor(block): refactor of VirtIOBlock

Simplified `DiskProperties` implementation.
Removed a lot of getters from `VirtIOBlock`
Removed `CacheTypeState` because it is same as `CacheType`

Signed-off-by: Egor Lazarchuk <[email protected]>

Author: ShadowCurse

src/vmm/src/builder.rs

@@ -889,19 +889,19 @@ fn attach_block_devices<'a, I: Iterator<Item = &'a BlockDeviceType> + Debug>(
             BlockDeviceType::VirtioBlock(block) => {
                 let id = {
                     let locked = block.lock().expect("Poisoned lock");
-                    if locked.is_root_device() {
-                        cmdline.insert_str(if let Some(partuuid) = locked.partuuid() {
+                    if locked.root_device {
+                        cmdline.insert_str(if let Some(ref partuuid) = locked.partuuid {
                             format!("root=PARTUUID={}", partuuid)
                         } else {
                             // If no PARTUUID was specified for the root device, try with the
                             // /dev/vda.
                             "root=/dev/vda".to_string()
                         })?;
 
-                        let flags = if locked.is_read_only() { "ro" } else { "rw" };
+                        let flags = if locked.read_only { "ro" } else { "rw" };
                         cmdline.insert_str(flags)?;
                     }
-                    locked.id().clone()
+                    locked.id.clone()
                 };
                 // The device mutex mustn't be locked here otherwise it will deadlock.
                 attach_virtio_device(event_manager, vmm, id, block.clone(), cmdline)?;