On Thu, May 02, 2019 at 07:33:28PM -0400, Christoph Hellwig wrote:
> @@ -1588,13 +1588,7 @@ static void iomap_dio_bio_end_io(struct bio *bio)
> if (should_dirty) {
> bio_check_pages_dirty(bio);
> } else {
> - if (!bio_flagged(bio, BIO_NO_PAGE_REF)) {
> - struct bvec_iter_all iter_all;
> - struct bio_vec *bvec;
> -
> - bio_for_each_segment_all(bvec, bio, iter_all)
> - put_page(bvec->bv_page);
> - }
> + bio_release_pages(bio);
Shouldn't this rather be:
if (!bio_flagged(bio, BIO_NO_PAGE_REF))
bio_release_pages(bio);
--
Johannes Thumshirn SUSE Labs Filesystems
jthumshirn@xxxxxxx +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Mary Higgins, Sri Rasiah
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
