Whamcloud - gitweb
LU-6179 llite: remove LOCKAHEAD_OLD compatibility
[fs/lustre-release.git] / lustre / utils / mount_lustre.c
index 6e7b75d..7ed149c 100644 (file)
  *
  * You should have received a copy of the GNU General Public License
  * version 2 along with this program; If not, see
- * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
- *
- * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
- * CA 95054 USA or visit www.sun.com if you need additional information or
- * have any questions.
+ * http://www.gnu.org/licenses/gpl-2.0.html
  *
  * GPL HEADER END
  */
@@ -27,7 +23,7 @@
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011, 2014, Intel Corporation.
+ * Copyright (c) 2011, 2017, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
 #include <errno.h>
 #include <string.h>
 #include <sys/mount.h>
-#include <lustre_ver.h>
+#include <sys/param.h>
+#include <linux/lustre/lustre_ver.h>
 #include <ctype.h>
 #include <limits.h>
-#include <lnet/nidstr.h>
-#include <lustre/lustre_idl.h>
+#include <linux/lnet/nidstr.h>
 #include <libcfs/util/string.h>
 
 #include "obdctl.h"
 #include "mount_utils.h"
 
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 10, 53, 0)
-/*
- * LU-1783
- * We only #include a kernel level include file here because
- * important MS_ flag #defines are missing from the SLES version
- * of sys/mount.h
- * In the future if SLES updates sys/mount.h to have a more complete
- * set of flag #defines we should stop including linux/fs.h
- */
-#if !defined(MS_RDONLY)
-#include <linux/fs.h>
-#endif
-#endif
-
 #ifdef HAVE_LIBMOUNT
 # define WITH_LIBMOUNT "(libmount)"
 #else
@@ -92,35 +74,49 @@ void usage(FILE *out)
 {
        fprintf(out, "\nThis mount helper should only be invoked via the "
                "mount (8) command,\ne.g. mount -t lustre dev dir\n\n");
-       fprintf(out, "usage: %s [-fhnvV] [-o <mntopt>] <device> <mountpt>\n",
+       fprintf(out, "usage: %s [-fhnvV] [-o <srvopt>] <device> <mountpt>\n",
+               progname);
+       fprintf(out, "usage: %s [-fhnvV] [-o <cliopt>] <mgstarget> <mountpt>\n",
                progname);
        fprintf(out,
-               "\t<device>: the disk device, or for a client:\n"
+               "\t<device>: the local disk device when mounting a server\n"
+               "\t<mgstarget>: the server MGS and filesystem for a client:\n"
                "\t\t<mgsnid>[:<altmgsnid>...]:/<filesystem>[/<subdir>]\n"
-               "\t<filesystem>: name of the Lustre filesystem (e.g. lustre1)\n"
+               "\t\t\t<mgsnid>: MGS LNet Node Identifier (e.g. mgs01@o2ib)\n"
+               "\t\t\t<filesystem>: Lustre filesystem name (e.g. lustre1)\n"
+               "\t\t\t<subdir>: subdirectory of the filesystem to mount\n"
                "\t<mountpt>: filesystem mountpoint (e.g. /mnt/lustre)\n"
-               "\t-f|--fake: fake mount (updates /etc/mtab)\n"
+               "\t-f|--fake: fake mount (only update /etc/mtab)\n"
                "\t-o force|--force: force mount even if already in /etc/mtab\n"
                "\t-h|--help: print this usage message\n"
                "\t-n|--nomtab: do not update /etc/mtab after mount\n"
                "\t-v|--verbose: print verbose config settings\n"
                "\t-V|--version: output build version of the utility and exit\n"
-               "\t<mntopt>: one or more comma separated of:\n"
-               "\t\t(no)flock,(no)user_xattr,(no)acl\n"
+               "\tdefault options are marked below with '*'\n"
+               "\t\t(no)flock: disable* or enable POSIX flock support\n"
+               "\t\t(no)user_xattr: disable or enable* user xattr namespace\n"
+               "\t<srvopt>: one or more comma separated server options:\n"
+               "\t\t(no)acl: disable or enable* POSIX ACL support completely\n"
                "\t\tabort_recov: abort server recovery handling\n"
-               "\t\tnosvc: only start MGC/MGS obds\n"
-               "\t\tnomgs: only start target obds, using existing MGS\n"
-               "\t\tnoscrub: NOT auto start OI scrub unless start explicitly\n"
-               "\t\tskip_lfsck: NOT auto resume the paused/crashed LFSCK\n"
-               "\t\texclude=<ostname>[:<ostname>] : colon-separated list of "
-               "inactive OSTs (e.g. lustre-OST0001)\n"
+               "\t\tnosvc: only start MGC/MGS without starting MDS/OSS\n"
+               "\t\tnomgs: only start target MDS/OSS, using existing MGS\n"
+               "\t\tnoscrub: do NOT auto start OI scrub unless requested\n"
+               "\t\tskip_lfsck: do NOT auto resume paused/crashed LFSCK\n"
+               "\t\tmd_stripe_cache_size=<num>: set MD RAID device stripe cache size\n"
+               "\t<cliopt>: one or more comma separated client options:\n"
+               "\t\texclude=<ostname>[:<ostname>]: list of inactive OSTs (e.g. lustre-OST0001)\n"
+               "\t\tlocalflock: enable POSIX flock only on local client\n"
+               "\t\tretry=<num>: number of times mount is retried by client\n"
 #ifdef HAVE_GSS
-               "\t\tskpath=<file|directory>: Path to a file or directory of"
-               "key configuration files to load into the kernel keyring\n"
+               "\t\tskpath=<file|directory>: path of keys to load into kernel keyring\n"
 #endif
-               "\t\tretry=<num>: number of times mount is retried by client\n"
-               "\t\tmd_stripe_cache_size=<num>: set the raid stripe cache "
-               "size for the underlying raid if present\n");
+               "\t\t(no)user_fid2path: disable* or enable user $MOUNT/.lustre/fid access\n"
+               "\t\t(no)checksum: disable or enable* data checksums\n"
+               "\t\t(no)lruresize: disable or enable* LDLM dynamic LRU size\n"
+               "\t\t(no)lazystatfs: disable or enable* statfs to work if OST is unavailable\n"
+               "\t\t32bitapi: return only 32-bit inode numbers to userspace\n"
+               "\t\t(no)verbose: disable or enable* messages at filesystem (un,re)mount\n"
+               );
        exit((out != stdout) ? EINVAL : 0);
 }
 
