On Thu, Oct 13, 2022 at 03:53:52PM +0200, Christoph Böhmwalder wrote: > + if (get_ldev(device)) { > + req->private_bio = bio_alloc_clone(device->ldev->backing_bdev, > + bio, GFP_NOIO, &drbd_io_bio_set); This adds an overly long line. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch at lst.de>