fuse2fs: don't record every errno in the superblock as an fs failure

fstests just blew up because somewhere in the fuse iomap code we
returned an ESTALE, which was then passed to translate_error.  That
function decided it was a Serious Error and wrote it to the superblock,
so every subsequent mount attempt failed.

I should go figure out why the iomap cache upsert operation returned
ESTALE, but that's not a sign that the *ondisk* filesystem is corrupt.
Prior to commit 71f046a788adba we wouldn't have written that to the
superblock either.

Fix this by isolating the handful of errno that usually mean corruption
problems in filesystems and writing those to the superblock; the other
errno are merely operational errors that can be passed back to the
kernel and up to userspace.

I'm not sure why e2fsck doesn't flag when s_error_count > 0.  That might
be an error on its own.

Cc: linux-ext4@vger.kernel.org # v1.47.3
Fixes: 71f046a788adba ("fuse2fs: correctly handle system errno values in __translate_error()")
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Link: https://lore.kernel.org/r/20250725155610.GP2672022@frogsfrogsfrogs
Signed-off-by: Theodore Ts'o <tytso@mit.edu>

diff --git a/misc/fuse2fs.c b/misc/fuse2fs.c
index 242bbfd221eb3aea9b9261276a03c407f5a933ee..18d8f426a5eb43e3e20913903fa6156470173c98 100644 (file)
--- a/misc/fuse2fs.c
+++ b/misc/fuse2fs.c
@@ -4969,9 +4969,23 @@ static int __translate_error(ext2_filsys fs, ext2_ino_t ino, errcode_t err,
                is_err = 1;
                ret = -EUCLEAN;
                break;
-       default:
+       case EIO:
+#ifdef EILSEQ
+       case EILSEQ:
+#endif
+       case EUCLEAN:
+               /* these errnos usually denote corruption or persistence fail */
                is_err = 1;
-               ret = (err < 256) ? -err : -EIO;
+               ret = -err;
+               break;
+       default:
+               if (err < 256) {
+                       /* other errno are usually operational errors */
+                       ret = -err;
+               } else {
+                       is_err = 1;
+                       ret = -EIO;
+               }
                break;
        }