David Howells <dhowells@xxxxxxxxxx> wrote: > +static inline struct inode *folio_inode(struct folio *folio) > +{ > + return folio_mapping(folio)->host; > +} I've changed my mind on how I implemented it. I think that if this is intended to be used from within filesystems, then it should be calling folio_file_mapping() instead - or possibly just doing folio->mapping->host. folio_mapping() seems unnecessarily complex for a page a filesystem is pretty sure belongs to it. David