diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2010-01-29 10:21:27 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2010-02-03 04:56:21 -0500 |
commit | 8f05228ee7c8f409ae3c6f9c3e13d7ccb9c18360 (patch) | |
tree | 34e8cf87485edf4ecb6878ade96704975e5d5bf5 /fs/gfs2/super.c | |
parent | e402746a945ceb9d0486a8e3d5917c9228fa4404 (diff) |
GFS2: Extend umount wait coverage to full glock lifetime
Although all glocks are, by the time of the umount glock wait,
scheduled for demotion, some of them haven't made it far
enough through the process for the original set of waiting
code to wait for them.
This extends the ref count to the whole glock lifetime in order
to ensure that the waiting does catch all glocks. It does make
it a bit more invasive, but it seems the only sensible solution
at the moment.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/super.c')
-rw-r--r-- | fs/gfs2/super.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c index 66242b32db5b..b9dd3da22c0a 100644 --- a/fs/gfs2/super.c +++ b/fs/gfs2/super.c | |||
@@ -861,8 +861,6 @@ restart: | |||
861 | gfs2_jindex_free(sdp); | 861 | gfs2_jindex_free(sdp); |
862 | /* Take apart glock structures and buffer lists */ | 862 | /* Take apart glock structures and buffer lists */ |
863 | gfs2_gl_hash_clear(sdp); | 863 | gfs2_gl_hash_clear(sdp); |
864 | /* Wait for dlm to reply to all our unlock requests */ | ||
865 | wait_event(sdp->sd_glock_wait, atomic_read(&sdp->sd_glock_disposal) == 0); | ||
866 | /* Unmount the locking protocol */ | 864 | /* Unmount the locking protocol */ |
867 | gfs2_lm_unmount(sdp); | 865 | gfs2_lm_unmount(sdp); |
868 | 866 | ||