Re: [PATCH v2] btrfs: fix free space calculation in dump_space_info()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Wang Xiaoguang posted on Mon, 11 Jul 2016 19:30:04 +0800 as excerpted:

> In btrfs, btrfs_space_info's bytes_may_use is treated as fs used space,
> as what we do in reserve_metadata_bytes() or
> btrfs_alloc_data_chunk_ondemand(), so in dump_space_info(), when
> calculating free space, we should also minus btrfs_space_info's
> bytes_may_use.
> 
> Signed-off-by: Wang Xiaoguang <wangxg.fnst@xxxxxxxxxxxxxx>
> ---
>  fs/btrfs/extent-tree.c | 4 ++--

I'm not a dev so won't evaluate the patch itself.  However, on kernel 
related mailing lists it's standard practice to include a short, often 
one-line per revision, revision changelog.  It helps reviewers keep track 
of what changed since the last time they looked at the patch.  See pretty 
much any on-list v2+ patch as an example.

-- 
Duncan - List replies preferred.   No HTML msgs.
"Every nonfree program has a lord, a master --
and if you use the program, he is your master."  Richard Stallman

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux