On Wed, Aug 23, 2023 at 03:30:22PM +0800, Yin Fengwei wrote: > On 8/22/23 04:20, Matthew Wilcox (Oracle) wrote: > > +static inline struct folio *vmalloc_to_folio(const void *addr) > > +{ > > + return page_folio(vmalloc_to_page(addr)); > I am wondering whether we should check the return value of vmalloc_to_page()? That's a good question. Almost every user of vmalloc_to_page() just assumes it works. But it'll cost very little to check for NULL, so I'll put that into the next version. Thanks!