Re: [PATCH 01/10] fs: add SB_I_OVL_USE_SYNC_FS

[Christoph Hellwig]

On Tue, Oct 27, 2026 at 09:09:27PM +0000, Josef Bacik wrote:
> sync_filesystem() writes every dirty inode of a filesystem and then calls
> ->sync_fs().  The data needs that.

That second sentence makes absolutely zero sence and barely passes the
grammer checker, what is that supposed to mean?

>Whether the metadata does depends on
> the filesystem.

The metadata does what?  Since when did data and metadata become
subjects that act on their own?

> ext2 copies an inode to its buffer in ->write_inode() and
> nowhere else.

Seriously, please rewrite this commit log to make sense.  It reads
like utter garabage without any meaning.