On 18/02/21 18:55, Jim Mattson wrote:
Got it now. It would sort of help, because while dumping the MSR load/store
area you could get hold of the real EFER, and use it to decide whether to
dump the PDPTRs.
EFER isn't guaranteed to be in the load list, either, e.g. if guest and host
have the same desired value.
The proper way to retrieve the effective EFER is to reuse the logic in
nested_vmx_calc_efer(), i.e. look at VM_ENTRY_IA32E_MODE if EFER isn't being
loaded via VMCS.
Shouldn't dump_vmcs() simply dump the contents of the VMCS, in its
entirety? What does it matter what the value of EFER is?
Currently it has some conditionals, but it wouldn't be a problem indeed
to remove them.
The MSR load list is missing state that dump_vmcs should print though.
Paolo