On Sun, Oct 25, 2015 at 04:34:27PM -0700, Hugh Dickins wrote: > I'll say > Acked-by: Hugh Dickins <hughd@xxxxxxxxxx> > as a gesture of goodwill, but in honesty I'm sitting on the fence, > and couldn't decide. I think I've gone back and forth on this in > my own mind in the past, worried that we might get stuck a long > time going back round to "again". In the past I've felt that to > give up with NULL is consistent with KSM's willingness to give way > to any obstruction; but if you're finding "goto again" a better > strategy, sure, go ahead. And at least there's a cond_resched() > just above the diff context shown. If a couple of large process exists and create lots of stale stable_nodes, we'll miss the opportunity to merge lots of unstable tree nodes for potentially many passes. So KSM gets an artificial latency in merging new unstable tree nodes. At the same time if we don't prune aggressively, we delay the freeing of the stale stable_nodes. Keeping stale stable_nodes around wastes memory and it can't provide any benefit. Ideally we should be doing a full rbtree walk to do a perfect pruning after each pass to be sure some stale stable nodes don't stay around forever, but walking the whole rtbree takes more memory and it'd use more CPU. So considering we're not perfect at pruning the tree after each pass, we can at least be more aggressive at pruning the tree during the lookup. > When a comment gets that long, in fact even if it were only one line, > I'd much prefer that block inside braces. I think I noticed Linus > feeling the same way a few days ago, when he fixed up someone's patch. Ok, I'll add braces, I don't mind either ways. -- 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>