dm snapshot: fix metadata corruption
authorMikulas Patocka <mpatocka@redhat.com>
Mon, 3 Mar 2014 22:19:22 +0000 (17:19 -0500)
committerMike Snitzer <snitzer@redhat.com>
Mon, 3 Mar 2014 22:58:13 +0000 (17:58 -0500)
commit2c945820cab96ebf265598d998e63ef22393d0d4
treebf738c96d1c98f6743abfcf4312449d9e021140d
parentc64d240df31513522901539a3206a41e9e6d00c8
dm snapshot: fix metadata corruption

Commit 55494bf2947dccdf2 ("dm snapshot: use dm-bufio") broke snapshots.
Before that 3.14-rc1 commit, loading a snapshot's list of exceptions
involved reading exception areas one by one into ps->area and inserting
those exceptions into the hash table.  Commit 55494bf2947dccdf2 changed
it so that dm-bufio with prefetch is used to load exceptions in batchs.
Exceptions are loaded correctly, but ps->area is left uninitialized.
When a new exception is allocated, it is stored in this uninitialized
ps->area which will be written to the disk.  This causes metadata
corruption.

Fix this corruption by copying the last area that was read via dm-bufio
into ps->area.

Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-snap-persistent.c