]> git.ipfire.org Git - thirdparty/mdadm.git/blobdiff - Assemble.c
mdadm: Fixup more broken logical operator formatting
[thirdparty/mdadm.git] / Assemble.c
index fa5fdbee125681eeb30ac25b2178ee15eeb04e96..afc6d9ca100e57dcd49a29af363da97517133e28 100644 (file)
@@ -149,6 +149,7 @@ static int select_devices(struct mddev_dev *devlist,
        struct mdinfo *content = NULL;
        int report_mismatch = ((inargv && c->verbose >= 0) || c->verbose > 0);
        struct domainlist *domains = NULL;
+       dev_t rdev;
 
        tmpdev = devlist; num_devs = 0;
        while (tmpdev) {
@@ -169,7 +170,6 @@ static int select_devices(struct mddev_dev *devlist,
             tmpdev = tmpdev ? tmpdev->next : NULL) {
                char *devname = tmpdev->devname;
                int dfd;
-               struct stat stb;
                struct supertype *tst;
                struct dev_policy *pol = NULL;
                int found_container = 0;
@@ -204,14 +204,7 @@ static int select_devices(struct mddev_dev *devlist,
                                pr_err("cannot open device %s: %s\n",
                                       devname, strerror(errno));
                        tmpdev->used = 2;
-               } else if (fstat(dfd, &stb)< 0) {
-                       /* Impossible! */
-                       pr_err("fstat failed for %s: %s\n",
-                              devname, strerror(errno));
-                       tmpdev->used = 2;
-               } else if ((stb.st_mode & S_IFMT) != S_IFBLK) {
-                       pr_err("%s is not a block device.\n",
-                              devname);
+               } else if (!fstat_is_blkdev(dfd, devname, &rdev)) {
                        tmpdev->used = 2;
                } else if (must_be_container(dfd)) {
                        if (st) {
@@ -222,21 +215,20 @@ static int select_devices(struct mddev_dev *devlist,
                                        pr_err("%s is a container, but we are looking for components\n",
                                               devname);
                                tmpdev->used = 2;
-#if !defined(MDASSEMBLE) || defined(MDASSEMBLE) && defined(MDASSEMBLE_AUTO)
                        } if (!tst && (tst = super_by_fd(dfd, NULL)) == NULL) {
                                if (report_mismatch)
                                        pr_err("not a recognisable container: %s\n",
                                               devname);
                                tmpdev->used = 2;
-#endif
-                       } else if (!tst->ss->load_container
-                                  || tst->ss->load_container(tst, dfd, NULL)) {
+                       } else if (!tst->ss->load_container ||
+                                  tst->ss->load_container(tst, dfd, NULL)) {
                                if (report_mismatch)
                                        pr_err("no correct container type: %s\n",
                                               devname);
                                tmpdev->used = 2;
                        } else if (auto_assem &&
-                                  !conf_test_metadata(tst->ss->name, (pol = devid_policy(stb.st_rdev)),
+                                  !conf_test_metadata(tst->ss->name,
+                                                      (pol = devid_policy(rdev)),
                                                       tst->ss->match_home(tst, c->homehost) == 1)) {
                                if (report_mismatch)
                                        pr_err("%s has metadata type %s for which auto-assembly is disabled\n",
@@ -263,7 +255,8 @@ static int select_devices(struct mddev_dev *devlist,
                                               tst->ss->name, devname);
                                tmpdev->used = 2;
                        } else if (auto_assem && st == NULL &&
-                                  !conf_test_metadata(tst->ss->name, (pol = devid_policy(stb.st_rdev)),
+                                  !conf_test_metadata(tst->ss->name,
+                                                      (pol = devid_policy(rdev)),
                                                       tst->ss->match_home(tst, c->homehost) == 1)) {
                                if (report_mismatch)
                                        pr_err("%s has metadata type %s for which auto-assembly is disabled\n",
@@ -486,7 +479,7 @@ static int select_devices(struct mddev_dev *devlist,
                /* Collect domain information from members only */
                if (tmpdev && tmpdev->used == 1) {
                        if (!pol)
-                               pol = devid_policy(stb.st_rdev);
+                               pol = devid_policy(rdev);
                        domain_merge(&domains, pol, tst?tst->ss->name:NULL);
                }
                dev_policy_free(pol);
@@ -519,15 +512,12 @@ static int select_devices(struct mddev_dev *devlist,
 
        /* Now reject spares that don't match domains of identified members */
        for (tmpdev = devlist; tmpdev; tmpdev = tmpdev->next) {
-               struct stat stb;
                if (tmpdev->used != 3)
                        continue;
-               if (stat(tmpdev->devname, &stb)< 0) {
-                       pr_err("fstat failed for %s: %s\n",
-                              tmpdev->devname, strerror(errno));
+               if (!stat_is_blkdev(tmpdev->devname, &rdev)) {
                        tmpdev->used = 2;
                } else {
-                       struct dev_policy *pol = devid_policy(stb.st_rdev);
+                       struct dev_policy *pol = devid_policy(rdev);
                        int dt = domain_test(domains, pol, NULL);
                        if (inargv && dt != 0)
                                /* take this spare as domains match
@@ -574,9 +564,7 @@ static int load_devices(struct devs *devices, char *devmap,
        struct mddev_dev *tmpdev;
        int devcnt = 0;
        int nextspare = 0;
-#ifndef MDASSEMBLE
        int bitmap_done = 0;
-#endif
        int most_recent = -1;
        int bestcnt = 0;
        int *best = *bestp;
@@ -592,7 +580,6 @@ static int load_devices(struct devs *devices, char *devmap,
                if (tmpdev->used != 1)
                        continue;
                /* looks like a good enough match to update the super block if needed */
-#ifndef MDASSEMBLE
                if (c->update) {
                        /* prepare useful information in info structures */
                        struct stat stb2;
@@ -683,9 +670,7 @@ static int load_devices(struct devs *devices, char *devmap,
                                else
                                        bitmap_done = 1;
                        }
-               } else
-#endif
-               {
+               } else {
                        dfd = dev_open(devname,
                                       tmpdev->disposition == 'I'
                                       ? O_RDWR : (O_RDWR|O_EXCL));
@@ -768,12 +753,12 @@ static int load_devices(struct devs *devices, char *devmap,
                                bestcnt = newbestcnt;
                        }
                        if (best[i] >=0 &&
-                           devices[best[i]].i.events
-                           == devices[devcnt].i.events
-                           && (devices[best[i]].i.disk.minor
-                               != devices[devcnt].i.disk.minor)
-                           && st->ss == &super0
-                           && content->array.level != LEVEL_MULTIPATH) {
+                           devices[best[i]].i.events ==
+                           devices[devcnt].i.events &&
+                           (devices[best[i]].i.disk.minor
+                            != devices[devcnt].i.disk.minor) &&
+                           st->ss == &super0 &&
+                           content->array.level != LEVEL_MULTIPATH) {
                                /* two different devices with identical superblock.
                                 * Could be a mis-detection caused by overlapping
                                 * partitions.  fail-safe.
@@ -791,9 +776,8 @@ static int load_devices(struct devs *devices, char *devmap,
                                *stp = st;
                                return -1;
                        }
-                       if (best[i] == -1
-                           || (devices[best[i]].i.events
-                               < devices[devcnt].i.events))
+                       if (best[i] == -1 || (devices[best[i]].i.events
+                                             < devices[devcnt].i.events))
                                best[i] = devcnt;
                }
                devcnt++;
@@ -816,14 +800,11 @@ static int force_array(struct mdinfo *content,
        int okcnt = 0;
        while (!enough(content->array.level, content->array.raid_disks,
                       content->array.layout, 1,
-                      avail)
-              ||
+                      avail) ||
               (content->reshape_active && content->delta_disks > 0 &&
                !enough(content->array.level, (content->array.raid_disks
                                               - content->delta_disks),
-                       content->new_layout, 1,
-                       avail)
-                      )) {
+                       content->new_layout, 1, avail))) {
                /* Choose the newest best drive which is
                 * not up-to-date, update the superblock
                 * and add it.
@@ -999,7 +980,7 @@ static int start_array(int mdfd,
        }
 
        /* First, add the raid disks, but add the chosen one last */
-       for (i=0; i<= bestcnt; i++) {
+       for (i = 0; i <= bestcnt; i++) {
                int j;
                if (i < bestcnt) {
                        j = best[i];
@@ -1009,8 +990,9 @@ static int start_array(int mdfd,
                        j = chosen_drive;
 
                if (j >= 0 && !devices[j].included) {
-                       int dfd = dev_open(devices[j].devname,
-                                          O_RDWR|O_EXCL);
+                       int dfd;
+
+                       dfd = dev_open(devices[j].devname, O_RDWR|O_EXCL);
                        if (dfd >= 0) {
                                remove_partitions(dfd);
                                close(dfd);
@@ -1019,28 +1001,30 @@ static int start_array(int mdfd,
 
                        if (rv) {
                                pr_err("failed to add %s to %s: %s\n",
-                                      devices[j].devname,
-                                      mddev,
+                                      devices[j].devname, mddev,
                                       strerror(errno));
-                               if (i < content->array.raid_disks * 2
-                                   || i == bestcnt)
+                               if (i < content->array.raid_disks * 2 ||
+                                   i == bestcnt)
                                        okcnt--;
                                else
                                        sparecnt--;
-                       } else if (c->verbose > 0)
+                       } else if (c->verbose > 0) {
                                pr_err("added %s to %s as %d%s%s\n",
                                       devices[j].devname, mddev,
                                       devices[j].i.disk.raid_disk,
                                       devices[j].uptodate?"":
                                       " (possibly out of date)",
-                                      (devices[j].i.disk.state & (1<<MD_DISK_REPLACEMENT))?" replacement":"");
+                                      (devices[j].i.disk.state &
+                                       (1<<MD_DISK_REPLACEMENT)) ?
+                                      " replacement":"");
+                       }
                } else if (j >= 0) {
                        if (c->verbose > 0)
                                pr_err("%s is already in %s as %d\n",
                                       devices[j].devname, mddev,
                                       devices[j].i.disk.raid_disk);
-               } else if (c->verbose > 0 && i < content->array.raid_disks*2
-                          && (i&1) == 0)
+               } else if (c->verbose > 0 &&
+                          i < content->array.raid_disks * 2 && (i & 1) == 0)
                        pr_err("no uptodate device for slot %d of %s\n",
                               i/2, mddev);
        }
@@ -1048,8 +1032,8 @@ static int start_array(int mdfd,
        if (content->array.level == LEVEL_CONTAINER) {
                if (c->verbose >= 0) {
                        pr_err("Container %s has been assembled with %d drive%s",
-                              mddev, okcnt+sparecnt+journalcnt,
-                              okcnt+sparecnt+journalcnt==1?"":"s");
+                              mddev, okcnt + sparecnt + journalcnt,
+                              okcnt + sparecnt + journalcnt == 1 ? "" : "s");
                        if (okcnt < (unsigned)content->array.raid_disks)
                                fprintf(stderr, " (out of %d)",
                                        content->array.raid_disks);
@@ -1058,10 +1042,13 @@ static int start_array(int mdfd,
 
                if (st->ss->validate_container) {
                        struct mdinfo *devices_list;
-                       struct mdinfo *info_devices = xmalloc(sizeof(struct mdinfo)*(okcnt+sparecnt));
+                       struct mdinfo *info_devices;
                        unsigned int count;
+
                        devices_list = NULL;
-                       for (count = 0; count < okcnt+sparecnt; count++) {
+                       info_devices = xmalloc(sizeof(struct mdinfo) *
+                                              (okcnt + sparecnt));
+                       for (count = 0; count < okcnt + sparecnt; count++) {
                                info_devices[count] = devices[count].i;
                                info_devices[count].next = devices_list;
                                devices_list = &info_devices[count];
@@ -1087,17 +1074,16 @@ static int start_array(int mdfd,
 
        if (c->runstop == 1 ||
            (c->runstop <= 0 &&
-            ( enough(content->array.level, content->array.raid_disks,
-                     content->array.layout, clean, avail) &&
-              (okcnt + rebuilding_cnt >= req_cnt || start_partial_ok)
-                    ))) {
+            (enough(content->array.level, content->array.raid_disks,
+                    content->array.layout, clean, avail) &&
+              (okcnt + rebuilding_cnt >= req_cnt || start_partial_ok)))) {
                /* This array is good-to-go.
                 * If a reshape is in progress then we might need to
                 * continue monitoring it.  In that case we start
                 * it read-only and let the grow code make it writable.
                 */
                int rv;
-#ifndef MDASSEMBLE
+
                if (content->reshape_active &&
                    !(content->reshape_active & RESHAPE_NO_BACKUP) &&
                    content->delta_disks <= 0) {
@@ -1117,12 +1103,11 @@ static int start_array(int mdfd,
                                                   c->backup_file, 0,
                                                   c->freeze_reshape);
                } else if (c->readonly &&
-                          sysfs_attribute_available(
-                                  content, NULL, "array_state")) {
+                          sysfs_attribute_available(content, NULL,
+                                                    "array_state")) {
                        rv = sysfs_set_str(content, NULL,
                                           "array_state", "readonly");
                } else
-#endif
                        rv = ioctl(mdfd, RUN_ARRAY, NULL);
                reopen_mddev(mdfd); /* drop O_EXCL */
                if (rv == 0) {
@@ -1130,13 +1115,19 @@ static int start_array(int mdfd,
                                pr_err("%s has been started with %d drive%s",
                                       mddev, okcnt, okcnt==1?"":"s");
                                if (okcnt < (unsigned)content->array.raid_disks)
-                                       fprintf(stderr, " (out of %d)", content->array.raid_disks);
+                                       fprintf(stderr, " (out of %d)",
+                                               content->array.raid_disks);
                                if (rebuilding_cnt)
-                                       fprintf(stderr, "%s %d rebuilding", sparecnt?",":" and", rebuilding_cnt);
+                                       fprintf(stderr, "%s %d rebuilding",
+                                               sparecnt?",":" and",
+                                               rebuilding_cnt);
                                if (sparecnt)
-                                       fprintf(stderr, " and %d spare%s", sparecnt, sparecnt==1?"":"s");
+                                       fprintf(stderr, " and %d spare%s",
+                                               sparecnt,
+                                               sparecnt == 1 ? "" : "s");
                                if (content->journal_clean)
-                                       fprintf(stderr, " and %d journal", journalcnt);
+                                       fprintf(stderr, " and %d journal",
+                                               journalcnt);
                                fprintf(stderr, ".\n");
                        }
                        if (content->reshape_active &&
@@ -1146,11 +1137,14 @@ static int start_array(int mdfd,
                                 * of the stripe cache - default is 256
                                 */
                                int chunk_size = content->array.chunk_size;
+
                                if (content->reshape_active &&
                                    content->new_chunk > chunk_size)
                                        chunk_size = content->new_chunk;
                                if (256 < 4 * ((chunk_size+4065)/4096)) {
-                                       struct mdinfo *sra = sysfs_read(mdfd, NULL, 0);
+                                       struct mdinfo *sra;
+
+                                       sra = sysfs_read(mdfd, NULL, 0);
                                        if (sra)
                                                sysfs_set_num(sra, NULL,
                                                              "stripe_cache_size",
@@ -1183,7 +1177,9 @@ static int start_array(int mdfd,
                        if (content->array.level == 6 &&
                            okcnt + 1 == (unsigned)content->array.raid_disks &&
                            was_forced) {
-                               struct mdinfo *sra = sysfs_read(mdfd, NULL, 0);
+                               struct mdinfo *sra;
+
+                               sra = sysfs_read(mdfd, NULL, 0);
                                if (sra)
                                        sysfs_set_str(sra, NULL,
                                                      "sync_action", "repair");
@@ -1191,45 +1187,47 @@ static int start_array(int mdfd,
                        }
                        return 0;
                }
-               pr_err("failed to RUN_ARRAY %s: %s\n",
-                      mddev, strerror(errno));
+               pr_err("failed to RUN_ARRAY %s: %s\n", mddev, strerror(errno));
 
                if (!enough(content->array.level, content->array.raid_disks,
                            content->array.layout, 1, avail))
                        pr_err("Not enough devices to start the array.\n");
                else if (!enough(content->array.level,
                                 content->array.raid_disks,
-                                content->array.layout, clean,
-                                avail))
+                                content->array.layout, clean, avail))
                        pr_err("Not enough devices to start the array while not clean - consider --force.\n");
 
                return 1;
        }
        if (c->runstop == -1) {
                pr_err("%s assembled from %d drive%s",
-                      mddev, okcnt, okcnt==1?"":"s");
+                      mddev, okcnt, okcnt == 1 ? "" : "s");
                if (okcnt != (unsigned)content->array.raid_disks)
-                       fprintf(stderr, " (out of %d)", content->array.raid_disks);
+                       fprintf(stderr, " (out of %d)",
+                               content->array.raid_disks);
                fprintf(stderr, ", but not started.\n");
                return 2;
        }
        if (c->verbose >= -1) {
-               pr_err("%s assembled from %d drive%s", mddev, okcnt, okcnt==1?"":"s");
+               pr_err("%s assembled from %d drive%s",
+                      mddev, okcnt, okcnt == 1 ? "" : "s");
                if (rebuilding_cnt)
-                       fprintf(stderr, "%s %d rebuilding", sparecnt?",":" and", rebuilding_cnt);
+                       fprintf(stderr, "%s %d rebuilding",
+                               sparecnt ? "," : " and", rebuilding_cnt);
                if (sparecnt)
-                       fprintf(stderr, " and %d spare%s", sparecnt, sparecnt==1?"":"s");
+                       fprintf(stderr, " and %d spare%s", sparecnt,
+                               sparecnt == 1 ? "" : "s");
                if (!enough(content->array.level, content->array.raid_disks,
                            content->array.layout, 1, avail))
                        fprintf(stderr, " - not enough to start the array.\n");
                else if (!enough(content->array.level,
                                 content->array.raid_disks,
-                                content->array.layout, clean,
-                                avail))
+                                content->array.layout, clean, avail))
                        fprintf(stderr, " - not enough to start the array while not clean - consider --force.\n");
                else {
                        if (req_cnt == (unsigned)content->array.raid_disks)
-                               fprintf(stderr, " - need all %d to start it", req_cnt);
+                               fprintf(stderr, " - need all %d to start it",
+                                       req_cnt);
                        else
                                fprintf(stderr, " - need %d to start", req_cnt);
                        fprintf(stderr, " (use --run to insist).\n");
@@ -1301,8 +1299,8 @@ int Assemble(struct supertype *st, char *mddev,
        int mdfd;
        int clean;
        int auto_assem = (mddev == NULL && !ident->uuid_set &&
-                         ident->super_minor == UnSet && ident->name[0] == 0
-                         && (ident->container == NULL || ident->member == NULL));
+                         ident->super_minor == UnSet && ident->name[0] == 0 &&
+                         (ident->container == NULL || ident->member == NULL));
        struct devs *devices;
        char *devmap;
        int *best = NULL; /* indexed by raid_disk */
@@ -1468,7 +1466,7 @@ try_again:
                        name = strchr(name, ':')+1;
 
                mdfd = create_mddev(mddev, name, ident->autof, trustworthy,
-                                   chosen_name);
+                                   chosen_name, 0);
        }
        if (mdfd < 0) {
                st->ss->free_super(st);
@@ -1477,13 +1475,6 @@ try_again:
                return 1;
        }
        mddev = chosen_name;
-       if (get_linux_version() < 2004000) {
-               pr_err("Assemble requires Linux 2.4 or later, and\n"
-                      "     md driver version 0.90.0 or later.\n"
-                      "    Upgrade your kernel or try --build\n");
-               close(mdfd);
-               return 1;
-       }
        if (pre_exist == NULL) {
                if (mddev_busy(fd2devnm(mdfd))) {
                        pr_err("%s already active, cannot restart it!\n",
@@ -1506,7 +1497,6 @@ try_again:
                ioctl(mdfd, STOP_ARRAY, NULL);
        }
 
-#ifndef MDASSEMBLE
        if (content != &info) {
                /* This is a member of a container.  Try starting the array. */
                int err;
@@ -1515,7 +1505,7 @@ try_again:
                close(mdfd);
                return err;
        }
-#endif
+
        /* Ok, no bad inconsistancy, we can try updating etc */
        devices = xcalloc(num_devs, sizeof(*devices));
        devmap = xcalloc(num_devs, content->array.raid_disks);
@@ -1668,14 +1658,13 @@ try_again:
                return 1;
        }
        st->ss->getinfo_super(st, content, NULL);
-#ifndef MDASSEMBLE
        if (sysfs_init(content, mdfd, NULL)) {
                pr_err("Unable to initialize sysfs\n");
                close(mdfd);
                free(devices);
                return 1;
        }
-#endif
+
        /* after reload context, store journal_clean in context */
        content->journal_clean = journal_clean;
        for (i=0; i<bestcnt; i++) {
@@ -1761,7 +1750,6 @@ try_again:
         * that was moved aside due to the reshape overwriting live data
         * The code of doing this lives in Grow.c
         */
-#ifndef MDASSEMBLE
        if (content->reshape_active &&
            !(content->reshape_active & RESHAPE_NO_BACKUP)) {
                int err = 0;
@@ -1813,7 +1801,6 @@ try_again:
                        return err;
                }
        }
-#endif
 
        /* Almost ready to actually *do* something */
        /* First, fill in the map, so that udev can find our name
@@ -1876,7 +1863,6 @@ try_again:
        return rv == 2 ? 0 : rv;
 }
 
-#ifndef MDASSEMBLE
 int assemble_container_content(struct supertype *st, int mdfd,
                               struct mdinfo *content, struct context *c,
                               char *chosen_name, int *result)
@@ -1901,7 +1887,7 @@ int assemble_container_content(struct supertype *st, int mdfd,
                    c->readonly &&
                    content->text_version[0] == '/')
                        content->text_version[0] = '-';
-               if (sysfs_set_array(content, md_get_version(mdfd)) != 0) {
+               if (sysfs_set_array(content, 9003) != 0) {
                        sysfs_free(sra);
                        return 1;
                }
@@ -2119,4 +2105,3 @@ int assemble_container_content(struct supertype *st, int mdfd,
        return err;
        /* FIXME should have an O_EXCL and wait for read-auto */
 }
-#endif