chore: Remove unused VsockError::BufDescMissing

With the vsock code no longer asserting that every vsock packet has two
packets, this error variant was no longer constructed. It's only use was
in a unittest elsewhere, that tested that a vsock error was properly
propagated (however, this test does not require this specific error, so
we can reuse a different variant as the dummy value).

Signed-off-by: Patrick Roy <[email protected]>

Author: roypat

src/vmm/src/devices/virtio/vsock/mod.rs

@@ -110,8 +110,6 @@ mod defs {
 pub enum VsockError {
     /// The vsock data/buffer virtio descriptor length is smaller than expected.
     BufDescTooSmall,
-    /// The vsock data/buffer virtio descriptor is expected, but missing.
-    BufDescMissing,
     /// Empty queue
     EmptyQueue,
     /// EventFd error: {0}

src/vmm/src/rpc_interface.rs

@@ -1012,7 +1012,7 @@ mod tests {
         pub fn set_vsock_device(&mut self, _: VsockDeviceConfig) -> Result<(), VsockConfigError> {
             if self.force_errors {
                 return Err(VsockConfigError::CreateVsockDevice(
-                    VsockError::BufDescMissing,
+                    VsockError::GuestMemoryBounds,
                 ));
             }
             self.vsock_set = true;
@@ -1471,7 +1471,7 @@ mod tests {
         check_preboot_request_err(
             req,
             VmmActionError::VsockConfig(VsockConfigError::CreateVsockDevice(
-                VsockError::BufDescMissing,
+                VsockError::GuestMemoryBounds,
             )),
         );
     }