| Message ID | 20210820120853.23134-1-lukas.bulwahn@gmail.com |
|---|---|
| State | Awaiting Upstream |
| Headers | show |
| Series | ext4: scope ret locally in ext4_try_to_trim_range() | expand |
On Fri, 20 Aug 2021 14:08:53 +0200, Lukas Bulwahn wrote: > As commit 6920b3913235 ("ext4: add new helper interface > ext4_try_to_trim_range()") moves some code into the separate function > ext4_try_to_trim_range(), the use of the variable ret within that > function is more limited and can be adjusted as well. > > Scope the use of the variable ret locally and drop dead assignments. > > [...] Applied, thanks! [1/1] ext4: scope ret locally in ext4_try_to_trim_range() commit: fd7a2ed7b32a4ee41477a5326941b95650575c84 Best regards,
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c index e89db3396203..f4c0a37431cd 100644 --- a/fs/ext4/mballoc.c +++ b/fs/ext4/mballoc.c @@ -6301,7 +6301,6 @@ __releases(ext4_group_lock_ptr(sb, e4b->bd_group)) { ext4_grpblk_t next, count, free_count; void *bitmap; - int ret = 0; bitmap = e4b->bd_bitmap; start = (e4b->bd_info->bb_first_free > start) ? @@ -6316,10 +6315,9 @@ __releases(ext4_group_lock_ptr(sb, e4b->bd_group)) next = mb_find_next_bit(bitmap, max + 1, start); if ((next - start) >= minblocks) { - ret = ext4_trim_extent(sb, start, next - start, e4b); + int ret = ext4_trim_extent(sb, start, next - start, e4b); if (ret && ret != -EOPNOTSUPP) break; - ret = 0; count += next - start; } free_count += next - start;
As commit 6920b3913235 ("ext4: add new helper interface ext4_try_to_trim_range()") moves some code into the separate function ext4_try_to_trim_range(), the use of the variable ret within that function is more limited and can be adjusted as well. Scope the use of the variable ret locally and drop dead assignments. No functional change. Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> --- fs/ext4/mballoc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)