On Wed, Dec 28, 2022 at 07:34:08PM +0800, Kefeng Wang wrote: > +static inline struct page *damon_get_page(unsigned long pfn) > +{ > + struct folio *folio = damon_get_folio(pfn); > + > + /* when folio is NULL, return &(0->page) mean return NULL */ > + return &folio->page; I really don't think this comment is needed. This is the standard idiom for converting from a folio to its head page.