On Wed, Jul 03, 2019 at 06:00:35AM -0700, Christoph Hellwig wrote: [snip] > phys_addr_t addr2 = page_to_phys(page) + offset + len - 1; > > if ((addr1 | mask) != (addr2 | mask)) > return false; > - > if (bv->bv_len + len > queue_max_segment_size(q)) > return false; > - > - return true; > + return __bio_try_merge_page(bio, page, len, offset, same_page); > } That's a lot of spurious whitespace changes here. -- 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