Address review comments.

Author: nitin966

src/daemon.rs

@@ -407,8 +407,8 @@ pub fn start_daemon_background(
         cmd.args(["--max-storage", &max.to_string()]);
     }
     cmd.stdin(Stdio::null())
-        .stdout(Stdio::inherit())
-        .stderr(Stdio::inherit())
+        .stdout(Stdio::null())
+        .stderr(Stdio::null())
         .spawn()?;
 
     // Wait for daemon to be ready

src/fs.rs

@@ -1549,7 +1549,6 @@ impl Filesystem for BranchFs {
         _out_size: u32,
         reply: ReplyIoctl,
     ) {
-        log::info!("ioctl: ino={}, cmd={:#x}", ino, cmd);
         // Resolve ino to the branch name this ctl fd refers to.
         let branch_name = if ino == CTL_INO {
             self.get_branch_name()

src/fs_helpers.rs

@@ -124,10 +124,7 @@ impl BranchFs {
 
     /// Return a synthetic ctl-file FileAttr.
     pub(crate) fn ctl_file_attr(&self, ino: u64) -> FileAttr {
-        // We set size to 0 here because if we report >0, the macOS kernel performs
-        // a Read-Modify-Write cycle on writes to the control file. This intercepts
-        // and corrupts commands like "commit" or "abort".
-        let size = 0;
+        let size = crate::platform::ctl_file_size(ino);
         FileAttr {
             ino,
             size,

src/platform/linux.rs

@@ -47,6 +47,11 @@ pub fn check_rename_noreplace(flags: u32) -> bool {
     flags & libc::RENAME_NOREPLACE != 0
 }
 
+pub fn ctl_file_size(ino: u64) -> u64 {
+    // On Linux we report 256 for CTL_INO to ensure the kernel issues read() calls.
+    if ino == crate::inode::CTL_INO { 256 } else { 0 }
+}
+
 pub struct PassthroughState {
     pub next_fh: AtomicU64,
     pub backing_ids: HashMap<u64, BackingId>,

src/platform/macos.rs

@@ -34,7 +34,14 @@ pub fn check_rename_flags(_flags: u32) -> Result<(), i32> {
 }
 
 pub fn check_rename_noreplace(_flags: u32) -> bool {
-    false // macOS does not have RENAME_NOREPLACE
+    // RENAME_NOREPLACE is Linux-only, macOS ignores this flag
+    false
+}
+
+pub fn ctl_file_size(_ino: u64) -> u64 {
+    // We set size to 0 on macOS because if we report >0, the kernel performs
+    // a Read-Modify-Write cycle on writes to the control file.
+    0
 }
 
 pub struct PassthroughState {}