lib/common/memory/src/checkfs.rs (1)

169-172: Create the directory before calling statfs to prevent false-negatives

Returning Unknown("Path does not exist") puts perfectly valid first-run deployments into the “unknown / non-POSIX” bucket and triggers the costly mmap probe (plus a scary log line).
Creating the directory (or propagating the error if creation fails) avoids that extra I/O, aligns with the earlier review suggestion, and removes noise from the logs.

 pub fn check_fs_info(path: impl AsRef<Path>) -> FsCheckResult {
-    if !path.as_ref().exists() {
-        return FsCheckResult::Unknown("Path does not exist".to_string());
-    }
+    if let Err(e) = create_dir_all(path.as_ref()) {
+        return FsCheckResult::Bad(format!("Cannot create storage directory: {e}"));
+    }

lib/common/memory/src/checkfs.rs (3)

87-87: Typo: hfs+, not htf+

The string "htf+" will never match a real filesystem name; it should be "hfs+".

-            "hfs" | "htf+" => Self::Hfs,
+            "hfs" | "hfs+" => Self::Hfs,

---

203-210: Drop the premature read-only mmap to save one mapping + syscall

mmap_seq is created before the file is populated and is never used; it is immediately dropped.
Opening it afterwards (or omitting it entirely) removes an unnecessary open + mmap round-trip.

-    let mmap_seq = open_read_mmap(
-        &magic_file_path,
-        AdviceSetting::Advice(Advice::Sequential),
-        false,
-    )?;
+    // Delay read-only check until after the write/flush cycle

(If the sequential‐advice read is important later, open it right before the final comparison.)

---

55-72: Minor clarity: use explicit pattern alternatives instead of bitwise-OR comments

The magic-number match arms already leverage Rust’s | pattern alternation.
Consider splitting multi-value arms onto separate lines for readability and to avoid confusing the | with a bitwise OR for newcomers.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro