xfs: skip always_cow inodes in xfs_reflink_trim_around_shared
authorChristoph Hellwig <hch@lst.de>
Sat, 14 Oct 2023 05:50:35 +0000 (07:50 +0200)
committerChristoph Hellwig <hch@lst.de>
Mon, 3 Mar 2025 15:16:44 +0000 (08:16 -0700)
xfs_reflink_trim_around_shared tries to find shared blocks in the
refcount btree.  Always_cow inodes don't have that tree, so don't
bother.

For the existing always_cow code this is a minor optimization.  For
the upcoming zoned code that can do COW without the rtreflink code it
avoids triggering a NULL pointer dereference.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
fs/xfs/xfs_reflink.c

index b21cb0d36dd44b139ab16bb504b02c2903e62299..fd65e5d7994a5e934e2f80704263267ad0b0187a 100644 (file)
@@ -235,7 +235,7 @@ xfs_reflink_trim_around_shared(
        int                     error = 0;
 
        /* Holes, unwritten, and delalloc extents cannot be shared */
-       if (!xfs_is_cow_inode(ip) || !xfs_bmap_is_written_extent(irec)) {
+       if (!xfs_is_reflink_inode(ip) || !xfs_bmap_is_written_extent(irec)) {
                *shared = false;
                return 0;
        }