dm: simplify dm_sumbit_bio_remap interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux