> @@ -1000,13 +1000,12 @@ static int __bio_iov_iter_get_pages(struct bio *bio, struct iov_iter *iter)
> struct page *page = pages[i];
>
> len = min_t(size_t, PAGE_SIZE - offset, left);
> -
> if (__bio_try_merge_page(bio, page, len, offset, &same_page)) {
> if (same_page)
> put_page(page);
> } else {
> if (WARN_ON_ONCE(bio_full(bio, len)))
> - return -EINVAL;
> + return -EINVAL;
> __bio_add_page(bio, page, len, offset);
spurious whitespace changes. They both actually look good to me,
but don't really belong into this patch.
Otherwise this looks good to me:
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
