* [PATCH v2] btrfs-progs: filesystem usage: add avail info from statfs()
@ 2020-11-10 0:52 Sidong Yang
2020-12-01 18:05 ` David Sterba
0 siblings, 1 reply; 2+ messages in thread
From: Sidong Yang @ 2020-11-10 0:52 UTC (permalink / raw)
To: dsterba, linux-btrfs; +Cc: Sidong Yang
dd available space information from statfs(). This can be different from
'Free (estimated)' in some cases. This patch provide more information
about filesystem usage like below. and update document for this.
Overall:
Device size: 5.00GiB
Device allocated: 1.02GiB
Device unallocated: 3.98GiB
Device missing: 0.00B
Used: 88.00KiB
Free (estimated): 4.48GiB (min: 2.49GiB)
Avail: 4.48GiB
Data ratio: 1.00
Metadata ratio: 2.00
Global reserve: 832.00KiB (used: 0.00B)
Multiple profiles: no
Issue: #306
Signed-off-by: Sidong Yang <realwakka@gmail.com>
---
Documentation/btrfs-filesystem.asciidoc | 3 +++
cmds/filesystem-usage.c | 10 ++++++++++
2 files changed, 13 insertions(+)
diff --git a/Documentation/btrfs-filesystem.asciidoc b/Documentation/btrfs-filesystem.asciidoc
index 6a5561ed..18623afe 100644
--- a/Documentation/btrfs-filesystem.asciidoc
+++ b/Documentation/btrfs-filesystem.asciidoc
@@ -276,6 +276,7 @@ Overall:
Device missing: 0.00B
Used: 1.14TiB
Free (estimated): 692.57GiB (min: 692.57GiB)
+ Avail: 692.57GiB
Data ratio: 1.00
Metadata ratio: 1.00
Global reserve: 512.00MiB (used: 0.00B)
@@ -295,6 +296,8 @@ including the reserved space
data, including currently allocated space and estimating the usage of the
unallocated space based on the block group profiles, the 'min' is the lower bound
of the estimate in case multiple profiles are present
+* 'Avail' -- the amount of space available for data. it's get by statfs() system
+call that can be different from 'Free (estimated)' in some cases
* 'Data ratio' -- ratio of total space for data including redundancy or parity to
the effectively usable data space, eg. single is 1.0, RAID1 is 2.0 and for RAID5/6
it depends on the number of devices
diff --git a/cmds/filesystem-usage.c b/cmds/filesystem-usage.c
index ab60d769..ed743a61 100644
--- a/cmds/filesystem-usage.c
+++ b/cmds/filesystem-usage.c
@@ -19,6 +19,7 @@
#include <string.h>
#include <unistd.h>
#include <sys/ioctl.h>
+#include <sys/vfs.h>
#include <errno.h>
#include <stdarg.h>
#include <getopt.h>
@@ -430,6 +431,7 @@ static int print_filesystem_usage_overall(int fd, struct chunk_info *chunkinfo,
u64 free_min = 0;
double max_data_ratio = 1.0;
int mixed = 0;
+ struct statfs statfs_buf;
sargs = load_space_info(fd, path);
if (!sargs) {
@@ -556,6 +558,12 @@ static int print_filesystem_usage_overall(int fd, struct chunk_info *chunkinfo,
if (unit_mode != UNITS_HUMAN)
width = 18;
+ ret = statfs(path, &statfs_buf);
+ if (ret) {
+ warning("cannot get space info with statfs() on '%s': %m", path);
+ ret = 0;
+ }
+
printf("Overall:\n");
printf(" Device size:\t\t%*s\n", width,
@@ -572,6 +580,8 @@ static int print_filesystem_usage_overall(int fd, struct chunk_info *chunkinfo,
width,
pretty_size_mode(free_estimated, unit_mode));
printf("min: %s)\n", pretty_size_mode(free_min, unit_mode));
+ printf(" Avail:\t\t\t%*s\n", width,
+ pretty_size_mode(statfs_buf.f_bavail * statfs_buf.f_bsize, unit_mode));
printf(" Data ratio:\t\t\t%*.2f\n",
width, data_ratio);
printf(" Metadata ratio:\t\t%*.2f\n",
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] btrfs-progs: filesystem usage: add avail info from statfs()
2020-11-10 0:52 [PATCH v2] btrfs-progs: filesystem usage: add avail info from statfs() Sidong Yang
@ 2020-12-01 18:05 ` David Sterba
0 siblings, 0 replies; 2+ messages in thread
From: David Sterba @ 2020-12-01 18:05 UTC (permalink / raw)
To: Sidong Yang; +Cc: dsterba, linux-btrfs
On Tue, Nov 10, 2020 at 12:52:21AM +0000, Sidong Yang wrote:
> dd available space information from statfs(). This can be different from
> 'Free (estimated)' in some cases. This patch provide more information
> about filesystem usage like below. and update document for this.
>
> Overall:
> Device size: 5.00GiB
> Device allocated: 1.02GiB
> Device unallocated: 3.98GiB
> Device missing: 0.00B
> Used: 88.00KiB
> Free (estimated): 4.48GiB (min: 2.49GiB)
> Avail: 4.48GiB
My idea was to print
Free (statfs, df): ...
'Avail' without context is useless.
> Data ratio: 1.00
> Metadata ratio: 2.00
> Global reserve: 832.00KiB (used: 0.00B)
> Multiple profiles: no
>
> Issue: #306
> Signed-off-by: Sidong Yang <realwakka@gmail.com>
> ---
> Documentation/btrfs-filesystem.asciidoc | 3 +++
> cmds/filesystem-usage.c | 10 ++++++++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/Documentation/btrfs-filesystem.asciidoc b/Documentation/btrfs-filesystem.asciidoc
> index 6a5561ed..18623afe 100644
> --- a/Documentation/btrfs-filesystem.asciidoc
> +++ b/Documentation/btrfs-filesystem.asciidoc
> @@ -276,6 +276,7 @@ Overall:
> Device missing: 0.00B
> Used: 1.14TiB
> Free (estimated): 692.57GiB (min: 692.57GiB)
> + Avail: 692.57GiB
> Data ratio: 1.00
> Metadata ratio: 1.00
> Global reserve: 512.00MiB (used: 0.00B)
> @@ -295,6 +296,8 @@ including the reserved space
> data, including currently allocated space and estimating the usage of the
> unallocated space based on the block group profiles, the 'min' is the lower bound
> of the estimate in case multiple profiles are present
> +* 'Avail' -- the amount of space available for data. it's get by statfs() system
> +call that can be different from 'Free (estimated)' in some cases
> * 'Data ratio' -- ratio of total space for data including redundancy or parity to
> the effectively usable data space, eg. single is 1.0, RAID1 is 2.0 and for RAID5/6
> it depends on the number of devices
> diff --git a/cmds/filesystem-usage.c b/cmds/filesystem-usage.c
> index ab60d769..ed743a61 100644
> --- a/cmds/filesystem-usage.c
> +++ b/cmds/filesystem-usage.c
> @@ -19,6 +19,7 @@
> #include <string.h>
> #include <unistd.h>
> #include <sys/ioctl.h>
> +#include <sys/vfs.h>
> #include <errno.h>
> #include <stdarg.h>
> #include <getopt.h>
> @@ -430,6 +431,7 @@ static int print_filesystem_usage_overall(int fd, struct chunk_info *chunkinfo,
> u64 free_min = 0;
> double max_data_ratio = 1.0;
> int mixed = 0;
> + struct statfs statfs_buf;
>
> sargs = load_space_info(fd, path);
> if (!sargs) {
> @@ -556,6 +558,12 @@ static int print_filesystem_usage_overall(int fd, struct chunk_info *chunkinfo,
> if (unit_mode != UNITS_HUMAN)
> width = 18;
>
> + ret = statfs(path, &statfs_buf);
> + if (ret) {
> + warning("cannot get space info with statfs() on '%s': %m", path);
> + ret = 0;
> + }
Yeah, that's better than hard failure.
With the text updated, patch added to devel, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-12-01 18:07 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-10 0:52 [PATCH v2] btrfs-progs: filesystem usage: add avail info from statfs() Sidong Yang
2020-12-01 18:05 ` David Sterba
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).