Re: [PATCH 1/4] bdi: Track users that require stable page writes

From: Christoph Hellwig
Date: Wed Nov 21 2012 - 05:52:52 EST


On Wed, Nov 21, 2012 at 02:54:35AM -0500, Christoph Hellwig wrote:
> On Tue, Nov 20, 2012 at 06:00:34PM -0800, Darrick J. Wong wrote:
> > This creates a per-backing-device counter that tracks the number of users which
> > require pages to be held immutable during writeout. Eventually it will be used
> > to waive wait_for_page_writeback() if nobody requires stable pages.
>
> Why are we going down this stupid route again now? Just let the block
> device say it needs stable writes and let the VM deal with it.

Seems like the series actually does that, and this paragraph was just
left over from earlier versions. Sorry for complaining too quickly, but
you really should update the description.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/