rust-vmm / vhost

Apache License 2.0
126 stars 64 forks source link

vhost_user: Remove support for FS_* requests #246

Closed aesteve-rh closed 1 week ago

aesteve-rh commented 1 week ago

Summary of the PR

Since these non-standard backend request message types are confirmed not to be used anymore, let's just remove them.

pub enum BackendReq {
    ...
    /// Virtio-fs draft: map file content into the window.
    FS_MAP = 100,
    /// Virtio-fs draft: unmap file content from the window.
    FS_UNMAP = 1001,
    /// Virtio-fs draft: sync file content.
    FS_SYNC = 1002,
    /// Virtio-fs draft: perform a read/write from an fd directly to GPA.
    FS_IO = 1003,
    ...
}

Closes: #213

Requirements

Before submitting your PR, please make sure you addressed the following requirements:

aesteve-rh commented 1 week ago

The best way to increase coverage, removing old code :)

germag commented 1 week ago

LGTM