dm: make flush bios explicitly sync
Commitb685d3d65a
("block: treat REQ_FUA and REQ_PREFLUSH as synchronous") removed REQ_SYNC flag from WRITE_{FUA|PREFLUSH|...} definitions. generic_make_request_checks() however strips REQ_FUA and REQ_PREFLUSH flags from a bio when the storage doesn't report volatile write cache and thus write effectively becomes asynchronous which can lead to performance regressions. Fix the problem by making sure all bios which are synchronous are properly marked with REQ_SYNC. Fixes:b685d3d65a
("block: treat REQ_FUA and REQ_PREFLUSH as synchronous") Cc: stable@vger.kernel.org Signed-off-by: Jan Kara <jack@suse.cz> Signed-off-by: Mike Snitzer <snitzer@redhat.com>
This commit is contained in:
@@ -783,7 +783,8 @@ static void write_journal(struct dm_integrity_c *ic, unsigned commit_start, unsi
|
||||
for (i = 0; i < commit_sections; i++)
|
||||
rw_section_mac(ic, commit_start + i, true);
|
||||
}
|
||||
rw_journal(ic, REQ_OP_WRITE, REQ_FUA, commit_start, commit_sections, &io_comp);
|
||||
rw_journal(ic, REQ_OP_WRITE, REQ_FUA | REQ_SYNC, commit_start,
|
||||
commit_sections, &io_comp);
|
||||
} else {
|
||||
unsigned to_end;
|
||||
io_comp.in_flight = (atomic_t)ATOMIC_INIT(2);
|
||||
|
Reference in New Issue
Block a user