From mboxrd@z Thu Jan  1 00:00:00 1970
From: Christoph Hellwig <hch@lst.de>
Date: Mon, 26 Nov 2018 14:11:20 +0100
Subject: [Cluster-devel] [PATCH V12 09/20] block: use
 bio_for_each_bvec() to compute multi-page bvec count
In-Reply-To: <20181126021720.19471-10-ming.lei@redhat.com>
References: <20181126021720.19471-1-ming.lei@redhat.com>
	<20181126021720.19471-10-ming.lei@redhat.com>
Message-ID: <20181126131120.GA6757@lst.de>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

Looks fine:

Reviewed-by: Christoph Hellwig <hch@lst.de>

Nitpick below:

> +{
> +	unsigned len = bv->bv_len;
> +	unsigned total_len = 0;
> +	unsigned new_nsegs = 0, seg_size = 0;
> +
> +	/*
> +	 * Multipage bvec may be too big to hold in one segment,
> +	 * so the current bvec has to be splitted as multiple
> +	 * segments.
> +	 */

Please use up all 80 chars in a line.