Whamcloud - gitweb
LU-13559 utils: fix lfs mirror delete error message
[fs/lustre-release.git] / lustre / utils / lfs.c
index 2edb399..1753019 100644 (file)
@@ -76,7 +76,7 @@
 #include <linux/lustre/lustre_ver.h>
 #include <linux/lustre/lustre_param.h>
 #include <linux/lnet/nidstr.h>
-#include <cyaml.h>
+#include <lnetconfig/cyaml.h>
 
 #ifndef ARRAY_SIZE
 # define ARRAY_SIZE(a) ((sizeof(a)) / (sizeof((a)[0])))
@@ -185,21 +185,27 @@ static inline int lfs_mirror_delete(int argc, char **argv)
        "                 [--overstripe-count|-C <stripe_count>]\n"     \
        "                 [--stripe-index|-i <start_ost_idx>]\n"        \
        "                 [--stripe-size|-S <stripe_size>]\n"           \
+       "                 [--extension-size|--ext-size|-z]\n"           \
        "                 [--layout|-L <pattern>]\n"                    \
-       "                 [--pool|-p <pool_name>]\n"                    \
+       "                 [--mirror_count|-N[mirror_count]]\n"          \
        "                 [--ost|-o <ost_indices>]\n"                   \
+       "                 [--pool|-p <pool_name>]\n"                    \
        "                 [--yaml|-y <yaml_template_file>]\n"           \
        "                 [--copy=<lustre_src>]\n"
 
 #define SSM_HELP_COMMON \
        "\tstripe_count: Number of OSTs to stripe over (0=fs default, -1 all)\n" \
        "\t              Using -C instead of -c allows overstriping, which\n" \
-        "\t             will place more than one stripe per OST if\n" \
-        "\t             stripe_count is greater than the number of OSTs\n" \
+       "\t              will place more than one stripe per OST if\n" \
+       "\t              stripe_count is greater than the number of OSTs\n" \
        "\tstart_ost_idx: OST index of first stripe (-1=default round robin)\n"\
        "\tstripe_size:  Number of bytes on each OST (0=fs default)\n" \
        "\t              Can be specified with K, M or G (for KB, MB, GB\n" \
        "\t              respectively)\n"                               \
+       "\textension_size:\n"                                           \
+       "\t              Number of bytes the previous component is extended\n" \
+       "\t              each time. Can be specified with K, M, G (for KB,\n" \
+       "\t              MB, GB respectively)\n"                        \
        "\tpool_name:    Name of OST pool to use (default none)\n"      \
        "\tlayout:       stripe pattern type: raid0, mdt (default raid0)\n"\
        "\tost_indices:  List of OST indices, can be repeated multiple times\n"\
@@ -207,7 +213,7 @@ static inline int lfs_mirror_delete(int argc, char **argv)
        "\t                -o <ost_1>,<ost_i>-<ost_j>,<ost_n>\n"        \
        "\t              Or:\n"                                         \
        "\t                -o <ost_1> -o <ost_i>-<ost_j> -o <ost_n>\n"  \
-       "\t              If --pool is set with --ost, then the OSTs\n"  \
+       "\t              If --pool is set with --ost then the OSTs\n"   \
        "\t              must be the members of the pool.\n"            \
        "\tcomp_end:     Extent end of component, start after previous end.\n"\
        "\t              Can be specified with K, M or G (for KB, MB, GB\n" \
@@ -227,7 +233,7 @@ static inline int lfs_mirror_delete(int argc, char **argv)
        "\t              it must follow the option without a space.\n"         \
        "\t              The option can also be repeated multiple times to\n"  \
        "\t              separate mirrors that have different layouts.\n"      \
-       "\tsetstripe options: Mirror layout\n"                                 \
+       "\tSETSTRIPE_OPTIONS: Mirror layout as with 'setstripe'\n"             \
        "\t              It can be a plain layout or a composite layout.\n"    \
        "\t              If not specified, the stripe options inherited\n"     \
        "\t              from the previous component will be used.\n"          \
@@ -244,8 +250,8 @@ static inline int lfs_mirror_delete(int argc, char **argv)
        "\t              file.\n"
 
 #define MIRROR_EXTEND_USAGE                                                   \
-       "                 <--mirror-count|-N[mirror_count]>\n"                 \
-       "                 [setstripe options|-f <victim_file>]\n"              \
+       "                 {--mirror-count|-N[mirror_count]}\n"                 \
+       "                 [SETSTRIPE_OPTIONS|-f|--file <victim_file>]\n"       \
        "                 [--no-verify]\n"
 
 #define SETSTRIPE_USAGE                                                        \
@@ -275,14 +281,14 @@ static inline int lfs_mirror_delete(int argc, char **argv)
        "\tstripe_count: stripe count of the striped directory\n"       \
        "\tmdt_index: MDT index of first stripe\n"                      \
        "\tmdt_hash:  hash type of the striped directory. mdt types:\n" \
-       "       fnv_1a_64 FNV-1a hash algorithm (default)\n"            \
+       "       crush     CRUSH hash algorithm (default)\n" \
+       "       fnv_1a_64 FNV-1a hash algorithm\n"              \
        "       all_char  sum of characters % MDT_COUNT (not recommended)\n" \
-       "       space     create subdirectories with balanced space usage\n" \
        "\tdefault_stripe: set default dirstripe of the directory\n"    \
        "\tmode: the file access permission of the directory (octal)\n" \
        "To create dir with a foreign (free format) layout :\n" \
        "setdirstripe|mkdir --foreign[=<foreign_type>] -x|-xattr <string> " \
-               "[--mode|-m mode] [--flags <hex>] <dir>\n" \
+               "[--mode|-o mode] [--flags <hex>] <dir>\n" \
        "\tmode: the mode of the directory\n" \
        "\tforeign_type: none or daos\n"
 
