diff options
| author |  Bob Peterson <rpeterso@redhat.com>
| 2022-08-18 13:32:37 -0500 |
|---|---|---|
| committer |  Andreas Gruenbacher <agruenba@redhat.com>
| 2022-08-25 17:11:14 +0200 |
| commit | 053640a73838400dca23087d66a9c0db579adafb (patch) | |
| tree | 42169cec2192266f57afe4c1228a3d3c96f471d2 /fs/gfs2/glock.c | |
| parent | gfs2: Prevent double iput for journal on error (diff) | |
| download | linux-053640a73838400dca23087d66a9c0db579adafb.tar.gz linux-053640a73838400dca23087d66a9c0db579adafb.tar.bz2 linux-053640a73838400dca23087d66a9c0db579adafb.zip | |
gfs2: Dequeue waiters when withdrawn
When a withdraw occurs, ordinary (not system) glocks may not be granted
anymore. Later, when the file system is unmounted, gfs2_gl_hash_clear()
tries to clear out all the glocks, but these un-grantable pending
waiters prevent some glocks from being freed. So the unmount hangs, at
least for its ten-minute timeout period.
This patch takes measures to remove any pending waiters from
the glocks that will never be granted. This allows the unmount to
proceed in a reasonable period of time.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Diffstat (limited to 'fs/gfs2/glock.c')
| -rw-r--r-- | fs/gfs2/glock.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 57dcfd05b362..858616afcae6 100644 --- a/fs/gfs2/glock.c +++ b/fs/gfs2/glock.c @@ -2196,6 +2196,20 @@ static void dump_glock_func(struct gfs2_glock *gl) dump_glock(NULL, gl, true); } +static void withdraw_dq(struct gfs2_glock *gl) +{ + spin_lock(&gl->gl_lockref.lock); + if (!__lockref_is_dead(&gl->gl_lockref) && + glock_blocked_by_withdraw(gl)) + do_error(gl, LM_OUT_ERROR); /* remove pending waiters */ + spin_unlock(&gl->gl_lockref.lock); +} + +void gfs2_gl_dq_holders(struct gfs2_sbd *sdp) +{ + glock_hash_walk(withdraw_dq, sdp); +} + /** * gfs2_gl_hash_clear - Empty out the glock hash table * @sdp: the filesystem |