[XFS] Fix a synchronous buftarg flush deadlock when freezing.
authorDavid Chinner <dgc@sgi.com>
Sat, 10 Feb 2007 07:32:29 +0000 (18:32 +1100)
committerTim Shimmin <tes@sgi.com>
Sat, 10 Feb 2007 07:32:29 +0000 (18:32 +1100)
commit585e6d8856526a846b90b485abf37ec40e5da1cf
tree8ecae5c3e10a1753fc178877ab11deadcf9625f3
parentdac61f521b1e4d2c6c48023f2f2743c6096b48ca
[XFS] Fix a synchronous buftarg flush deadlock when freezing.

At the last stage of a freeze, we flush the buftarg synchronously over and
over again until it succeeds twice without skipping any buffers.

The delwri list flush skips pinned buffers, but tries to flush all others.
It removes the buffers from the delwri list, then tries to lock them one
at a time as it traverses the list to issue the I/O. It holds them locked
until we issue all of the I/O and then unlocks them once we've waited for
it to complete.

The problem is that during a freeze, the filesystem may still be doing
stuff - like flushing delalloc data buffers - in the background and hence
we can be trying to lock buffers that were on the delwri list at the same
time. Hence we can get ABBA deadlocks between threads doing allocation and
the buftarg flush (freeze) thread.

Fix it by skipping locked (and pinned) buffers as we traverse the delwri
buffer list.

SGI-PV: 957195
SGI-Modid: xfs-linux-melb:xfs-kern:27535a

Signed-off-by: David Chinner <dgc@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
fs/xfs/linux-2.6/xfs_buf.c