From 6ba4551b434fcadb36570e96a4621e69208b0b42 Mon Sep 17 00:00:00 2001 From: Daniel Verkamp Date: Mon, 7 Mar 2022 16:19:58 -0800 Subject: [PATCH] devices: vfio: remove unnecessary MemSlot casts The values that were cast in these expressions were already MemSlot, so there is no need to cast back and forth between MemSlot and u32. BUG=None TEST=tools/presubmit Change-Id: I8a215bc50d10a93b21f6d2fe7e4ed7267242a50b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3508320 Reviewed-by: Alexandre Courbot Tested-by: kokoro Commit-Queue: Daniel Verkamp --- devices/src/pci/vfio_pci.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/devices/src/pci/vfio_pci.rs b/devices/src/pci/vfio_pci.rs index b01c19156c..0f5a30d41a 100644 --- a/devices/src/pci/vfio_pci.rs +++ b/devices/src/pci/vfio_pci.rs @@ -918,7 +918,7 @@ impl VfioPciDevice { }; match response { VmMemoryResponse::RegisterMemory { pfn: _, slot } => { - mmaps_slots.push(slot as MemSlot); + mmaps_slots.push(slot); } _ => break, } @@ -931,7 +931,7 @@ impl VfioPciDevice { fn remove_bar_mmap(&self, mmap_slot: &MemSlot) { if self .vm_socket_mem - .send(&VmMemoryRequest::UnregisterMemory(*mmap_slot as u32)) + .send(&VmMemoryRequest::UnregisterMemory(*mmap_slot)) .is_err() { error!("failed to send UnregisterMemory request");