On Sat, Mar 01, 2025 at 09:15:53AM +0800, Chao Yu wrote:
> > struct page *f2fs_get_node_page(struct f2fs_sb_info *sbi, pgoff_t nid)
> > {
> > - return __get_node_page(sbi, nid, NULL, 0);
> > + struct folio *folio = __get_node_folio(sbi, nid, NULL, 0);
> > +
>
> if (IS_ERR(folio))
> return ERR_CAST(folio));
>
> > + return &folio->page;
No need. It'll probably generate the saame code (or if not, it'll
generate worse code) and this wrapper function has to be deleted in
the next six to nine months anyway. We use this idiom extensively.
