summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven Whitehouse <swhiteho@redhat.com>2007-02-07 10:25:59 -0500
committerSteven Whitehouse <swhiteho@redhat.com>2007-02-07 10:25:59 -0500
commit2d72e7101cc7fff5c1eb21bfcbba51c8002418d2 (patch)
tree4f67388942d5716fb18732c04b8a8d74bdff0ff8
parent549ae0ac3d574a682e82b02e79259a65445a675b (diff)
downloadop-kernel-dev-2d72e7101cc7fff5c1eb21bfcbba51c8002418d2.zip
op-kernel-dev-2d72e7101cc7fff5c1eb21bfcbba51c8002418d2.tar.gz
[GFS2] Unlock page on prepare_write try lock failure
When the try lock of the glock failed in prepare_write we were incorrectly exiting this function with the page still locked. This was resulting in further I/O to this page hanging. Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
-rw-r--r--fs/gfs2/ops_address.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/gfs2/ops_address.c b/fs/gfs2/ops_address.c
index 9ddf975..5e9653c 100644
--- a/fs/gfs2/ops_address.c
+++ b/fs/gfs2/ops_address.c
@@ -360,8 +360,10 @@ static int gfs2_prepare_write(struct file *file, struct page *page,
gfs2_holder_init(ip->i_gl, LM_ST_EXCLUSIVE, GL_ATIME|LM_FLAG_TRY_1CB, &ip->i_gh);
error = gfs2_glock_nq_atime(&ip->i_gh);
if (unlikely(error)) {
- if (error == GLR_TRYFAILED)
+ if (error == GLR_TRYFAILED) {
+ unlock_page(page);
error = AOP_TRUNCATED_PAGE;
+ }
goto out_uninit;
}
OpenPOWER on IntegriCloud