@@ -128,38 +124,38 @@ void usage(FILE *out)
 #define MAXNIDSTR 1024
 static char *convert_hostnames(char *s1)
 {
-        char *converted, *s2 = 0, *c;
-        char sep;
-        int left = MAXNIDSTR;
-        lnet_nid_t nid;
-
-        converted = malloc(left);
-        if (converted == NULL) {
-                fprintf(stderr, "out of memory: needed %d bytes\n",
-                        MAXNIDSTR);
-                return NULL;
-        }
-        c = converted;
-        while ((left > 0) && (*s1 != '/')) {
-                s2 = strpbrk(s1, ",:");
-                if (!s2)
-                        goto out_free;
-                sep = *s2;
-                *s2 = '\0';
-                nid = libcfs_str2nid(s1);
-                *s2 = sep;                      /* back to original string */
-                if (nid == LNET_NID_ANY)
-                        goto out_free;
-                c += snprintf(c, left, "%s%c", libcfs_nid2str(nid), sep);
-                left = converted + MAXNIDSTR - c;
-                s1 = s2 + 1;
-        }
-        snprintf(c, left, "%s", s1);
-        return converted;
+       char *converted, *s2 = 0, *c;
+       char sep;
+       int left = MAXNIDSTR;
+       lnet_nid_t nid;
+
+       converted = malloc(left);
+       if (converted == NULL) {
+               fprintf(stderr, "out of memory: needed %d bytes\n",
+                       MAXNIDSTR);
+               return NULL;
+       }
+       c = converted;
+       while ((left > 0) && (*s1 != '/')) {
+               s2 = strpbrk(s1, ",:");
+               if (!s2)
+                       goto out_free;
+               sep = *s2;
+               *s2 = '\0';
+               nid = libcfs_str2nid(s1);
+               *s2 = sep;                      /* back to original string */
+               if (nid == LNET_NID_ANY)
+                       goto out_free;
+               c += scnprintf(c, left, "%s%c", libcfs_nid2str(nid), sep);
+               left = converted + MAXNIDSTR - c;
+               s1 = s2 + 1;
+       }
+       snprintf(c, left, "%s", s1);
+       return converted;
 out_free:
-        fprintf(stderr, "%s: Can't parse NID '%s'\n", progname, s1);
-        free(converted);
-        return NULL;
+       fprintf(stderr, "%s: Can't parse NID '%s'\n", progname, s1);
+       free(converted);
+       return NULL;
 }
 
 /*****************************************************************************
@@ -176,42 +172,45 @@ struct opt_map {
 };
 
 static const struct opt_map opt_map[] = {
-  /*"optname", inv,ms_mask */
-  /* These flags are parsed by mount, not lustre */
-  { "defaults", 0, 0         },      /* default options */
-  { "remount",  0, MS_REMOUNT},      /* remount with different options */
-  { "rw",       1, MS_RDONLY },      /* read-write */
-  { "ro",       0, MS_RDONLY },      /* read-only */
-  { "exec",     1, MS_NOEXEC },      /* permit execution of binaries */
-  { "noexec",   0, MS_NOEXEC },      /* don't execute binaries */
-  { "suid",     1, MS_NOSUID },      /* honor suid executables */
-  { "nosuid",   0, MS_NOSUID },      /* don't honor suid executables */
-  { "dev",      1, MS_NODEV  },      /* interpret device files  */
-  { "nodev",    0, MS_NODEV  },      /* don't interpret devices */
-  { "sync",     0, MS_SYNCHRONOUS},  /* synchronous I/O */
-  { "async",    1, MS_SYNCHRONOUS},  /* asynchronous I/O */
-  { "atime",    1, MS_NOATIME  },    /* set file access time on read */
-  { "noatime",  0, MS_NOATIME  },    /* do not set file access time on read */
+/* these flags are parsed by mount, not Lustre */
+{ .opt = "async",   .mask = MS_SYNCHRONOUS, .inv = 1 }, /* asynchronous I/O */
+{ .opt = "atime",   .mask = MS_NOATIME,            .inv = 1 }, /* set access time */
+{ .opt = "auto" },                                     /* allow auto mount */
+{ .opt = "defaults" },                                 /* default options */
+{ .opt = "dev",            .mask = MS_NODEV,       .inv = 1 }, /* interpret devs */
+{ .opt = "exec",    .mask = MS_NOEXEC,     .inv = 1 }, /* allow execution */
+{ .opt = "loop" },
+{ .opt = "noatime", .mask = MS_NOATIME },              /* do not set atime */
+{ .opt = "noauto" },                                   /* mount explicitly */
+{ .opt = "nodev",   .mask = MS_NODEV },                        /* no interpret devs */
+{ .opt = "noowner",                        .inv = 1 }, /* no special privs */
+{ .opt = "nosuid",  .mask = MS_NOSUID },               /* do not honor suid */
+{ .opt = "nouser",                         .inv = 1 }, /* users cannot mount */
+{ .opt = "nousers",                        .inv = 1 }, /* users cannot mount */
+{ .opt = "_netdev" },                                  /* network only */
+{ .opt = "noexec",  .mask = MS_NOEXEC },               /* no execute */
+{ .opt = "remount", .mask = MS_REMOUNT },              /* remount */
+{ .opt = "ro",     .mask = MS_RDONLY },                /* read-only */
+{ .opt = "rw",     .mask = MS_RDONLY,      .inv = 1 }, /* read-write */
+{ .opt = "suid",    .mask = MS_NOSUID,     .inv = 1 }, /* honor suid */
+{ .opt = "sync",    .mask = MS_SYNCHRONOUS },          /* synchronous I/O */
 #ifdef MS_NODIRATIME
