> - mapping = page_mapping(page); > + mapping = folio_mapping(folio); > if (mapping && mapping->a_ops->is_dirty_writeback) > - mapping->a_ops->is_dirty_writeback(page, dirty, writeback); > + mapping->a_ops->is_dirty_writeback(&folio->page, dirty, writeback); This adds an overly long line. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>