]> git.ipfire.org Git - people/ms/linux.git/commit - block/bdev.c
fs: add freeze_super/thaw_super fs hooks
authorBenjamin Marzinski <bmarzins@redhat.com>
Fri, 14 Nov 2014 02:42:03 +0000 (20:42 -0600)
committerSteven Whitehouse <swhiteho@redhat.com>
Mon, 17 Nov 2014 10:35:17 +0000 (10:35 +0000)
commit48b6bca6b7b8309697fc8a101793befe92d249d9
tree2cec8b867e5058c6672e425e2ac9a0d25ae60cf1
parent98f1a696a1b2f6cf0c1ce6850b1516beda9c1269
fs: add freeze_super/thaw_super fs hooks

Currently, freezing a filesystem involves calling freeze_super, which locks
sb->s_umount and then calls the fs-specific freeze_fs hook. This makes it
hard for gfs2 (and potentially other cluster filesystems) to use the vfs
freezing code to do freezes on all the cluster nodes.

In order to communicate that a freeze has been requested, and to make sure
that only one node is trying to freeze at a time, gfs2 uses a glock
(sd_freeze_gl). The problem is that there is no hook for gfs2 to acquire
this lock before calling freeze_super. This means that two nodes can
attempt to freeze the filesystem by both calling freeze_super, acquiring
the sb->s_umount lock, and then attempting to grab the cluster glock
sd_freeze_gl. Only one will succeed, and the other will be stuck in
freeze_super, making it impossible to finish freezing the node.

To solve this problem, this patch adds the freeze_super and thaw_super
hooks.  If a filesystem implements these hooks, they are called instead of
the vfs freeze_super and thaw_super functions. This means that every
filesystem that implements these hooks must call the vfs freeze_super and
thaw_super functions itself within the hook function to make use of the vfs
freezing code.

Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
fs/block_dev.c
fs/ioctl.c
include/linux/fs.h