diff options
author | NeilBrown <neilb@suse.de> | 2010-04-06 22:14:43 -0400 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2010-05-18 01:27:56 -0400 |
commit | d754c5ae1ff76b20d3ecde8ad666d7865eada8ae (patch) | |
tree | 60523d76c5e437ea39ce158295d75ea4247f70e1 /drivers/md/raid1.c | |
parent | 9e35b99c7efacfddc748c89a0c53b1122b0ee72c (diff) |
md/raid1: fix confusing 'redirect sector' message.
This message seems to suggest the named device is the one on which a
read failed, however it is actually the device that the read will be
redirected to.
So make the message a little clearer.
Reported-by: Tim Burgess <ozburgess@gmail.com>
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md/raid1.c')
-rw-r--r-- | drivers/md/raid1.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index 5ff75c4d3af6..2e08e48b02d9 100644 --- a/drivers/md/raid1.c +++ b/drivers/md/raid1.c | |||
@@ -1689,10 +1689,10 @@ static void raid1d(mddev_t *mddev) | |||
1689 | r1_bio->bios[r1_bio->read_disk] = bio; | 1689 | r1_bio->bios[r1_bio->read_disk] = bio; |
1690 | rdev = conf->mirrors[disk].rdev; | 1690 | rdev = conf->mirrors[disk].rdev; |
1691 | if (printk_ratelimit()) | 1691 | if (printk_ratelimit()) |
1692 | printk(KERN_ERR "raid1: %s: redirecting sector %llu to" | 1692 | printk(KERN_ERR "raid1: redirecting sector %llu to" |
1693 | " another mirror\n", | 1693 | " other mirror: %s\n", |
1694 | bdevname(rdev->bdev,b), | 1694 | (unsigned long long)r1_bio->sector, |
1695 | (unsigned long long)r1_bio->sector); | 1695 | bdevname(rdev->bdev,b)); |
1696 | bio->bi_sector = r1_bio->sector + rdev->data_offset; | 1696 | bio->bi_sector = r1_bio->sector + rdev->data_offset; |
1697 | bio->bi_bdev = rdev->bdev; | 1697 | bio->bi_bdev = rdev->bdev; |
1698 | bio->bi_end_io = raid1_end_read_request; | 1698 | bio->bi_end_io = raid1_end_read_request; |