diff options
| author | Brian Foster <bfoster@redhat.com> | 2013-02-11 16:08:22 +0100 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-02-15 00:24:53 +0100 |
| commit | fa5566e4ffb918131a054413eb42075a77a41413 (patch) | |
| tree | 48feb3221aa163294a36e19355703fe6bba07a4e /fs/xfs | |
| parent | xfs: recheck buffer pinned status after push trylock failure (diff) | |
| download | linux-fa5566e4ffb918131a054413eb42075a77a41413.tar.xz linux-fa5566e4ffb918131a054413eb42075a77a41413.zip | |
xfs: remove log force from xfs_buf_trylock()
The trylock log force invoked via xfs_buf_item_push() can attempt
to acquire xa_lock, thus leading to a recursion bug when called
with xa_lock held.
This log force was originally added to xfs_buf_trylock() to address
xfsaild stalls due to pinned and stale buffers. Since the addition
of this behavior, the log item pushing code had been reworked to
detect and track pinned items to inform xfsaild to issue a log
force itself when necessary. As such, the log force on trylock
failure is redundant and safe to remove.
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/xfs')
| -rw-r--r-- | fs/xfs/xfs_buf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c index fbbb9eb92e32..4e8f0df82d02 100644 --- a/fs/xfs/xfs_buf.c +++ b/fs/xfs/xfs_buf.c @@ -951,8 +951,6 @@ xfs_buf_trylock( locked = down_trylock(&bp->b_sema) == 0; if (locked) XB_SET_OWNER(bp); - else if (atomic_read(&bp->b_pin_count) && (bp->b_flags & XBF_STALE)) - xfs_log_force(bp->b_target->bt_mount, 0); trace_xfs_buf_trylock(bp, _RET_IP_); return locked; |