[Ocfs2-devel] [RFC PATCH 3/3] ocfs2: only return the shared extents if FIEMAP_FLAG_COW is set

Jeff Liu jeff.liu at oracle.com
Tue Feb 19 19:59:29 PST 2013


Teach OCFS2 to be aware of FIEMAP_FLAG_COW - fill the extent map array with
those extents that have OCFS2_EXT_REFCOUNTED in rec.re_flags.

Signed-off-by: Jie Liu <jeff.liu at oracle.com>
Cc: Alexander Viro <viro at zeniv.linux.org.uk>
Cc: Andreas Dilger <adilger at dilger.ca>
Cc: Dave Chinner <david at fromorbit.com>
Cc: Mark Fasheh <mfasheh at suse.com>
Cc: Joel Becker <jlbec at evilplan.org>
Cc: Jan Kara <jack at suse.de>
Cc: Chris Mason <chris.mason at fusionio.com>
Cc: Christoph Hellwig <hch at infradead.org>

---
 fs/ocfs2/extent_map.c |   11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/fs/ocfs2/extent_map.c b/fs/ocfs2/extent_map.c
index f487aa3..1eaae3f 100644
--- a/fs/ocfs2/extent_map.c
+++ b/fs/ocfs2/extent_map.c
@@ -745,7 +745,7 @@ static int ocfs2_fiemap_inline(struct inode *inode, struct buffer_head *di_bh,
 	return 0;
 }
 
-#define OCFS2_FIEMAP_FLAGS	(FIEMAP_FLAG_SYNC)
+#define OCFS2_FIEMAP_FLAGS	(FIEMAP_FLAG_SYNC | FIEMAP_FLAG_COW)
 
 int ocfs2_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
 		 u64 map_start, u64 map_len)
@@ -810,11 +810,20 @@ int ocfs2_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
 		phys_bytes = le64_to_cpu(rec.e_blkno) << osb->sb->s_blocksize_bits;
 		virt_bytes = (u64)le32_to_cpu(rec.e_cpos) << osb->s_clustersize_bits;
 
+		/*
+		 * If the FIEMAP_FLAG_COW is set, fill the fiemap_extent array
+		 * with the extents in copy-on-write state only.
+		 */
+		if ((fieinfo->fi_flags & FIEMAP_FLAG_COW) &&
+		    !(fe_flags & FIEMAP_EXTENT_SHARED))
+			goto next_map;
+
 		ret = fiemap_fill_next_extent(fieinfo, virt_bytes, phys_bytes,
 					      len_bytes, fe_flags);
 		if (ret)
 			break;
 
+next_map:
 		cpos = le32_to_cpu(rec.e_cpos)+ le16_to_cpu(rec.e_leaf_clusters);
 	}
 
-- 
1.7.9.5



More information about the Ocfs2-devel mailing list