btrfs-progs: alias btrfs device delete to btrfs device remove
[platform/upstream/btrfs-progs.git] / cmds-scrub.c
index b0fcde6..b7aa809 100644 (file)
@@ -24,6 +24,7 @@
 #include <sys/types.h>
 #include <sys/socket.h>
 #include <sys/un.h>
+#include <sys/syscall.h>
 #include <poll.h>
 #include <sys/file.h>
 #include <uuid/uuid.h>
@@ -58,8 +59,18 @@ struct scrub_stats {
        u64 duration;
        u64 finished;
        u64 canceled;
+       int in_progress;
 };
 
+/* TBD: replace with #include "linux/ioprio.h" in some years */
+#if !defined (IOPRIO_H)
+#define IOPRIO_WHO_PROCESS 1
+#define IOPRIO_CLASS_SHIFT 13
+#define IOPRIO_PRIO_VALUE(class, data) \
+               (((class) << IOPRIO_CLASS_SHIFT) | (data))
+#define IOPRIO_CLASS_IDLE 3
+#endif
+
 struct scrub_progress {
        struct btrfs_ioctl_scrub_args scrub_args;
        int fd;
@@ -69,6 +80,8 @@ struct scrub_progress {
        struct scrub_file_record *resumed;
        int ioctl_errno;
        pthread_mutex_t progress_mutex;
+       int ioprio_class;
+       int ioprio_classdata;
 };
 
 struct scrub_file_record {
@@ -127,7 +140,6 @@ static void print_scrub_summary(struct btrfs_scrub_progress *p)
 {
        u64 err_cnt;
        u64 err_cnt2;
-       char *bytes;
 
        err_cnt = p->read_errors +
                        p->csum_errors +
@@ -139,10 +151,11 @@ static void print_scrub_summary(struct btrfs_scrub_progress *p)
        if (p->malloc_errors)
                printf("*** WARNING: memory allocation failed while scrubbing. "
                       "results may be inaccurate\n");
-       bytes = pretty_sizes(p->data_bytes_scrubbed + p->tree_bytes_scrubbed);
-       printf("\ttotal bytes scrubbed: %s with %llu errors\n", bytes,
+
+       printf("\ttotal bytes scrubbed: %s with %llu errors\n",
+               pretty_size(p->data_bytes_scrubbed + p->tree_bytes_scrubbed),
                max(err_cnt, err_cnt2));
-       free(bytes);
+
        if (err_cnt || err_cnt2) {
                printf("\terror details:");
                PRINT_SCRUB_ERROR(p->read_errors, "read");
@@ -216,6 +229,8 @@ static void _print_scrub_ss(struct scrub_stats *ss)
 {
        char t[4096];
        struct tm tm;
+       time_t seconds;
+       unsigned hours;
 
        if (!ss || !ss->t_start) {
                printf("\tno stats available\n");
@@ -232,14 +247,21 @@ static void _print_scrub_ss(struct scrub_stats *ss)
                t[sizeof(t) - 1] = '\0';
                printf("\tscrub started at %s", t);
        }
+
+       seconds = ss->duration;
+       hours = ss->duration / (60 * 60);
+       gmtime_r(&seconds, &tm);
+       strftime(t, sizeof(t), "%M:%S", &tm);
        if (ss->finished && !ss->canceled) {
-               printf(" and finished after %llu seconds\n",
-                      ss->duration);
+               printf(" and finished after %02u:%s\n", hours, t);
        } else if (ss->canceled) {
-               printf(" and was aborted after %llu seconds\n",
-                      ss->duration);
+               printf(" and was aborted after %02u:%s\n", hours, t);
        } else {
-               printf(", running for %llu seconds\n", ss->duration);
+               if (ss->in_progress)
+                       printf(", running for %02u:%s\n", hours, t);
+               else
+                       printf(", interrupted after %02u:%s, not running\n",
+                                       hours, t);
        }
 }
 
@@ -273,7 +295,7 @@ static void print_fs_stat(struct scrub_fs_stat *fs_stat, int raw)
 static void free_history(struct scrub_file_record **last_scrubs)
 {
        struct scrub_file_record **l = last_scrubs;
-       if (!l)
+       if (!l || IS_ERR(l))
                return;
        while (*l)
                free(*l++);
@@ -368,7 +390,7 @@ static int scrub_open_file(const char *datafile, int m)
 static int scrub_open_file_r(const char *fn_base, const char *fn_local)
 {
        int ret;
-       char datafile[BTRFS_PATH_NAME_MAX + 1];
+       char datafile[PATH_MAX];
        ret = scrub_datafile(fn_base, fn_local, NULL,
                                datafile, sizeof(datafile));
        if (ret < 0)
@@ -380,7 +402,7 @@ static int scrub_open_file_w(const char *fn_base, const char *fn_local,
                                const char *tmp)
 {
        int ret;
-       char datafile[BTRFS_PATH_NAME_MAX + 1];
+       char datafile[PATH_MAX];
        ret = scrub_datafile(fn_base, fn_local, tmp,
                                datafile, sizeof(datafile));
        if (ret < 0)
@@ -392,8 +414,8 @@ static int scrub_rename_file(const char *fn_base, const char *fn_local,
                                const char *tmp)
 {
        int ret;
-       char datafile_old[BTRFS_PATH_NAME_MAX + 1];
-       char datafile_new[BTRFS_PATH_NAME_MAX + 1];
+       char datafile_old[PATH_MAX];
+       char datafile_new[PATH_MAX];
        ret = scrub_datafile(fn_base, fn_local, tmp,
                                datafile_old, sizeof(datafile_old));
        if (ret < 0)
@@ -462,9 +484,6 @@ static struct scrub_file_record **scrub_read_file(int fd, int report_errors)
        char empty_uuid[BTRFS_FSID_SIZE] = {0};
        struct scrub_file_record **p = NULL;
 
-       if (fd < 0)
-               return ERR_PTR(-EINVAL);
-
 again:
        old_avail = avail - i;
        BUG_ON(old_avail < 0);
@@ -541,7 +560,7 @@ again:
                                ;
                        if (i + j + 1 >= avail)
                                _SCRUB_INVALID;
-                       if (j != 36)
+                       if (j != BTRFS_UUID_UNPARSED_SIZE - 1)
                                _SCRUB_INVALID;
                        l[i + j] = '\0';
                        ret = uuid_parse(l + i, p[curr]->fsid);
@@ -761,31 +780,31 @@ static int scrub_write_progress(pthread_mutex_t *m, const char *fsid,
        int fd = -1;
        int old;
 
-       ret = pthread_mutex_lock(m);
+       ret = pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, &old);
        if (ret) {
                err = -ret;
-               goto out;
+               goto out3;
        }
 
-       ret = pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, &old);
+       ret = pthread_mutex_lock(m);
        if (ret) {
                err = -ret;
-               goto out;
+               goto out2;
        }
 
        fd = scrub_open_file_w(SCRUB_DATA_FILE, fsid, "tmp");
        if (fd < 0) {
                err = fd;
-               goto out;
+               goto out1;
        }
        err = scrub_write_file(fd, fsid, data, n);
        if (err)
-               goto out;
+               goto out1;
        err = scrub_rename_file(SCRUB_DATA_FILE, fsid, "tmp");
        if (err)
-               goto out;
+               goto out1;
 
-out:
+out1:
        if (fd >= 0) {
                ret = close(fd);
                if (ret)
@@ -796,10 +815,12 @@ out:
        if (ret && !err)
                err = -ret;
 
+out2:
        ret = pthread_setcancelstate(PTHREAD_CANCEL_ENABLE, &old);
        if (ret && !err)
                err = -ret;
 
+out3:
        return err;
 }
 
@@ -813,6 +834,14 @@ static void *scrub_one_dev(void *ctx)
        sp->stats.duration = 0;
        sp->stats.finished = 0;
 
+       ret = syscall(SYS_ioprio_set, IOPRIO_WHO_PROCESS, 0,
+                     IOPRIO_PRIO_VALUE(sp->ioprio_class,
+                                       sp->ioprio_classdata));
+       if (ret)
+               fprintf(stderr,
+                       "WARNING: setting ioprio failed: %s (ignored).\n",
+                       strerror(errno));
+
        ret = ioctl(sp->fd, BTRFS_IOC_SCRUB, &sp->scrub_args);
        gettimeofday(&tv, NULL);
        sp->ret = ret;
@@ -843,11 +872,11 @@ static void *progress_one_dev(void *ctx)
 /* nb: returns a negative errno via ERR_PTR */
 static void *scrub_progress_cycle(void *ctx)
 {
-       int ret;
+       int ret = 0;
        int  perr = 0;  /* positive / pthread error returns */
        int old;
        int i;
-       char fsid[37];
+       char fsid[BTRFS_UUID_UNPARSED_SIZE];
        struct scrub_progress *sp;
        struct scrub_progress *sp_last;
        struct scrub_progress *sp_shared;
@@ -923,6 +952,10 @@ static void *scrub_progress_cycle(void *ctx)
                         * result we got for the current write and go
                         * on. flag should be set on next cycle, then.
                         */
+                       perr = pthread_setcancelstate(
+                                       PTHREAD_CANCEL_DISABLE, &old);
+                       if (perr)
+                               goto out;
                        perr = pthread_mutex_lock(&sp_shared->progress_mutex);
                        if (perr)
                                goto out;
@@ -931,12 +964,20 @@ static void *scrub_progress_cycle(void *ctx)
                                                &sp_shared->progress_mutex);
                                if (perr)
                                        goto out;
+                               perr = pthread_setcancelstate(
+                                               PTHREAD_CANCEL_ENABLE, &old);
+                               if (perr)
+                                       goto out;
                                memcpy(sp, sp_last, sizeof(*sp));
                                continue;
                        }
                        perr = pthread_mutex_unlock(&sp_shared->progress_mutex);
                        if (perr)
                                goto out;
+                       perr = pthread_setcancelstate(
+                                       PTHREAD_CANCEL_ENABLE, &old);
+                       if (perr)
+                               goto out;
                        memcpy(sp, sp_shared, sizeof(*sp));
                        memcpy(sp_last, sp_shared, sizeof(*sp));
                }
@@ -987,7 +1028,7 @@ static struct scrub_file_record *last_dev_scrub(
        return NULL;
 }
 
-int mkdir_p(char *path)
+static int mkdir_p(char *path)
 {
        int i;
        int ret;
@@ -998,13 +1039,52 @@ int mkdir_p(char *path)
                path[i] = '\0';
                ret = mkdir(path, 0777);
                if (ret && errno != EEXIST)
-                       return 1;
+                       return -errno;
                path[i] = '/';
        }
 
        return 0;
 }
 
+static int is_scrub_running_on_fs(struct btrfs_ioctl_fs_info_args *fi_args,
+                                 struct btrfs_ioctl_dev_info_args *di_args,
+                                 struct scrub_file_record **past_scrubs)
+{
+       int i;
+
+       if (!fi_args || !di_args || !past_scrubs)
+               return 0;
+
+       for (i = 0; i < fi_args->num_devices; i++) {
+               struct scrub_file_record *sfr =
+                       last_dev_scrub(past_scrubs, di_args[i].devid);
+
+               if (!sfr)
+                       continue;
+               if (!(sfr->stats.finished || sfr->stats.canceled))
+                       return 1;
+       }
+       return 0;
+}
+
+static int is_scrub_running_in_kernel(int fd,
+               struct btrfs_ioctl_dev_info_args *di_args, u64 max_devices)
+{
+       struct scrub_progress sp;
+       int i;
+       int ret;
+
+       for (i = 0; i < max_devices; i++) {
+               memset(&sp, 0, sizeof(sp));
+               sp.scrub_args.devid = di_args[i].devid;
+               ret = ioctl(fd, BTRFS_IOC_SCRUB_PROGRESS, &sp.scrub_args);
+               if (!ret)
+                       return 1;
+       }
+
+       return 0;
+}
+
 static const char * const cmd_scrub_start_usage[];
 static const char * const cmd_scrub_resume_usage[];
 
@@ -1029,6 +1109,8 @@ static int scrub_start(int argc, char **argv, int resume)
        int do_record = 1;
        int readonly = 0;
        int do_stats_per_dev = 0;
+       int ioprio_class = IOPRIO_CLASS_IDLE;
+       int ioprio_classdata = 0;
        int n_start = 0;
        int n_skip = 0;
        int n_resume = 0;
@@ -1042,19 +1124,21 @@ static int scrub_start(int argc, char **argv, int resume)
        };
        pthread_t *t_devs = NULL;
        pthread_t t_prog;
-       pthread_attr_t t_attr;
        struct scrub_file_record **past_scrubs = NULL;
        struct scrub_file_record *last_scrub = NULL;
        char *datafile = strdup(SCRUB_DATA_FILE);
-       char fsid[37];
-       char sock_path[BTRFS_PATH_NAME_MAX + 1] = "";
+       char fsid[BTRFS_UUID_UNPARSED_SIZE];
+       char sock_path[PATH_MAX] = "";
        struct scrub_progress_cycle spc;
        pthread_mutex_t spc_write_mutex = PTHREAD_MUTEX_INITIALIZER;
        void *terr;
        u64 devid;
+       DIR *dirstream = NULL;
+       int force = 0;
+       int nothing_to_resume = 0;
 
        optind = 1;
-       while ((c = getopt(argc, argv, "BdqrR")) != -1) {
+       while ((c = getopt(argc, argv, "BdqrRc:n:f")) != -1) {
                switch (c) {
                case 'B':
                        do_background = 0;
@@ -1073,6 +1157,15 @@ static int scrub_start(int argc, char **argv, int resume)
                case 'R':
                        print_raw = 1;
                        break;
+               case 'c':
+                       ioprio_class = (int)strtol(optarg, NULL, 10);
+                       break;
+               case 'n':
+                       ioprio_classdata = (int)strtol(optarg, NULL, 10);
+                       break;
+               case 'f':
+                       force = 1;
+                       break;
                case '?':
                default:
                        usage(resume ? cmd_scrub_resume_usage :
@@ -1101,13 +1194,20 @@ static int scrub_start(int argc, char **argv, int resume)
 
        path = argv[optind];
 
-       fdmnt = open_file_or_dir(path);
+       fdmnt = open_path_or_dev_mnt(path, &dirstream);
+
        if (fdmnt < 0) {
-               ERR(!do_quiet, "ERROR: can't access '%s'\n", path);
-               return 12;
+               if (errno == EINVAL)
+                       ERR(!do_quiet,
+                           "ERROR: '%s' is not a mounted btrfs device\n",
+                           path);
+               else
+                       ERR(!do_quiet, "ERROR: can't access '%s': %s\n",
+                           path, strerror(errno));
+               return 1;
        }
 
-       ret = get_fs_info(fdmnt, path, &fi_args, &di_args);
+       ret = get_fs_info(path, &fi_args, &di_args);
        if (ret) {
                ERR(!do_quiet, "ERROR: getting dev info for scrub failed: "
                    "%s\n", strerror(-ret));
@@ -1133,6 +1233,34 @@ static int scrub_start(int argc, char **argv, int resume)
                close(fdres);
        }
 
+       /*
+        * Check for stale information in the status file, ie. if it's
+        * canceled=0, finished=0 but no scrub is running.
+        */
+       if (!is_scrub_running_in_kernel(fdmnt, di_args, fi_args.num_devices))
+               force = 1;
+
+       /*
+        * check whether any involved device is already busy running a
+        * scrub. This would cause damaged status messages and the state
+        * "aborted" without the explanation that a scrub was already
+        * running. Therefore check it first, prevent it and give some
+        * feedback to the user if scrub is already running.
+        * Note that if scrub is started with a block device as the
+        * parameter, only that particular block device is checked. It
+        * is a normal mode of operation to start scrub on multiple
+        * single devices, there is no reason to prevent this.
+        */
+       if (!force && is_scrub_running_on_fs(&fi_args, di_args, past_scrubs)) {
+               ERR(!do_quiet,
+                   "ERROR: scrub is already running.\n"
+                   "To cancel use 'btrfs scrub cancel %s'.\n"
+                   "To see the status use 'btrfs scrub status [-d] %s'.\n",
+                   path, path);
+               err = 1;
+               goto out;
+       }
+
        t_devs = malloc(fi_args.num_devices * sizeof(*t_devs));
        sp = calloc(fi_args.num_devices, sizeof(*sp));
        spc.progress = calloc(fi_args.num_devices * 2, sizeof(*spc.progress));
@@ -1143,14 +1271,6 @@ static int scrub_start(int argc, char **argv, int resume)
                goto out;
        }
 
-       ret = pthread_attr_init(&t_attr);
-       if (ret) {
-               ERR(!do_quiet, "ERROR: pthread_attr_init failed: %s\n",
-                   strerror(ret));
-               err = 1;
-               goto out;
-       }
-
        for (i = 0; i < fi_args.num_devices; ++i) {
                devid = di_args[i].devid;
                ret = pthread_mutex_init(&sp[i].progress_mutex, NULL);
@@ -1181,13 +1301,15 @@ static int scrub_start(int argc, char **argv, int resume)
                sp[i].skip = 0;
                sp[i].scrub_args.end = (u64)-1ll;
                sp[i].scrub_args.flags = readonly ? BTRFS_SCRUB_READONLY : 0;
+               sp[i].ioprio_class = ioprio_class;
+               sp[i].ioprio_classdata = ioprio_classdata;
        }
 
        if (!n_start && !n_resume) {
                if (!do_quiet)
                        printf("scrub: nothing to resume for %s, fsid %s\n",
                               path, fsid);
-               err = 0;
+               nothing_to_resume = 1;
                goto out;
        }
 
@@ -1296,7 +1418,7 @@ static int scrub_start(int argc, char **argv, int resume)
                devid = di_args[i].devid;
                gettimeofday(&tv, NULL);
                sp[i].stats.t_start = tv.tv_sec;
-               ret = pthread_create(&t_devs[i], &t_attr,
+               ret = pthread_create(&t_devs[i], NULL,
                                        scrub_one_dev, &sp[i]);
                if (ret) {
                        if (do_print)
@@ -1314,7 +1436,7 @@ static int scrub_start(int argc, char **argv, int resume)
        spc.write_mutex = &spc_write_mutex;
        spc.shared_progress = sp;
        spc.fi = &fi_args;
-       ret = pthread_create(&t_prog, &t_attr, scrub_progress_cycle, &spc);
+       ret = pthread_create(&t_prog, NULL, scrub_progress_cycle, &spc);
        if (ret) {
                if (do_print)
                        fprintf(stderr, "ERROR: creating progress thread "
@@ -1422,25 +1544,35 @@ out:
                if (sock_path[0])
                        unlink(sock_path);
        }
-       close(fdmnt);
+       close_file_or_dir(fdmnt, dirstream);
 
        if (err)
                return 1;
+       if (nothing_to_resume)
+               return 2;
+       if (e_uncorrectable) {
+               ERR(!do_quiet, "ERROR: There are uncorrectable errors.\n");
+               return 3;
+       }
        if (e_correctable)
-               return 7;
-       if (e_uncorrectable)
-               return 8;
+               ERR(!do_quiet, "WARNING: errors detected during scrubbing, corrected.\n");
+
        return 0;
 }
 
 static const char * const cmd_scrub_start_usage[] = {
-       "btrfs scrub start [-Bdqr] <path>|<device>",
-       "Start a new scrub",
+       "btrfs scrub start [-BdqrRf] [-c ioprio_class -n ioprio_classdata] <path>|<device>",
+       "Start a new scrub. If a scrub is already running, the new one fails.",
        "",
        "-B     do not background",
        "-d     stats per device (-B only)",
        "-q     be quiet",
        "-r     read only mode",
+       "-R     raw print mode, print full data instead of summary",
+       "-c     set ioprio class (see ionice(1) manpage)",
+       "-n     set ioprio classdata (see ionice(1) manpage)",
+       "-f     force starting new scrub even if a scrub is already running",
+       "       this is useful when scrub stats record file is damaged",
        NULL
 };
 
@@ -1459,66 +1591,58 @@ static int cmd_scrub_cancel(int argc, char **argv)
 {
        char *path;
        int ret;
-       int fdmnt;
-       int err;
-       char mp[BTRFS_PATH_NAME_MAX + 1];
-       struct btrfs_fs_devices *fs_devices_mnt = NULL;
+       int fdmnt = -1;
+       DIR *dirstream = NULL;
 
        if (check_argc_exact(argc, 2))
                usage(cmd_scrub_cancel_usage);
 
        path = argv[1];
 
-again:
-       fdmnt = open_file_or_dir(path);
+       fdmnt = open_path_or_dev_mnt(path, &dirstream);
        if (fdmnt < 0) {
-               perror("ERROR: scrub cancel failed:");
-               return 1;
-       }
-
-       ret = ioctl(fdmnt, BTRFS_IOC_SCRUB_CANCEL, NULL);
-       err = errno;
-
-       if (ret && err == EINVAL) {
-               /* path is not a btrfs mount point.  See if it's a device. */
-               ret = check_mounted_where(fdmnt, path, mp, sizeof(mp),
-                                         &fs_devices_mnt);
-               if (ret > 0) {
-                       /* It's a mounted btrfs device; retry w/ mountpoint. */
-                       close(fdmnt);
-                       path = mp;
-                       goto again;
-               } else {
-                       /* It's not a mounted btrfs device either */
+               if (errno == EINVAL)
                        fprintf(stderr,
-                               "ERROR: %s is not a mounted btrfs device\n",
+                               "ERROR: '%s' is not a mounted btrfs device\n",
                                path);
-                       ret = 1;
-                       err = EINVAL;
-               }
+               else
+                       fprintf(stderr, "ERROR: can't access '%s': %s\n",
+                               path, strerror(errno));
+               ret = 1;
+               goto out;
        }
 
-       close(fdmnt);
+       ret = ioctl(fdmnt, BTRFS_IOC_SCRUB_CANCEL, NULL);
 
-       if (ret) {
+       if (ret < 0) {
                fprintf(stderr, "ERROR: scrub cancel failed on %s: %s\n", path,
-                       err == ENOTCONN ? "not running" : strerror(err));
-               return 1;
+                       errno == ENOTCONN ? "not running" : strerror(errno));
+               if (errno == ENOTCONN)
+                       ret = 2;
+               else
+                       ret = 1;
+               goto out;
        }
 
+       ret = 0;
        printf("scrub cancelled\n");
 
-       return 0;
+out:
+       close_file_or_dir(fdmnt, dirstream);
+       return ret;
 }
 
 static const char * const cmd_scrub_resume_usage[] = {
-       "btrfs scrub resume [-Bdqr] <path>|<device>",
+       "btrfs scrub resume [-BdqrR] [-c ioprio_class -n ioprio_classdata] <path>|<device>",
        "Resume previously canceled or interrupted scrub",
        "",
        "-B     do not background",
        "-d     stats per device (-B only)",
        "-q     be quiet",
        "-r     read only mode",
+       "-R     raw print mode, print full data instead of summary",
+       "-c     set ioprio class (see ionice(1) manpage)",
+       "-n     set ioprio classdata (see ionice(1) manpage)",
        NULL
 };
 
@@ -1547,15 +1671,17 @@ static int cmd_scrub_status(int argc, char **argv)
        struct sockaddr_un addr = {
                .sun_family = AF_UNIX,
        };
+       int in_progress;
        int ret;
        int i;
        int fdmnt;
        int print_raw = 0;
        int do_stats_per_dev = 0;
        int c;
-       char fsid[37];
+       char fsid[BTRFS_UUID_UNPARSED_SIZE];
        int fdres = -1;
        int err = 0;
+       DIR *dirstream = NULL;
 
        optind = 1;
        while ((c = getopt(argc, argv, "dR")) != -1) {
@@ -1577,13 +1703,20 @@ static int cmd_scrub_status(int argc, char **argv)
 
        path = argv[optind];
 
-       fdmnt = open_file_or_dir(path);
+       fdmnt = open_path_or_dev_mnt(path, &dirstream);
+
        if (fdmnt < 0) {
-               fprintf(stderr, "ERROR: can't access to '%s'\n", path);
-               return 12;
+               if (errno == EINVAL)
+                       fprintf(stderr,
+                               "ERROR: '%s' is not a mounted btrfs device\n",
+                               path);
+               else
+                       fprintf(stderr, "ERROR: can't access '%s': %s\n",
+                               path, strerror(errno));
+               return 1;
        }
 
-       ret = get_fs_info(fdmnt, path, &fi_args, &di_args);
+       ret = get_fs_info(path, &fi_args, &di_args);
        if (ret) {
                fprintf(stderr, "ERROR: getting dev info for scrub failed: "
                                "%s\n", strerror(-ret));
@@ -1628,6 +1761,7 @@ static int cmd_scrub_status(int argc, char **argv)
                        fprintf(stderr, "WARNING: failed to read status: %s\n",
                                strerror(-PTR_ERR(past_scrubs)));
        }
+       in_progress = is_scrub_running_in_kernel(fdmnt, di_args, fi_args.num_devices);
 
        printf("scrub status for %s\n", fsid);
 
@@ -1640,6 +1774,7 @@ static int cmd_scrub_status(int argc, char **argv)
                                                NULL, NULL);
                                continue;
                        }
+                       last_scrub->stats.in_progress = in_progress;
                        print_scrub_dev(&di_args[i], &last_scrub->p, print_raw,
                                        last_scrub->stats.finished ?
                                                        "history" : "status",
@@ -1647,6 +1782,7 @@ static int cmd_scrub_status(int argc, char **argv)
                }
        } else {
                init_fs_stat(&fs_stat);
+               fs_stat.s.in_progress = in_progress;
                for (i = 0; i < fi_args.num_devices; ++i) {
                        last_scrub = last_dev_scrub(past_scrubs,
                                                        di_args[i].devid);
@@ -1663,17 +1799,21 @@ out:
        free(di_args);
        if (fdres > -1)
                close(fdres);
+       close_file_or_dir(fdmnt, dirstream);
 
-       return err;
+       return !!err;
 }
 
+static const char scrub_cmd_group_info[] =
+"verify checksums of data and metadata";
+
 const struct cmd_group scrub_cmd_group = {
-       scrub_cmd_group_usage, NULL, {
+       scrub_cmd_group_usage, scrub_cmd_group_info, {
                { "start", cmd_scrub_start, cmd_scrub_start_usage, NULL, 0 },
                { "cancel", cmd_scrub_cancel, cmd_scrub_cancel_usage, NULL, 0 },
                { "resume", cmd_scrub_resume, cmd_scrub_resume_usage, NULL, 0 },
                { "status", cmd_scrub_status, cmd_scrub_status_usage, NULL, 0 },
-               { 0, 0, 0, 0, 0 }
+               NULL_CMD_STRUCT
        }
 };