Re: [PATCH 3/6] folio: Add a function to get the host inode for a folio

From: David Howells
Date: Tue Aug 24 2021 - 10:12:02 EST


Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote:

> > +static inline struct inode *folio_inode(struct folio *folio)
> > +{
> > + return folio_file_mapping(folio)->host;
>
> You're contradicting yourself here. If you're allowed to call this
> function for swap cache pages, then the documentation needs to change.
> If you're not, then we can just use folio->mapping->host.

Um. I don't know. I'll do the latter, then, for now.

David