mdadm: load default sysfs attributes after assemblation

[thirdparty/mdadm.git] / Examine.c

diff --git a/Examine.c b/Examine.c
index 90b25ec4421bb03a41eeff62eea5d901362df918..7013480d6dd80846cc2f464d3ba23c9e8e21284d 100644 (file)
--- a/Examine.c
+++ b/Examine.c
@@ -1,7 +1,7 @@
 /*
  * mdadm - manage Linux "md" devices aka RAID arrays.
  *
- * Copyright (C) 2001-2012 Neil Brown <neilb@suse.de>
+ * Copyright (C) 2001-2013 Neil Brown <neilb@suse.de>
  *
  *
  *    This program is free software; you can redistribute it and/or modify
@@ -53,7 +53,6 @@ int Examine(struct mddev_dev *devlist,
         */
        int fd;
        int rv = 0;
-       int err = 0;
 
        struct array {
                struct supertype *st;
@@ -66,6 +65,8 @@ int Examine(struct mddev_dev *devlist,
        for (; devlist ; devlist = devlist->next) {
                struct supertype *st;
                int have_container = 0;
+               int err = 0;
+               int container = 0;
 
                fd = dev_open(devlist->devname, O_RDONLY);
                if (fd < 0) {
@@ -74,44 +75,46 @@ int Examine(struct mddev_dev *devlist,
                                       devlist->devname, strerror(errno));
                                rv = 1;
                        }
-                       err = 1;
+                       continue;
                }
-               else {
-                       int container = 0;
-                       if (forcest)
-                               st = dup_super(forcest);
-                       else if (must_be_container(fd)) {
-                               /* might be a container */
-                               st = super_by_fd(fd, NULL);
-                               container = 1;
-                       } else
-                               st = guess_super(fd);
-                       if (st) {
-                               err = 1;
-                               st->ignore_hw_compat = 1;
-                               if (!container)
-                                       err = st->ss->load_super(st, fd,
-                                                                (c->brief||c->scan) ? NULL
-                                                                :devlist->devname);
-                               if (err && st->ss->load_container) {
-                                       err = st->ss->load_container(st, fd,
-                                                                (c->brief||c->scan) ? NULL
-                                                                :devlist->devname);
-                                       if (!err)
-                                               have_container = 1;
-                               }
-                               st->ignore_hw_compat = 0;
-                       } else {
-                               if (!c->brief) {
-                                       pr_err("No md superblock detected on %s.\n", devlist->devname);
-                                       rv = 1;
-                               }
-                               err = 1;
+
+               if (forcest)
+                       st = dup_super(forcest);
+               else if (must_be_container(fd)) {
+                       /* might be a container */
+                       st = super_by_fd(fd, NULL);
+                       container = 1;
+               } else
+                       st = guess_super(fd);
+               if (st) {
+                       err = 1;
+                       st->ignore_hw_compat = 1;
+                       if (!container)
+                               err = st->ss->load_super(st, fd,
+                                                        (c->brief||c->scan) ? NULL
+                                                        :devlist->devname);
+                       if (err && st->ss->load_container) {
+                               err = st->ss->load_container(st, fd,
+                                                            (c->brief||c->scan) ? NULL
+                                                            :devlist->devname);
+                               if (!err)
+                                       have_container = 1;
                        }
-                       close(fd);
+                       st->ignore_hw_compat = 0;
+               } else {
+                       if (!c->brief) {
+                               pr_err("No md superblock detected on %s.\n", devlist->devname);
+                               rv = 1;
+                       }
+                       err = 1;
                }
-               if (err)
+               close(fd);
+
+               if (err) {
+                       if (st)
+                               st->ss->free_super(st);
                        continue;
+               }
 
                if (c->SparcAdjust)
                        st->ss->update_super(st, NULL, "sparc2.2",
@@ -121,7 +124,7 @@ int Examine(struct mddev_dev *devlist,
                if (c->brief && st->ss->brief_examine_super == NULL) {
                        if (!c->scan)
                                pr_err("No brief listing for %s on %s\n",
-                                       st->ss->name, devlist->devname);
+                                      st->ss->name, devlist->devname);
                } else if (c->brief) {
                        struct array *ap;
                        char *d;