On Fri, 2025-02-14 at 19:34 +0000, Matthew Wilcox wrote:
> On Fri, Feb 14, 2025 at 07:10:26PM +0000, Viacheslav Dubeyko wrote:
> > > - cache_ctl->page = find_lock_page(&dir->i_data, ptr_pgoff);
> > > - if (!cache_ctl->page) {
> > > + cache_ctl->folio = filemap_lock_folio(&dir->i_data, ptr_pgoff);
> > > + if (IS_ERR(cache_ctl->folio)) {
> > > + cache_ctl->folio = NULL;
> > > doutc(cl, " page %lu not found\n", ptr_pgoff);
> >
> > Maybe, we need to change debug output here too?
> >
> > doutc(cl, " folio %lu not found\n", ptr_pgoff);
>
> I'm happy to make that change for the next version, or for somebody to
> make that change while applying the patches.
It's not critical one. The patch looks good.
Reviewed-by: Viacheslav Dubeyko <Slava.Dubeyko@xxxxxxx>
Thanks,
Slava.
