Re: [PATCH 002 of 2] md: Make sure bi_max_vecs is set properly in bio_split

From: Jens Axboe
Date: Mon May 22 2006 - 03:01:07 EST


On Mon, May 22 2006, NeilBrown wrote:
>
> Else a subsequence bio_clone might make a mess.
>
> Signed-off-by: Neil Brown <neilb@xxxxxxx>
> Cc: "Don Dupuis" <dondster@xxxxxxxxx>
> Cc: Jens Axboe <axboe@xxxxxxx>
> ### Diffstat output
> ./fs/bio.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff ./fs/bio.c~current~ ./fs/bio.c
> --- ./fs/bio.c~current~ 2006-05-22 16:12:46.000000000 +1000
> +++ ./fs/bio.c 2006-05-22 16:12:16.000000000 +1000
> @@ -1103,6 +1103,9 @@ struct bio_pair *bio_split(struct bio *b
> bp->bio1.bi_io_vec = &bp->bv1;
> bp->bio2.bi_io_vec = &bp->bv2;
>
> + bp->bio1.bi_max_vecs = 1;
> + bp->bio2.bi_max_vecs = 1;
> +
> bp->bio1.bi_end_io = bio_pair_end_1;
> bp->bio2.bi_end_io = bio_pair_end_2;
>

Obviously correct, thanks Neil.

--
Jens Axboe

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/