@@ -294,22 +300,24 @@ command_t mirror_cmdlist[] = {
          .pc_help = "Create a mirrored file.\n"
                "usage: lfs mirror create "
                "<--mirror-count|-N[mirror_count]> "
-               "[setstripe options] ... <filename|directory> ...\n"
+               "[SETSTRIPE_OPTIONS] ... <filename|directory> ...\n"
          MIRROR_CREATE_HELP },
        { .pc_name = "delete", .pc_func = lfs_mirror_delete,
-         .pc_help = "delete a mirror from a file.\n"
-               "usage: lfs mirror delete {--comp-id|-I <comp_id>|-p <pool>} <mirrored_file> ...\n"
+         .pc_help = "Delete a mirror from a file.\n"
+       "usage: lfs mirror delete {--mirror-id <mirror_id> |\n"
+       "\t               --component-id|--comp-id|-I <comp_id> |\n"
+       "\t               -p <pool>} <mirrored_file> ...\n"
        },
        { .pc_name = "extend", .pc_func = lfs_mirror_extend,
          .pc_help = "Extend a mirrored file.\n"
                "usage: lfs mirror extend "
                "<--mirror-count|-N[mirror_count]> [--no-verify] "
-               "[setstripe options|-f <victim_file>] ... <filename> ...\n"
+               "[SETSTRIPE_OPTIONS|-f <victim_file>] ... <filename> ...\n"
          MIRROR_EXTEND_HELP },
        { .pc_name = "split", .pc_func = lfs_mirror_split,
          .pc_help = "Split a mirrored file.\n"
        "usage: lfs mirror split <--mirror-id <mirror_id> |\n"
-       "\t             <--component-id|-I <comp_id>|-p <pool>> [--destroy|-d]\n"
+       "\t             --component-id|-I <comp_id>|-p <pool>> [--destroy|-d]\n"
        "\t             [-f <new_file>] <mirrored_file> ...\n"
        "\tmirror_id:   The numerical unique identifier for a mirror. It\n"
        "\t             can be fetched by lfs getstripe command.\n"
@@ -392,10 +400,10 @@ command_t cmdlist[] = {
         SSM_CMD_COMMON("setstripe --component-add")
         SSM_HELP_COMMON
         "To totally delete the default striping from an existing directory:\n"
-        "usage: setstripe -d <directory>\n"
+        "usage: setstripe [--delete|-d] <directory>\n"
         " or\n"
         "To create a mirrored file or set s default mirror layout on a directory:\n"
-        "usage: setstripe -N[mirror_count] [STRIPE_OPTIONS] <directory|filename>\n"
+        "usage: setstripe {--mirror-count|-N}[mirror_count] [SETSTRIPE_OPTIONS] <directory|filename>\n"
         " or\n"
         "To delete the last component(s) from an existing composite file\n"
         "(note that this will also delete any data in those components):\n"
@@ -428,8 +436,8 @@ command_t cmdlist[] = {
         "                  [!] --mirror-id=[+-]<id>]\n"
         "                 <directory|filename> ..."},
        {"setdirstripe", lfs_setdirstripe, 0,
-        "To create a striped directory on a specified MDT. This can only\n"
-        "be done on MDT0 with the right of administrator.\n"
+        "Create striped directory on specified MDT, same as mkdir.\n"
+        "May be restricted to root or group users, depending on settings.\n"
         "usage: setdirstripe [OPTION] <directory>\n"
         SETDIRSTRIPE_USAGE},
        {"getdirstripe", lfs_getdirstripe, 0,
@@ -440,8 +448,7 @@ command_t cmdlist[] = {
         "                    [--recursive|-r] [--yaml|-y]\n"
         "                    [--verbose|-v] [--default|-D] <dir> ..."},
        {"mkdir", lfs_setdirstripe, 0,
-        "To create a striped directory on a specified MDT. This can only\n"
-        "be done on MDT0 with the right of administrator.\n"
+        "Create striped directory on specified MDT, same as setdirstripe.\n"
         "usage: mkdir [OPTION] <directory>\n"
         SETDIRSTRIPE_USAGE},
        {"rm_entry", lfs_rmentry, 0,
@@ -457,14 +464,16 @@ command_t cmdlist[] = {
         "find files matching given attributes recursively in directory tree.\n"
         "usage: find <directory|filename> ...\n"
         "     [[!] --atime|-A [+-]N[smhdwy]] [[!] --ctime|-C [+-]N[smhdwy]]\n"
-        "     [[!] --mtime|-M [+-]N[smhdwy]] [[!] --blocks|-b N]\n"
-        "     [[!] --newer[XY] <reference>]\n"
+        "     [[!] --mtime|-M [+-]N[smhdwy]]\n"
+        "     [[!] --btime|--Btime|-B [+-]N[smhdwy]]\n"
+        "     [[!] --newer[XY] <reference>] [[!] --blocks|-b N]\n"
         "     [--maxdepth|-D N] [[!] --mdt-index|--mdt|-m <uuid|index,...>]\n"
         "     [[!] --name|-n <pattern>] [[!] --ost|-O <uuid|index,...>]\n"
         "     [--print|-P] [--print0|-0] [[!] --size|-s [+-]N[bkMGTPE]]\n"
         "     [[!] --stripe-count|-c [+-]<stripes>]\n"
         "     [[!] --stripe-index|-i <index,...>]\n"
         "     [[!] --stripe-size|-S [+-]N[kMGT]] [[!] --type|-t <filetype>]\n"
+        "     [[!] --extension-size|--ext-size|-z [+-]N[kMGT]]\n"
         "     [[!] --gid|-g|--group|-G <gid>|<gname>]\n"
         "     [[!] --uid|-u|--user|-U <uid>|<uname>] [[!] --pool <pool>]\n"
         "     [[!] --projid <projid>]\n"
@@ -474,7 +483,7 @@ command_t cmdlist[] = {
         "     [[!] --component-count [+-]<comp_cnt>]\n"
         "     [[!] --component-start [+-]N[kMGTPE]]\n"
         "     [[!] --component-end|-E [+-]N[kMGTPE]]\n"
-        "     [[!] --component-flags <comp_flags>]\n"
+        "     [[!] --component-flags {init,stale,prefer,offline,nosync,extension}]\n"
         "     [[!] --mirror-count|-N [+-]<n>]\n"
         "     [[!] --mirror-state <[^]state>]\n"
         "     [[!] --mdt-count|-T [+-]<stripes>]\n"
@@ -497,7 +506,8 @@ command_t cmdlist[] = {
        {"df", lfs_df, 0,
         "report filesystem disk space usage or inodes usage "
         "of each MDS and all OSDs or a batch belonging to a specific pool.\n"
-        "Usage: df [-i] [-h] [--lazy|-l] [--pool|-p <fsname>[.<pool>] [path]"},
+        "Usage: df [--inodes|-i] [--human-readable|-h] [--lazy|-l]\n"
+        "          [--pool|-p <fsname>[.<pool>]] [path]"},
        {"getname", lfs_getname, 0,
         "list instances and specified mount points [for specified path only]\n"
         "Usage: getname [--help|-h] [--instance|-i] [--fsname|-n] [path ...]"},
@@ -512,8 +522,8 @@ command_t cmdlist[] = {
          "                [--inode-softlimit <inode-softlimit>]\n"
          "                [--inode-hardlimit <inode-hardlimit>] <filesystem>\n"
         "       setquota [-t] <-u|--user|-g|--group|-p|--projid>\n"
-         "                [--block-grace <block-grace>]\n"
-         "                [--inode-grace <inode-grace>] <filesystem>\n"
+        "                [--block-grace 'notify'|<block-grace>]\n"
+        "                [--inode-grace 'notify'|<inode-grace>] <filesystem>\n"
         "       setquota <-U|-G|-P>\n"
         "                -b <block-softlimit> -B <block-hardlimit>\n"
         "                -i <inode-softlimit> -I <inode-hardlimit> <filesystem>\n"
@@ -536,8 +546,11 @@ command_t cmdlist[] = {
         "      Quota space rebalancing process will stop when this mininum\n"
         "      value is reached. As a result, quota exceeded can be returned\n"
         "      while many targets still have 1MB or 1K inodes of spare\n"
-        "      quota space."},
-        {"quota", lfs_quota, 0, "Display disk usage and limits.\n"
+        "      quota space.\n\n"
+        "      When setting the grace time, 'notify' can be used as grace to\n"
+        "      be notified after the quota is over soft limit but prevents\n"
+        "      the soft limit from becoming the hard limit."},
+       {"quota", lfs_quota, 0, "Display disk usage and limits.\n"
         "usage: quota [-q] [-v] [-h] [-o <obd_uuid>|-i <mdt_idx>|-I "
                       "<ost_idx>]\n"
         "             [<-u|-g|-p> <uname>|<uid>|<gname>|<gid>|<projid>] <filesystem>\n"
@@ -574,7 +587,7 @@ command_t cmdlist[] = {
        {"rmfid", lfs_rmfid, 0, "Remove file(s) by FID(s)\n"
         "usage: rmfid <fsname|rootpath> <fid> ..."},
        {"data_version", lfs_data_version, 0, "Display file data version for "
-        "a given path.\n" "usage: data_version -[n|r|w] <path>"},
+        "a given path.\n" "usage: data_version [-n|-r|-w] <path>"},
        {"hsm_state", lfs_hsm_state, 0, "Display the HSM information (states, "
         "undergoing actions) for given files.\n usage: hsm_state <file> ..."},
        {"hsm_set", lfs_hsm_set, 0, "Set HSM user flag on specified files.\n"
@@ -623,8 +636,9 @@ command_t cmdlist[] = {
         "                      it's the MDT index of first stripe\n"
         "\tmdt_count:  number of MDTs to stripe a directory over\n"
         "\tmdt_hash:   hash type of the striped directory. mdt types:\n"
-        "                      fnv_1a_64 FNV-1a hash algorithm (default)\n"
-        "                      all_char  sum of characters % MDT_COUNT\n"
+        "              all_char  (type 1)sum of characters % MDT_COUNT\n"
+        "              fnv_1a_64 (type 2)FNV-1a hash algorithm (default)\n"
+        "              crush     (type 3)CRUSH hash algorithm\n"
         "\n"
         "migrate file objects from one OST "
         "layout\nto another (may be not safe with concurent writes).\n"
@@ -706,8 +720,14 @@ command_t cmdlist[] = {
 
 static int check_hashtype(const char *hashtype)
 {
+       int type_num = atoi(hashtype);
        int i;
 
+       /* numeric hash type */
+       if (hashtype && strlen(hashtype) == 1 &&
+           (type_num > 0 && type_num < LMV_HASH_TYPE_MAX))
+               return type_num;
+       /* string hash type */
        for (i = LMV_HASH_TYPE_ALL_CHARS; i < LMV_HASH_TYPE_MAX; i++)
                if (strcmp(hashtype, mdt_hash_name[i]) == 0)
                        return i;
@@ -816,12 +836,12 @@ migrate_open_files(const char *name, __u64 migration_flags,
                }
 
                /* create, open a volatile file, use caching (ie no directio) */
-               if (param != NULL)
-                       fdv = llapi_file_open_param(volatile_file, open_flags,
-                                                   open_mode, param);
-               else
+               if (layout)
                        fdv = lfs_component_create(volatile_file, open_flags,
                                                   open_mode, layout);
+               else
+                       fdv = llapi_file_open_param(volatile_file, open_flags,
+                                                   open_mode, param);
        } while (fdv < 0 && (rc = fdv) == -EEXIST);
 
        if (rc < 0) {
@@ -1375,6 +1395,7 @@ struct mirror_args {
        struct llapi_layout     *m_layout;
        const char              *m_file;
        struct mirror_args      *m_next;
+       bool                    m_inherit;
 };
 
 /**
@@ -1726,14 +1747,36 @@ out:
        return rc;
 }
 
-static int mirror_extend_layout(char *name, struct llapi_layout *layout)
+static int mirror_extend_layout(char *name, struct llapi_layout *m_layout,
+                               bool inherit)
 {
+       struct llapi_layout *f_layout = NULL;
        struct ll_ioc_lease *data = NULL;
        int fd = -1;
        int fdv = -1;
-       int rc;
+       int rc = 0;
 
-       rc = migrate_open_files(name, 0, NULL, layout, &fd, &fdv);
+       if (inherit) {
+               f_layout = llapi_layout_get_by_path(name, 0);
+               if (f_layout == NULL) {
+                       fprintf(stderr, "%s: cannot get layout\n", progname);
+                       goto out;
+               }
+               rc = llapi_layout_get_last_init_comp(f_layout);
+               if (rc) {
+                       fprintf(stderr, "%s: cannot get the last init comp\n",
+                               progname);
+                       goto out;
+               }
+               rc = llapi_layout_mirror_inherit(f_layout, m_layout);
+               if (rc) {
+                       fprintf(stderr,
+                               "%s: cannot inherit from the last init comp\n",
+                               progname);
+                       goto out;
+               }
+       }
+       rc = migrate_open_files(name, 0, NULL, m_layout, &fd, &fdv);
        if (rc < 0)
                goto out;
 
@@ -1801,7 +1844,8 @@ static int mirror_extend(char *fname, struct mirror_args *mirror_list,
 
                        while (mirror_count > 0) {
                                rc = mirror_extend_layout(fname,
-                                                       mirror_list->m_layout);
+                                                       mirror_list->m_layout,
+                                                       mirror_list->m_inherit);
                                if (rc)
                                        break;
 
@@ -1948,6 +1992,7 @@ static int mirror_split(const char *fname, __u32 id, const char *pool,
        char *ptr;
        struct ll_ioc_lease *data;
        uint16_t mirror_count;
+       __u32 mirror_id;
        int mdt_index;
        int fd, fdv;
        int rc;
@@ -1986,22 +2031,35 @@ static int mirror_split(const char *fname, __u32 id, const char *pool,
 
                rc = llapi_layout_comp_iterate(layout, find_comp_id_by_pool,
                                               &data);
-               id = data.id;
+               mirror_id = data.id;
        } else if (mflags & MF_COMP_ID) {
                rc = llapi_layout_comp_iterate(layout, find_comp_id, &id);
-               id = mirror_id_of(id);
+               mirror_id = mirror_id_of(id);
        } else {
                rc = llapi_layout_comp_iterate(layout, find_mirror_id, &id);
+               mirror_id = id;
        }
        if (rc < 0) {
                fprintf(stderr, "error %s: failed to iterate layout of '%s'\n",
                        progname, fname);
                goto free_layout;
        } else if (rc == LLAPI_LAYOUT_ITER_CONT) {
-               fprintf(stderr,
-                    "error %s: file '%s' does not contain mirror with id %u\n",
-                       progname, fname, id);
-               goto free_layout;
+               if (mflags & MF_COMP_POOL) {
+                       fprintf(stderr,
+                               "error %s: file '%s' does not contain mirror with pool '%s'\n",
+                               progname, fname, pool);
+                       goto free_layout;
+               } else if (mflags & MF_COMP_ID) {
+                       fprintf(stderr,
+                               "error %s: file '%s' does not contain mirror with comp-id %u\n",
+                               progname, fname, id);
+                       goto free_layout;
+               } else {
+                       fprintf(stderr,
+                               "error %s: file '%s' does not contain mirror with id %u\n",
+                               progname, fname, id);
+                       goto free_layout;
+               }
        }
 
        fd = open(fname, O_RDWR);
@@ -2044,7 +2102,7 @@ static int mirror_split(const char *fname, __u32 id, const char *pool,
        if (victim_file == NULL) {
                /* use a temp file to store the splitted layout */
                if (mflags & MF_DESTROY) {
-                       if (last_non_stale_mirror(id, layout)) {
+                       if (last_non_stale_mirror(mirror_id, layout)) {
                                rc = -EUCLEAN;
                                fprintf(stderr,
                                        "%s: cannot destroy the last non-stale mirror of file '%s'\n",
@@ -2056,7 +2114,7 @@ static int mirror_split(const char *fname, __u32 id, const char *pool,
                                                        O_LOV_DELAY_CREATE);
                } else {
                        snprintf(victim, sizeof(victim), "%s.mirror~%u",
-                                fname, id);
+                                fname, mirror_id);
                        fdv = open(victim, flags, S_IRUSR | S_IWUSR);
                }
        } else {
@@ -2091,7 +2149,7 @@ static int mirror_split(const char *fname, __u32 id, const char *pool,
        data->lil_flags = LL_LEASE_LAYOUT_SPLIT;
        data->lil_count = 2;
        data->lil_ids[0] = fdv;
-       data->lil_ids[1] = id;
+       data->lil_ids[1] = mirror_id;
        rc = llapi_lease_set(fd, data);
        if (rc <= 0) {
                if (rc == 0) /* lost lease lock */
@@ -2385,13 +2443,12 @@ new_comp:
                        return rc;
                }
 
-               if (lsa->lsa_first_comp)
+               if (lsa->lsa_first_comp) {
                        prev_end = 0;
-
-               if (lsa->lsa_first_comp)
                        rc = llapi_layout_add_first_comp(layout);
-               else
+               } else {
                        rc = llapi_layout_comp_add(layout);
+               }
                if (rc) {
                        fprintf(stderr, "Add component failed. %s\n",
                                strerror(errno));
@@ -2582,26 +2639,75 @@ static int build_component(struct llapi_layout **layout,
        return rc;
 }
 
+static int build_prev_component(struct llapi_layout **layout,
+                               struct lfs_setstripe_args *prev,
+                               struct lfs_setstripe_args *lsa,
+                               bool set_extent)
+{
+       int extension = lsa->lsa_comp_flags & LCME_FL_EXTENSION;
+       int rc;
+
+       if (prev->lsa_stripe_size) {
+               if (extension) {
+                       prev->lsa_comp_end = lsa->lsa_comp_end;
+                       prev->lsa_extension_size = lsa->lsa_extension_size;
+                       prev->lsa_extension_comp = true;
+               }
+
+               rc = build_component(layout, prev, true);
+               if (rc)
+                       return rc;
+       }
+
+       /* Copy lsa to previous lsa;
+        * if this is an extension component, make the previous invalid; */
+       if (extension)
+               prev->lsa_stripe_size = 0;
+       else
+               *prev = *lsa;
+
+       return 0;
+}
+
 static int build_layout_from_yaml_node(struct cYAML *node,
                                       struct llapi_layout **layout,
                                       struct lfs_setstripe_args *lsa,
-                                      __u32 *osts)
+                                      struct lfs_setstripe_args *prevp)
 {
+       struct lfs_setstripe_args prev = { 0 };
+       __u32 *osts = lsa->lsa_tgts;
        char *string;
        int rc = 0;
 
+       if (prevp == NULL)
+               prevp = &prev;
+
        while (node) {
+               string = node->cy_string;
+
                if (node->cy_type == CYAML_TYPE_OBJECT) {
                        /* go deep to sub blocks */
+                       if (string && !strncmp(string, "component", 9) &&
+                           strncmp(string, "component0", 10) &&
+                           strncmp(string, "components", 10)) {
+                               rc = build_prev_component(layout, prevp, lsa, true);
+                               if (rc)
+                                       return rc;
+
+                               /* initialize lsa. */
+                               setstripe_args_init(lsa);
+                               lsa->lsa_first_comp = false;
+                               lsa->lsa_tgts = osts;
+                       }
+
                        rc = build_layout_from_yaml_node(node->cy_child, layout,
-                                                        lsa, osts);
+                                                        lsa, prevp);
                        if (rc)
                                return rc;
                } else {
                        if (node->cy_string == NULL)
                                return -EINVAL;
 
-                       string = node->cy_string;
                        /* skip leading lmm_ if present, to simplify parsing */
                        if (strncmp(string, "lmm_", 4) == 0)
                                string += 4;
@@ -2635,18 +2741,8 @@ static int build_layout_from_yaml_node(struct cYAML *node,
                                if (!strcmp(string, "lcm_mirror_count")) {
                                        lsa->lsa_mirror_count = node->cy_valueint;
                                } else if (!strcmp(string, "lcme_extent.e_start")) {
-                                       if (node->cy_valueint != 0 || *layout != NULL) {
-                                               rc = build_component(layout, lsa, true);
-                                               if (rc)
-                                                       return rc;
-                                       }
-
                                        if (node->cy_valueint == 0)
                                                lsa->lsa_first_comp = true;
-
-                                       /* initialize lsa */
-                                       setstripe_args_init(lsa);
-                                       lsa->lsa_tgts = osts;
                                } else if (!strcmp(string, "lcme_extent.e_end")) {
                                        if (node->cy_valueint == -1)
                                                lsa->lsa_comp_end = LUSTRE_EOF;
@@ -2656,6 +2752,9 @@ static int build_layout_from_yaml_node(struct cYAML *node,
                                        lsa->lsa_stripe_count = node->cy_valueint;
                                } else if (!strcmp(string, "stripe_size")) {
                                        lsa->lsa_stripe_size = node->cy_valueint;
+                               } else if (!strcmp(string, "extension_size")) {
+                                       lsa->lsa_extension_size = node->cy_valueint;
+                                       lsa->lsa_extension_comp = true;
                                } else if (!strcmp(string, "stripe_offset")) {
                                        lsa->lsa_stripe_off = node->cy_valueint;
                                } else if (!strcmp(string, "l_ost_idx")) {
@@ -2667,6 +2766,15 @@ static int build_layout_from_yaml_node(struct cYAML *node,
                node = node->cy_next;
        }
 
+       if (prevp == &prev) {
+               rc = build_prev_component(layout, prevp, lsa, true);
+               if (rc)
+                       return rc;
+
+               if (!(lsa->lsa_comp_flags & LCME_FL_EXTENSION))
+                       rc = build_component(layout, lsa, *layout != NULL);
+       }
+
        return rc;
 }
 
@@ -2694,13 +2802,11 @@ static int lfs_comp_create_from_yaml(char *template,
        setstripe_args_init(lsa);
        lsa->lsa_tgts = osts;
 
-       rc = build_layout_from_yaml_node(tree, layout, lsa, osts);
+       rc = build_layout_from_yaml_node(tree, layout, lsa, NULL);
        if (rc) {
                fprintf(stderr, "%s: cannot build layout from YAML file %s.\n",
                        progname, template);
                goto err;
-       } else {
-               rc = build_component(layout, lsa, *layout != NULL);
        }
        /* clean clean lsa */
        setstripe_args_init(lsa);
@@ -2799,7 +2905,7 @@ static int layout_adjust_first_extent(char *fname, struct llapi_layout *layout,
        uint64_t end;
        int rc, ret = 0;
 
-       if (layout == NULL)
+       if (layout == NULL || !(comp_add || llapi_layout_is_composite(layout)))
                return 0;
 
        errno = 0;
@@ -2916,8 +3022,10 @@ static struct mirror_args *lfs_mirror_alloc(void)
 
        while (1) {
                mirror = calloc(1, sizeof(*mirror));
-               if (mirror != NULL)
+               if (mirror != NULL) {
+                       mirror->m_inherit = false;
                        break;
+               }
 
                sleep(1);
        }
@@ -3033,6 +3141,7 @@ static int lfs_setstripe_internal(int argc, char **argv,
 /* find        { .val = 'A',   .name = "atime",        .has_arg = required_argument }*/
        /* --block is only valid in migrate mode */
        { .val = 'b',   .name = "block",        .has_arg = no_argument },
+/* find        { .val = 'B',   .name = "btime",        .has_arg = required_argument }*/
        { .val = LFS_COMP_ADD_OPT,
                        .name = "comp-add",     .has_arg = no_argument },
        { .val = LFS_COMP_ADD_OPT,
@@ -3255,6 +3364,12 @@ static int lfs_setstripe_internal(int argc, char **argv,
                        migration_block = true;
                        break;
                case 'C':
+                       if (lsa.lsa_pattern == LLAPI_LAYOUT_MDT) {
+                               fprintf(stderr,
+                                       "%s %s: -C|--overstripe-count incompatible with DoM layout\n",
+                                       progname, argv[0]);
+                               goto usage_error;
+                       }
                        lsa.lsa_pattern = LLAPI_LAYOUT_OVERSTRIPING;
                        /* fall through */
                case 'c':
@@ -3457,6 +3572,8 @@ static int lfs_setstripe_internal(int argc, char **argv,
 
                        if (last_mirror != NULL) {
                                /* wrap up last mirror */
+                               if (!setstripe_args_specified(&lsa))
+                                       last_mirror->m_inherit = true;
                                if (lsa.lsa_comp_end == 0)
                                        lsa.lsa_comp_end = LUSTRE_EOF;
 
@@ -3480,6 +3597,12 @@ static int lfs_setstripe_internal(int argc, char **argv,
                                fprintf(stderr, "warning: '--ost-list' is "
                                        "deprecated, use '--ost' instead\n");
 #endif
+                       if (lsa.lsa_pattern == LLAPI_LAYOUT_MDT) {
+                               fprintf(stderr,
+                                       "%s %s: -o|--ost incompatible with DoM layout\n",
+                                       progname, argv[0]);
+                               goto usage_error;
+                       }
                        /* -o allows overstriping, and must note it because
                         * parse_targets is shared with MDT striping, which
                         * does not allow duplicates
@@ -3565,6 +3688,11 @@ static int lfs_setstripe_internal(int argc, char **argv,
                goto usage_error;
        }
 
+       /* lfs migrate $filename should keep the file's layout by default */
+       if (migrate_mode && !setstripe_args_specified(&lsa) && !layout &&
+           !from_yaml)
+               from_copy = true;
+
        if (xattr && !foreign_mode) {
                /* only print a warning as this is harmless and will be ignored
                 */
@@ -3597,6 +3725,8 @@ static int lfs_setstripe_internal(int argc, char **argv,
        }
 
        if (mirror_mode) {
+               if (!setstripe_args_specified(&lsa))
+                       last_mirror->m_inherit = true;
                if (lsa.lsa_comp_end == 0)
                        lsa.lsa_comp_end = LUSTRE_EOF;
        }
@@ -3691,21 +3821,6 @@ static int lfs_setstripe_internal(int argc, char **argv,
                }
        }
 
-       if (layout != NULL || mirror_list != NULL) {
-               if (mirror_list)
-                       result = mirror_adjust_first_extents(mirror_list);
-               else
-                       result = layout_adjust_first_extent(fname, layout,
-                                                           comp_add);
-               if (result == -ENODATA)
-                       comp_add = 0;
-               else if (result != 0) {
-                       fprintf(stderr, "error: %s: invalid layout\n",
-                               progname);
-                       goto error;
-               }
-       }
-
        if (from_yaml && from_copy) {
                fprintf(stderr,
                        "%s: can't specify --yaml and --copy together\n",
@@ -3715,8 +3830,8 @@ static int lfs_setstripe_internal(int argc, char **argv,
 
        if ((from_yaml || from_copy) &&
            (setstripe_args_specified(&lsa) || layout != NULL)) {
-               fprintf(stderr, "error: %s: can't specify --yaml with "
-                       "-c, -S, -i, -o, -p or -E options.\n",
+               fprintf(stderr, "error: %s: can't specify --yaml or --copy with"
+                       " -c, -S, -i, -o, -p or -E options.\n",
                        argv[0]);
                goto error;
        }
@@ -3762,7 +3877,7 @@ static int lfs_setstripe_internal(int argc, char **argv,
                if (lsa.lsa_pattern != LLAPI_LAYOUT_RAID0)
                        lmu->lum_hash_type = lsa.lsa_pattern;
                else
-                       lmu->lum_hash_type = LMV_HASH_TYPE_DEFAULT;
+                       lmu->lum_hash_type = LMV_HASH_TYPE_UNKNOWN;
                if (lsa.lsa_pool_name) {
                        strncpy(lmu->lum_pool_name, lsa.lsa_pool_name,
                                sizeof(lmu->lum_pool_name) - 1);
@@ -3856,17 +3971,34 @@ static int lfs_setstripe_internal(int argc, char **argv,
                                argv[0], template);
                        goto error;
                }
-       } else if (from_copy) {
-               layout = llapi_layout_get_by_path(template, 0);
-               if (layout == NULL) {
-                       fprintf(stderr,
-                           "%s: can't create composite layout from file %s.\n",
-                               progname, template);
+       }
+
+       if (layout != NULL || mirror_list != NULL) {
+               if (mirror_list)
+                       result = mirror_adjust_first_extents(mirror_list);
+               else
+                       result = layout_adjust_first_extent(fname, layout,
+                                                           comp_add);
+               if (result == -ENODATA)
+                       comp_add = 0;
+               else if (result != 0) {
+                       fprintf(stderr, "error: %s: invalid layout\n",
+                               progname);
                        goto error;
                }
        }
 
        for (fname = argv[optind]; fname != NULL; fname = argv[++optind]) {
+               if (from_copy) {
+                       layout = llapi_layout_get_by_path(template ?: fname, 0);
+                       if (layout == NULL) {
+                               fprintf(stderr, "%s: can't create composite "
+                                       "layout from file %s.\n",
+                                       progname, template ?: fname);
+                               goto error;
+                       }
+               }
+
                if (migrate_mdt_mode) {
                        result = llapi_migrate_mdt(fname, &migrate_mdt_param);
                } else if (migrate_mode) {
@@ -4109,6 +4241,8 @@ static int lfs_find(int argc, char **argv)
         struct option long_opts[] = {
        { .val = 'A',   .name = "atime",        .has_arg = required_argument },
        { .val = 'b',   .name = "blocks",       .has_arg = required_argument },
+       { .val = 'B',   .name = "btime",        .has_arg = required_argument },
+       { .val = 'B',   .name = "Btime",        .has_arg = required_argument },
        { .val = LFS_COMP_COUNT_OPT,
                        .name = "comp-count",   .has_arg = required_argument },
        { .val = LFS_COMP_COUNT_OPT,
@@ -4137,23 +4271,49 @@ static int lfs_find(int argc, char **argv)
        { .val = LFS_NEWERXY_OPT,
                        .name = "newerac",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
+                       .name = "newerab",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
                        .name = "newerma",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
                        .name = "newermm",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
                        .name = "newermc",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
+                       .name = "newermb",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
                        .name = "newerca",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
                        .name = "newercm",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
                        .name = "newercc",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
+                       .name = "newercb",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerba",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerbm",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerbc",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerbb",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerBa",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerBm",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerBc",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerBB",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
                        .name = "newerat",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
                        .name = "newermt",      .has_arg = required_argument},
        { .val = LFS_NEWERXY_OPT,
                        .name = "newerct",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerbt",      .has_arg = required_argument},
+       { .val = LFS_NEWERXY_OPT,
+                       .name = "newerBt",      .has_arg = required_argument},
        { .val = 'c',   .name = "stripe-count", .has_arg = required_argument },
        { .val = 'c',   .name = "stripe_count", .has_arg = required_argument },
        { .val = 'C',   .name = "ctime",        .has_arg = required_argument },
@@ -4219,8 +4379,8 @@ static int lfs_find(int argc, char **argv)
 
        /* when getopt_long_only() hits '!' it returns 1, puts "!" in optarg */
        while ((c = getopt_long_only(argc, argv,
-                       "-0A:b:c:C:D:E:g:G:H:i:L:m:M:n:N:O:Ppqrs:S:t:T:u:U:vz:",
-                       long_opts, &optidx)) >= 0) {
+               "-0A:b:B:c:C:D:E:g:G:H:i:L:m:M:n:N:O:Ppqrs:S:t:T:u:U:vz:",
+               long_opts, &optidx)) >= 0) {
                 xtime = NULL;
                 xsign = NULL;
                 if (neg_opt)
@@ -4259,6 +4419,13 @@ static int lfs_find(int argc, char **argv)
                        xtime = &param.fp_atime;
                        xsign = &param.fp_asign;
                        param.fp_exclude_atime = !!neg_opt;
+                       /* no break, this falls through to 'B' for btime */
+               case 'B':
+                       if (c == 'B') {
+                               xtime = &param.fp_btime;
+                               xsign = &param.fp_bsign;
+                               param.fp_exclude_btime = !!neg_opt;
+                       }
                        /* no break, this falls through to 'C' for ctime */
                case 'C':
                        if (c == 'C') {
@@ -4503,6 +4670,24 @@ static int lfs_find(int argc, char **argv)
                                }
 
                                ref = mktime(&tm);
+                       } else if (y == 'b' || y == 'B') {
+                               lstatx_t stx;
+
+                               rc = llapi_get_lum_file(optarg, NULL, &stx,
+                                                       NULL, 0);
+                               if (rc || !(stx.stx_mask & STATX_BTIME)) {
+                                       if (!(stx.stx_mask & STATX_BTIME))
+                                               ret = -EOPNOTSUPP;
+                                       else
+                                               ret = -errno;
+                                       fprintf(stderr,
+                                               "%s: get btime failed '%s': %s\n",
+                                               progname, optarg,
+                                               strerror(-ret));
+                                       goto err;
+                               }
+
+                               ref = stx.stx_btime.tv_sec;
                        } else {
                                struct stat statbuf;
 
@@ -4544,6 +4729,10 @@ static int lfs_find(int argc, char **argv)
                        case 'c':
                                xidx = NEWERXY_CTIME;
                                break;
+                       case 'b':
+                       case 'B':
+                               xidx = NEWERXY_BTIME;
+                               break;
                        default:
                                fprintf(stderr,
                                        "%s: invalid X argument: '%c'\n",
@@ -4915,6 +5104,8 @@ static int lfs_getstripe_internal(int argc, char **argv,
        struct option long_opts[] = {
 /* find        { .val = 'A',   .name = "atime",        .has_arg = required_argument }*/
 /* find        { .val = 'b',   .name = "blocks",       .has_arg = required_argument }*/
+/* find        { .val = 'B',   .name = "btime",        .has_arg = required_argument }*/
+/* find        { .val = 'B',   .name = "Btime",        .has_arg = required_argument }*/
        { .val = LFS_COMP_COUNT_OPT,
                        .name = "comp-count",   .has_arg = no_argument },
        { .val = LFS_COMP_COUNT_OPT,
@@ -5467,15 +5658,15 @@ static struct obd_statfs_state_names {
        const char              osn_name;
        bool                    osn_err;
 } oss_names[] = {
-       { .osn_state = OS_STATE_DEGRADED,    .osn_name = 'D', .osn_err = true },
-       { .osn_state = OS_STATE_READONLY,    .osn_name = 'R', .osn_err = true },
-       { .osn_state = OS_STATE_NOPRECREATE, .osn_name = 'N', .osn_err = true },
-       { .osn_state = OS_STATE_UNUSED1,     .osn_name = '?', .osn_err = true },
-       { .osn_state = OS_STATE_UNUSED2,     .osn_name = '?', .osn_err = true },
-       { .osn_state = OS_STATE_ENOSPC,      .osn_name = 'S', .osn_err = true },
-       { .osn_state = OS_STATE_ENOINO,      .osn_name = 'I', .osn_err = true },
-       { .osn_state = OS_STATE_SUM,         .osn_name = 'a', /* aggregate */ },
-       { .osn_state = OS_STATE_NONROT,      .osn_name = 'f', /* flash */     },
+       { .osn_state = OS_STATFS_DEGRADED,   .osn_name = 'D', .osn_err = true },
+       { .osn_state = OS_STATFS_READONLY,   .osn_name = 'R', .osn_err = true },
+       { .osn_state = OS_STATFS_NOPRECREATE,.osn_name = 'N', .osn_err = true },
+       { .osn_state = OS_STATFS_UNUSED1,    .osn_name = '?', .osn_err = true },
+       { .osn_state = OS_STATFS_UNUSED2,    .osn_name = '?', .osn_err = true },
+       { .osn_state = OS_STATFS_ENOSPC,     .osn_name = 'S', .osn_err = true },
+       { .osn_state = OS_STATFS_ENOINO,     .osn_name = 'I', .osn_err = true },
+       { .osn_state = OS_STATFS_SUM,        .osn_name = 'a', /* aggregate */ },
+       { .osn_state = OS_STATFS_NONROT,     .osn_name = 'f', /* flash */     },
 };
 
 static int showdf(char *mntdir, struct obd_statfs *stat,
@@ -5989,6 +6180,11 @@ static int lfs_setdirstripe(int argc, char **argv)
                return CMD_HELP;
        }
 
+       /* if "lfs setdirstripe -D -i -1" is used, assume 1-stripe directory */
+       if (default_stripe && lsa.lsa_stripe_off == -1 &&
+           (lsa.lsa_stripe_count == LLAPI_LAYOUT_DEFAULT ||
+            lsa.lsa_stripe_count == 0))
+               lsa.lsa_stripe_count = 1;
        if (lsa.lsa_stripe_count != LLAPI_LAYOUT_DEFAULT)
                param->lsp_stripe_count = lsa.lsa_stripe_count;
        if (lsa.lsa_stripe_off == LLAPI_LAYOUT_DEFAULT)
@@ -5998,7 +6194,7 @@ static int lfs_setdirstripe(int argc, char **argv)
        if (lsa.lsa_pattern != LLAPI_LAYOUT_RAID0)
                param->lsp_stripe_pattern = lsa.lsa_pattern;
        else
-               param->lsp_stripe_pattern = LMV_HASH_TYPE_DEFAULT;
+               param->lsp_stripe_pattern = LMV_HASH_TYPE_UNKNOWN;
        param->lsp_pool = lsa.lsa_pool_name;
        param->lsp_is_specific = false;
        if (lsa.lsa_nr_tgts > 1) {
@@ -6122,6 +6318,7 @@ static int lfs_mv(int argc, char **argv)
                return CMD_HELP;
        }
 
+       lmu.lum_hash_type = LMV_HASH_TYPE_UNKNOWN;
 
        /* initialize migrate mdt parameters */
        param.fp_lmv_md = &lmu;
@@ -6129,7 +6326,7 @@ static int lfs_mv(int argc, char **argv)
        rc = llapi_migrate_mdt(argv[optind], &param);
        if (rc != 0)
                fprintf(stderr, "%s mv: cannot migrate '%s' to MDT%04x: %s\n",
-                       progname, argv[optind], param.fp_mdt_index,
+                       progname, argv[optind], lmu.lum_stripe_offset,
                        strerror(-rc));
        return rc;
 }
@@ -6203,43 +6400,33 @@ static int lfs_df(int argc, char **argv)
        return rc;
 }
 
-static int print_instance(const char *mntdir, char *fsname, size_t fsnamelen,
+static int print_instance(const char *mntdir, char *buf, size_t buflen,
                          bool opt_instance, bool opt_fsname, bool opt_mntdir)
 {
-       char *buf = fsname;
-
-       /* llapi_search_mounts() fills "fsname", but that is not called if
-        * explicit paths are specified on the command-line
-        */
-       if (opt_instance || (opt_fsname && fsname[0] == '\0')) {
-               int rc = llapi_getname(mntdir, fsname, fsnamelen);
+       int rc = 0;
 
-               if (rc < 0) {
-                       fprintf(stderr, "cannot get instance for '%s': %s\n",
-                               mntdir, strerror(-rc));
-                       return rc;
-               }
-               buf = fsname;
-               if (!opt_instance) {
-                       /* print only the fsname name */
-                       buf = strchr(fsname, '-');
-                       if (buf)
-                               *buf = '\0';
-                       buf = fsname;
-               } else if (!opt_fsname) {
-                       /* print only the instance name */
-                       buf = strchr(fsname, '-');
-                       if (buf)
-                               buf++;
-                       else
-                               buf = fsname;
-               }
+       if (opt_fsname == opt_instance) { /* both true or both false */
+               rc = llapi_getname(mntdir, buf, buflen);
        } else if (opt_fsname) {
-               /* print only the fsname */
-               buf = fsname;
+               /* llapi_search_mounts() fills @buf with fsname, but that is not
+                * called if explicit paths are specified on the command-line
+                */
+               if (buf[0] == '\0')
+                       rc = llapi_get_fsname(mntdir, buf, buflen);
+       } else /* if (opt_instance) */ {
+               rc = llapi_get_instance(mntdir, buf, buflen);
        }
 
-       printf("%s %s\n", buf, opt_mntdir ? mntdir : "");
+       if (rc < 0) {
+               fprintf(stderr, "cannot get instance for '%s': %s\n",
+                       mntdir, strerror(-rc));
+               return rc;
+       }
+
+       if (opt_mntdir)
+               printf("%s %s\n", buf, mntdir);
+       else
+               printf("%s\n", buf);
 
        return 0;
 }
@@ -6269,10 +6456,6 @@ static int lfs_getname(int argc, char **argv)
                }
        }
 
-       /* If neither option is given, print both instance and fsname */
-       if (!opt_instance && !opt_fsname)
-               opt_instance = opt_fsname = true;
-
        if (optind == argc) { /* no paths specified, get all paths. */
                char mntdir[PATH_MAX] = "", path[PATH_MAX] = "";
                int index = 0;
@@ -6455,28 +6638,38 @@ static inline int has_times_option(int argc, char **argv)
         return 0;
 }
 
-int lfs_setquota_times(int argc, char **argv)
+static inline int lfs_verify_poolarg(char *pool)
 {
-        int c, rc;
-        struct if_quotactl qctl;
-        char *mnt, *obd_type = (char *)qctl.obd_type;
-        struct obd_dqblk *dqb = &qctl.qc_dqblk;
-        struct obd_dqinfo *dqi = &qctl.qc_dqinfo;
-        struct option long_opts[] = {
+       if (strnlen(optarg, LOV_MAXPOOLNAME + 1) > LOV_MAXPOOLNAME) {
+               fprintf(stderr,
+                       "Pool name '%.*s' is longer than %d\n",
+                       LOV_MAXPOOLNAME, pool, LOV_MAXPOOLNAME);
+               return 1;
+       }
+       return 0;
+}
+
+int lfs_setquota_times(int argc, char **argv, struct if_quotactl *qctl)
+{
+       int c, rc;
+       char *mnt, *obd_type = (char *)qctl->obd_type;
+       struct obd_dqblk *dqb = &qctl->qc_dqblk;
+       struct obd_dqinfo *dqi = &qctl->qc_dqinfo;
+       struct option long_opts[] = {
        { .val = 'b',   .name = "block-grace",  .has_arg = required_argument },
        { .val = 'g',   .name = "group",        .has_arg = no_argument },
        { .val = 'i',   .name = "inode-grace",  .has_arg = required_argument },
        { .val = 'p',   .name = "projid",       .has_arg = no_argument },
        { .val = 't',   .name = "times",        .has_arg = no_argument },
        { .val = 'u',   .name = "user",         .has_arg = no_argument },
+       { .val = 'o',   .name = "pool",         .has_arg = required_argument },
        { .name = NULL } };
        int qtype;
 
-       memset(&qctl, 0, sizeof(qctl));
-       qctl.qc_cmd  = LUSTRE_Q_SETINFO;
-       qctl.qc_type = ALLQUOTA;
+       qctl->qc_cmd  = LUSTRE_Q_SETINFO;
+       qctl->qc_type = ALLQUOTA;
 
-       while ((c = getopt_long(argc, argv, "b:gi:ptu",
+       while ((c = getopt_long(argc, argv, "b:gi:ptuo:",
                                long_opts, NULL)) != -1) {
                switch (c) {
                case 'u':
@@ -6488,63 +6681,78 @@ int lfs_setquota_times(int argc, char **argv)
                case 'p':
                        qtype = PRJQUOTA;
 quota_type:
-                       if (qctl.qc_type != ALLQUOTA) {
+                       if (qctl->qc_type != ALLQUOTA) {
                                fprintf(stderr, "error: -u/g/p can't be used "
                                                 "more than once\n");
                                return CMD_HELP;
                        }
-                       qctl.qc_type = qtype;
+                       qctl->qc_type = qtype;
                        break;
-                case 'b':
-                        if ((dqi->dqi_bgrace = str2sec(optarg)) == ULONG_MAX) {
-                                fprintf(stderr, "error: bad block-grace: %s\n",
-                                        optarg);
-                                return CMD_HELP;
-                        }
-                        dqb->dqb_valid |= QIF_BTIME;
-                        break;
-                case 'i':
-                        if ((dqi->dqi_igrace = str2sec(optarg)) == ULONG_MAX) {
-                                fprintf(stderr, "error: bad inode-grace: %s\n",
-                                        optarg);
-                                return CMD_HELP;
-                        }
-                        dqb->dqb_valid |= QIF_ITIME;
-                        break;
-                case 't': /* Yes, of course! */
-                        break;
-                default: /* getopt prints error message for us when opterr != 0 */
-                        return CMD_HELP;
-                }
-        }
+               case 'b':
+                       if (strncmp(optarg, NOTIFY_GRACE,
+                                   strlen(NOTIFY_GRACE)) == 0) {
+                               dqi->dqi_bgrace = NOTIFY_GRACE_TIME;
+                       } else {
+                               dqi->dqi_bgrace = str2sec(optarg);
+                               if (dqi->dqi_bgrace >= NOTIFY_GRACE_TIME) {
+                                       fprintf(stderr, "error: bad "
+                                               "block-grace: %s\n", optarg);
+                                       return CMD_HELP;
+                               }
+                       }
+                       dqb->dqb_valid |= QIF_BTIME;
+                       break;
+               case 'i':
+                       if (strncmp(optarg, NOTIFY_GRACE,
+                                   strlen(NOTIFY_GRACE)) == 0) {
+                               dqi->dqi_igrace = NOTIFY_GRACE_TIME;
+                       } else {
+                               dqi->dqi_igrace = str2sec(optarg);
+                               if (dqi->dqi_igrace >= NOTIFY_GRACE_TIME) {
+                                       fprintf(stderr, "error: bad "
+                                               "inode-grace: %s\n", optarg);
+                                       return CMD_HELP;
+                               }
+                       }
+                       dqb->dqb_valid |= QIF_ITIME;
+                       break;
+               case 't': /* Yes, of course! */
+                       break;
+               case 'o':
+                       if (lfs_verify_poolarg(optarg))
+                               return -1;
+                       fprintf(stdout,
+                               "Trying to set grace for pool %s\n", optarg);
+                       strncpy(qctl->qc_poolname, optarg, LOV_MAXPOOLNAME);
+                       qctl->qc_cmd  = LUSTRE_Q_SETINFOPOOL;
+                       break;
+               /* getopt prints error message for us when opterr != 0 */
+               default:
+                       return CMD_HELP;
+               }
+       }
 
-       if (qctl.qc_type == ALLQUOTA) {
+       if (qctl->qc_type == ALLQUOTA) {
                fprintf(stderr, "error: neither -u, -g nor -p specified\n");
-                return CMD_HELP;
-        }
-
-        if (optind != argc - 1) {
-                fprintf(stderr, "error: unexpected parameters encountered\n");
-                return CMD_HELP;
-        }
+               return CMD_HELP;
+       }
 
-       if ((dqb->dqb_valid | QIF_BTIME && dqi->dqi_bgrace >= UINT_MAX) ||
-           (dqb->dqb_valid | QIF_ITIME && dqi->dqi_igrace >= UINT_MAX)) {
-               fprintf(stderr, "error: grace time is too large\n");
+       if (optind != argc - 1) {
+               fprintf(stderr, "error: unexpected parameters encountered\n");
                return CMD_HELP;
        }
 
-        mnt = argv[optind];
-        rc = llapi_quotactl(mnt, &qctl);
-        if (rc) {
-                if (*obd_type)
-                        fprintf(stderr, "%s %s ", obd_type,
-                                obd_uuid2str(&qctl.obd_uuid));
-                fprintf(stderr, "setquota failed: %s\n", strerror(-rc));
-                return rc;
-        }
+       mnt = argv[optind];
+       rc = llapi_quotactl(mnt, qctl);
+       if (rc) {
+               if (*obd_type)
+                       fprintf(stderr, "%s %s ", obd_type,
+                               obd_uuid2str(&qctl->obd_uuid));
+               fprintf(stderr, "setquota failed: %s\n", strerror(-rc));
+               return rc;
+       }
 
-        return 0;
+       return 0;
 }
 
 #define BSLIMIT (1 << 0)
@@ -6555,9 +6763,9 @@ quota_type:
 int lfs_setquota(int argc, char **argv)
 {
        int c, rc = 0;
-       struct if_quotactl qctl;
-       char *mnt, *obd_type = (char *)qctl.obd_type;
-       struct obd_dqblk *dqb = &qctl.qc_dqblk;
+       struct if_quotactl *qctl;
+       char *mnt, *obd_type;
+       struct obd_dqblk *dqb;
        struct option long_opts[] = {
        { .val = 'b',   .name = "block-softlimit",
                                                .has_arg = required_argument },
@@ -6574,79 +6782,93 @@ int lfs_setquota(int argc, char **argv)
        { .val = 'P',   .name = "default-prj",  .has_arg = no_argument },
        { .val = 'u',   .name = "user",         .has_arg = required_argument },
        { .val = 'U',   .name = "default-usr",  .has_arg = no_argument },
+       { .val = 'o',   .name = "pool",         .has_arg = required_argument },
        { .name = NULL } };
        unsigned limit_mask = 0;
        char *endptr;
        bool use_default = false;
-       int qtype;
+       int qtype, qctl_len;
+
+       qctl_len = sizeof(*qctl) + LOV_MAXPOOLNAME + 1;
+       qctl = malloc(qctl_len);
+       if (!qctl)
+               return -ENOMEM;
 
-       if (has_times_option(argc, argv))
-               return lfs_setquota_times(argc, argv);
+       memset(qctl, 0, qctl_len);
+       obd_type = (char *)qctl->obd_type;
+       dqb = &qctl->qc_dqblk;
 
-       memset(&qctl, 0, sizeof(qctl));
-       qctl.qc_cmd  = LUSTRE_Q_SETQUOTA;
-       qctl.qc_type = ALLQUOTA; /* ALLQUOTA makes no sense for setquota,
+       if (has_times_option(argc, argv)) {
+               rc = lfs_setquota_times(argc, argv, qctl);
+               goto out;
+       }
+
+       qctl->qc_cmd  = LUSTRE_Q_SETQUOTA;
+       qctl->qc_type = ALLQUOTA; /* ALLQUOTA makes no sense for setquota,
                                  * so it can be used as a marker that qc_type
                                  * isn't reinitialized from command line */
 
-       while ((c = getopt_long(argc, argv, "b:B:dg:Gi:I:p:Pu:U",
+       while ((c = getopt_long(argc, argv, "b:B:dg:Gi:I:p:Pu:Uo:",
                long_opts, NULL)) != -1) {
                switch (c) {
                case 'U':
-                       qctl.qc_cmd = LUSTRE_Q_SETDEFAULT;
+                       qctl->qc_cmd = LUSTRE_Q_SETDEFAULT;
                        qtype = USRQUOTA;
-                       qctl.qc_id = 0;
+                       qctl->qc_id = 0;
                        goto quota_type_def;
                case 'u':
                        qtype = USRQUOTA;
-                       rc = name2uid(&qctl.qc_id, optarg);
+                       rc = name2uid(&qctl->qc_id, optarg);
                        goto quota_type;
                case 'G':
-                       qctl.qc_cmd = LUSTRE_Q_SETDEFAULT;
+                       qctl->qc_cmd = LUSTRE_Q_SETDEFAULT;
                        qtype = GRPQUOTA;
-                       qctl.qc_id = 0;
+                       qctl->qc_id = 0;
                        goto quota_type_def;
                 case 'g':
                        qtype = GRPQUOTA;
-                       rc = name2gid(&qctl.qc_id, optarg);
+                       rc = name2gid(&qctl->qc_id, optarg);
                        goto quota_type;
                case 'P':
-                       qctl.qc_cmd = LUSTRE_Q_SETDEFAULT;
+                       qctl->qc_cmd = LUSTRE_Q_SETDEFAULT;
                        qtype = PRJQUOTA;
-                       qctl.qc_id = 0;
+                       qctl->qc_id = 0;
                        goto quota_type_def;
                case 'p':
                        qtype = PRJQUOTA;
-                       rc = name2projid(&qctl.qc_id, optarg);
+                       rc = name2projid(&qctl->qc_id, optarg);
 quota_type:
                        if (rc) {
-                               qctl.qc_id = strtoul(optarg, &endptr, 10);
+                               qctl->qc_id = strtoul(optarg, &endptr, 10);
                                if (*endptr != '\0') {
                                        fprintf(stderr, "%s setquota: invalid"
                                                " id '%s'\n", progname, optarg);
-                                       return -1;
+                                       rc = -1;
+                                       goto out;
                                }
                        }
 
-                       if (qctl.qc_id == 0) {
+                       if (qctl->qc_id == 0) {
                                fprintf(stderr, "%s setquota: can't set quota"
                                        " for root usr/group/project.\n",
                                        progname);
-                               return -1;
+                               rc = -1;
+                               goto out;
                        }
 
 quota_type_def:
-                       if (qctl.qc_type != ALLQUOTA) {
+                       if (qctl->qc_type != ALLQUOTA) {
                                fprintf(stderr,
                                        "%s setquota: only one of -u, -U, -g,"
                                        " -G, -p or -P may be specified\n",
                                        progname);
-                               return CMD_HELP;
+                               rc = CMD_HELP;
+                               goto out;
                        }
-                       qctl.qc_type = qtype;
+                       qctl->qc_type = qtype;
                        break;
                case 'd':
-                       qctl.qc_cmd = LUSTRE_Q_SETDEFAULT;
+                       qctl->qc_cmd = LUSTRE_Q_SETDEFAULT;
                        use_default = true;
                        break;
                case 'b':
@@ -6699,26 +6921,39 @@ quota_type_def:
                                        (unsigned long long)dqb->dqb_ihardlimit,
                                        progname);
                        break;
+               case 'o':
+                       if (lfs_verify_poolarg(optarg)) {
+                               rc = -1;
+                               goto out;
+                       }
+                       fprintf(stdout,
+                               "Trying to set quota for pool %s\n", optarg);
+                       strncpy(qctl->qc_poolname, optarg, LOV_MAXPOOLNAME);
+                       qctl->qc_cmd  = LUSTRE_Q_SETQUOTAPOOL;
+                       break;
                default:
                        fprintf(stderr,
                                "%s setquota: unrecognized option '%s'\n",
                                progname, argv[optind - 1]);
-                       return CMD_HELP;
+                       rc = CMD_HELP;
+                       goto out;
                }
        }
 
-       if (qctl.qc_type == ALLQUOTA) {
+       if (qctl->qc_type == ALLQUOTA) {
                fprintf(stderr,
                        "%s setquota: either -u or -g must be specified\n",
                        progname);
-               return CMD_HELP;
+               rc = CMD_HELP;
+               goto out;
        }
 
        if (!use_default && limit_mask == 0) {
                fprintf(stderr,
                        "%s setquota: at least one limit must be specified\n",
                        progname);
-               return CMD_HELP;
+               rc = CMD_HELP;
+               goto out;
        }
 
        if (use_default && limit_mask != 0) {
@@ -6726,22 +6961,25 @@ quota_type_def:
                        "%s setquota: limits should not be specified when"
                        " using default quota\n",
                        progname);
-               return CMD_HELP;
+               rc = CMD_HELP;
+               goto out;
        }
 
-       if (use_default && qctl.qc_id == 0) {
+       if (use_default && qctl->qc_id == 0) {
                fprintf(stderr,
                        "%s setquota: can not set default quota for root"
                        " user/group/project\n",
                        progname);
-               return CMD_HELP;
+               rc = CMD_HELP;
+               goto out;
        }
 
        if (optind != argc - 1) {
                fprintf(stderr,
                        "%s setquota: filesystem not specified or unexpected argument '%s'\n",
                        progname, argv[optind]);
-               return CMD_HELP;
+               rc = CMD_HELP;
+               goto out;
        }
 
         mnt = argv[optind];
@@ -6758,12 +6996,12 @@ quota_type_def:
                   (!(limit_mask & IHLIMIT) ^ !(limit_mask & ISLIMIT))) {
                /* sigh, we can't just set blimits/ilimits */
                struct if_quotactl tmp_qctl = {.qc_cmd  = LUSTRE_Q_GETQUOTA,
-                                              .qc_type = qctl.qc_type,
-                                              .qc_id   = qctl.qc_id};
+                                              .qc_type = qctl->qc_type,
+                                              .qc_id   = qctl->qc_id};
 
                rc = llapi_quotactl(mnt, &tmp_qctl);
                if (rc < 0)
-                       return rc;
+                       goto out;
 
                if (!(limit_mask & BHLIMIT))
                        dqb->dqb_bhardlimit = tmp_qctl.qc_dqblk.dqb_bhardlimit;
@@ -6789,17 +7027,17 @@ quota_type_def:
        dqb->dqb_valid |= (limit_mask & (BHLIMIT | BSLIMIT)) ? QIF_BLIMITS : 0;
        dqb->dqb_valid |= (limit_mask & (IHLIMIT | ISLIMIT)) ? QIF_ILIMITS : 0;
 
-       rc = llapi_quotactl(mnt, &qctl);
+       rc = llapi_quotactl(mnt, qctl);
        if (rc) {
                if (*obd_type)
                        fprintf(stderr,
                                "%s setquota: cannot quotactl '%s' '%s': %s",
                                progname, obd_type,
-                               obd_uuid2str(&qctl.obd_uuid), strerror(-rc));
-               return rc;
+                               obd_uuid2str(&qctl->obd_uuid), strerror(-rc));
        }
-
-       return 0;
+out:
+       free(qctl);
+       return rc;
 }
 
 /* Converts seconds value into format string
@@ -6908,6 +7146,7 @@ static void print_quota(char *mnt, struct if_quotactl *qctl, int type,
        time(&now);
 
        if (qctl->qc_cmd == LUSTRE_Q_GETQUOTA || qctl->qc_cmd == Q_GETOQUOTA ||
+           qctl->qc_cmd == LUSTRE_Q_GETQUOTAPOOL ||
            qctl->qc_cmd == LUSTRE_Q_GETDEFAULT) {
                int bover = 0, iover = 0;
                struct obd_dqblk *dqb = &qctl->qc_dqblk;
@@ -7005,17 +7244,23 @@ static void print_quota(char *mnt, struct if_quotactl *qctl, int type,
                else
                        printf(" %7s %7s %7s %7s", "-", "-", "-", "-");
                printf("\n");
+       } else if (qctl->qc_cmd == LUSTRE_Q_GETINFO || LUSTRE_Q_GETINFOPOOL ||
+                  qctl->qc_cmd == Q_GETOINFO) {
+               char bgtimebuf[40];
+               char igtimebuf[40];
 
-        } else if (qctl->qc_cmd == LUSTRE_Q_GETINFO ||
-                   qctl->qc_cmd == Q_GETOINFO) {
-                char bgtimebuf[40];
-                char igtimebuf[40];
+               if (qctl->qc_dqinfo.dqi_bgrace == NOTIFY_GRACE_TIME)
+                       strncpy(bgtimebuf, NOTIFY_GRACE, 40);
+               else
+                       sec2str(qctl->qc_dqinfo.dqi_bgrace, bgtimebuf, rc);
+               if (qctl->qc_dqinfo.dqi_igrace == NOTIFY_GRACE_TIME)
+                       strncpy(igtimebuf, NOTIFY_GRACE, 40);
+               else
+                       sec2str(qctl->qc_dqinfo.dqi_igrace, igtimebuf, rc);
 
-                sec2str(qctl->qc_dqinfo.dqi_bgrace, bgtimebuf, rc);
-                sec2str(qctl->qc_dqinfo.dqi_igrace, igtimebuf, rc);
-                printf("Block grace time: %s; Inode grace time: %s\n",
-                       bgtimebuf, igtimebuf);
-        }
+               printf("Block grace time: %s; Inode grace time: %s\n",
+                      bgtimebuf, igtimebuf);
+       }
 }
 
 static int print_obd_quota(char *mnt, struct if_quotactl *qctl, int is_mdt,
@@ -7024,6 +7269,8 @@ static int print_obd_quota(char *mnt, struct if_quotactl *qctl, int is_mdt,
         int rc = 0, rc1 = 0, count = 0;
         __u32 valid = qctl->qc_valid;
 
+       /* TODO: for commands LUSTRE_Q_"S\|G"ETQUOTAPOOL we need
+        * to go only through OSTs that belong to requested pool. */
         rc = llapi_get_obd_count(mnt, &count, is_mdt);
         if (rc) {
                 fprintf(stderr, "can not get %s count: %s\n",
@@ -7112,6 +7359,7 @@ static int get_print_quota(char *mnt, char *name, struct if_quotactl *qctl,
        }
 
        if ((qctl->qc_cmd == LUSTRE_Q_GETQUOTA ||
+            qctl->qc_cmd == LUSTRE_Q_GETQUOTAPOOL ||
             qctl->qc_cmd == LUSTRE_Q_GETDEFAULT) && !quiet)
                print_quota_title(name, qctl, human_readable, show_default);
 
@@ -7121,14 +7369,16 @@ static int get_print_quota(char *mnt, char *name, struct if_quotactl *qctl,
        if (qctl->qc_valid != QC_GENERAL)
                mnt = "";
 
-       inacc = (qctl->qc_cmd == LUSTRE_Q_GETQUOTA) &&
+       inacc = (qctl->qc_cmd == LUSTRE_Q_GETQUOTA ||
+                qctl->qc_cmd == LUSTRE_Q_GETQUOTAPOOL) &&
                ((qctl->qc_dqblk.dqb_valid & (QIF_LIMITS|QIF_USAGE)) !=
                 (QIF_LIMITS|QIF_USAGE));
 
        print_quota(mnt, qctl, QC_GENERAL, rc1, human_readable, show_default);
 
        if (!show_default && verbose &&
-           qctl->qc_valid == QC_GENERAL && qctl->qc_cmd != LUSTRE_Q_GETINFO) {
+           qctl->qc_valid == QC_GENERAL && qctl->qc_cmd != LUSTRE_Q_GETINFO &&
+           qctl->qc_cmd != LUSTRE_Q_GETINFOPOOL) {
                char strbuf[STRBUF_LEN];
 
                rc2 = print_obd_quota(mnt, qctl, 1, human_readable,
@@ -7325,17 +7575,28 @@ static int lfs_quota(int argc, char **argv)
 {
        int c;
        char *mnt, *name = NULL;
-       struct if_quotactl qctl = { .qc_cmd = LUSTRE_Q_GETQUOTA,
-                                   .qc_type = ALLQUOTA };
-       char *obd_uuid = (char *)qctl.obd_uuid.uuid;
+       struct if_quotactl *qctl;
+       char *obd_uuid;
        int rc = 0, rc1 = 0, verbose = 0, quiet = 0;
        char *endptr;
        __u32 valid = QC_GENERAL, idx = 0;
        bool human_readable = false;
        bool show_default = false;
        int qtype;
+       struct option long_opts[] = {
+       { .val = 1,     .name = "pool", .has_arg = required_argument },
+       { .name = NULL } };
+
+       qctl = calloc(1, sizeof(*qctl) + LOV_MAXPOOLNAME + 1);
+       if (!qctl)
+               return -ENOMEM;
+
+       qctl->qc_cmd = LUSTRE_Q_GETQUOTA;
+       qctl->qc_type = ALLQUOTA;
+       obd_uuid = (char *)qctl->obd_uuid.uuid;
 
-       while ((c = getopt(argc, argv, "gGi:I:o:pPqtuUvh")) != -1) {
+       while ((c = getopt_long(argc, argv, "gGi:I:o:pPqtuUvh",
+               long_opts, NULL)) != -1) {
                switch (c) {
                case 'U':
                        show_default = true;
@@ -7352,39 +7613,42 @@ static int lfs_quota(int argc, char **argv)
                case 'p':
                        qtype = PRJQUOTA;
 quota_type:
-                       if (qctl.qc_type != ALLQUOTA) {
+                       if (qctl->qc_type != ALLQUOTA) {
                                fprintf(stderr,
                                        "%s quota: only one of -u, -g, or -p may be specified\n",
                                        progname);
-                               return CMD_HELP;
+                               rc = CMD_HELP;
+                               goto out;
                        }
-                       qctl.qc_type = qtype;
+                       qctl->qc_type = qtype;
                        break;
                 case 't':
-                        qctl.qc_cmd = LUSTRE_Q_GETINFO;
+                       qctl->qc_cmd = LUSTRE_Q_GETINFO;
                         break;
                 case 'o':
-                        valid = qctl.qc_valid = QC_UUID;
-                       snprintf(obd_uuid, sizeof(qctl.obd_uuid), "%s", optarg);
+                       valid = qctl->qc_valid = QC_UUID;
+                       snprintf(obd_uuid, sizeof(*obd_uuid), "%s", optarg);
                         break;
                case 'i':
-                       valid = qctl.qc_valid = QC_MDTIDX;
-                       idx = qctl.qc_idx = atoi(optarg);
+                       valid = qctl->qc_valid = QC_MDTIDX;
+                       idx = qctl->qc_idx = atoi(optarg);
                        if (idx == 0 && *optarg != '0') {
                                fprintf(stderr,
                                        "%s quota: invalid MDT index '%s'\n",
                                        progname, optarg);
-                               return CMD_HELP;
+                               rc = CMD_HELP;
+                               goto out;
                        }
                        break;
                case 'I':
-                       valid = qctl.qc_valid = QC_OSTIDX;
-                       idx = qctl.qc_idx = atoi(optarg);
+                       valid = qctl->qc_valid = QC_OSTIDX;
+                       idx = qctl->qc_idx = atoi(optarg);
                        if (idx == 0 && *optarg != '0') {
                                fprintf(stderr,
                                        "%s quota: invalid OST index '%s'\n",
                                        progname, optarg);
-                               return CMD_HELP;
+                               rc = CMD_HELP;
+                               goto out;
                        }
                        break;
                 case 'v':
@@ -7396,92 +7660,110 @@ quota_type:
                case 'h':
                        human_readable = true;
                        break;
+               case 1:
+                       if (lfs_verify_poolarg(optarg)) {
+                               rc = -1;
+                               goto out;
+                       }
+                       strncpy(qctl->qc_poolname, optarg, LOV_MAXPOOLNAME);
+                       qctl->qc_cmd = qctl->qc_cmd == LUSTRE_Q_GETINFO ?
+                                               LUSTRE_Q_GETINFOPOOL :
+                                               LUSTRE_Q_GETQUOTAPOOL;
+                       break;
                default:
                        fprintf(stderr, "%s quota: unrecognized option '%s'\n",
                                progname, argv[optind - 1]);
-                       return CMD_HELP;
+                       rc = CMD_HELP;
+                       goto out;
                }
        }
 
         /* current uid/gid info for "lfs quota /path/to/lustre/mount" */
-       if (qctl.qc_cmd == LUSTRE_Q_GETQUOTA && qctl.qc_type == ALLQUOTA &&
-           optind == argc - 1 && !show_default) {
+       if ((qctl->qc_cmd == LUSTRE_Q_GETQUOTA ||
+            qctl->qc_cmd == LUSTRE_Q_GETQUOTAPOOL) &&
+            qctl->qc_type == ALLQUOTA &&
+            optind == argc - 1 && !show_default) {
 
-               qctl.qc_cmd = LUSTRE_Q_GETQUOTA;
-               qctl.qc_idx = idx;
+               qctl->qc_idx = idx;
 
                for (qtype = USRQUOTA; qtype <= GRPQUOTA; qtype++) {
-                       qctl.qc_type = qtype;
-                       qctl.qc_valid = valid;
+                       qctl->qc_type = qtype;
+                       qctl->qc_valid = valid;
                        if (qtype == USRQUOTA) {
-                               qctl.qc_id = geteuid();
-                               rc = uid2name(&name, qctl.qc_id);
+                               qctl->qc_id = geteuid();
+                               rc = uid2name(&name, qctl->qc_id);
                        } else {
-                               qctl.qc_id = getegid();
-                               rc = gid2name(&name, qctl.qc_id);
-                               memset(&qctl.qc_dqblk, 0,
-                                      sizeof(qctl.qc_dqblk));
+                               qctl->qc_id = getegid();
+                               rc = gid2name(&name, qctl->qc_id);
+                               memset(&qctl->qc_dqblk, 0,
+                                      sizeof(qctl->qc_dqblk));
                        }
                        if (rc)
                                name = "<unknown>";
                        mnt = argv[optind];
-                       rc1 = get_print_quota(mnt, name, &qctl, verbose, quiet,
+                       rc1 = get_print_quota(mnt, name, qctl, verbose, quiet,
                                              human_readable, show_default);
                        if (rc1 && !rc)
                                rc = rc1;
                }
-               return rc;
+               goto out;
        /* lfs quota -u username /path/to/lustre/mount */
-       } else if (qctl.qc_cmd == LUSTRE_Q_GETQUOTA) {
+       } else if (qctl->qc_cmd == LUSTRE_Q_GETQUOTA ||
+                  qctl->qc_cmd == LUSTRE_Q_GETQUOTAPOOL) {
                /* options should be followed by u/g-name and mntpoint */
                if ((!show_default && optind + 2 != argc) ||
                    (show_default && optind + 1 != argc) ||
-                   qctl.qc_type == ALLQUOTA) {
+                   qctl->qc_type == ALLQUOTA) {
                        fprintf(stderr,
                                "%s quota: name and mount point must be specified\n",
                                progname);
-                       return CMD_HELP;
+                       rc = CMD_HELP;
+                       goto out;
                }
 
                if (!show_default) {
                        name = argv[optind++];
-                       switch (qctl.qc_type) {
+                       switch (qctl->qc_type) {
                        case USRQUOTA:
-                               rc = name2uid(&qctl.qc_id, name);
+                               rc = name2uid(&qctl->qc_id, name);
                                break;
                        case GRPQUOTA:
-                               rc = name2gid(&qctl.qc_id, name);
+                               rc = name2gid(&qctl->qc_id, name);
                                break;
                        case PRJQUOTA:
-                               rc = name2projid(&qctl.qc_id, name);
+                               rc = name2projid(&qctl->qc_id, name);
                                break;
                        default:
                                rc = -ENOTSUP;
                                break;
                        }
                } else {
-                       qctl.qc_valid = QC_GENERAL;
-                       qctl.qc_cmd = LUSTRE_Q_GETDEFAULT;
-                       qctl.qc_id = 0;
+                       qctl->qc_valid = QC_GENERAL;
+                       qctl->qc_cmd = LUSTRE_Q_GETDEFAULT;
+                       qctl->qc_id = 0;
                }
 
                if (rc) {
-                       qctl.qc_id = strtoul(name, &endptr, 10);
+                       qctl->qc_id = strtoul(name, &endptr, 10);
                        if (*endptr != '\0') {
                                fprintf(stderr, "%s quota: invalid id '%s'\n",
                                        progname, name);
-                               return CMD_HELP;
+                               rc = CMD_HELP;
+                               goto out;
                        }
                }
-       } else if (optind + 1 != argc || qctl.qc_type == ALLQUOTA) {
+       } else if (optind + 1 != argc || qctl->qc_type == ALLQUOTA) {
                fprintf(stderr, "%s quota: missing quota info argument(s)\n",
                        progname);
-               return CMD_HELP;
+               rc = CMD_HELP;
+               goto out;
        }
 
        mnt = argv[optind];
-       rc = get_print_quota(mnt, name, &qctl, verbose, quiet,
+       rc = get_print_quota(mnt, name, qctl, verbose, quiet,
                             human_readable, show_default);
+out:
+       free(qctl);
        return rc;
 }
 #endif /* HAVE_SYS_QUOTA_H! */
@@ -9433,22 +9715,16 @@ static inline int verify_mirror_id_by_fd(int fd, __u16 mirror_id)
  * \retval     1  not the same file
  * \retval     <0 error code
  */
-static inline int check_same_file(const char *f1, const char *f2)
+static inline int check_same_file(int fd, const char *f2)
 {
        struct stat stbuf1;
        struct stat stbuf2;
 
-       if (stat(f1, &stbuf1) < 0) {
-               fprintf(stderr, "%s: cannot stat file '%s': %s\n",
-                       progname, f1, strerror(errno));
+       if (fstat(fd, &stbuf1) < 0)
                return -errno;
-       }
 
-       if (stat(f2, &stbuf2) < 0) {
-               fprintf(stderr, "%s: cannot stat file '%s': %s\n",
-                       progname, f2, strerror(errno));
-               return -errno;
-       }
+       if (stat(f2, &stbuf2) < 0)
+               return 1;
 
        if (stbuf1.st_rdev == stbuf2.st_rdev &&
            stbuf1.st_ino == stbuf2.st_ino)
@@ -9514,19 +9790,6 @@ static inline int lfs_mirror_read(int argc, char **argv)
 
        /* open mirror file */
        fname = argv[optind];
-
-       if (outfile) {
-               rc = check_same_file(fname, outfile);
-               if (rc == 0) {
-                       fprintf(stderr,
-                       "%s %s: output file cannot be the mirrored file\n",
-                               progname, argv[0]);
-                       return -EINVAL;
-               }
-               if (rc < 0)
-                       return rc;
-       }
-
        fd = open(fname, O_DIRECT | O_RDONLY);
        if (fd < 0) {
                fprintf(stderr, "%s %s: cannot open '%s': %s\n",
@@ -9543,7 +9806,7 @@ static inline int lfs_mirror_read(int argc, char **argv)
                goto close_fd;
        }
 
-       /* open output file */
+       /* open output file - O_EXCL ensures output is not the same as input */
        if (outfile) {
                outfd = open(outfile, O_EXCL | O_WRONLY | O_CREAT, 0644);
                if (outfd < 0) {
@@ -9683,19 +9946,6 @@ static inline int lfs_mirror_write(int argc, char **argv)
 
        /* open mirror file */
        fname = argv[optind];
-
-       if (inputfile) {
-               rc = check_same_file(fname, inputfile);
-               if (rc == 0) {
-                       fprintf(stderr,
-                       "%s %s: input file cannot be the mirrored file\n",
-                               progname, argv[0]);
-                       return -EINVAL;
-               }
-               if (rc < 0)
-                       return rc;
-       }
-
        fd = open(fname, O_DIRECT | O_WRONLY);
        if (fd < 0) {
                fprintf(stderr, "%s %s: cannot open '%s': %s\n",
@@ -9714,6 +9964,16 @@ static inline int lfs_mirror_write(int argc, char **argv)
 
        /* open input file */
        if (inputfile) {
+               rc = check_same_file(fd, inputfile);
+               if (rc == 0) {
+                       fprintf(stderr,
+                       "%s %s: input file cannot be the mirrored file\n",
+                               progname, argv[0]);
+                       goto close_fd;
+               }
+               if (rc < 0)
+                       goto close_fd;
+
                inputfd = open(inputfile, O_RDONLY, 0644);
                if (inputfd < 0) {
                        fprintf(stderr, "%s %s: cannot open file '%s': %s\n",