GFS2: Make "try" lock not try quite so hard
authorSteven Whitehouse <swhiteho@redhat.com>
Fri, 23 Jul 2010 13:05:51 +0000 (14:05 +0100)
committerSteven Whitehouse <swhiteho@redhat.com>
Thu, 29 Jul 2010 08:37:38 +0000 (09:37 +0100)
commitd5341a92416706808dc5cd847826f28c08063c8c
tree48f6680c25d2fab956c52cd5021110f47f2ddf54
parent4244b52e18be959ced77b984f268e46a0a7654e3
GFS2: Make "try" lock not try quite so hard

This looks like a big change, but in reality its only a single line of actual
code change, the rest is just moving a function to before its new caller.
The "try" flag for glocks is a rather subtle and delicate setting since it
requires that the state machine tries just hard enough to ensure that it has
a good chance of getting the requested lock, but no so hard that the
request can land up blocked behind another.

The patch adds in an additional check which will fail any queued try
locks if there is another request blocking the try lock request which
is not granted and compatible, nor in progress already. The check is made
only after all pending locks which may be granted have been granted.

I've checked this with the reproducer for the reported flock bug which
this is intended to fix, and it now passes.

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
fs/gfs2/glock.c