On Thu, Sep 05, 2019 at 06:27:21PM +0200, Christoph Hellwig wrote: > > + if (dio->dops && dio->dops->submit_io) { > > + dio->dops->submit_io(bio, file_inode(dio->iocb->ki_filp), > > + pos); > > pos would still fit on the previously line as-is. > > > + dio->submit.cookie = BLK_QC_T_NONE; > > But I think you should return the cookie from the submit function for > completeness, even if btrfs would currently always return BLK_QC_T_NONE. Yeah, that looked funny to me too. --D