refactor: split some logic

Signed-off-by: akitaSummer <[email protected]>

src/api/server/sync_io.rs

@@ -143,9 +143,9 @@ impl<F: FileSystem + Sync> Server<F> {
             x if x == Opcode::Rename2 as u32 => self.rename2(ctx),
             #[cfg(target_os = "linux")]
             x if x == Opcode::Lseek as u32 => self.lseek(ctx),
-            #[cfg(feature = "virtiofs")]
+            #[cfg(all(target_os = "linux", feature = "virtiofs"))]
             x if x == Opcode::SetupMapping as u32 => self.setupmapping(ctx, vu_req),
-            #[cfg(feature = "virtiofs")]
+            #[cfg(all(target_os = "linux", feature = "virtiofs"))]
             x if x == Opcode::RemoveMapping as u32 => self.removemapping(ctx, vu_req),
             // Group reqeusts don't need reply together
             x => match x {

src/passthrough/file_handle.rs

@@ -320,8 +320,10 @@ impl OpenableFileHandle {
 #[cfg(test)]
 mod tests {
     use super::*;
+    #[cfg(target_os = "macos")]
     use nix::unistd::getuid;
     use std::ffi::CString;
+    #[cfg(target_os = "macos")]
     use std::io::Read;
 
     fn generate_c_file_handle(

src/passthrough/inode_store.rs

@@ -7,7 +7,7 @@ use std::sync::Arc;
 #[cfg(target_os = "linux")]
 use super::file_handle::FileHandle;
 #[cfg(target_os = "macos")]
-use super::stat::Stat;
+use super::stat::Stat as StatExt;
 #[cfg(target_os = "linux")]
 use super::statx::StatExt;
 
@@ -25,24 +25,15 @@ pub struct InodeId {
 }
 
 impl InodeId {
-    #[cfg(target_os = "linux")]
     #[inline]
     pub(super) fn from_stat(st: &StatExt) -> Self {
         InodeId {
             ino: st.st.st_ino,
             dev: st.st.st_dev,
+            #[cfg(target_os = "linux")]
             mnt: st.mnt_id,
         }
     }
-
-    #[cfg(target_os = "macos")]
-    #[inline]
-    pub(super) fn from_stat(st: &Stat) -> Self {
-        InodeId {
-            ino: st.st.st_ino,
-            dev: st.st.st_dev,
-        }
-    }
 }
 
 #[derive(Default)]
@@ -125,7 +116,9 @@ mod test {
     use super::super::*;
     use super::*;
 
+    #[cfg(target_os = "linux")]
     use std::ffi::CStr;
+    #[cfg(target_os = "linux")]
     use std::mem::MaybeUninit;
     use std::os::unix::io::AsRawFd;
     use std::sync::atomic::Ordering;
@@ -134,6 +127,7 @@ mod test {
     #[cfg(target_os = "linux")]
     use vmm_sys_util::tempfile::TempFile;
 
+    #[cfg(target_os = "macos")]
     use stat::stat;
 
     impl PartialEq for InodeData {