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
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/dm-devel
