On Thu, 5 Jul 2012 14:50:00 +0300 Gleb Natapov <gleb@xxxxxxxxxx> wrote: > > Note that "if (!nr_to_scan--)" check is removed since we do not try to > > free mmu pages from more than one VM. > > > IIRC this was proposed in the past that we should iterate over vm list > until freeing something eventually, but Avi was against it. I think the > probability of a VM with kvm->arch.n_used_mmu_pages == 0 is low, so > it looks OK to drop nr_to_scan to me. Since our batch size is 128, the minimum positive @nr_to_scan, it's almost impossible to see the effect of the check. Thanks, Takuya -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html