[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