Looks fine:

Reviewed-by: Christoph Hellwig <[email protected]>

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.

--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to