Re: [PATCH 2/2] KVM: x86/mmu: Don't step down in the TDP iterator when zapping all SPTEs

From: Paolo Bonzini
Date: Thu Aug 12 2021 - 13:15:10 EST


On 12/08/21 19:07, Sean Christopherson wrote:
Yeah, I was/am on the fence too, I almost included a blurb in the cover letter
saying as much. I'll do that for v2 and let Paolo decide.

I think it makes sense to have it. You can even use the ternary operator

/*
* When zapping everything, all entries at the top level
* ultimately go away, and the levels below go down with them.
* So do not bother iterating all the way down to the leaves.
*/
int min_level = zap_all ? root->role.level : PG_LEVEL_4K;

Paolo