ext4: return FIEMAP_EXTENT_UNKNOWN if an extent is delayed allocations

PATCH Applied Jeff Liu 3 participants 3 messages Jun 6, 2013 – Jun 18, 2013 ext4 file system

lore

Share & tools

lore

Jeff Liu

author

2013-06-06 13:31 UTC #0 lore raw

PATCH 1/1 ext4: return FIEMAP_EXTENT_UNKNOWN if an extent is delayed allocations

From: Jie Liu <[email protected]>

Return the FIEMAP_EXTENT_UNKNOWN flag as well except the FIEMAP_EXTENT_DELALLOC because the data location of an delayed allocation extent is unknown.

Signed-off-by Jie Liu <[email protected]>

---
 fs/ext4/extents.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

fs/ext4/extents.c +2 -1

index bc0f191..47e17b1 100644
@@ -2125,7 +2125,8 @@ static int ext4_fill_fiemap_extents(struct inode *inode,
 		next_del = ext4_find_delayed_extent(inode, &es);
 		if (!exists && next_del) {
 			exists = 1;
-			flags |= FIEMAP_EXTENT_DELALLOC;
+			flags |= (FIEMAP_EXTENT_DELALLOC |
+				  FIEMAP_EXTENT_UNKNOWN);
 		}
 		up_read(&EXT4_I(inode)->i_data_sem);

Signature · 1 line

1.7.9.5

Zheng Liu

Reviewed-by on v1 1/1

2013-06-07 11:36 UTC #1 lore raw

Quoted 7 lines · Jeff Liu

From: Jie Liu <[email protected]>
Return the FIEMAP_EXTENT_UNKNOWN flag as well except the FIEMAP_EXTENT_DELALLOC because the data location of an delayed allocation extent is unknown.
Signed-off-by Jie Liu <[email protected]>

Reviewed-by Zheng Liu <[email protected]>

- Zheng

Quoted 24 lines

---
 fs/ext4/extents.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

fs/ext4/extents.c +2 -1

index bc0f191..47e17b1 100644
@@ -2125,7 +2125,8 @@ static int ext4_fill_fiemap_extents(struct inode *inode,
 		next_del = ext4_find_delayed_extent(inode, &es);
 		if (!exists && next_del) {
 			exists = 1;
-			flags |= FIEMAP_EXTENT_DELALLOC;
+			flags |= (FIEMAP_EXTENT_DELALLOC |
+				  FIEMAP_EXTENT_UNKNOWN);
 		}
 		up_read(&EXT4_I(inode)->i_data_sem);

Signature · 5 lines

1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Theodore Ts'o

maintainer Applied

2013-06-18 15:08 UTC #2 lore raw

Quoted 10 lines · Zheng Liu

Quoted 7 lines · Jeff Liu
From: Jie Liu <[email protected]>
Return the FIEMAP_EXTENT_UNKNOWN flag as well except the FIEMAP_EXTENT_DELALLOC because the data location of an delayed allocation extent is unknown.
Signed-off-by Jie Liu <[email protected]>
Reviewed-by Zheng Liu <[email protected]>

Thanks, applied.

- Ted