Paolo Bonzini <[email protected]> writes:

> On 04/05/2017 00:14, Bandan Das wrote:
>> +    if (vmx->nested.pml_full) {
>> +            exit_reason = EXIT_REASON_PML_FULL;
>> +            vmx->nested.pml_full = false;
>> +    } else if (fault->error_code & PFERR_RSVD_MASK)
>>              exit_reason = EXIT_REASON_EPT_MISCONFIG;
>>      else
>>              exit_reason = EXIT_REASON_EPT_VIOLATION;
>> +    /*
>> +     * The definition of bit 12 for EPT violations and PML
>> +     * full event is the same, so pass it through since
>> +     * the rest of the bits are undefined.
>> +     */
>
> Please zero all other bits instead.  It's as easy as adding an "u64
> exit_qualification" local variable.

Will do, thanks for the review.

Bandan

> Paolo
>
>>      nested_vmx_vmexit(vcpu, exit_reason, 0, vcpu->arch.exit_qualification);

Reply via email to