Re: [syzbot ci] Re: ext4: deferred iput framework for EA inodes

[Zhou, Yun]

Hi Jan,

On 7/3/26 22:02, Jan Kara wrote:
> On Thu 02-07-26 15:57:01, syzbot ci wrote:
> > syzbot ci has tested the following series
> >
> > [v13] ext4: deferred iput framework for EA inodes
> > https://lore.kernel.org/all/20260702152225.4071746-1-yun.zhou@xxxxxxxxxxxxx
> > * [PATCH v13 1/4] fs: add iput_if_not_last() helper
> > * [PATCH v13 2/4] ext4: introduce ext4_put_ea_inode() for safe deferred iput
> > * [PATCH v13 3/4] ext4: convert all EA inode iput() calls to ext4_put_ea_inode()
> > * [PATCH v13 4/4] ext4: remove ea_inode_array mechanism in favor of ext4_put_ea_inode()
> >
> > and found the following issue:
> > WARNING in ext4_journal_check_start
>
> Good catch. We need to flush the pending work with iputs also in
> ext4_sync_fs() which should also take care of this race with remount
> read-only (and the same issue with filesystem freezing etc.).

Sorry for the late reply—I missed it because I receive too many emails 
from syzbot. I've now reconfigured my filtering rules.

Regarding the issue reported by syzbot CI, would it be better to add 
flush_delayed_work in __ext4_remount before sb->s_flags |= SB_RDONLY? 
That would make the race window smaller.

Also, should I resend the v14 series, or submit a separate patch 
specifically for this fix?

BR,
Yun