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. Best regards, Pavel -- DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Attachment:
signature.asc
Description: PGP signature