On Wed, Jul 31, 2024 at 1:19 PM Hao Peng <flyingpenghao@xxxxxxxxx> wrote: > > So if anything you could check list_empty(&kvm->arch.active_mmu_pages) > > before the loop of kvm_zap_obsolete_pages(), similar to what is done in > > kvm_mmu_zap_oldest_mmu_pages(). I doubt it can have any practical > > benefit, though. > > I did some tests, when ept=0, kvm_zap_obsolete_pages was called 42 > times, and only 17 times > active_mmu_page list was not empty. When tdp_mmu was enabled, > active_mmu_page list > was always empty. Did you also test with nested virtual machines running? In any case, we're talking of a difference of about 100 instructions at most, so it's irrelevant. Paolo