dm integrity: decouple common code in dm_integrity_map_continue()
authorMikulas Patocka <mpatocka@redhat.com>
Tue, 3 Jul 2018 18:13:26 +0000 (20:13 +0200)
committerMike Snitzer <snitzer@redhat.com>
Fri, 27 Jul 2018 19:24:21 +0000 (15:24 -0400)
Decouple how dm_integrity_map_continue() responds to being out of free
sectors and when add_new_range() fails.

This has no functional change, but helps prepare for the next commit.

Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-integrity.c

index 0a8a4c2aa3ea525fe8ef8123f2268c8d6e5517c6..0829f18d91a33a140a1fb8db2a8b70bb8eb63c06 100644 (file)
@@ -1599,8 +1599,12 @@ retry:
 
                        dio->range.n_sectors = min(dio->range.n_sectors,
                                                   ic->free_sectors << ic->sb->log2_sectors_per_block);
-                       if (unlikely(!dio->range.n_sectors))
-                               goto sleep;
+                       if (unlikely(!dio->range.n_sectors)) {
+                               if (from_map)
+                                       goto offload_to_thread;
+                               sleep_on_endio_wait(ic);
+                               goto retry;
+                       }
                        range_sectors = dio->range.n_sectors >> ic->sb->log2_sectors_per_block;
                        ic->free_sectors -= range_sectors;
                        journal_section = ic->free_section;
@@ -1660,8 +1664,8 @@ retry:
                 * stall bios on current->bio_list.
                 * So, we offload the bio to a workqueue if we have to sleep.
                 */
-sleep:
                if (from_map) {
+offload_to_thread:
                        spin_unlock_irq(&ic->endio_wait.lock);
                        INIT_WORK(&dio->work, integrity_bio_wait);
                        queue_work(ic->wait_wq, &dio->work);