diff options
| author | Ming Lei <tom.leiming@gmail.com> | 2017-02-14 16:29:03 +0100 |
|---|---|---|
| committer | Shaohua Li <shli@fb.com> | 2017-02-15 20:24:54 +0100 |
| commit | d7a1030839d35c04a620e841f406b9b2a8600041 (patch) | |
| tree | 293b09d7b92528436daa8953c9202cf669702037 /drivers/md/raid5.c | |
| parent | md: remove unnecessary check on mddev (diff) | |
| download | linux-d7a1030839d35c04a620e841f406b9b2a8600041.tar.xz linux-d7a1030839d35c04a620e841f406b9b2a8600041.zip | |
md: fast clone bio in bio_clone_mddev()
Firstly bio_clone_mddev() is used in raid normal I/O and isn't
in resync I/O path.
Secondly all the direct access to bvec table in raid happens on
resync I/O except for write behind of raid1, in which we still
use bio_clone() for allocating new bvec table.
So this patch replaces bio_clone() with bio_clone_fast()
in bio_clone_mddev().
Also kill bio_clone_mddev() and call bio_clone_fast() directly, as
suggested by Christoph Hellwig.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Ming Lei <tom.leiming@gmail.com>
Signed-off-by: Shaohua Li <shli@fb.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/md/raid5.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index e2fa88fc6a1b..b193316804d4 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c @@ -5056,9 +5056,9 @@ static int raid5_read_one_chunk(struct mddev *mddev, struct bio *raid_bio) return 0; } /* - * use bio_clone_mddev to make a copy of the bio + * use bio_clone_fast to make a copy of the bio */ - align_bi = bio_clone_mddev(raid_bio, GFP_NOIO, mddev); + align_bi = bio_clone_fast(raid_bio, GFP_NOIO, mddev->bio_set); if (!align_bi) return 0; /* |