>
> What prevents vfs_quota_off() from racing with unmount?
>
> If you look at, say, sync_filesystems() you'll see that we take ->s_umount
> and then test ->s_root to check that we didn't race with an unmount
> attempt.
I thought that the sync_fs and stuff is guarded by the fact that quota
code is holding references to inodes on the filesystem. And invalidate_bdev()
should not care about the filesystem. Or do I miss something? Anyway
adding a comment about this is really a good idea...
I've found a subtle bug though - there can be vfs_quota_off() racing
against vfs_quota_on() resulting in the flags on the inode being set
wrong... I'll fix that one.
Thanks for checking
Honza
--
Jan Kara <[EMAIL PROTECTED]>
SuSE CR Labs