md/raid1: use rdev in raid1_write_request directly
authorGuoqing Jiang <guoqing.jiang@linux.dev>
Mon, 4 Oct 2021 15:34:50 +0000 (23:34 +0800)
committerJens Axboe <axboe@kernel.dk>
Mon, 18 Oct 2021 20:50:28 +0000 (14:50 -0600)
We already get rdev from conf->mirrors[i].rdev at the beginning of the
loop, so just use it.

Signed-off-by: Guoqing Jiang <guoqing.jiang@linux.dev>
Signed-off-by: Song Liu <songliubraving@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/md/raid1.c

index 6ba12f0..7dc8026 100644 (file)
@@ -1529,13 +1529,12 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio,
 
                r1_bio->bios[i] = mbio;
 
-               mbio->bi_iter.bi_sector = (r1_bio->sector +
-                                  conf->mirrors[i].rdev->data_offset);
-               bio_set_dev(mbio, conf->mirrors[i].rdev->bdev);
+               mbio->bi_iter.bi_sector = (r1_bio->sector + rdev->data_offset);
+               bio_set_dev(mbio, rdev->bdev);
                mbio->bi_end_io = raid1_end_write_request;
                mbio->bi_opf = bio_op(bio) | (bio->bi_opf & (REQ_SYNC | REQ_FUA));
-               if (test_bit(FailFast, &conf->mirrors[i].rdev->flags) &&
-                   !test_bit(WriteMostly, &conf->mirrors[i].rdev->flags) &&
+               if (test_bit(FailFast, &rdev->flags) &&
+                   !test_bit(WriteMostly, &rdev->flags) &&
                    conf->raid_disks - mddev->degraded > 1)
                        mbio->bi_opf |= MD_FAILFAST;
                mbio->bi_private = r1_bio;
@@ -1546,7 +1545,7 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio,
                        trace_block_bio_remap(mbio, disk_devt(mddev->gendisk),
                                              r1_bio->sector);
                /* flush_pending_writes() needs access to the rdev so...*/
-               mbio->bi_bdev = (void *)conf->mirrors[i].rdev;
+               mbio->bi_bdev = (void *)rdev;
 
                cb = blk_check_plugged(raid1_unplug, mddev, sizeof(*plug));
                if (cb)