On Fri, Aug 25, 2023 at 09:12:19PM +0100, Matthew Wilcox (Oracle) wrote: > +++ b/fs/ceph/addr.c > @@ -1608,29 +1608,30 @@ static vm_fault_t ceph_filemap_fault(struct vm_fault *vmf) > ret = VM_FAULT_SIGBUS; > } else { > struct address_space *mapping = inode->i_mapping; > - struct page *page; > + struct folio *folio; > > filemap_invalidate_lock_shared(mapping); > - page = find_or_create_page(mapping, 0, > + folio = __filemap_get_folio(mapping, 0, > + FGP_LOCK|FGP_ACCESSED|FGP_CREAT, > mapping_gfp_constraint(mapping, ~__GFP_FS)); > - if (!page) { > + if (!folio) { This needs to be "if (IS_ERR(folio))". Meant to fix that but forgot.