diff options
author | Guoqing Jiang <guoqing.jiang@linux.dev> | 2022-08-17 14:05:13 +0200 |
---|---|---|
committer | Song Liu <song@kernel.org> | 2022-08-24 20:19:23 +0200 |
commit | 1d258758cf06a0734482989911d184dd5837ed4e (patch) | |
tree | 3d5c95e14f7b4a4f4b77c2b7690628e359e6ea89 /drivers/md/md.c | |
parent | md: Flush workqueue md_rdev_misc_wq in md_alloc() (diff) | |
download | linux-1d258758cf06a0734482989911d184dd5837ed4e.tar.xz linux-1d258758cf06a0734482989911d184dd5837ed4e.zip |
Revert "md-raid: destroy the bitmap after destroying the thread"
This reverts commit e151db8ecfb019b7da31d076130a794574c89f6f. Because it
obviously breaks clustered raid as noticed by Neil though it fixed KASAN
issue for dm-raid, let's revert it and fix KASAN issue in next commit.
[1]. https://lore.kernel.org/linux-raid/a6657e08-b6a7-358b-2d2a-0ac37d49d23a@linux.dev/T/#m95ac225cab7409f66c295772483d091084a6d470
Fixes: e151db8ecfb0 ("md-raid: destroy the bitmap after destroying the thread")
Signed-off-by: Guoqing Jiang <guoqing.jiang@linux.dev>
Signed-off-by: Song Liu <song@kernel.org>
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 71d221601bf8..107c4c953c35 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -6239,11 +6239,11 @@ static void mddev_detach(struct mddev *mddev) static void __md_stop(struct mddev *mddev) { struct md_personality *pers = mddev->pers; + md_bitmap_destroy(mddev); mddev_detach(mddev); /* Ensure ->event_work is done */ if (mddev->event_work.func) flush_workqueue(md_misc_wq); - md_bitmap_destroy(mddev); spin_lock(&mddev->lock); mddev->pers = NULL; spin_unlock(&mddev->lock); |