On Wed, Oct 13, 2021 at 10:49:36AM -0600, Jens Axboe wrote: > +extern void __bio_advance(struct bio *, unsigned); No need for the extern, but it would be nice to spell out the argument names. > +static inline void bio_advance(struct bio *bio, unsigned int nbytes) > +{ The kerneldoc comment for bio_advance needs to move here now.