-  { "diratime", 1, MS_NODIRATIME },  /* set file access time on read */
-  { "nodiratime",0,MS_NODIRATIME },  /* do not set file access time on read */
+{ .opt = "diratime",                                   /* set access time */
+                   .mask = MS_NODIRATIME,  .inv = 1 }, /* on read */
+{ .opt = "nodiratime",                                 /* do not set access */
+                   .mask = MS_NODIRATIME },            /* time on read */
 #endif
 #ifdef MS_RELATIME
-  { "relatime", 0, MS_RELATIME },  /* set file access time on read */
-  { "norelatime",1,MS_RELATIME },  /* do not set file access time on read */
+{ .opt = "norelatime",                                 /* do not set rel */
+                   .mask = MS_RELATIME,    .inv = 1 }, /* access time */
+{ .opt = "relatime",                                   /* set relative */
+                   .mask = MS_RELATIME },              /* access time */
 #endif
 #ifdef MS_STRICTATIME
-  { "strictatime",0,MS_STRICTATIME },  /* update access time strictly */
+{ .opt = "strictatime",
+                   .mask = MS_STRICTATIME },           /* strict access time */
 #endif
-  { "auto",     0, 0         },      /* Can be mounted using -a */
-  { "noauto",   0, 0         },      /* Can only be mounted explicitly */
-  { "nousers",  1, 0         },      /* Forbid ordinary user to mount */
-  { "nouser",   1, 0         },      /* Forbid ordinary user to mount */
-  { "noowner",  1, 0         },      /* Device owner has no special privs */
-  { "_netdev",  0, 0         },      /* Device accessible only via network */
-  { "loop",     0, 0         },
-  { NULL,       0, 0         }
-};
+{ .opt = NULL } };
 /****************************************************************************/
 
 /* 1  = don't pass on to lustre
@@ -236,20 +235,38 @@ static int parse_one_option(const char *check, int *flagp)
         return 0;
 }
 
+static size_t merge_strings(char *dst, const char *src, size_t size)
+{
+       size_t dsize = strlen(dst);
+       size_t len = strlen(src);
+       size_t ret = dsize + len;
+
+       dst  += dsize;
+       size -= dsize;
+       if (len >= size)
+               len = size - 1;
+       memcpy(dst, src, len);
+       dst[len] = '\0';
+       return ret;
+}
+
 static int append_option(char *options, size_t options_len,
                         const char *param, const char *value)
 {
        int rc;
+
        if (options[0] != '\0') {
-               rc = strlcat(options, ",", options_len);
+               rc = merge_strings(options, ",", options_len);
                if (rc >= options_len)
                        goto out_err;
        }
-       rc = strlcat(options, param, options_len);
+
+       rc = merge_strings(options, param, options_len);
        if (rc >= options_len)
                goto out_err;
+
        if (value != NULL) {
-               rc = strlcat(options, value, options_len);
+               rc = merge_strings(options, value, options_len);
                if (rc >= options_len)
                        goto out_err;
        }
@@ -268,6 +285,9 @@ int parse_options(struct mount_opts *mop, char *orig_options,
        int rc = 0;
 
        options = calloc(strlen(orig_options) + 1, 1);
+       if (options == NULL)
+               return ENOMEM;
+
        *flagp = 0;
        nextopt = orig_options;
        while ((opt = strsep(&nextopt, ","))) {
@@ -275,14 +295,16 @@ int parse_options(struct mount_opts *mop, char *orig_options,
                        /* empty option */
                        continue;
 
