* Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
* Use is subject to license terms.
*
- * Copyright (c) 2011, 2015, Intel Corporation.
+ * Copyright (c) 2011, 2017, Intel Corporation.
*/
/*
* This file is part of Lustre, http://www.lustre.org/
#include <getopt.h>
#include <limits.h>
#include <ctype.h>
-#include <lnet/nidstr.h>
-#include <lustre_disk.h>
-#include <lustre_param.h>
-#include <lnet/lnetctl.h>
-#include <lustre_ver.h>
+#include <linux/lnet/nidstr.h>
+#include <linux/lnet/lnetctl.h>
+#include <linux/lustre/lustre_user.h>
+#include <linux/lustre/lustre_ver.h>
#include "mount_utils.h"
char *progname;
int verbose = 1;
int version;
-static int print_only = 0;
+static int print_only;
#ifdef HAVE_LDISKFS_OSD
#define FSLIST_LDISKFS "ldiskfs"
"cache, log)\n"
#endif
"\n"
-#ifndef TUNEFS
"\ttarget types:\n"
"\t\t--mgs: configuration management service\n"
+ "\t\t--nomgs: turn off MGS service on this MDT\n"
+#ifndef TUNEFS
"\t\t--mdt: metadata storage, mutually exclusive with ost\n"
"\t\t--ost: object storage, mutually exclusive with mdt, mgs\n"
#endif
"\t\t--replace: replace an old target with the same index\n"
"\t\t--stripe-count-hint=#N: for optimizing MDT inode size\n"
#else
+ "\t\t--erase-param <key>: erase all instances of a parameter\n"
"\t\t--erase-params: erase all old parameter settings\n"
- "\t\t--nomgs: turn off MGS service on this MDT\n"
"\t\t--writeconf: erase all config logs for this fs.\n"
"\t\t--quota: enable space accounting on old 2.x device.\n"
+ "\t\t--rename: rename the filesystem name\n"
#endif
"\t\t--comment=<user comment>: arbitrary string (%d bytes)\n"
"\t\t--dryrun: report what we would do; don't write to disk\n"
"\t\t\texit\n"
"\t\t--quiet\n",
(int)sizeof(((struct lustre_disk_data *)0)->ldd_userdata));
- return;
}
/* ==================== Lustre config functions =============*/
-void print_ldd(char *str, struct lustre_disk_data *ldd)
+void print_ldd(char *str, struct mkfs_opts *mop)
{
- printf("\n %s:\n", str);
- printf("Target: %s\n", ldd->ldd_svname);
- if (ldd->ldd_svindex == INDEX_UNASSIGNED)
- printf("Index: unassigned\n");
- else
- printf("Index: %d\n", ldd->ldd_svindex);
- if (ldd->ldd_uuid[0])
- printf("UUID: %s\n", (char *)ldd->ldd_uuid);
- printf("Lustre FS: %s\n", ldd->ldd_fsname);
- printf("Mount type: %s\n", MT_STR(ldd));
- printf("Flags: %#x\n", ldd->ldd_flags);
- printf(" (%s%s%s%s%s%s%s%s%s)\n",
- IS_MDT(ldd) ? "MDT ":"",
- IS_OST(ldd) ? "OST ":"",
- IS_MGS(ldd) ? "MGS ":"",
- ldd->ldd_flags & LDD_F_NEED_INDEX ? "needs_index ":"",
- ldd->ldd_flags & LDD_F_VIRGIN ? "first_time ":"",
- ldd->ldd_flags & LDD_F_UPDATE ? "update ":"",
- ldd->ldd_flags & LDD_F_WRITECONF ? "writeconf ":"",
- ldd->ldd_flags & LDD_F_NO_PRIMNODE? "no_primnode ":"",
- ldd->ldd_flags & LDD_F_UPGRADE14 ? "upgrade1.4 ":"");
- printf("Persistent mount opts: %s\n", ldd->ldd_mount_opts);
- printf("Parameters:%s\n", ldd->ldd_params);
- if (ldd->ldd_userdata[0])
- printf("Comment: %s\n", ldd->ldd_userdata);
- printf("\n");
+ struct lustre_disk_data *ldd = &mop->mo_ldd;
+
+ printf("\n %s:\n", str);
+ printf("Target: %s\n", ldd->ldd_svname);
+ if (ldd->ldd_svindex == INDEX_UNASSIGNED)
+ printf("Index: unassigned\n");
+ else
+ printf("Index: %d\n", ldd->ldd_svindex);
+ if (ldd->ldd_uuid[0])
+ printf("UUID: %s\n", (char *)ldd->ldd_uuid);
+ printf("Lustre FS: %s\n", ldd->ldd_fsname);
+ printf("Mount type: %s\n", MT_STR(ldd));
+ printf("Flags: %#x\n", ldd->ldd_flags);
+ printf(" (%s%s%s%s%s%s%s%s)\n",
+ IS_MDT(ldd) ? "MDT " : "",
+ IS_OST(ldd) ? "OST " : "",
+ IS_MGS(ldd) ? "MGS " : "",
+ ldd->ldd_flags & LDD_F_NEED_INDEX ? "needs_index " : "",
+ ldd->ldd_flags & LDD_F_VIRGIN ? "first_time " : "",
+ ldd->ldd_flags & LDD_F_UPDATE ? "update " : "",
+ ldd->ldd_flags & LDD_F_WRITECONF ? "writeconf " : "",
+ ldd->ldd_flags & LDD_F_NO_PRIMNODE ? "no_primnode " : "");
+ printf("Persistent mount opts: %s\n", ldd->ldd_mount_opts);
+ osd_print_ldd_params(mop);
+ if (ldd->ldd_userdata[0])
+ printf("Comment: %s\n", ldd->ldd_userdata);
+ printf("\n");
}
void set_defaults(struct mkfs_opts *mop)
mop->mo_pool_vdevs = NULL;
}
+/* Make the mdt/ost server obd name based on the filesystem name */
+static bool server_make_name(__u32 flags, __u16 index, const char *fs,
+ char *name_buf, size_t name_buf_size)
+{
+ bool invalid_flag = false;
+
+ if (flags & (LDD_F_SV_TYPE_MDT | LDD_F_SV_TYPE_OST)) {
+ if (!(flags & LDD_F_SV_ALL))
+ snprintf(name_buf, name_buf_size, "%.8s%c%s%04x", fs,
+ (flags & LDD_F_VIRGIN) ? ':' :
+ ((flags & LDD_F_WRITECONF) ? '=' : '-'),
+ (flags & LDD_F_SV_TYPE_MDT) ? "MDT" : "OST",
+ index);
+ } else if (flags & LDD_F_SV_TYPE_MGS) {
+ snprintf(name_buf, name_buf_size, "MGS");
+ } else {
+ fprintf(stderr, "unknown server type %#x\n", flags);
+ invalid_flag = true;
+ }
+ return invalid_flag;
+}
+
static inline void badopt(const char *opt, char *type)
{
- fprintf(stderr, "%s: '--%s' only valid for %s\n",
- progname, opt, type);
- usage(stderr);
+ fprintf(stderr, "%s: '--%s' only valid for %s\n",
+ progname, opt, type);
+ usage(stderr);
}
+#ifdef TUNEFS
+/**
+ * Removes all existing instances of the parameter passed in \a param,
+ * which are in the form of "key=<value>", from the buffer at \a buf.
+ *
+ * The parameter can be either in the form of "key" when passed by option
+ * "--erase-param", or in the form of "key=<value>" when passed by option
+ * "--param".
+ *
+ * \param buf the buffer holding on-disk server parameters.
+ * \param param the parameter whose instances are to be removed from \a buf.
+ * \param withval true means the parameter is in the form of "key=<value>"
+ * false means the parameter is in the form of "key"
+ *
+ * \retval 0 success, parameter was erased,
+ * \retval 1 success, parameter was not found, don't need to do erase_ldd,
+ * \retval EINVAL failure, invalid input parameter.
+ */
+static int erase_param(const char *const buf, const char *const param,
+ bool withval)
+{
+ char search[PARAM_MAX + 1] = "";
+ char *buffer = (char *)buf;
+ bool found = false;
+
+ if (strlen(param) > PARAM_MAX) {
+ fprintf(stderr, "%s: param to erase is too long-\n%s\n",
+ progname, param);
+ return EINVAL;
+ }
+
+ /* add_param() writes a space as the first character in ldd_params */
+ search[0] = ' ';
+
+ /* "key" or "key=<value>" */
+ if (withval) {
+ char *keyend;
+
+ keyend = strchr(param, '=');
+ if (!keyend)
+ return EINVAL;
+ strncpy(&search[1], param, keyend - param + 1);
+ } else {
+ snprintf(search + 1, sizeof(search) - 1, "%s=", param);
+ }
+
+ while (1) {
+ char *space;
+
+ buffer = strstr(buffer, search);
+ if (!buffer)
+ return found == true ? 0 : 1;
+ found = true;
+ space = strchr(buffer + 1, ' ');
+ if (space) {
+ memmove(buffer, space, strlen(space) + 1);
+ } else {
+ *buffer = '\0';
+ return 0;
+ }
+ }
+}
+#endif
+
/* from mount_lustre */
/* Get rid of symbolic hostnames for tcp, since kernel can't do lookups */
#define MAXNIDSTR 1024
static char *convert_hostnames(char *s1)
{
- char *converted, *s2 = 0, *c, *end, sep;
- int left = MAXNIDSTR;
- lnet_nid_t nid;
-
- converted = malloc(left);
- if (converted == NULL) {
- return NULL;
- }
-
- end = s1 + strlen(s1);
- c = converted;
- while ((left > 0) && (s1 < end)) {
- s2 = strpbrk(s1, ",:");
- if (!s2)
- s2 = end;
- sep = *s2;
- *s2 = '\0';
- nid = libcfs_str2nid(s1);
+ char *converted, *s2 = 0, *c, *end, sep;
+ int left = MAXNIDSTR;
+ lnet_nid_t nid;
+
+ converted = malloc(left);
+ if (!converted)
+ return NULL;
+
+ end = s1 + strlen(s1);
+ c = converted;
+ while ((left > 0) && (s1 < end)) {
+ s2 = strpbrk(s1, ",:");
+ if (!s2)
+ s2 = end;
+ sep = *s2;
+ *s2 = '\0';
+ nid = libcfs_str2nid(s1);
*s2 = sep;
- if (nid == LNET_NID_ANY) {
+ if (nid == LNET_NID_ANY) {
fprintf(stderr, "%s: Cannot resolve hostname '%s'.\n",
progname, s1);
- free(converted);
- return NULL;
- }
- if (strncmp(libcfs_nid2str(nid), "127.0.0.1",
- strlen("127.0.0.1")) == 0) {
- fprintf(stderr, "%s: The NID '%s' resolves to the "
- "loopback address '%s'. Lustre requires a "
- "non-loopback address.\n",
- progname, s1, libcfs_nid2str(nid));
- free(converted);
- return NULL;
- }
-
- c += snprintf(c, left, "%s%c", libcfs_nid2str(nid), sep);
- left = converted + MAXNIDSTR - c;
- s1 = s2 + 1;
- }
- return converted;
+ free(converted);
+ return NULL;
+ }
+ if (strncmp(libcfs_nid2str(nid), "127.0.0.1",
+ strlen("127.0.0.1")) == 0) {
+ fprintf(stderr,
+ "%s: The NID '%s' resolves to the loopback address '%s'. Lustre requires a non-loopback address.\n",
+ progname, s1, libcfs_nid2str(nid));
+ free(converted);
+ return NULL;
+ }
+
+ c += snprintf(c, left, "%s%c", libcfs_nid2str(nid), sep);
+ left = converted + MAXNIDSTR - c;
+ s1 = s2 + 1;
+ }
+ return converted;
}
int parse_opts(int argc, char *const argv[], struct mkfs_opts *mop,
- char **mountopts)
+ char **mountopts, char *old_fsname)
{
- static struct option long_opt[] = {
- { "backfs-mount-opts", required_argument, NULL, 'B' },
- { "failnode", required_argument, NULL, 'f' },
- { "failover", required_argument, NULL, 'f' },
- { "help", no_argument, NULL, 'h' },
- { "index", required_argument, NULL, 'i' },
- { "fsname", required_argument, NULL, 'L' },
- { "mgsnode", required_argument, NULL, 'm' },
- { "mgsnid", required_argument, NULL, 'm' },
- { "dryrun", no_argument, NULL, 'n' },
- { "mountfsoptions", required_argument, NULL, 'o' },
- { "param", required_argument, NULL, 'p' },
- { "quiet", no_argument, NULL, 'q' },
- { "servicenode", required_argument, NULL, 's' },
- { "network", required_argument, NULL, 't' },
- { "comment", required_argument, NULL, 'u' },
- { "force-nohostid", no_argument, NULL, 'U' },
- { "verbose", no_argument, NULL, 'v' },
- { "version", no_argument, NULL, 'V' },
+ static struct option long_opts[] = {
+ { .val = 'B', .name = "backfs-mount-opts",
+ .has_arg = required_argument},
+ { .val = 'f', .name = "failnode", .has_arg = required_argument},
+ { .val = 'f', .name = "failover", .has_arg = required_argument},
+ { .val = 'G', .name = "mgs", .has_arg = no_argument},
+ { .val = 'h', .name = "help", .has_arg = no_argument},
+ { .val = 'i', .name = "index", .has_arg = required_argument},
+ { .val = 'L', .name = "fsname", .has_arg = required_argument},
+ { .val = 'm', .name = "mgsnode", .has_arg = required_argument},
+ { .val = 'm', .name = "mgsnid", .has_arg = required_argument},
+ { .val = 'n', .name = "dryrun", .has_arg = no_argument},
+ { .val = 'N', .name = "nomgs", .has_arg = no_argument},
+ { .val = 'o', .name = "mountfsoptions",
+ .has_arg = required_argument},
+ { .val = 'p', .name = "param", .has_arg = required_argument},
+ { .val = 'q', .name = "quiet", .has_arg = no_argument},
+ { .val = 's', .name = "servicenode", .has_arg = required_argument},
+ { .val = 't', .name = "network", .has_arg = required_argument},
+ { .val = 'u', .name = "comment", .has_arg = required_argument},
+ { .val = 'U', .name = "force-nohostid",
+ .has_arg = no_argument},
+ { .val = 'v', .name = "verbose", .has_arg = no_argument},
+ { .val = 'V', .name = "version", .has_arg = no_argument},
#ifndef TUNEFS
- { "backfstype", required_argument, NULL, 'b' },
- { "stripe-count-hint", required_argument, NULL, 'c' },
- { "device-size", required_argument, NULL, 'd' },
- { "mgs", no_argument, NULL, 'G' },
- { "mkfsoptions", required_argument, NULL, 'k' },
- { "mdt", no_argument, NULL, 'M' },
- { "nomgs", no_argument, NULL, 'N' },
- { "ost", no_argument, NULL, 'O' },
- { "reformat", no_argument, NULL, 'r' },
- { "replace", no_argument, NULL, 'R' },
+ { .val = 'b', .name = "backfstype", .has_arg = required_argument},
+ { .val = 'c', .name = "stripe-count-hint",
+ .has_arg = required_argument},
+ { .val = 'd', .name = "device-size", .has_arg = required_argument},
+ { .val = 'k', .name = "mkfsoptions", .has_arg = required_argument},
+ { .val = 'M', .name = "mdt", .has_arg = no_argument},
+ { .val = 'O', .name = "ost", .has_arg = no_argument},
+ { .val = 'r', .name = "reformat", .has_arg = no_argument},
+ { .val = 'R', .name = "replace", .has_arg = no_argument},
#else
- { "erase-params", no_argument, NULL, 'e' },
- { "quota", no_argument, NULL, 'Q' },
- { "writeconf", no_argument, NULL, 'w' },
+ { .val = 'E', .name = "erase-param", .has_arg = required_argument},
+ { .val = 'e', .name = "erase-params",
+ .has_arg = no_argument},
+ { .val = 'Q', .name = "quota", .has_arg = no_argument},
+ { .val = 'R', .name = "rename", .has_arg = optional_argument},
+ { .val = 'w', .name = "writeconf", .has_arg = no_argument},
#endif
- { 0, 0, NULL, 0 }
- };
- char *optstring = "B:f:hi:L:m:no:p:qs:t:u:vV"
+ { .name = NULL } };
+ char *short_opts = "B:f:Ghi:L:m:nNo:p:qs:t:u:vV"
#ifndef TUNEFS
- "b:c:d:Gk:MNOrR";
+ "b:c:d:k:MOrR";
#else
- "eQw";
+ "E:eQR::w";
#endif
struct lustre_disk_data *ldd = &mop->mo_ldd;
char new_fsname[16] = { 0 };
int replace = 0;
bool index_option = false;
- while ((opt = getopt_long(argc, argv, optstring, long_opt, &longidx)) !=
- EOF) {
+#ifdef TUNEFS
+ /*
+ * For the right semantics, if '-e'/'--erase-params' is specified,
+ * it must be picked out and all old parameters should be erased
+ * before any other changes are done.
+ */
+ while ((opt = getopt_long(argc, argv, short_opts, long_opts,
+ &longidx)) != EOF) {
+ switch (opt) {
+ case 'e':
+ ldd->ldd_params[0] = '\0';
+ mop->mo_flags |= MO_ERASE_ALL;
+ ldd->ldd_flags |= LDD_F_UPDATE;
+ break;
+ default:
+ break;
+ }
+ if (mop->mo_flags & MO_ERASE_ALL)
+ break;
+ }
+ optind = 0;
+#endif
+ while ((opt = getopt_long(argc, argv, short_opts, long_opts,
+ &longidx)) != EOF) {
switch (opt) {
case 'B':
mop->mo_mountopts = optarg;
if ((opt == 'f' && servicenode_set) ||
(opt == 's' && failnode_set)) {
fprintf(stderr, "%s: %s cannot use with --%s\n",
- progname, long_opt[longidx].name,
+ progname, long_opts[longidx].name,
opt == 'f' ? "servicenode" :
"failnode");
return 1;
}
nids = convert_hostnames(optarg);
- if (nids == NULL)
+ if (!nids)
return 1;
rc = append_param(ldd->ldd_params, PARAM_FAILNODE,
mop->mo_flags |= MO_FAILOVER;
break;
}
+ case 'G':
+ ldd->ldd_flags |= LDD_F_SV_TYPE_MGS;
+ break;
case 'h':
usage(stdout);
return 1;
case 'i': {
char *endptr = NULL;
int base;
+
index_option = true;
/* LU-2374: check whether it is OST/MDT later */
base = (strlen(optarg) > 1 &&
mop->mo_ldd.ldd_svindex = strtoul(optarg,
&endptr, base);
if (*endptr != '\0') {
- fprintf(stderr, "%s: wrong index %s. "
- "Target index must be decimal or "
- "hexadecimal.\n",
+ fprintf(stderr,
+ "%s: wrong index %s. Target index must be decimal or hexadecimal.\n",
progname, optarg);
return 1;
}
if (ldd->ldd_svindex >= INDEX_UNASSIGNED) {
- fprintf(stderr, "%s: wrong index %u. "
- "Target index must be less than %u.\n",
+ fprintf(stderr,
+ "%s: wrong index %u. Target index must be less than %u.\n",
progname, ldd->ldd_svindex,
INDEX_UNASSIGNED);
return 1;
break;
}
case 'L': {
- rc = lustre_is_fsname_valid(optarg, 1,
- LUSTRE_MAXFSNAME);
- if (rc < 0) {
- fprintf(stderr, "%s: filesystem name must be "
- "1-%d chars\n", progname,
- LUSTRE_MAXFSNAME);
- return 1;
- } else if (rc > 0) {
- fprintf(stderr, "%s: char '%c' not allowed in "
- "filesystem name\n", progname, rc);
+ const char *tmp;
+ size_t len;
+
+ len = strlen(optarg);
+ if (len < 1 || len > LUSTRE_MAXFSNAME) {
+ fprintf(stderr,
+ "%s: filesystem name must be 1-%d chars\n",
+ progname, LUSTRE_MAXFSNAME);
return 1;
}
+ for (tmp = optarg; *tmp != '\0'; ++tmp) {
+ if (isalnum(*tmp) || *tmp == '_' || *tmp == '-')
+ continue;
+ else
+ break;
+ }
+ if (*tmp != '\0') {
+ fprintf(stderr,
+ "%s: char '%c' not allowed in filesystem name\n",
+ progname, *tmp);
+ return 1;
+ }
strscpy(new_fsname, optarg, sizeof(new_fsname));
break;
}
case 'm': {
char *nids = convert_hostnames(optarg);
- if (nids == NULL)
+ if (!nids)
return 1;
rc = append_param(ldd->ldd_params, PARAM_MGSNODE,
case 'n':
print_only++;
break;
+ case 'N':
+ ldd->ldd_flags &= ~LDD_F_SV_TYPE_MGS;
+ break;
case 'o':
*mountopts = optarg;
break;
case 'p':
+#ifdef TUNEFS
+ /*
+ * Removes all existing instances of the parameter
+ * before adding new values.
+ */
+ rc = erase_param(ldd->ldd_params, optarg, true);
+ if (rc > 1)
+ return rc;
+#endif
rc = add_param(ldd->ldd_params, NULL, optarg);
if (rc != 0)
return rc;
-
/* Must update the mgs logs */
ldd->ldd_flags |= LDD_F_UPDATE;
break;
break;
case 't':
if (!IS_MDT(ldd) && !IS_OST(ldd)) {
- badopt(long_opt[longidx].name, "MDT,OST");
+ badopt(long_opts[longidx].name, "MDT,OST");
return 1;
}
- if (optarg == NULL)
+ if (!optarg)
return 1;
rc = add_param(ldd->ldd_params, PARAM_NETWORK, optarg);
} while (++i < LDD_MT_LAST);
if (i == LDD_MT_LAST) {
- fprintf(stderr, "%s: invalid backend filesystem"
- " type %s\n", progname, optarg);
+ fprintf(stderr,
+ "%s: invalid backend filesystem type %s\n",
+ progname, optarg);
return 1;
}
break;
int stripe_count = atol(optarg);
if (stripe_count <= 0) {
- fprintf(stderr, "%s: bad stripe count "
- "%d\n", progname, stripe_count);
+ fprintf(stderr,
+ "%s: bad stripe count %s\n",
+ progname, optarg);
return 1;
}
mop->mo_stripe_count = stripe_count;
} else {
- badopt(long_opt[longidx].name, "MDT");
+ badopt(long_opts[longidx].name, "MDT");
return 1;
}
break;
case 'd':
mop->mo_device_kb = atol(optarg);
break;
- case 'G':
- ldd->ldd_flags |= LDD_F_SV_TYPE_MGS;
- break;
case 'k':
strscpy(mop->mo_mkfsopts, optarg,
sizeof(mop->mo_mkfsopts));
case 'M':
ldd->ldd_flags |= LDD_F_SV_TYPE_MDT;
break;
- case 'N':
- ldd->ldd_flags &= ~LDD_F_SV_TYPE_MGS;
- break;
case 'O':
ldd->ldd_flags |= LDD_F_SV_TYPE_OST;
break;
case 'R':
replace = 1;
break;
-#else /* !TUNEFS */
- case 'e':
- ldd->ldd_params[0] = '\0';
+#else /* TUNEFS */
+ case 'E':
+ rc = erase_param(ldd->ldd_params, optarg, false);
+ /*
+ * (rc == 1) means not found, so don't need to
+ * call osd_erase_ldd().
+ */
+ if (rc > 1)
+ return rc;
+ if (!rc) {
+ rc = osd_erase_ldd(mop, optarg);
+ if (rc)
+ return rc;
+ }
/* Must update the mgs logs */
ldd->ldd_flags |= LDD_F_UPDATE;
break;
+ case 'e':
+ /* Already done in the beginning */
+ break;
case 'Q':
mop->mo_flags |= MO_QUOTA;
break;
+ case 'R': {
+ char *tmp;
+
+ mop->mo_flags |= MO_RENAME;
+ if (!optarg) {
+ if (IS_SEPARATED_MGS(ldd)) {
+ fprintf(stderr,
+ "%s: must specify the old fsname to be renamed for separated MGS\n",
+ progname);
+ return 1;
+ }
+ break;
+ }
+
+ if ((strlen(optarg) < 1) || (strlen(optarg) > 8)) {
+ fprintf(stderr,
+ "%s: filesystem name must be 1-8 chars\n",
+ progname);
+ return 1;
+ }
+
+ tmp = strpbrk(optarg, "/:");
+ if (tmp) {
+ fprintf(stderr,
+ "%s: char '%c' not allowed in filesystem name\n",
+ progname, *tmp);
+ return 1;
+ }
+
+ if (IS_SEPARATED_MGS(ldd)) {
+ strscpy(old_fsname, optarg,
+ sizeof(ldd->ldd_fsname));
+ } else if (strlen(old_fsname) != strlen(optarg) ||
+ strcmp(old_fsname, optarg) != 0) {
+ fprintf(stderr,
+ "%s: the given fsname '%s' to be renamed does not exist\n",
+ progname, optarg);
+ return 1;
+ }
+ break;
+ }
case 'w':
ldd->ldd_flags |= LDD_F_WRITECONF;
break;
}
if (strlen(new_fsname) > 0) {
- if (!(mop->mo_flags & MO_FORCEFORMAT) &&
- (!(ldd->ldd_flags &
- (LDD_F_UPGRADE14 | LDD_F_VIRGIN | LDD_F_WRITECONF)))) {
- fprintf(stderr, "%s: cannot change the name "
- "of a registered target\n", progname);
+ if (!(mop->mo_flags & (MO_FORCEFORMAT | MO_RENAME)) &&
+ (!(ldd->ldd_flags & (LDD_F_VIRGIN | LDD_F_WRITECONF)))) {
+ fprintf(stderr,
+ "%s: cannot change the name of a registered target\n",
+ progname);
return 1;
}
}
if (index_option && !(mop->mo_ldd.ldd_flags &
- (LDD_F_UPGRADE14 | LDD_F_VIRGIN |
- LDD_F_WRITECONF))) {
- fprintf(stderr, "%s: cannot change the index of"
- " a registered target\n", progname);
+ (LDD_F_VIRGIN | LDD_F_WRITECONF))) {
+ fprintf(stderr,
+ "%s: cannot change the index of a registered target\n",
+ progname);
return 1;
}
+#ifdef TUNEFS
+ if (mop->mo_flags & MO_RENAME) {
+ if (new_fsname[0] == '\0') {
+ fprintf(stderr,
+ "%s: need to specify new fsname for renaming case\n",
+ progname);
+ return 1;
+ }
+
+ if (strcmp(old_fsname, new_fsname) == 0) {
+ fprintf(stderr,
+ "%s: cannot rename fsname '%s' to the same name\n",
+ progname, old_fsname);
+ return 1;
+ }
+ }
+#endif
+
/* Need to clear this flag after parsing 'L' and 'i' options. */
if (replace)
ldd->ldd_flags &= ~LDD_F_VIRGIN;
if (optind == argc) {
/* The user didn't specify device name */
fatal();
- fprintf(stderr, "Not enough arguments - device name or "
- "pool/dataset name not specified.\n");
+ fprintf(stderr,
+ "Not enough arguments - device name or pool/dataset name not specified.\n");
return EINVAL;
- } else {
- /* The device or pool/filesystem name */
- strscpy(mop->mo_device, argv[optind], sizeof(mop->mo_device));
-
- /* Followed by optional vdevs */
- if (optind < argc - 1)
- mop->mo_pool_vdevs = (char **) &argv[optind + 1];
}
+ /* The device or pool/filesystem name */
+ strscpy(mop->mo_device, argv[optind], sizeof(mop->mo_device));
+
+ /* Followed by optional vdevs */
+ if (optind < argc - 1)
+ mop->mo_pool_vdevs = (char **)&argv[optind + 1];
+
return 0;
}
struct lustre_disk_data *ldd = &mop.mo_ldd;
char *mountopts = NULL;
char wanted_mountopts[512] = "";
- unsigned mount_type;
+ char old_fsname[16] = "";
+ unsigned int mount_type;
int ret = 0;
int ret2 = 0;
progname = strrchr(argv[0], '/');
- if (progname != NULL)
+ if (progname)
progname++;
else
progname = argv[0];
return ret;
#ifdef TUNEFS
- /* For tunefs, we must read in the old values before parsing any
- new ones. */
+ /*
+ * For tunefs, we must read in the old values before parsing any
+ * new ones.
+ */
/* Check whether the disk has already been formatted by mkfs.lustre */
ret = osd_is_lustre(mop.mo_device, &mount_type);
if (ret == 0) {
fatal();
- fprintf(stderr, "Device %s has not been formatted with "
- "mkfs.lustre\n", mop.mo_device);
+ fprintf(stderr,
+ "Device %s has not been formatted with mkfs.lustre\n",
+ mop.mo_device);
ret = ENODEV;
goto out;
}
ret = osd_read_ldd(mop.mo_device, ldd);
if (ret != 0) {
fatal();
- fprintf(stderr, "Failed to read previous Lustre data from %s "
- "(%d)\n", mop.mo_device, ret);
+ fprintf(stderr,
+ "Failed to read previous Lustre data from %s (%d)\n",
+ mop.mo_device, ret);
goto out;
}
+ strscpy(old_fsname, ldd->ldd_fsname, sizeof(ldd->ldd_fsname));
ldd->ldd_flags &= ~(LDD_F_WRITECONF | LDD_F_VIRGIN);
/* svname of the form lustre:OST1234 means never registered */
mop.mo_mgs_failnodes++;
if (verbose > 0)
- print_ldd("Read previous values", ldd);
+ print_ldd("Read previous values", &mop);
#endif /* TUNEFS */
- ret = parse_opts(argc, argv, &mop, &mountopts);
+ ret = parse_opts(argc, argv, &mop, &mountopts, old_fsname);
if (ret != 0 || version)
goto out;
ldd->ldd_flags &= ~LDD_F_NEED_INDEX;
}
- if ((ldd->ldd_flags & (LDD_F_NEED_INDEX | LDD_F_UPGRADE14)) ==
- (LDD_F_NEED_INDEX | LDD_F_UPGRADE14)) {
- fatal();
- fprintf(stderr, "Can't find the target index, "
- "specify with --index\n");
- ret = EINVAL;
- goto out;
- }
-
if (ldd->ldd_flags & LDD_F_NEED_INDEX)
- fprintf(stderr, "warning: %s: for Lustre 2.4 and later, the "
- "target index must be specified with --index\n",
+ fprintf(stderr,
+ "warning: %s: for Lustre 2.4 and later, the target index must be specified with --index\n",
mop.mo_device);
/* If no index is supplied for MDT by default set index to zero */
ldd->ldd_flags &= ~LDD_F_NEED_INDEX;
ldd->ldd_svindex = 0;
}
+#ifndef TUNEFS
if (!IS_MGS(ldd) && (mop.mo_mgs_failnodes == 0)) {
+#else
+ /*
+ * Don't check --mgs or --mgsnode if print_only is set or
+ * --erase-params is set.
+ */
+ if (!IS_MGS(ldd) && (mop.mo_mgs_failnodes == 0) && !print_only &&
+ !(mop.mo_flags & MO_ERASE_ALL)) {
+#endif
fatal();
if (IS_MDT(ldd))
fprintf(stderr, "Must specify --mgs or --mgsnode\n");
}
if (server_make_name(ldd->ldd_flags, ldd->ldd_svindex,
- ldd->ldd_fsname, ldd->ldd_svname)) {
+ ldd->ldd_fsname, ldd->ldd_svname,
+ sizeof(ldd->ldd_svname))) {
printf("unknown server type %#x\n", ldd->ldd_flags);
goto out;
}
if (verbose >= 0)
- print_ldd("Permanent disk data", ldd);
+ print_ldd("Permanent disk data", &mop);
if (print_only) {
printf("exiting before disk write.\n");
}
if (check_mtab_entry(mop.mo_device, mop.mo_device, NULL, NULL))
- return(EEXIST);
+ return EEXIST;
/* Create the loopback file */
if (mop.mo_flags & MO_IS_LOOP) {
if (ret != 0) {
fatal();
fprintf(stderr, "Loop device setup for %s failed: %s\n",
- mop.mo_device, strerror(ret));
+ mop.mo_device, strerror(ret));
goto out;
}
}
ret = osd_is_lustre(mop.mo_device, &mount_type);
if (ret != 0) {
fatal();
- fprintf(stderr, "Device %s was previously formatted "
- "for lustre. Use --reformat to reformat it, "
- "or tunefs.lustre to modify.\n",
+ fprintf(stderr,
+ "Device %s was previously formatted for lustre. Use --reformat to reformat it, or tunefs.lustre to modify.\n",
mop.mo_device);
goto out;
}
fprintf(stderr, "mkfs failed %d\n", ret);
goto out;
}
-#else /* !TUNEFS */
+#else /* TUNEFS */
/* update svname with '=' to refresh config */
if (ldd->ldd_flags & LDD_F_WRITECONF) {
struct mount_opts opts;
+
opts.mo_ldd = *ldd;
opts.mo_source = mop.mo_device;
- (void) osd_label_lustre(&opts);
+ (void)osd_label_lustre(&opts);
+ }
+
+ /* Rename filesystem fsname */
+ if (mop.mo_flags & MO_RENAME) {
+ ret = osd_rename_fsname(&mop, old_fsname);
+ if (ret)
+ goto out;
}
/* Enable quota accounting */
fprintf(stderr, "failed to write local files\n");
goto out;
}
-
out:
osd_fini();
ret2 = loop_cleanup(&mop);