On Tue, Jun 22, 2021 at 01:15:33PM +0100, Matthew Wilcox (Oracle) wrote: > Reimplement __set_page_dirty_nobuffers() as a wrapper around > filemap_dirty_folio(). This can use a cast to struct folio > because we know that the ->set_page_dirty address space op > is always called with a page pointer that happens to also be > a folio pointer. Saves 7 bytes of kernel text. Modulo the cast comment from the last patch this looks fine: Reviewed-by: Christoph Hellwig <hch@xxxxxx>