diff options
| author | Bob Peterson <rpeterso@redhat.com> | 2010-04-14 17:58:16 +0200 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2010-04-14 17:48:05 +0200 |
| commit | 1a0eae8848cde6e0734360f6456496c995ee1e23 (patch) | |
| tree | 536f944468ef5bc6f47ec09325422adbc5a3907a /fs/gfs2/export.c | |
| parent | GFS2: Clean up stuffed file copying (diff) | |
| download | linux-1a0eae8848cde6e0734360f6456496c995ee1e23.tar.xz linux-1a0eae8848cde6e0734360f6456496c995ee1e23.zip | |
GFS2: glock livelock
This patch fixes a couple gfs2 problems with the reclaiming of
unlinked dinodes. First, there were a couple of livelocks where
everything would come to a halt waiting for a glock that was
seemingly held by a process that no longer existed. In fact, the
process did exist, it just had the wrong pid number in the holder
information. Second, there was a lock ordering problem between
inode locking and glock locking. Third, glock/inode contention
could sometimes cause inodes to be improperly marked invalid by
iget_failed.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2/export.c')
| -rw-r--r-- | fs/gfs2/export.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/export.c b/fs/gfs2/export.c index d15876e9aa26..d81bc7e90e98 100644 --- a/fs/gfs2/export.c +++ b/fs/gfs2/export.c @@ -169,7 +169,7 @@ static struct dentry *gfs2_get_dentry(struct super_block *sb, if (error) goto fail; - inode = gfs2_inode_lookup(sb, DT_UNKNOWN, inum->no_addr, 0, 0); + inode = gfs2_inode_lookup(sb, DT_UNKNOWN, inum->no_addr, 0); if (IS_ERR(inode)) { error = PTR_ERR(inode); goto fail; |