Re: [PATCH] block: Add bvec_folio()
Christoph Hellwig <[email protected]>
| Newsgroups | org.kernel.vger.io-uring,org.kernel.vger.linux-block,org.kernel.vger.linux-kernel,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On Tue, May 26, 2026 at 06:47:30PM +0100, Matthew Wilcox wrote: > How about: > > /** > * bvec_folio - Return the first folio referenced by this bvec > * @bv: bvec to access > * > * bvecs can contain non-folio memory, so this should only be called by > * the creator of the bvec; drivers have no business looking at the owner > * of the memory. It may not even be the right interface for the caller > * to use as bvecs can span multiple folios. You may be better off using > * something like bio_for_each_folio_all() which iterates over all folios. > */ Sounds good, although I'd captialize the first word in the sentence. (Not that anyone should follow my spelling advice in general)