-               /* Handle retries in a slightly different
-                * manner */
+               /* Handle retries in a slightly different manner */
                arg = opt;
                val = strchr(opt, '=');
                /* please note that some ldiskfs mount options are also in
                 * the form of param=value. We should pay attention not to
                 * remove those mount options, see bug 22097. */
-               if (val && strncmp(arg, "md_stripe_cache_size", 20) == 0) {
+               if (val && strncmp(arg, "max_sectors_kb", 14) == 0) {
+                       mop->mo_max_sectors_kb = atoi(val + 1);
+               } else if (val &&
+                          strncmp(arg, "md_stripe_cache_size", 20) == 0) {
                        mop->mo_md_stripe_cache_size = atoi(val + 1);
                } else if (val && strncmp(arg, "retry", 5) == 0) {
                        mop->mo_retry = atoi(val + 1);
@@ -310,9 +332,10 @@ int parse_options(struct mount_opts *mop, char *orig_options,
                                        "%s: shared key path too long\n",
                                        progname);
                                free(options);
-                               return -1;
+                               return EINVAL;
                        }
-                       strncpy(mop->mo_skpath, val + 1, strlen(val + 1));
+                       strncpy(mop->mo_skpath, val + 1,
+                               sizeof(mop->mo_skpath) - 1);
 #endif
                } else if (parse_one_option(opt, flagp) == 0) {
                        /* pass this on as an option */
@@ -345,6 +368,7 @@ out_options:
        return rc;
 }
 
+#ifdef HAVE_SERVER_SUPPORT
 /* Add mgsnids from ldd params */
 static int add_mgsnids(struct mount_opts *mop, char *options,
                       const char *params, size_t options_len)
