xfs: re-enable xfsaild idle mode and fix associated races
authorBrian Foster <bfoster@redhat.com>
Thu, 28 Jun 2012 10:52:56 +0000 (06:52 -0400)
committerBen Myers <bpm@sgi.com>
Sun, 29 Jul 2012 21:27:57 +0000 (16:27 -0500)
commit8375f922aaa6e7a880022529202fb486315568c3
tree5fd3385ae7860ef9dabc84619f64fb85878fa864
parent4f59af758f9092bc7b266ca919ce6067170e5172
xfs: re-enable xfsaild idle mode and fix associated races

xfsaild idle mode logic currently leads to a couple hangs:

1.) If xfsaild is rescheduled in during an incremental scan
    (i.e., tout != 0) and the target has been updated since
    the previous run, we can hit the new target and go into
    idle mode with a still populated ail.
2.) A wake up is only issued when the target is pushed forward.
    The wake up can race with xfsaild if it is currently in the
    process of entering idle mode, causing future wake up
    events to be lost.

These hangs have been reproduced and verified as fixed by
running xfstests 273 in a loop on a slightly modified upstream
kernel. The kernel is modified to re-enable idle mode as
previously implemented (when count == 0) and with a revert of
commit 670ce93f, which includes performance improvements that
make this harder to reproduce.

The solution, the algorithm for which has been outlined by
Dave Chinner, is to modify xfsaild to enter idle mode only when
the ail is empty and the push target has not been moved forward
since the last push.

Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Ben Myers <bpm@sgi.com>
fs/xfs/xfs_trans_ail.c
fs/xfs/xfs_trans_priv.h