| Message ID | 5446ba76bd7fa527447a23fe6e3262b36533d4c7.1712287421.git.wqu@suse.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | btrfs: more explaination on extent_map members | expand |
On Fri, Apr 5, 2024 at 4:28 AM Qu Wenruo <wqu@suse.com> wrote: > > With the tree-checker ensuring all inline file extents starts at file > offset 0 and has a length no larger than sectorsize, we can simplify the > calculation to assigned those fixes values directly. > > Signed-off-by: Qu Wenruo <wqu@suse.com> Reviewed-by: Filipe Manana <fdmanana@suse.com> Looks good now, thanks. > --- > fs/btrfs/file-item.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/fs/btrfs/file-item.c b/fs/btrfs/file-item.c > index e58fb5347e65..844439f19949 100644 > --- a/fs/btrfs/file-item.c > +++ b/fs/btrfs/file-item.c > @@ -1265,20 +1265,19 @@ void btrfs_extent_item_to_extent_map(struct btrfs_inode *inode, > struct extent_buffer *leaf = path->nodes[0]; > const int slot = path->slots[0]; > struct btrfs_key key; > - u64 extent_start, extent_end; > + u64 extent_start; > u64 bytenr; > u8 type = btrfs_file_extent_type(leaf, fi); > int compress_type = btrfs_file_extent_compression(leaf, fi); > > btrfs_item_key_to_cpu(leaf, &key, slot); > extent_start = key.offset; > - extent_end = btrfs_file_extent_end(path); > em->ram_bytes = btrfs_file_extent_ram_bytes(leaf, fi); > em->generation = btrfs_file_extent_generation(leaf, fi); > if (type == BTRFS_FILE_EXTENT_REG || > type == BTRFS_FILE_EXTENT_PREALLOC) { > em->start = extent_start; > - em->len = extent_end - extent_start; > + em->len = btrfs_file_extent_end(path) - extent_start; > em->orig_start = extent_start - > btrfs_file_extent_offset(leaf, fi); > em->orig_block_len = btrfs_file_extent_disk_num_bytes(leaf, fi); > @@ -1299,9 +1298,12 @@ void btrfs_extent_item_to_extent_map(struct btrfs_inode *inode, > em->flags |= EXTENT_FLAG_PREALLOC; > } > } else if (type == BTRFS_FILE_EXTENT_INLINE) { > + /* Tree-checker has ensured this. */ > + ASSERT(extent_start == 0); > + > em->block_start = EXTENT_MAP_INLINE; > - em->start = extent_start; > - em->len = extent_end - extent_start; > + em->start = 0; > + em->len = fs_info->sectorsize; > /* > * Initialize orig_start and block_len with the same values > * as in inode.c:btrfs_get_extent(). > @@ -1334,12 +1336,10 @@ u64 btrfs_file_extent_end(const struct btrfs_path *path) > ASSERT(key.type == BTRFS_EXTENT_DATA_KEY); > fi = btrfs_item_ptr(leaf, slot, struct btrfs_file_extent_item); > > - if (btrfs_file_extent_type(leaf, fi) == BTRFS_FILE_EXTENT_INLINE) { > - end = btrfs_file_extent_ram_bytes(leaf, fi); > - end = ALIGN(key.offset + end, leaf->fs_info->sectorsize); > - } else { > + if (btrfs_file_extent_type(leaf, fi) == BTRFS_FILE_EXTENT_INLINE) > + end = leaf->fs_info->sectorsize; > + else > end = key.offset + btrfs_file_extent_num_bytes(leaf, fi); > - } > > return end; > } > -- > 2.44.0 > >
diff --git a/fs/btrfs/file-item.c b/fs/btrfs/file-item.c index e58fb5347e65..844439f19949 100644 --- a/fs/btrfs/file-item.c +++ b/fs/btrfs/file-item.c @@ -1265,20 +1265,19 @@ void btrfs_extent_item_to_extent_map(struct btrfs_inode *inode, struct extent_buffer *leaf = path->nodes[0]; const int slot = path->slots[0]; struct btrfs_key key; - u64 extent_start, extent_end; + u64 extent_start; u64 bytenr; u8 type = btrfs_file_extent_type(leaf, fi); int compress_type = btrfs_file_extent_compression(leaf, fi); btrfs_item_key_to_cpu(leaf, &key, slot); extent_start = key.offset; - extent_end = btrfs_file_extent_end(path); em->ram_bytes = btrfs_file_extent_ram_bytes(leaf, fi); em->generation = btrfs_file_extent_generation(leaf, fi); if (type == BTRFS_FILE_EXTENT_REG || type == BTRFS_FILE_EXTENT_PREALLOC) { em->start = extent_start; - em->len = extent_end - extent_start; + em->len = btrfs_file_extent_end(path) - extent_start; em->orig_start = extent_start - btrfs_file_extent_offset(leaf, fi); em->orig_block_len = btrfs_file_extent_disk_num_bytes(leaf, fi); @@ -1299,9 +1298,12 @@ void btrfs_extent_item_to_extent_map(struct btrfs_inode *inode, em->flags |= EXTENT_FLAG_PREALLOC; } } else if (type == BTRFS_FILE_EXTENT_INLINE) { + /* Tree-checker has ensured this. */ + ASSERT(extent_start == 0); + em->block_start = EXTENT_MAP_INLINE; - em->start = extent_start; - em->len = extent_end - extent_start; + em->start = 0; + em->len = fs_info->sectorsize; /* * Initialize orig_start and block_len with the same values * as in inode.c:btrfs_get_extent(). @@ -1334,12 +1336,10 @@ u64 btrfs_file_extent_end(const struct btrfs_path *path) ASSERT(key.type == BTRFS_EXTENT_DATA_KEY); fi = btrfs_item_ptr(leaf, slot, struct btrfs_file_extent_item); - if (btrfs_file_extent_type(leaf, fi) == BTRFS_FILE_EXTENT_INLINE) { - end = btrfs_file_extent_ram_bytes(leaf, fi); - end = ALIGN(key.offset + end, leaf->fs_info->sectorsize); - } else { + if (btrfs_file_extent_type(leaf, fi) == BTRFS_FILE_EXTENT_INLINE) + end = leaf->fs_info->sectorsize; + else end = key.offset + btrfs_file_extent_num_bytes(leaf, fi); - } return end; }
With the tree-checker ensuring all inline file extents starts at file offset 0 and has a length no larger than sectorsize, we can simplify the calculation to assigned those fixes values directly. Signed-off-by: Qu Wenruo <wqu@suse.com> --- fs/btrfs/file-item.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)