[dm-devel] dm: simplify dm_sumbit_bio_remap interface
Mikulas Patocka
mpatocka at redhat.com
Mon Mar 14 11:40:29 UTC 2022
Hi
The patch b7f8dff09827c96032c34a945ee7757e394b5952 ("dm: simplify
dm_sumbit_bio_remap interface") removes the "from_wq" parameter and adds
io->map_task and compares it to the "current" pointer.
I think that this is not safe, because the original task that submitted
the bio may exit, a new task with the same "current" pointer may be
created and the test "if (io->map_task == current)" may spuriously return
true.
Or, do you have some reason why do you think that returning true
spuriously in this test doesn't matter?
Mikulas
More information about the dm-devel
mailing list