Re: [PATCH v2 09/17] KVM: x86/mmu: Use '0' as the one and only value for an invalid PAE root

From: Paolo Bonzini
Date: Fri Mar 05 2021 - 12:53:43 EST


On 05/03/21 02:10, Sean Christopherson wrote:
Use '0' to denote an invalid pae_root instead of '0' or INVALID_PAGE.
Unlike root_hpa, the pae_roots hold permission bits and thus are
guaranteed to be non-zero. Having to deal with both values leads to
bugs, e.g. failing to set back to INVALID_PAGE, warning on the wrong
value, etc...

I don't dispute this is a good idea, but it deserves one or more comments.

Paolo