On Thu, Jul 11, 2024 at 10:37:50AM +0530, Kundan Kumar wrote: > +static inline void bio_release_folio(struct bio *bio, struct folio *folio, > + unsigned long npages) > +{ > + unpin_user_folio(folio, npages); > +} Why doesn't this need to check BIO_PAGE_PINNED like bio_release_page() does? That should be explained, certainly in the commit message or maybe in a comment.