diff options
author | NeilBrown <neilb@suse.de> | 2014-12-14 20:56:58 -0500 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2015-02-03 16:35:52 -0500 |
commit | afa0f557cb15176570a18fb2a093e348a793afd4 (patch) | |
tree | 7ae3f6a37ae3e6bcfbea51faadad924df8020095 /drivers/md/md.h | |
parent | 5aa61f427e4979be733e4847b9199ff9cc48a47e (diff) |
md: rename ->stop to ->free
Now that the ->stop function only frees the private data,
rename is accordingly.
Also pass in the private pointer as an arg rather than using
mddev->private. This flexibility will be useful in level_store().
Finally, don't clear ->private. It doesn't make sense to clear
it seeing that isn't what we free, and it is no longer necessary
to clear ->private (it was some time ago before ->to_remove was
introduced).
Setting ->to_remove in ->free() is a bit of a wart, but not a
big problem at the moment.
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md/md.h')
-rw-r--r-- | drivers/md/md.h | 2 |
1 files changed, 1 insertions, 1 deletions
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 | |||
465 | struct module *owner; | 465 | struct module *owner; |
466 | void (*make_request)(struct mddev *mddev, struct bio *bio); | 466 | void (*make_request)(struct mddev *mddev, struct bio *bio); |
467 | int (*run)(struct mddev *mddev); | 467 | int (*run)(struct mddev *mddev); |
468 | int (*stop)(struct mddev *mddev); | 468 | void (*free)(struct mddev *mddev, void *priv); |
469 | void (*status)(struct seq_file *seq, struct mddev *mddev); | 469 | void (*status)(struct seq_file *seq, struct mddev *mddev); |
470 | /* error_handler must set ->faulty and clear ->in_sync | 470 | /* error_handler must set ->faulty and clear ->in_sync |
471 | * if appropriate, and should abort recovery if needed | 471 | * if appropriate, and should abort recovery if needed |