9 files changed, 35 insertions, 50 deletions

diff --git a/drivers/md/faulty.c b/drivers/md/faulty.c
index e8b4574956c7..1277eb26b58a 100644
--- a/drivers/md/faulty.c
+++ b/drivers/md/faulty.c
@@ -332,13 +332,11 @@ static int run(struct mddev *mddev)
         return 0;
 }
 
-static int stop(struct mddev *mddev)
+static void faulty_free(struct mddev *mddev, void *priv)
 {
-        struct faulty_conf *conf = mddev->private;
+        struct faulty_conf *conf = priv;
 
         kfree(conf);
-        mddev->private = NULL;
-        return 0;
 }
 
 static struct md_personality faulty_personality =
@@ -348,7 +346,7 @@ static struct md_personality faulty_personality =
         .owner          = THIS_MODULE,
         .make_request   = make_request,
         .run            = run,
-        .stop           = stop,
+        .free           = faulty_free,
         .status         = status,
         .check_reshape  = reshape,
         .size           = faulty_size,
diff --git a/drivers/md/linear.c b/drivers/md/linear.c
index c201555b9c6c..fa7d577f3d12 100644
--- a/drivers/md/linear.c
+++ b/drivers/md/linear.c
@@ -249,14 +249,11 @@ static int linear_add(struct mddev *mddev, struct md_rdev *rdev)
         return 0;
 }
 
-static int linear_stop (struct mddev *mddev)
+static void linear_free(struct mddev *mddev, void *priv)
 {
-        struct linear_conf *conf = mddev->private;
+        struct linear_conf *conf = priv;
 
         kfree(conf);
-        mddev->private = NULL;
-
-        return 0;
 }
 
 static void linear_make_request(struct mddev *mddev, struct bio *bio)
@@ -335,7 +332,7 @@ static struct md_personality linear_personality =
         .owner          = THIS_MODULE,
         .make_request   = linear_make_request,
         .run            = linear_run,
-        .stop           = linear_stop,
+        .free           = linear_free,
         .status         = linear_status,
         .hot_add_disk   = linear_add,
         .size           = linear_size,
diff --git a/drivers/md/md.c b/drivers/md/md.c
index 58f140bef999..2920fd004865 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -293,8 +293,8 @@ static void md_make_request(struct request_queue *q, struct bio *bio)
 /* mddev_suspend makes sure no new requests are submitted
  * to the device, and that any requests that have been submitted
  * are completely handled.
- * Once ->stop is called and completes, the module will be completely
- * unused.
+ * Once mddev_detach() is called and completes, the module will be
+ * completely unused.
  */
 void mddev_suspend(struct mddev *mddev)
 {
@@ -3374,7 +3374,7 @@ level_store(struct mddev *mddev, const char *buf, size_t len)
         /* Looks like we have a winner */
         mddev_suspend(mddev);
         mddev_detach(mddev);
-        mddev->pers->stop(mddev);
+        mddev->pers->free(mddev, mddev->private);
 
         if (mddev->pers->sync_request == NULL &&
             pers->sync_request != NULL) {
@@ -4940,7 +4940,7 @@ int md_run(struct mddev *mddev)
         }
         if (err) {
                 mddev_detach(mddev);
-                mddev->pers->stop(mddev);
+                mddev->pers->free(mddev, mddev->private);
                 module_put(mddev->pers->owner);
                 mddev->pers = NULL;
                 bitmap_destroy(mddev);
@@ -5137,7 +5137,7 @@ static void __md_stop(struct mddev *mddev)
 {
         mddev->ready = 0;
         mddev_detach(mddev);
-        mddev->pers->stop(mddev);
+        mddev->pers->free(mddev, mddev->private);
         if (mddev->pers->sync_request && mddev->to_remove == NULL)
                 mddev->to_remove = &md_redundancy_group;
         module_put(mddev->pers->owner);
diff --git a/drivers/md/md.h b/drivers/md/md.h
index bee5b852c33f..37e7c17e56a6 100644
--- a/drivers/md/md.h
+++ b/drivers/md/md.h
@@ -465,7 +465,7 @@ struct md_personality
         struct module *owner;
         void (*make_request)(struct mddev *mddev, struct bio *bio);
         int (*run)(struct mddev *mddev);
-        int (*stop)(struct mddev *mddev);
+        void (*free)(struct mddev *mddev, void *priv);
         void (*status)(struct seq_file *seq, struct mddev *mddev);
         /* error_handler must set ->faulty and clear ->in_sync
          * if appropriate, and should abort recovery if needed
diff --git a/drivers/md/multipath.c b/drivers/md/multipath.c
index 9fe34453835b..ac3ede2bd00e 100644
--- a/drivers/md/multipath.c
+++ b/drivers/md/multipath.c
@@ -399,7 +399,7 @@ static int multipath_run (struct mddev *mddev)
         /*
          * copy the already verified devices into our private MULTIPATH
          * bookkeeping area. [whatever we allocate in multipath_run(),
-         * should be freed in multipath_stop()]
+         * should be freed in multipath_free()]
          */
 
         conf = kzalloc(sizeof(struct mpconf), GFP_KERNEL);
@@ -500,15 +500,13 @@ out:
         return -EIO;
 }
 
-static int multipath_stop (struct mddev *mddev)
+static void multipath_free(struct mddev *mddev, void *priv)
 {
-        struct mpconf *conf = mddev->private;
+        struct mpconf *conf = priv;
 
         mempool_destroy(conf->pool);
         kfree(conf->multipaths);
         kfree(conf);
-        mddev->private = NULL;
-        return 0;
 }
 
 static struct md_personality multipath_personality =
@@ -518,7 +516,7 @@ static struct md_personality multipath_personality =
         .owner          = THIS_MODULE,
         .make_request   = multipath_make_request,
         .run            = multipath_run,
-        .stop           = multipath_stop,
+        .free           = multipath_free,
         .status         = multipath_status,
         .error_handler  = multipath_error,
         .hot_add_disk   = multipath_add_disk,
diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c
index 01dfca94b663..a13f738a7b39 100644
--- a/drivers/md/raid0.c
+++ b/drivers/md/raid0.c
@@ -415,7 +415,7 @@ static sector_t raid0_size(struct mddev *mddev, sector_t sectors, int raid_disks
         return array_sectors;
 }
 
-static int raid0_stop(struct mddev *mddev);
+static void raid0_free(struct mddev *mddev, void *priv);
 
 static int raid0_run(struct mddev *mddev)
 {
@@ -468,20 +468,18 @@ static int raid0_run(struct mddev *mddev)
 
         ret = md_integrity_register(mddev);
         if (ret)
-                raid0_stop(mddev);
+                raid0_free(mddev, conf);
 
         return ret;
 }
 
-static int raid0_stop(struct mddev *mddev)
+static void raid0_free(struct mddev *mddev, void *priv)
 {