[GFS2] Fix bug where we called drop_bh incorrectly
authorSteven Whitehouse <swhiteho@redhat.com>
Tue, 12 Feb 2008 15:28:21 +0000 (15:28 +0000)
committerSteven Whitehouse <swhiteho@redhat.com>
Mon, 31 Mar 2008 09:41:01 +0000 (10:41 +0100)
As a result of an earlier patch, drop_bh was being called in cases
when it shouldn't have been. Since we never have a gh in the drop
case and we always have a gh in the promote case, we can use that
extra information to tell which case has been seen.

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

index d00dc37e3d51bda4e1410fffe50aad2af92f5ca5..63981e2fb835f3c21a404e1eb71ca1181c1b20f7 100644 (file)
@@ -765,7 +765,6 @@ static void drop_bh(struct gfs2_glock *gl, unsigned int ret)
 {
        struct gfs2_sbd *sdp = gl->gl_sbd;
        const struct gfs2_glock_operations *glops = gl->gl_ops;
-       struct gfs2_holder *gh = gl->gl_req_gh;
 
        gfs2_assert_warn(sdp, test_bit(GLF_LOCK, &gl->gl_flags));
        gfs2_assert_warn(sdp, list_empty(&gl->gl_holders));
@@ -776,23 +775,11 @@ static void drop_bh(struct gfs2_glock *gl, unsigned int ret)
        if (glops->go_inval)
                glops->go_inval(gl, DIO_METADATA);
 
-       if (gh) {
-               spin_lock(&gl->gl_spin);
-               list_del_init(&gh->gh_list);
-               gh->gh_error = 0;
-               spin_unlock(&gl->gl_spin);
-       }
-
        spin_lock(&gl->gl_spin);
        gfs2_demote_wake(gl);
-       gl->gl_req_gh = NULL;
        clear_bit(GLF_LOCK, &gl->gl_flags);
        spin_unlock(&gl->gl_spin);
-
        gfs2_glock_put(gl);
-
-       if (gh)
-               gfs2_holder_wake(gh);
 }
 
 /**
@@ -810,7 +797,7 @@ static void xmote_bh(struct gfs2_glock *gl, unsigned int ret)
        int prev_state = gl->gl_state;
        int op_done = 1;
 
-       if ((ret & LM_OUT_ST_MASK) == LM_ST_UNLOCKED) {
+       if (!gh && (ret & LM_OUT_ST_MASK) == LM_ST_UNLOCKED) {
                drop_bh(gl, ret);
                return;
        }