| From | David Howells <> | Subject | Re: [PATCH v11 15/33] mm/filemap: Add folio_offset() and folio_file_offset() | Date | Wed, 16 Jun 2021 11:05:21 +0100 |
| |
Matthew Wilcox (Oracle) <willy@infradead.org> wrote:
> These are just wrappers around their page counterpart. No change to > generated code. > > Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org> > Reviewed-by: Christoph Hellwig <hch@lst.de> > Acked-by: Jeff Layton <jlayton@kernel.org> > Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com> > Acked-by: Vlastimil Babka <vbabka@suse.cz> > Reviewed-by: William Kucharski <william.kucharski@oracle.com>
I wonder if folio_fpos() would be better a better name than folio_offset(). "Offset" is a bit overloaded.
Reviewed-by: David Howells <dhowells@redhat.com>
|