xfs: revert to using a kthread for AIL pushing
authorChristoph Hellwig <hch@infradead.org>
Tue, 11 Oct 2011 15:14:10 +0000 (11:14 -0400)
committerAlex Elder <aelder@sgi.com>
Tue, 11 Oct 2011 16:02:49 +0000 (11:02 -0500)
commit0030807c66f058230bcb20d2573bcaf28852e804
tree40c98d94085b6556cbbb33a7f5fca5a2cce153a2
parent17b38471c3c07a49f0bbc2ecc2e92050c164e226
xfs: revert to using a kthread for AIL pushing

Currently we have a few issues with the way the workqueue code is used to
implement AIL pushing:

 - it accidentally uses the same workqueue as the syncer action, and thus
   can be prevented from running if there are enough sync actions active
   in the system.
 - it doesn't use the HIGHPRI flag to queue at the head of the queue of
   work items

At this point I'm not confident enough in getting all the workqueue flags and
tweaks right to provide a perfectly reliable execution context for AIL
pushing, which is the most important piece in XFS to make forward progress
when the log fills.

Revert back to use a kthread per filesystem which fixes all the above issues
at the cost of having a task struct and stack around for each mounted
filesystem.  In addition this also gives us much better ways to diagnose
any issues involving hung AIL pushing and removes a small amount of code.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reported-by: Stefan Priebe <s.priebe@profihost.ag>
Tested-by: Stefan Priebe <s.priebe@profihost.ag>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
fs/xfs/xfs_linux.h
fs/xfs/xfs_super.c
fs/xfs/xfs_trans_ail.c
fs/xfs/xfs_trans_priv.h