brd: use radix_tree_maybe_preload instead of radix_tree_preload
authorPankaj Raghav <p.raghav@samsung.com>
Fri, 17 Feb 2023 12:14:44 +0000 (17:44 +0530)
committerJens Axboe <axboe@kernel.dk>
Fri, 17 Feb 2023 13:15:53 +0000 (06:15 -0700)
commit0aa2988e4fd23c0c8b33999d7b47dfbc5e6bf24b
tree0b58f8f20c95007f4a260ab06d8d111b554b1670
parentf3ca73862453ac1e64fc6968a14bf66d839cd2d8
brd: use radix_tree_maybe_preload instead of radix_tree_preload

Unconditionally calling radix_tree_preload_end() results in a OOPS
message as the preload is only conditionally called for
gfpflags_allow_blocking().

[   20.267323] BUG: using smp_processor_id() in preemptible [00000000] code: fio/416
[   20.267837] caller is brd_insert_page.part.0+0xbe/0x190 [brd]
[   20.269436] Call Trace:
[   20.269598]  <TASK>
[   20.269742]  dump_stack_lvl+0x32/0x50
[   20.269982]  check_preemption_disabled+0xd1/0xe0
[   20.270289]  brd_insert_page.part.0+0xbe/0x190 [brd]
[   20.270664]  brd_submit_bio+0x33f/0xf40 [brd]

Use radix_tree_maybe_preload() which does preload only if
gfpflags_allow_blocking() is true but also takes the lock. Therefore,
unconditionally calling radix_tree_preload_end() should not create any
issues and the message disappears.

Fixes: 6ded703c56c2 ("brd: check for REQ_NOWAIT and set correct page allocation mask")
Signed-off-by: Pankaj Raghav <p.raghav@samsung.com>
Link: https://lore.kernel.org/r/20230217121442.33914-1-p.raghav@samsung.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/brd.c