diff options
author | Bob Peterson <rpeterso@redhat.com> | 2013-07-25 09:53:49 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2013-09-04 10:58:21 -0400 |
commit | 068213f7d3378d3e33d0f1b9415b2fdc3e9efa14 (patch) | |
tree | 306c039eacdae598cb7f401171212e1e2b659f96 /fs/gfs2 | |
parent | 9d35814355e2baba9de523941e2ce52235359670 (diff) |
GFS2: Remove unnecessary memory barrier
Function test_and_clear_bit implies a memory barrier, so subsequent
memory barriers are unnecessary.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2')
-rw-r--r-- | fs/gfs2/glock.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index ce7078d5aa97..722329cac98f 100644 --- a/fs/gfs2/glock.c +++ b/fs/gfs2/glock.c | |||
@@ -1411,7 +1411,6 @@ __acquires(&lru_lock) | |||
1411 | if (demote_ok(gl)) | 1411 | if (demote_ok(gl)) |
1412 | handle_callback(gl, LM_ST_UNLOCKED, 0, false); | 1412 | handle_callback(gl, LM_ST_UNLOCKED, 0, false); |
1413 | WARN_ON(!test_and_clear_bit(GLF_LOCK, &gl->gl_flags)); | 1413 | WARN_ON(!test_and_clear_bit(GLF_LOCK, &gl->gl_flags)); |
1414 | smp_mb__after_clear_bit(); | ||
1415 | if (queue_delayed_work(glock_workqueue, &gl->gl_work, 0) == 0) | 1414 | if (queue_delayed_work(glock_workqueue, &gl->gl_work, 0) == 0) |
1416 | gfs2_glock_put_nolock(gl); | 1415 | gfs2_glock_put_nolock(gl); |
1417 | spin_unlock(&gl->gl_spin); | 1416 | spin_unlock(&gl->gl_spin); |