ext4: Correct ext4_journalled_dirty_folio() conversion

Matthew Wilcox (Oracle) · Matthew Wilcox (Oracle) · commit 0f2523366386 · 2022-04-01T14:40:44.000-04:00
This should use the new folio_buffers() instead of page_has_buffers().

Signed-off-by: Matthew Wilcox (Oracle) &lt;willy@infradead.org&gt;
Reviewed-by: Christoph Hellwig &lt;hch@lst.de&gt;
Reviewed-by: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
Acked-by: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
@@ -3589,7 +3589,7 @@ const struct iomap_ops ext4_iomap_report_ops = {
 static bool ext4_journalled_dirty_folio(struct address_space *mapping,
 		struct folio *folio)
 {
-	WARN_ON_ONCE(!page_has_buffers(&folio->page));
+	WARN_ON_ONCE(!folio_buffers(folio));
 	folio_set_checked(folio);
 	return filemap_dirty_folio(mapping, folio);
 }

Original file line number	Diff line number	Diff line change
`@@ -3589,7 +3589,7 @@ const struct iomap_ops ext4_iomap_report_ops = {`
`3589`	`3589`	`static bool ext4_journalled_dirty_folio(struct address_space *mapping,`
`3590`	`3590`	`struct folio *folio)`
`3591`	`3591`	`{`
`3592`		`- WARN_ON_ONCE(!page_has_buffers(&folio->page));`
	`3592`	`+ WARN_ON_ONCE(!folio_buffers(folio));`
`3593`	`3593`	`folio_set_checked(folio);`
`3594`	`3594`	`return filemap_dirty_folio(mapping, folio);`
`3595`	`3595`	`}`