Status reports of the checking process should be printed to stdout
instead of stderr, as that is normal program output and not related to
problems in btrfsck. This patch changes this behaviour and adds the
output "Done!" after each of the parts.
Signed-off-by: Dieter Ries <mail@xxxxxxxxxxxxxx>
---
btrfsck.c | 20 +++++++++++++++-----
1 files changed, 15 insertions(+), 5 deletions(-)
diff --git a/btrfsck.c b/btrfsck.c
index 516bcdf..83275cd 100644
--- a/btrfsck.c
+++ b/btrfsck.c
@@ -3559,7 +3559,7 @@ int main(int ac, char **av)
root = info->fs_root;
- fprintf(stderr, "checking extents\n");
+ printf("checking extents... ");
if (rw)
trans = btrfs_start_transaction(root, 1);
@@ -3567,22 +3567,32 @@ int main(int ac, char **av)
fprintf(stderr, "Reinit crc root\n");
ret = btrfs_fsck_reinit_root(trans, info->csum_root);
if (ret) {
+ printf("\n");
fprintf(stderr, "crc root initialization failed\n");
return -EIO;
}
goto out;
}
ret = check_extents(trans, root, repair);
- if (ret)
+ if (ret) {
fprintf(stderr, "Errors found in extent allocation tree\n");
+ printf("\n");
+ }
+ else
+ printf("Done!\n");
- fprintf(stderr, "checking fs roots\n");
+ printf("checking fs roots... ");
ret = check_fs_roots(root, &root_cache);
- if (ret)
+ if (ret) {
+ printf("\n");
goto out;
+ }
+ else
+ printf("Done!\n");
- fprintf(stderr, "checking root refs\n");
+ printf("checking root refs... ");
ret = check_root_refs(root, &root_cache);
+ printf("Done!\n");
out:
free_root_recs(&root_cache);
if (rw) {
--
1.7.3.GIT
--
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
