Re: [PATCH] btrfs: drop math for block_reserved which is block_rsv size

David Sterba
2020-02-24T19:08:49Z

On Tue, Feb 04, 2020 at 08:46:04PM +0800, Anand Jain wrote:
>
>
> On 2/4/20 7:10 PM, Nikolay Borisov wrote:
> >
> >
> > On 4.02.20 г. 13:05 ч., Anand Jain wrote:
> >> In btrfs_update_global_block_rsv the lines
> >> num_bytes = block_rsv->size - block_rsv->reserved;
> >> block_rsv->reserved += num_bytes;
> >> imply
> >> block_rsv->reserved = block_rsv->size;
> >>
> >> Just assign block_rsv->size to block_rsv->reserved instead of the math.
> >>
> >> Signed-off-by: Anand Jain <anand.jain@oracle.com>
> >> ---
> >> fs/btrfs/block-rsv.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/fs/btrfs/block-rsv.c b/fs/btrfs/block-rsv.c
> >> index 6dacde9a7e93..62e0885c1e5d 100644
> >> --- a/fs/btrfs/block-rsv.c
> >> +++ b/fs/btrfs/block-rsv.c
> >> @@ -304,9 +304,9 @@ void btrfs_update_global_block_rsv(struct btrfs_fs_info *fs_info)
> >>
> >> if (block_rsv->reserved < block_rsv->size) {
> >> num_bytes = block_rsv->size - block_rsv->reserved;
> >> - block_rsv->reserved += num_bytes;
> >> btrfs_space_info_update_bytes_may_use(fs_info, sinfo,
> >> num_bytes);
> >> + block_rsv->reserved = block_rsv->size;
> >
> > Any particular reason why you put the assignment after
> > btrfs_space_info_update_bytes_may_use and not before?
>
> Oh. To make it similar to the else if part below.

Makes sense. For cleanup patches, such things can be mentioned in the
changelog, reordering lines may have indirect implications eg. if the
block_rsv would be used in btrfs_space_info_update_bytes_may_use (but it
is not). Patch added to misc-next, thanks.