]> www.infradead.org Git - users/hch/xfsprogs.git/commitdiff
xfs: record health problems with the metadata directory
authorDarrick J. Wong <djwong@kernel.org>
Wed, 29 May 2024 04:11:03 +0000 (21:11 -0700)
committerDarrick J. Wong <djwong@kernel.org>
Tue, 30 Jul 2024 00:13:16 +0000 (17:13 -0700)
Make a report to the health monitoring subsystem any time we encounter
something in the metadata directory tree that looks like corruption.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
libxfs/xfs_fs.h
libxfs/xfs_health.h
libxfs/xfs_metadir.c

index cb7563d330d0ff6f3570b3c97aa9560e2eda833c..6f5aebaf47ac814e2f43de624b641e9df22c6e67 100644 (file)
@@ -197,6 +197,7 @@ struct xfs_fsop_geom {
 #define XFS_FSOP_GEOM_SICK_RT_SUMMARY  (1 << 5)  /* realtime summary */
 #define XFS_FSOP_GEOM_SICK_QUOTACHECK  (1 << 6)  /* quota counts */
 #define XFS_FSOP_GEOM_SICK_NLINKS      (1 << 7)  /* inode link counts */
+#define XFS_FSOP_GEOM_SICK_METADIR     (1 << 8)  /* metadata directory */
 
 /* Output for XFS_FS_COUNTS */
 typedef struct xfs_fsop_counts {
index b0edb4288e59297ea5a6a54c67da1d70ea471d0c..0ded0cd93ce633775e227b2d5a12fdbfc306038b 100644 (file)
@@ -60,6 +60,7 @@ struct xfs_da_args;
 #define XFS_SICK_FS_PQUOTA     (1 << 3)  /* project quota */
 #define XFS_SICK_FS_QUOTACHECK (1 << 4)  /* quota counts */
 #define XFS_SICK_FS_NLINKS     (1 << 5)  /* inode link counts */
+#define XFS_SICK_FS_METADIR    (1 << 6)  /* metadata directory tree */
 
 /* Observable health issues for realtime volume metadata. */
 #define XFS_SICK_RT_BITMAP     (1 << 0)  /* realtime bitmap */
@@ -103,7 +104,8 @@ struct xfs_da_args;
                                 XFS_SICK_FS_GQUOTA | \
                                 XFS_SICK_FS_PQUOTA | \
                                 XFS_SICK_FS_QUOTACHECK | \
-                                XFS_SICK_FS_NLINKS)
+                                XFS_SICK_FS_NLINKS | \
+                                XFS_SICK_FS_METADIR)
 
 #define XFS_SICK_RT_PRIMARY    (XFS_SICK_RT_BITMAP | \
                                 XFS_SICK_RT_SUMMARY)
index 80e1d856011771588a3a20b99aad2671eaa88a3c..f099789c06bafb73608eae88c5ac52d7a5143390 100644 (file)
@@ -27,6 +27,7 @@
 #include "xfs_dir2.h"
 #include "xfs_dir2_priv.h"
 #include "xfs_parent.h"
+#include "xfs_health.h"
 
 /*
  * Metadata Directory Tree
@@ -93,8 +94,10 @@ xfs_metadir_lookup(
        };
        int                     error;
 
-       if (!S_ISDIR(VFS_I(dp)->i_mode))
+       if (!S_ISDIR(VFS_I(dp)->i_mode)) {
+               xfs_fs_mark_sick(mp, XFS_SICK_FS_METADIR);
                return -EFSCORRUPTED;
+       }
        if (xfs_is_shutdown(mp))
                return -EIO;
 
@@ -102,10 +105,14 @@ xfs_metadir_lookup(
        if (error)
                return error;
 
-       if (!xfs_verify_ino(mp, args.inumber))
+       if (!xfs_verify_ino(mp, args.inumber)) {
+               xfs_fs_mark_sick(mp, XFS_SICK_FS_METADIR);
                return -EFSCORRUPTED;
-       if (xname->type != XFS_DIR3_FT_UNKNOWN && xname->type != args.filetype)
+       }
+       if (xname->type != XFS_DIR3_FT_UNKNOWN && xname->type != args.filetype) {
+               xfs_fs_mark_sick(mp, XFS_SICK_FS_METADIR);
                return -EFSCORRUPTED;
+       }
 
        trace_xfs_metadir_lookup(dp, xname, args.inumber);
        *ino = args.inumber;