Commit f2859af6 authored by Dan Williams's avatar Dan Williams Committed by NeilBrown

md: allow integers to be passed to md/level

e.g. allow md to interpret 'echo 4 > md/level' as a request for raid4.
Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>
parent bb7f8d22
...@@ -2934,9 +2934,10 @@ level_show(mddev_t *mddev, char *page) ...@@ -2934,9 +2934,10 @@ level_show(mddev_t *mddev, char *page)
static ssize_t static ssize_t
level_store(mddev_t *mddev, const char *buf, size_t len) level_store(mddev_t *mddev, const char *buf, size_t len)
{ {
char level[16]; char clevel[16];
ssize_t rv = len; ssize_t rv = len;
struct mdk_personality *pers; struct mdk_personality *pers;
long level;
void *priv; void *priv;
mdk_rdev_t *rdev; mdk_rdev_t *rdev;
...@@ -2969,19 +2970,22 @@ level_store(mddev_t *mddev, const char *buf, size_t len) ...@@ -2969,19 +2970,22 @@ level_store(mddev_t *mddev, const char *buf, size_t len)
} }
/* Now find the new personality */ /* Now find the new personality */
if (len == 0 || len >= sizeof(level)) if (len == 0 || len >= sizeof(clevel))
return -EINVAL; return -EINVAL;
strncpy(level, buf, len); strncpy(clevel, buf, len);
if (level[len-1] == '\n') if (clevel[len-1] == '\n')
len--; len--;
level[len] = 0; clevel[len] = 0;
if (strict_strtol(clevel, 10, &level))
level = LEVEL_NONE;
request_module("md-%s", level); if (request_module("md-%s", clevel) != 0)
request_module("md-level-%s", clevel);
spin_lock(&pers_lock); spin_lock(&pers_lock);
pers = find_pers(LEVEL_NONE, level); pers = find_pers(level, clevel);
if (!pers || !try_module_get(pers->owner)) { if (!pers || !try_module_get(pers->owner)) {
spin_unlock(&pers_lock); spin_unlock(&pers_lock);
printk(KERN_WARNING "md: personality %s not loaded\n", level); printk(KERN_WARNING "md: personality %s not loaded\n", clevel);
return -EINVAL; return -EINVAL;
} }
spin_unlock(&pers_lock); spin_unlock(&pers_lock);
...@@ -2994,7 +2998,7 @@ level_store(mddev_t *mddev, const char *buf, size_t len) ...@@ -2994,7 +2998,7 @@ level_store(mddev_t *mddev, const char *buf, size_t len)
if (!pers->takeover) { if (!pers->takeover) {
module_put(pers->owner); module_put(pers->owner);
printk(KERN_WARNING "md: %s: %s does not support personality takeover\n", printk(KERN_WARNING "md: %s: %s does not support personality takeover\n",
mdname(mddev), level); mdname(mddev), clevel);
return -EINVAL; return -EINVAL;
} }
...@@ -3010,7 +3014,7 @@ level_store(mddev_t *mddev, const char *buf, size_t len) ...@@ -3010,7 +3014,7 @@ level_store(mddev_t *mddev, const char *buf, size_t len)
mddev->delta_disks = 0; mddev->delta_disks = 0;
module_put(pers->owner); module_put(pers->owner);
printk(KERN_WARNING "md: %s: %s would not accept array\n", printk(KERN_WARNING "md: %s: %s would not accept array\n",
mdname(mddev), level); mdname(mddev), clevel);
return PTR_ERR(priv); return PTR_ERR(priv);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment