brd: use cond_resched instead of cond_resched_rcu
authorPankaj Raghav <p.raghav@samsung.com>
Wed, 14 Jun 2023 13:35:38 +0000 (15:35 +0200)
committerJens Axboe <axboe@kernel.dk>
Wed, 14 Jun 2023 17:13:07 +0000 (11:13 -0600)
The body of the loop is run without RCU lock held. Use the regular
cond_resched() instead of cond_resched_rcu().

Fixes: 786bb0245881 ("brd: use XArray instead of radix-tree to index backing pages")
Suggested-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Pankaj Raghav <p.raghav@samsung.com>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Link: https://lore.kernel.org/r/20230614133538.1279369-1-p.raghav@samsung.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/brd.c

index 2f71376..970bd6f 100644 (file)
@@ -111,7 +111,7 @@ static void brd_free_pages(struct brd_device *brd)
 
        xa_for_each(&brd->brd_pages, idx, page) {
                __free_page(page);
-               cond_resched_rcu();
+               cond_resched();
        }
 
        xa_destroy(&brd->brd_pages);