[PATCH 5/5] repair: limit auto-striding concurrency apprpriately

Dave Chinner david at fromorbit.com
Thu Dec 12 15:00:40 CST 2013


On Thu, Dec 12, 2013 at 10:29:08AM -0800, Christoph Hellwig wrote:
> Looks reasonable, but it might be worth to add a blurb on how you
> arrived at the magic 8 threads per cpu.

OK - that was simply that processing with 8 threads is more than
enough to saturate a single CPU on fast devices so we don't need any
more parallelism in this case. And when you have large slow devices,
8 threads is more than enough to saturate the IO subsystem....

I'll add a comment mentioning that.

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list