dm thin metadata: don't issue prefetches if a transaction abort has failed
authorJoe Thornber <ejt@redhat.com>
Tue, 1 Mar 2016 10:58:44 +0000 (10:58 +0000)
committerMike Snitzer <snitzer@redhat.com>
Thu, 10 Mar 2016 22:12:09 +0000 (17:12 -0500)
If a transaction abort has failed then we can no longer use the metadata
device.  Typically this happens if the superblock is unreadable.

This fix addresses a crash seen during metadata device failure testing.

Fixes: 8a01a6af75 ("dm thin: prefetch missing metadata pages")
Cc: stable@vger.kernel.org # 3.19+
Signed-off-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-thin-metadata.c

index dc947f54824ebd3604a3053efa614b439f416c34..43824d73366d07723e09acd3b5007e46a64a6d32 100644 (file)
@@ -1981,5 +1981,8 @@ bool dm_pool_metadata_needs_check(struct dm_pool_metadata *pmd)
 
 void dm_pool_issue_prefetches(struct dm_pool_metadata *pmd)
 {
-       dm_tm_issue_prefetches(pmd->tm);
+       down_read(&pmd->root_lock);
+       if (!pmd->fail_io)
+               dm_tm_issue_prefetches(pmd->tm);
+       up_read(&pmd->root_lock);
 }