diff options
author | NeilBrown <neilb@suse.de> | 2009-12-13 20:49:46 -0500 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2009-12-13 20:49:46 -0500 |
commit | aa5cbd103887011b4830355f88fb055f9ad2d556 (patch) | |
tree | 92ef7c80999c6d0bfc6fef9a9cd3e4a75ee16c06 /drivers/md/bitmap.h | |
parent | f40542532e96dda5506eb76badea322f2ae4731c (diff) |
md/bitmap: protect against bitmap removal while being updated.
A write intent bitmap can be removed from an array while the
array is active.
When this happens, all IO is suspended and flushed before the
bitmap is removed.
However it is possible that bitmap_daemon_work is still running to
clear old bits from the bitmap. If it is, it can dereference the
bitmap after it has been freed.
So introduce a new mutex to protect bitmap_daemon_work and get it
before destroying a bitmap.
This is suitable for any current -stable kernel.
Signed-off-by: NeilBrown <neilb@suse.de>
Cc: stable@kernel.org
Diffstat (limited to 'drivers/md/bitmap.h')
-rw-r--r-- | drivers/md/bitmap.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/bitmap.h b/drivers/md/bitmap.h index e98900671ca9..7e38d13ddcac 100644 --- a/drivers/md/bitmap.h +++ b/drivers/md/bitmap.h | |||
@@ -282,7 +282,7 @@ void bitmap_close_sync(struct bitmap *bitmap); | |||
282 | void bitmap_cond_end_sync(struct bitmap *bitmap, sector_t sector); | 282 | void bitmap_cond_end_sync(struct bitmap *bitmap, sector_t sector); |
283 | 283 | ||
284 | void bitmap_unplug(struct bitmap *bitmap); | 284 | void bitmap_unplug(struct bitmap *bitmap); |
285 | void bitmap_daemon_work(struct bitmap *bitmap); | 285 | void bitmap_daemon_work(mddev_t *mddev); |
286 | #endif | 286 | #endif |
287 | 287 | ||
288 | #endif | 288 | #endif |