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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Discussing this on another thread, but officially I'll NAK this here.

Joel

On Wed, Feb 20, 2013 at 11:59:29AM +0800, Jeff Liu wrote:
> 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@xxxxxxxxxx>
> Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>
> Cc: Andreas Dilger <adilger@xxxxxxxxx>
> Cc: Dave Chinner <david@xxxxxxxxxxxxx>
> Cc: Mark Fasheh <mfasheh@xxxxxxxx>
> Cc: Joel Becker <jlbec@xxxxxxxxxxxx>
> Cc: Jan Kara <jack@xxxxxxx>
> Cc: Chris Mason <chris.mason@xxxxxxxxxxxx>
> Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>
> 
> ---
>  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

-- 

Life's Little Instruction Book #306

	"Take a nap on Sunday afternoons."

			http://www.jlbec.org/
			jlbec@xxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux