On Fri, Mar 18, 2011 at 03:06:48PM +1100, Dave Chinner wrote: > It gets used by a second caller in the next patch that uses a > timeout of zero. The idea of adding a delay to a normal push is to > rate limit the number of times we do work so we always work on > batches rather a few items at a time in multiple executions of the > work. > > I'll see if it's simpler to just do this work directly in teh > callers, though. I don't think hiding this delay (uncommented) in the workqueue use is a good idea. xlog_grant_push_ail has all the logics about when to push the AIL, so any batching should be grouped with that logic, and documented there. It in fact already has some comments static that a min/max watermark scheme would be useful. _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
