GFS2: Fix & clean up GFS2 rename
authorSteven Whitehouse <swhiteho@redhat.com>
Tue, 26 Aug 2008 08:38:26 +0000 (09:38 +0100)
committerSteven Whitehouse <swhiteho@redhat.com>
Wed, 27 Aug 2008 12:33:10 +0000 (13:33 +0100)
commit0188d6c5807b65e2e20dcb75a668efbe5418b27e
tree419ee0e74cded399d2e4b9adf4dacf9f9dd31a3e
parent72dbf4790fc6736f9cb54424245114acf0b0038c
GFS2: Fix & clean up GFS2 rename

This patch fixes a locking issue in the rename code by ensuring that we hold
the per sb rename lock over both directory and "other" renames which involve
different parent directories.

At the same time, this moved the (only called from one place) function
gfs2_ok_to_move into the file that its called from, so we can mark it
static. This should make a code a bit easier to follow.

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Peter Staubach <staubach@redhat.com>
fs/gfs2/inode.c
fs/gfs2/inode.h
fs/gfs2/ops_inode.c