Re: [PATCH 3/4] xfs: revert to using a kthread for AIL pushing

Christoph Hellwig
2011-10-10T13:26:17Z

On Mon, Oct 10, 2011 at 07:55:46AM +0200, Markus Trippelsdorf wrote:
> Wouldn't it be possible to verify that the problem also goes away with
> this simple one liner?

We've been through a few variants, and none fixed it while Stefan had
to try them on production machines.

To be honest I'm not convinced at all that a workqueue was such a good
idea for the ail in particular. It works extremly well for things were
we can easily define a work item, e.g. an object that gets queued up
and a method on it gets exectured. But for the AIL we really have
a changing target that needs more or less constant pushing, and the
target keeps changing while executing our work. Conceptually it fits
the idea of an thread much better, with the added benefit of not relying
on finding a combination of workqueue flags that gets the exact
behaviour (exectuion ASAP without any limits because of other items
or required memory allocation).

And unlike the various per-cpu threads we used to have it is only one
thread per filesystem anyway.