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