Prevent DM RAID from loading bitmap twice.
authorJonathan Brassow <jbrassow@redhat.com>
Fri, 27 Jan 2012 20:53:53 +0000 (14:53 -0600)
committerNeilBrown <neilb@suse.de>
Mon, 30 Jan 2012 22:43:41 +0000 (09:43 +1100)
commit34f8ac6d79e5446c6242e4bcb474f152c857c5c6
tree5a0be3b7910428e52603304aa53e8da4476fb4b2
parent307729c8bc5b5a41361af8af95906eee7552acb1
Prevent DM RAID from loading bitmap twice.

The life cycle of a device-mapper target is:
1) create
2) resume
3) suspend
*) possibly repeat from 2
4) destroy

The dm-raid target is unconditionally calling MD's bitmap_load function upon
every resume.  If steps 2 & 3 above are repeated, bitmap_load is called
multiple times.  It is only written to be called once; otherwise, it allocates
new memory for the bitmap (without freeing the old) and incrementing the number
of pages it thinks it has without zeroing first.  This ultimately leads to
access beyond allocated memory and lost memory.

Simply avoiding the bitmap_load call upon resume is not sufficient.  If the
target was suspended while the initial recovery was only partially complete,
it needs to be restarted when the target is resumed.  This is why
'md_wakeup_thread' is called before issuing the 'mddev_resume'.

Signed-off-by: Jonathan Brassow <jbrassow@redhat.com>
Signed-off-by: NeilBrown <neilb@suse.de>
drivers/md/dm-raid.c
This page took 0.026558 seconds and 5 git commands to generate.