]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
xfs: constify feature checks
authorChristoph Hellwig <hch@lst.de>
Mon, 13 Jan 2025 04:31:20 +0000 (05:31 +0100)
committerCarlos Maiolino <cem@kernel.org>
Mon, 13 Jan 2025 13:57:08 +0000 (14:57 +0100)
They will eventually be needed to be const for zoned growfs, but even
now having such simpler helpers as const as possible is a good thing.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
fs/xfs/libxfs/xfs_rtgroup.c
fs/xfs/scrub/scrub.h
fs/xfs/xfs_mount.h

index a6468e591232c3350871fe47fff22928d1d4766a..d84d32f1b48fde0bcc7f21525bc5523d160650c6 100644 (file)
@@ -338,7 +338,7 @@ struct xfs_rtginode_ops {
        unsigned int            fmt_mask; /* all valid data fork formats */
 
        /* Does the fs have this feature? */
-       bool                    (*enabled)(struct xfs_mount *mp);
+       bool                    (*enabled)(const struct xfs_mount *mp);
 
        /* Create this rtgroup metadata inode and initialize it. */
        int                     (*create)(struct xfs_rtgroup *rtg,
index a1086f1f06d0d943fd9a5e0f85df25b5bd538bc7..a3f1abc91390351e5cdb60847cec71062aec97a5 100644 (file)
@@ -96,7 +96,7 @@ struct xchk_meta_ops {
        int             (*repair_eval)(struct xfs_scrub *sc);
 
        /* Decide if we even have this piece of metadata. */
-       bool            (*has)(struct xfs_mount *);
+       bool            (*has)(const struct xfs_mount *);
 
        /* type describing required/allowed inputs */
        enum xchk_type  type;
index 9a1516080e6361f4585a5f65ee745e2e6c019f30..fbed172d677087c070a841fdb2dcfc5064620f24 100644 (file)
@@ -357,7 +357,7 @@ typedef struct xfs_mount {
 #define XFS_FEAT_NOUUID                (1ULL << 63)    /* ignore uuid during mount */
 
 #define __XFS_HAS_FEAT(name, NAME) \
-static inline bool xfs_has_ ## name (struct xfs_mount *mp) \
+static inline bool xfs_has_ ## name (const struct xfs_mount *mp) \
 { \
        return mp->m_features & XFS_FEAT_ ## NAME; \
 }
@@ -393,25 +393,25 @@ __XFS_HAS_FEAT(large_extent_counts, NREXT64)
 __XFS_HAS_FEAT(exchange_range, EXCHANGE_RANGE)
 __XFS_HAS_FEAT(metadir, METADIR)
 
-static inline bool xfs_has_rtgroups(struct xfs_mount *mp)
+static inline bool xfs_has_rtgroups(const struct xfs_mount *mp)
 {
        /* all metadir file systems also allow rtgroups */
        return xfs_has_metadir(mp);
 }
 
-static inline bool xfs_has_rtsb(struct xfs_mount *mp)
+static inline bool xfs_has_rtsb(const struct xfs_mount *mp)
 {
        /* all rtgroups filesystems with an rt section have an rtsb */
        return xfs_has_rtgroups(mp) && xfs_has_realtime(mp);
 }
 
-static inline bool xfs_has_rtrmapbt(struct xfs_mount *mp)
+static inline bool xfs_has_rtrmapbt(const struct xfs_mount *mp)
 {
        return xfs_has_rtgroups(mp) && xfs_has_realtime(mp) &&
               xfs_has_rmapbt(mp);
 }
 
-static inline bool xfs_has_rtreflink(struct xfs_mount *mp)
+static inline bool xfs_has_rtreflink(const struct xfs_mount *mp)
 {
        return xfs_has_metadir(mp) && xfs_has_realtime(mp) &&
               xfs_has_reflink(mp);