Hey guys,
I'm checking through the archives and I didn't see any replies to this
patch, nor does it appear in our tree that I can see. We should
probably let him know what we're doing with it. :)
-Greg
On Mon, Jun 25, 2012 at 12:37 AM, Guangliang Zhao <gzhao@xxxxxxxx> wrote:
> Signed-off-by: Guangliang Zhao <gzhao@xxxxxxxx>
> ---
> drivers/block/rbd.c | 10 ++++------
> 1 files changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
> index 65665c9..3d6dfc8 100644
> --- a/drivers/block/rbd.c
> +++ b/drivers/block/rbd.c
> @@ -719,8 +719,6 @@ static struct bio *bio_chain_clone(struct bio **old, struct bio **next,
> goto err_out;
>
> if (total + old_chain->bi_size > len) {
> - struct bio_pair *bp;
> -
> /*
> * this split can only happen with a single paged bio,
> * split_bio will BUG_ON if this is not the case
> @@ -732,13 +730,13 @@ static struct bio *bio_chain_clone(struct bio **old, struct bio **next,
>
> /* split the bio. We'll release it either in the next
> call, or it will have to be released outside */
> - bp = bio_split(old_chain, (len - total) / SECTOR_SIZE);
> - if (!bp)
> + *bp = bio_split(old_chain, (len - total) / SECTOR_SIZE);
> + if (!*bp)
> goto err_out;
>
> - __bio_clone(tmp, &bp->bio1);
> + __bio_clone(tmp, &(*bp)->bio1);
>
> - *next = &bp->bio2;
> + *next = &(*bp)->bio2;
> } else {
> __bio_clone(tmp, old_chain);
> *next = old_chain->bi_next;
> --
> 1.7.3.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
