diff options
| author | Ye Bin | 2022-09-24 10:12:10 +0800 |
|---|---|---|
| committer | Theodore Ts'o | 2022-09-30 23:46:54 -0400 |
| commit | b6a750c01925954d0dc6c360e91b5fcb53664a26 (patch) | |
| tree | 1bdb8991a3faac69709e27802602b46e54e752e0 /fs | |
| parent | 27cd49780381c6ccbf248798e5e8fd076200ffba (diff) | |
ext4: remove unnecessary drop path references in mext_check_coverage()
According to Jan Kara's suggestion:
"The use in mext_check_coverage() can be actually removed
- get_ext_path() -> ext4_find_extent() takes care of dropping the references."
So remove unnecessary call ext4_ext_drop_refs() in mext_check_coverage().
Signed-off-by: Ye Bin <yebin10@huawei.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20220924021211.3831551-2-yebin10@huawei.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/ext4/move_extent.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index 2b83621d16e2..93ceee6d4caa 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -103,7 +103,6 @@ mext_check_coverage(struct inode *inode, ext4_lblk_t from, ext4_lblk_t count, if (unwritten != ext4_ext_is_unwritten(ext)) goto out; from += ext4_ext_get_actual_len(ext); - ext4_ext_drop_refs(path); } ret = 1; out: |