On Wed, Apr 22, 2020 at 10:13 AM Chengguang Xu <cgxu519@xxxxxxxxxxxx> wrote: > > ---- 在 星期三, 2020-04-22 15:29:33 Miklos Szeredi <miklos@xxxxxxxxxx> 撰写 ---- > > On Wed, Apr 22, 2020 at 6:29 AM Chengguang Xu <cgxu519@xxxxxxxxxxxx> wrote: > > > > > > sync_filesystem() does not sync dirty data for readonly > > > filesystem during umount, so before changing to readonly > > > filesystem we should sync dirty data for data integrity. > > > > Isn't the same true for ->put_super()? > > > > Before getting into ->put_super(), dirty data have been synced through below functions, > so I think we don't have to worry about ->put_super(). > > kill_anon_super() > generic_shutdown_super() > sync_filesystem() Okay, makes sense. Thanks, Miklos