diff options
| author | NeilBrown <neilb@suse.de> | 2012-07-03 09:56:52 +0400 |
|---|---|---|
| committer | NeilBrown <neilb@suse.de> | 2012-07-03 09:56:52 +0400 |
| commit | 0232605d987d8230b254aa139805bbb56a7ca30c (patch) | |
| tree | 18537d016ecc0f12b41b4589818b4c047cd58bcf /drivers/md/md.c | |
| parent | 055d3747dbf00ce85c6872ecca4d466638e80c22 (diff) | |
| download | linux-0232605d987d8230b254aa139805bbb56a7ca30c.tar.xz | |
md: make 'name' arg to md_register_thread non-optional.
Having the 'name' arg optional and defaulting to the current
personality name is no necessary and leads to errors, as when
changing the level of an array we can end up using the
name of the old level instead of the new one.
So make it non-optional and always explicitly pass the name
of the level that the array will be.
Reported-by: majianpeng <majianpeng@gmail.com>
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md/md.c')
| -rw-r--r-- | drivers/md/md.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 973aa8459e98..c601c4be77c7 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -6751,7 +6751,7 @@ struct md_thread *md_register_thread(void (*run) (struct mddev *), struct mddev thread->tsk = kthread_run(md_thread, thread, "%s_%s", mdname(thread->mddev), - name ?: mddev->pers->name); + name); if (IS_ERR(thread->tsk)) { kfree(thread); return NULL; |