diff options
author | Mike Snitzer <snitzer@redhat.com> | 2012-12-21 15:23:34 -0500 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2012-12-21 15:23:34 -0500 |
commit | a5bd968aeb87af4a3a96ecdd78912832997e8a9d (patch) | |
tree | cb12dce916dfdbde42d1cf283d6756590bfd82fb | |
parent | 3a0f9aaee02857609d79b20c809c02a8b7c39d06 (diff) |
dm block manager: reinstate message when validator fails
Reinstate a useful error message when the block manager buffer validator fails.
This was mistakenly eliminated when the block manager was converted to use
dm-bufio.
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
-rw-r--r-- | drivers/md/persistent-data/dm-block-manager.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/md/persistent-data/dm-block-manager.c b/drivers/md/persistent-data/dm-block-manager.c index 5ba277768d9..47b683e67a7 100644 --- a/drivers/md/persistent-data/dm-block-manager.c +++ b/drivers/md/persistent-data/dm-block-manager.c | |||
@@ -428,8 +428,11 @@ static int dm_bm_validate_buffer(struct dm_block_manager *bm, | |||
428 | if (!v) | 428 | if (!v) |
429 | return 0; | 429 | return 0; |
430 | r = v->check(v, (struct dm_block *) buf, dm_bufio_get_block_size(bm->bufio)); | 430 | r = v->check(v, (struct dm_block *) buf, dm_bufio_get_block_size(bm->bufio)); |
431 | if (unlikely(r)) | 431 | if (unlikely(r)) { |
432 | DMERR("%s validator check failed for block %llu", v->name, | ||
433 | (unsigned long long) dm_bufio_get_block_number(buf)); | ||
432 | return r; | 434 | return r; |
435 | } | ||
433 | aux->validator = v; | 436 | aux->validator = v; |
434 | } else { | 437 | } else { |
435 | if (unlikely(aux->validator != v)) { | 438 | if (unlikely(aux->validator != v)) { |