@@ -440,6 +464,7 @@ static int parse_ldd(char *source, struct mount_opts *mop,
 {
        struct lustre_disk_data *ldd = &mop->mo_ldd;
        char *cur, *start;
+       char *temp_options;
        int rc = 0;
 
        rc = osd_is_lustre(source, &ldd->ldd_mount_type);
@@ -466,18 +491,25 @@ static int parse_ldd(char *source, struct mount_opts *mop,
                return EINVAL;
        }
 
-       if (ldd->ldd_flags & LDD_F_UPGRADE14) {
-               fprintf(stderr, "%s: we cannot upgrade %s from this (very old) "
-                       "Lustre version\n", progname, source);
-               return EINVAL;
-       }
-
        if (ldd->ldd_flags & LDD_F_UPDATE)
                clear_update_ondisk(source, ldd);
 
        /* Since we never rewrite ldd, ignore temp flags */
        ldd->ldd_flags &= ~(LDD_F_VIRGIN | LDD_F_WRITECONF);
 
+       /* This is to make sure default options go first */
+       temp_options = strdup(options);
+       if (temp_options == NULL) {
+               fprintf(stderr, "%s: can't allocate memory for temp_options\n",
+                       progname);
+               return ENOMEM;
+       }
+       strncpy(options, ldd->ldd_mount_opts, options_len);
+       rc = append_option(options, options_len, temp_options, NULL);
+       free(temp_options);
+       if (rc != 0)
+               return rc;
+
        /* svname of the form lustre:OST1234 means never registered */
        rc = strlen(ldd->ldd_svname);
        if (strcmp(ldd->ldd_svname, "MGS") != 0) {
@@ -499,10 +531,6 @@ static int parse_ldd(char *source, struct mount_opts *mop,
        if (rc != 0)
                return rc;
 
-       rc = append_option(options, options_len, ldd->ldd_mount_opts, NULL);
-       if (rc != 0)
-               return rc;
-
        if (!mop->mo_have_mgsnid) {
                /* Only use disk data if mount -o mgsnode=nid wasn't
                 * specified */
@@ -568,6 +596,7 @@ static int parse_ldd(char *source, struct mount_opts *mop,
 
        return rc;
 }
+#endif /* HAVE_SERVER_SUPPORT */
 
 static void set_defaults(struct mount_opts *mop)
 {
@@ -582,20 +611,20 @@ static void set_defaults(struct mount_opts *mop)
        mop->mo_md_stripe_cache_size = 16384;
        mop->mo_orig_options = "";
        mop->mo_nosvc = 0;
+       mop->mo_max_sectors_kb = -1;
 }
 
 static int parse_opts(int argc, char *const argv[], struct mount_opts *mop)
 {
-       static struct option long_opt[] = {
-               {"fake", 0, 0, 'f'},
-               {"force", 0, 0, 1},
-               {"help", 0, 0, 'h'},
-               {"nomtab", 0, 0, 'n'},
-               {"options", 1, 0, 'o'},
-               {"verbose", 0, 0, 'v'},
-               {"version", 0, 0, 'V'},
-               {0, 0, 0, 0}
-       };
+       static struct option long_opts[] = {
+       { .val = 1,     .name = "force",        .has_arg = no_argument },
+       { .val = 'f',   .name = "fake",         .has_arg = no_argument },
+       { .val = 'h',   .name = "help",         .has_arg = no_argument },
+       { .val = 'n',   .name = "nomtab",       .has_arg = no_argument },
+       { .val = 'o',   .name = "options",      .has_arg = required_argument },
+       { .val = 'v',   .name = "verbose",      .has_arg = no_argument },
+       { .val = 'V',   .name = "version",      .has_arg = no_argument },
+       { .name = NULL } };
        char real_path[PATH_MAX] = {'\0'};
        FILE *f;
        char path[256], name[256];
@@ -604,7 +633,7 @@ static int parse_opts(int argc, char *const argv[], struct mount_opts *mop)
        int opt, rc;
 
        while ((opt = getopt_long(argc, argv, "fhno:vV",
-                                 long_opt, NULL)) != EOF){
+                                 long_opts, NULL)) != EOF){
                switch (opt) {
                case 1:
                        ++mop->mo_force;
@@ -689,6 +718,37 @@ static int parse_opts(int argc, char *const argv[], struct mount_opts *mop)
        return 0;
 }
 
+#ifdef HAVE_SERVER_SUPPORT
+/* change label from <fsname>:<index> to
+ * <fsname>-<index> to indicate the device has
+ * been registered. only if the label is
+ * supposed to be changed and target service
+ * is supposed to start */
+static void label_lustre(struct mount_opts *mop)
+{
+       if (mop->mo_nosvc)
+               return;
+
+       if (mop->mo_ldd.ldd_flags & (LDD_F_VIRGIN | LDD_F_WRITECONF)) {
+               (void)osd_label_lustre(mop);
+       } else {
+               struct lustre_disk_data ldd;
+               int rc;
+
+               /* device label could be changed after journal recovery,
+                * it should also be relabeled for mount has succeeded. */
+               memset(&ldd, 0, sizeof(ldd));
+               ldd.ldd_mount_type = mop->mo_ldd.ldd_mount_type;
+               rc = osd_read_ldd(mop->mo_source, &ldd);
+               if (rc == 0) {
+                       rc = strlen(ldd.ldd_svname);
+                       if (rc >= 8 && ldd.ldd_svname[rc - 8] != '-')
+                               (void)osd_label_lustre(mop);
+               }
+       }
+}
+#endif /* HAVE_SERVER_SUPPORT */
+
 int main(int argc, char *const argv[])
 {
        struct mount_opts mop;
@@ -742,7 +802,6 @@ int main(int argc, char *const argv[])
         if (rc) {
                 fprintf(stderr, "%s: can't parse options: %s\n",
                         progname, options);
-               rc = EINVAL;
                goto out_options;
         }
 
@@ -777,6 +836,7 @@ int main(int argc, char *const argv[])
 
        client = (strstr(mop.mo_usource, ":/") != NULL);
        if (!client) {
+#ifdef HAVE_SERVER_SUPPORT
                rc = osd_init();
                if (rc)
                        goto out_options;
@@ -784,6 +844,10 @@ int main(int argc, char *const argv[])
                rc = parse_ldd(mop.mo_source, &mop, options, maxopt_len);
                if (rc)
                        goto out_osd;
+#else
+               rc = -EINVAL;
+               goto out_options;
+#endif
        }
 
        /* In Linux 2.4, the target device doesn't get passed to any of our
@@ -796,13 +860,14 @@ int main(int argc, char *const argv[])
                printf("mounting device %s at %s, flags=%#x options=%s\n",
                       mop.mo_source, mop.mo_target, flags, options);
 
+#ifdef HAVE_SERVER_SUPPORT
        if (!client && osd_tune_lustre(mop.mo_source, &mop)) {
                if (verbose)
                        fprintf(stderr, "%s: unable to set tunables for %s"
                                        " (may cause reduced IO performance)\n",
                                        argv[0], mop.mo_source);
        }
-
+#endif
 #ifdef HAVE_GSS
        if (mop.mo_skpath[0] != '\0') {
                /* Treat shared key failures as fatal */
@@ -823,18 +888,7 @@ int main(int argc, char *const argv[])
                for (i = 0, rc = -EAGAIN; i <= mop.mo_retry && rc != 0; i++) {
                        rc = mount(mop.mo_source, mop.mo_target, "lustre",
                                   flags, (void *)options);
-                       if (rc == 0) {
-                               /* change label from <fsname>:<index> to
-                                * <fsname>-<index> to indicate the device has
-                                *  been registered. only if the label is
-                                *  supposed to be changed and target service
-                                *  is supposed to start */
-                               if (mop.mo_ldd.ldd_flags &
-                                  (LDD_F_VIRGIN | LDD_F_WRITECONF)) {
-                                       if (mop.mo_nosvc == 0)
-                                               (void)osd_label_lustre(&mop);
-                               }
-                       } else {
+                       if (rc != 0) {
                                 if (verbose) {
                                         fprintf(stderr, "%s: mount %s at %s "
                                                 "failed: %s retries left: "
@@ -851,6 +905,11 @@ int main(int argc, char *const argv[])
                                } else {
                                        rc = errno;
                                }
+#ifdef HAVE_SERVER_SUPPORT
+                       } else {
+                               if (!client)
+                                       label_lustre(&mop);
+#endif
                        }
                }
        }
@@ -932,9 +991,10 @@ int main(int argc, char *const argv[])
        }
 
 out_osd:
+#ifdef HAVE_SERVER_SUPPORT
        if (!client)
                osd_fini();
-
+#endif
 out_options:
        free(options);