Although per device usage cannot be shown without root privilege,
per profile usage can be shown.

To achieve this, we just basically need to remove the check of nullness
of chunkinfo in print_filesystem_usage_by_chunk(), because other
functions except print_unused() properly handles chunkinfo by
chunkcount, which is 0 if chunkinfo is null.

As a result, "fi usage" always includes the information of "fi df".

Reviewed-by: Qu Wenruo <[email protected]>
Signed-off-by: Tomohiro Misono <[email protected]>
---
 cmds-fi-usage.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/cmds-fi-usage.c b/cmds-fi-usage.c
index f2307fe0..2d45b3bb 100644
--- a/cmds-fi-usage.c
+++ b/cmds-fi-usage.c
@@ -923,9 +923,11 @@ static void _cmd_filesystem_usage_linear(unsigned 
unit_mode,
                printf("\n");
        }
 
-       printf("Unallocated:\n");
-       print_unused(info_ptr, info_count, device_info_ptr, device_info_count,
-                       unit_mode | UNITS_NEGATIVE);
+       if (info_count) {
+               printf("Unallocated:\n");
+               print_unused(info_ptr, info_count, device_info_ptr,
+                               device_info_count, unit_mode | UNITS_NEGATIVE);
+       }
 }
 
 static int print_filesystem_usage_by_chunk(int fd,
@@ -936,9 +938,6 @@ static int print_filesystem_usage_by_chunk(int fd,
        struct btrfs_ioctl_space_args *sargs;
        int ret = 0;
 
-       if (!chunkinfo)
-               return 0;
-
        sargs = load_space_info(fd, path);
        if (!sargs) {
                ret = 1;
-- 
2.14.3

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

Reply via email to