On Fri, Nov 25, 2011 at 03:25:16PM -0500, Mikulas Patocka wrote: > The following patch fixes the deadlock. When the quota subsystem takes s_umount, > it checks if the filesystem is frozen. If it is, we drop s_umount, wait for > the filesystem to resume and retry. > > Signed-off-by: Mikulas Patocka <mpatocka@xxxxxxxxxx> > CC: stable@xxxxxxxxxx So basically you want a variant of get_super() that would get you a superblock for this bdev, locked and unfrozen? Fair enough, but * that should be a proper helper in super.c, rather than open-coded in fs/quota/quota.c, of all places * what about other existing callers get_super() and its friends? and while we are at it, why in damnation name is it exported? The only caller outside of core VFS is under #if 0... -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel