Re: [PATCH AUTOSEL 5.10 9/9] ext4: enable the lazy init thread when remounting read/write

[Sasha Levin]

On Fri, Jun 16, 2023 at 09:41:04PM +0200, Pavel Machek wrote:
> Hi!
>
> > [ Upstream commit eb1f822c76beeaa76ab8b6737ab9dc9f9798408c ]
> >
> > In commit a44be64bbecb ("ext4: don't clear SB_RDONLY when remounting
> > r/w until quota is re-enabled") we defer clearing tyhe SB_RDONLY flag
> > in struct super.  However, we didn't defer when we checked sb_rdonly()
> > to determine the lazy itable init thread should be enabled, with the
> > next result that the lazy inode table initialization would not be
> > properly started.  This can cause generic/231 to fail in ext4's
> > nojournal mode.
> >
> > Fix this by moving when we decide to start or stop the lazy itable
> > init thread to after we clear the SB_RDONLY flag when we are
> > remounting the file system read/write.
> >
> > Fixes a44be64bbecb ("ext4: don't clear SB_RDONLY when remounting r/w until...")
> >
> > Signed-off-by: Theodore Ts'o <tytso@xxxxxxx>
> > Link: https://lore.kernel.org/r/20230527035729.1001605-1-tytso@xxxxxxx
> > Signed-off-by: Theodore Ts'o <tytso@xxxxxxx>
> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
>
> Normally "Fixes" would be "Fixes:" and in the signed-off block. Plus,
> two consecutive sign-offs from tytso are probably wrong, too.

I'm really not sure what you want us to do here, or in other places
where you've commented about issues with the upstream patch...

--
Thanks,
Sasha