file-systems: Fix ‘bcachefs fsck’ exit value logic.
authorTobias Geerinckx-Rice <me@tobias.gr>
Sun, 6 Dec 2020 10:14:48 +0000 (11:14 +0100)
committerTobias Geerinckx-Rice <me@tobias.gr>
Sun, 6 Dec 2020 15:11:43 +0000 (16:11 +0100)
Bit 1 means the target device was mounted read-only whilst checking.
This should never happen in an initrd context but is not an error.

* gnu/build/file-systems.scm (check-bcachefs-file-system): Ignore status
bits that don't signal an error.  Remove the 'reboot-required case.

gnu/build/file-systems.scm

index b762e82..ddf6117 100644 (file)
@@ -262,14 +262,16 @@ bytevector."
 
 (define (check-bcachefs-file-system device)
   "Return the health of a bcachefs file system on DEVICE."
-  (match (status:exit-val
+  (let ((ignored-bits (logior 2))       ; DEVICE was mounted read-only
+        (status
+         (status:exit-val
           (apply system* "bcachefs" "fsck" "-p" "-v"
                  ;; Make each multi-device member a separate argument.
-                 (string-split device #\:)))
-    (0 'pass)
-    (1 'errors-corrected)
-    (2 'reboot-required)
-    (_ 'fatal-error)))
+                 (string-split device #\:)))))
+    (match (logand (lognot ignored-bits) status)
+      (0 'pass)
+      (1 'errors-corrected)
+      (_ 'fatal-error))))
 
 \f
 ;;;