diff options
author | Neil Brown <neilb@suse.com> | 2015-09-30 13:11:22 +1000 |
---|---|---|
committer | Jiri Slaby <jslaby@suse.cz> | 2015-10-05 22:02:55 +0200 |
commit | e460fa8bdc1f744a92dd20a968ac652c6cacaf89 (patch) | |
tree | 3dad78f3ad30cbbbadddd7fd85bda77c7f61cf24 | |
parent | 2ca1ae468673a36d1a2dc5946995ba617f8cba96 (diff) | |
download | lwn-e460fa8bdc1f744a92dd20a968ac652c6cacaf89.tar.gz lwn-e460fa8bdc1f744a92dd20a968ac652c6cacaf89.zip |
md: flush ->event_work before stopping array.
commit ee5d004fd0591536a061451eba2b187092e9127c upstream.
The 'event_work' worker used by dm-raid may still be running
when the array is stopped. This can result in an oops.
So flush the workqueue on which it is run after detaching
and before destroying the device.
Reported-by: Heinz Mauelshagen <heinzm@redhat.com>
Signed-off-by: NeilBrown <neilb@suse.com>
Fixes: 9d09e663d550 ("dm: raid456 basic support")
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
-rw-r--r-- | drivers/md/md.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 1c512dc1f17f..81bf511b3182 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -5315,6 +5315,8 @@ EXPORT_SYMBOL_GPL(md_stop_writes); static void __md_stop(struct mddev *mddev) { mddev->ready = 0; + /* Ensure ->event_work is done */ + flush_workqueue(md_misc_wq); mddev->pers->stop(mddev); if (mddev->pers->sync_request && mddev->to_remove == NULL) mddev->to_remove = &md_redundancy_group; |