commit 6ba4551b434fcadb36570e96a4621e69208b0b42
author Daniel Verkamp <dverkamp@chromium.org> Mon Mar 07 16:19:58 2022 -0800
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Mar 23 22:41:23 2022 +0000
tree d8f69a234276416ffa572e309031625d880b7cf9
parent 4440db2370d563bafd4a13f431e977182e3d545e

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 <acourbot@chromium.org>
Tested-by: kokoro <noreply+kokoro@google.com>
Commit-Queue: Daniel Verkamp <dverkamp@chromium.org>

devices/src/pci/vfio_pci.rs

diff --git a/devices/src/pci/vfio_pci.rs b/devices/src/pci/vfio_pci.rs
index b01c191..0f5a30d 100644
--- a/devices/src/pci/vfio_pci.rs
+++ b/devices/src/pci/vfio_pci.rs
@@ -918,7 +918,7 @@
                 };
                 match response {
                     VmMemoryResponse::RegisterMemory { pfn: _, slot } => {
-                        mmaps_slots.push(slot as MemSlot);
+                        mmaps_slots.push(slot);
                     }
                     _ => break,
                 }
@@ -931,7 +931,7 @@
     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");