dm thin: fix bug where bio that overwrites thin block ignores FUA
authorNikos Tsironis <ntsironis@arrikto.com>
Thu, 14 Feb 2019 18:38:47 +0000 (20:38 +0200)
committerMike Snitzer <snitzer@redhat.com>
Fri, 15 Feb 2019 00:02:29 +0000 (19:02 -0500)
commit4ae280b4ee3463fa57bbe6eede26b97daff8a0f1
treeafe587e40391199bed1d9b5fdff20f32db8bad28
parentff0c129d3b5ecb3df7c8f5e2236582bf745b6c5f
dm thin: fix bug where bio that overwrites thin block ignores FUA

When provisioning a new data block for a virtual block, either because
the block was previously unallocated or because we are breaking sharing,
if the whole block of data is being overwritten the bio that triggered
the provisioning is issued immediately, skipping copying or zeroing of
the data block.

When this bio completes the new mapping is inserted in to the pool's
metadata by process_prepared_mapping(), where the bio completion is
signaled to the upper layers.

This completion is signaled without first committing the metadata.  If
the bio in question has the REQ_FUA flag set and the system crashes
right after its completion and before the next metadata commit, then the
write is lost despite the REQ_FUA flag requiring that I/O completion for
this request must only be signaled after the data has been committed to
non-volatile storage.

Fix this by deferring the completion of overwrite bios, with the REQ_FUA
flag set, until after the metadata has been committed.

Cc: stable@vger.kernel.org
Signed-off-by: Nikos Tsironis <ntsironis@arrikto.com>
Acked-by: Joe Thornber <ejt@redhat.com>
Acked-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-thin.c