dm snapshot: flush merged data before committing metadata
authorAkilesh Kailash <akailash@google.com>
Mon, 28 Dec 2020 07:14:07 +0000 (07:14 +0000)
committerMike Snitzer <snitzer@redhat.com>
Wed, 6 Jan 2021 23:28:34 +0000 (18:28 -0500)
commitfcc42338375a1e67b8568dbb558f8b784d0f3b01
treea8d0aa235499bc36dc1697cd560a11075f349262
parentd68b29584c25dbacd01ed44a3e45abb35353f1de
dm snapshot: flush merged data before committing metadata

If the origin device has a volatile write-back cache and the following
events occur:

1: After finishing merge operation of one set of exceptions,
   merge_callback() is invoked.
2: Update the metadata in COW device tracking the merge completion.
   This update to COW device is flushed cleanly.
3: System crashes and the origin device's cache where the recent
   merge was completed has not been flushed.

During the next cycle when we read the metadata from the COW device,
we will skip reading those metadata whose merge was completed in
step (1). This will lead to data loss/corruption.

To address this, flush the origin device post merge IO before
updating the metadata.

Cc: stable@vger.kernel.org
Signed-off-by: Akilesh Kailash <akailash@google.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-snap.c