refactor(iov): remove SmallVec

As we don't create IoVecBuffer(Mut) types
at runtime, but reuse existing ones in both
virtio-net and virtio-vsock, we don't need to
use SmallVec type anymore.
With this we remove the type alias.

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

Author: ShadowCurse

Cargo.lock

@@ -34,7 +34,6 @@ semver = { version = "1.0.23", features = ["serde"] }
 serde = { version = "1.0.210", features = ["derive", "rc"] }
 serde_json = "1.0.128"
 slab = "0.4.7"
-smallvec = "1.11.2"
 thiserror = "1.0.64"
 timerfd = "1.5.0"
 userfaultfd = "0.8.1"

src/vmm/Cargo.toml

@@ -4,7 +4,6 @@
 use std::io::ErrorKind;
 
 use libc::{c_void, iovec, size_t};
-use smallvec::SmallVec;
 use vm_memory::bitmap::Bitmap;
 use vm_memory::{
     GuestMemory, GuestMemoryError, ReadVolatile, VolatileMemoryError, VolatileSlice, WriteVolatile,
@@ -25,14 +24,6 @@ pub enum IoVecError {
     GuestMemory(#[from] GuestMemoryError),
 }
 
-// Using SmallVec in the kani proofs causes kani to use unbounded amounts of memory
-// during post-processing, and then crash.
-// TODO: remove new-type once kani performance regression are resolved
-#[cfg(kani)]
-type IoVecVec = Vec<iovec>;
-#[cfg(not(kani))]
-type IoVecVec = SmallVec<[iovec; 4]>;
-
 /// This is essentially a wrapper of a `Vec<libc::iovec>` which can be passed to `libc::writev`.
 ///
 /// It describes a buffer passed to us by the guest that is scattered across multiple
@@ -41,7 +32,7 @@ type IoVecVec = SmallVec<[iovec; 4]>;
 #[derive(Debug, Default)]
 pub struct IoVecBuffer {
     // container of the memory regions included in this IO vector
-    vecs: IoVecVec,
+    vecs: Vec<iovec>,
     // Total length of the IoVecBuffer
     len: u32,
 }
@@ -222,7 +213,7 @@ impl IoVecBuffer {
 #[derive(Debug, Default)]
 pub struct IoVecBufferMut {
     // container of the memory regions included in this IO vector
-    vecs: IoVecVec,
+    vecs: Vec<iovec>,
     // Total length of the IoVecBufferMut
     len: u32,
 }
@@ -406,8 +397,7 @@ mod tests {
                 vecs: vec![iovec {
                     iov_base: buf.as_ptr() as *mut c_void,
                     iov_len: buf.len(),
-                }]
-                .into(),
+                }],
                 len: buf.len().try_into().unwrap(),
             }
         }
@@ -437,8 +427,7 @@ mod tests {
                 vecs: vec![iovec {
                     iov_base: buf.as_mut_ptr().cast::<c_void>(),
                     iov_len: buf.len(),
-                }]
-                .into(),
+                }],
                 len: buf.len().try_into().unwrap(),
             }
         }
@@ -690,7 +679,7 @@ mod verification {
     use vm_memory::bitmap::BitmapSlice;
     use vm_memory::VolatileSlice;
 
-    use super::{IoVecBuffer, IoVecBufferMut, IoVecVec};
+    use super::{IoVecBuffer, IoVecBufferMut};
 
     // Maximum memory size to use for our buffers. For the time being 1KB.
     const GUEST_MEMORY_SIZE: usize = 1 << 10;
@@ -702,7 +691,7 @@ mod verification {
     // >= 1.
     const MAX_DESC_LENGTH: usize = 4;
 
-    fn create_iovecs(mem: *mut u8, size: usize, nr_descs: usize) -> (IoVecVec, u32) {
+    fn create_iovecs(mem: *mut u8, size: usize, nr_descs: usize) -> (Vec<iovec>, u32) {
         let mut vecs: Vec<iovec> = Vec::with_capacity(nr_descs);
         let mut len = 0u32;
         for _ in 0..nr_descs {