]> git.baikalelectronics.ru Git - kernel.git/commit
dm: fix bio length of empty flush
authorShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Fri, 15 Apr 2022 08:45:13 +0000 (17:45 +0900)
committerMike Snitzer <snitzer@kernel.org>
Fri, 15 Apr 2022 20:16:09 +0000 (16:16 -0400)
commitad82f50bf7eb0a99317cef8c73ea9c446a4884c7
tree88be1698935aecf7d069367065459eb5e6514c6f
parent5a6027329ef0ec9e1902b40f530541524d3d4aba
dm: fix bio length of empty flush

The commit 2139a37d694d ("dm: use bio_clone_fast in alloc_io/alloc_tio")
removed bio_clone_fast() call from alloc_tio() when ci->io->tio is
available. In this case, ci->bio is not copied to ci->io->tio.clone.
This is fine since init_clone_info() sets same values to ci->bio and
ci->io->tio.clone.

However, when incoming bios have REQ_PREFLUSH flag, __send_empty_flush()
prepares a zero length bio on stack and set it to ci->bio. At this time,
ci->io->tio.clone still keeps non-zero length. When alloc_tio() chooses
this ci->io->tio.clone as the bio to map, it is passed to targets as
non-empty flush bio. It causes bio length check failure in dm-zoned and
unexpected operation such as dm_accept_partial_bio() call.

To avoid the non-empty flush bio, set zero length to ci->io->tio.clone
in __send_empty_flush().

Fixes: 2139a37d694d ("dm: use bio_clone_fast in alloc_io/alloc_tio")
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
drivers/md/dm.c