Hi, On Tue, Jun 10, 2014 at 04:43:17PM +0900, Joonsoo Kim wrote: > You mentioned that disabling per cpu arrays would degrade performance. > But, this patch is implemented to disable per cpu arrays. Is there any > reason to do like this? How about not disabling per cpu arrays and > others? Leaving it as is makes the patch less intrusive and has low > impact on performance. I guess that amount of reclaimed memory has no > big difference between both approaches. Frankly, I incline to shrinking dead SLAB caches periodically from cache_reap too, because it looks neater and less intrusive to me. Also it has zero performance impact, which is nice. However, Christoph proposed to disable per cpu arrays for dead caches, similarly to SLUB, and I decided to give it a try, just to see the end code we'd have with it. I'm still not quite sure which way we should choose though... Thanks. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>