mdadm/mdopen: create new function create_named_array for writing to new_array

Split 'write to new_array' out into a function named create_named_array.
And fixed a trivial compiling warning 'warn_unused_result' against commit:
fdbf7aaa19 (mdopen: call "modprobe md_mod" if it might be needed.)

Suggested-by: NeilBrown <neilb@suse.com>
Signed-off-by: Zhilong Liu <zlliu@suse.com>
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
This commit is contained in:
Zhilong Liu 2017-10-16 15:54:18 +08:00 committed by Jes Sorensen
parent 1b21c449e6
commit 7105228e19
1 changed files with 27 additions and 20 deletions

View File

@ -100,6 +100,31 @@ void make_parts(char *dev, int cnt)
free(name);
}
int create_named_array(char *devnm)
{
int fd;
int n = -1;
static const char new_array_file[] = {
"/sys/module/md_mod/parameters/new_array"
};
fd = open(new_array_file, O_WRONLY);
if (fd < 0 && errno == ENOENT) {
if (system("modprobe md_mod") == 0)
fd = open(new_array_file, O_WRONLY);
}
if (fd >= 0) {
n = write(fd, devnm, strlen(devnm));
close(fd);
}
if (fd < 0 || n != (int)strlen(devnm)) {
pr_err("Fail create %s when using %s\n", devnm, new_array_file);
return 0;
}
return 1;
}
/*
* We need a new md device to assemble/build/create an array.
* 'dev' is a name given us by the user (command line or mdadm.conf)
@ -306,37 +331,19 @@ int create_mddev(char *dev, char *name, int autof, int trustworthy,
devnm[0] = 0;
if (num < 0 && cname && ci->names) {
int fd;
int n = -1;
sprintf(devnm, "md_%s", cname);
if (block_udev)
udev_block(devnm);
fd = open("/sys/module/md_mod/parameters/new_array", O_WRONLY);
if (fd < 0 && errno == ENOENT) {
system("modprobe md_mod");
fd = open("/sys/module/md_mod/parameters/new_array", O_WRONLY);
}
if (fd >= 0) {
n = write(fd, devnm, strlen(devnm));
close(fd);
}
if (n < 0) {
if (!create_named_array(devnm)) {
devnm[0] = 0;
udev_unblock();
}
}
if (num >= 0) {
int fd;
int n = -1;
sprintf(devnm, "md%d", num);
if (block_udev)
udev_block(devnm);
fd = open("/sys/module/md_mod/parameters/new_array", O_WRONLY);
if (fd >= 0) {
n = write(fd, devnm, strlen(devnm));
close(fd);
}
if (n < 0) {
if (!create_named_array(devnm)) {
devnm[0] = 0;
udev_unblock();
}