(2012/12/10 10:30), Dave Chinner wrote:
> On Mon, Dec 10, 2012 at 08:42:15AM +0900, Satoru Takeuchi wrote:
>> From: Satoru Takeuchi <takeuchi_satoru@xxxxxxxxxxxxxx>
>> xfsbufd is removed since commit 43ff2122e6492bcc88b065c433453dce88223b30.
>> But there stille be two sysctl parameters about it
>> This patch marks these parameters as obsolete. In adition, prints
>> a warning message if someone tries to use these parameters.
> While we currently don't use them, I'm not sure they are obsolete.
> We need to integrate the AIL flushing with time-based log idling
> (i.e. flushing to an empty state), and part of that work is making
> the AIL log items have a age-based flush component.
> i.e. how often it will wake to try to flush/idle the log when there
> is no pressure, and how long items should wait before being flushed
> even when there is activity.
> Those two parameters are exactly what the xfsbufd parameters have
> historically controlled, so rather than introduce new sysctls for
> the same purpose, I was planning on just re-using them. Hence I
> suspect that marking them obsolete is probably the wrong thing to do
> right now - adding the time-based component to the AIL flushing
> would be a better solution.
OK, got it.