From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jim Mattson Subject: Re: [PATCH] kvm: vmx: Properly handle machine check during VM-entry Date: Mon, 22 May 2017 08:14:22 -0700 Message-ID: References: <20170518230239.86518-1-jmattson@google.com> <61f6cb5f-9912-c95d-4a45-04691a970ddd@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Xiao Guangrong , kvm To: Wanpeng Li Return-path: Received: from mail-wm0-f52.google.com ([74.125.82.52]:35836 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934647AbdEVPOY (ORCPT ); Mon, 22 May 2017 11:14:24 -0400 Received: by mail-wm0-f52.google.com with SMTP id b84so301755958wmh.0 for ; Mon, 22 May 2017 08:14:24 -0700 (PDT) In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: I don't understand the confusion here. Is it because I used Intel's name for the VMCS field rather than the local variable in this function? Is it "unmodified" versus "invalid"? In any case, feel free to use whatever verbiage you are comfortable with. On Mon, May 22, 2017 at 2:06 AM, Wanpeng Li wrote: > 2017-05-22 16:26 GMT+08:00 Xiao Guangrong : >> >> >> On 05/19/2017 07:02 AM, Jim Mattson wrote: >>> >>> When bit 31 of the exit reason is set to indicate a VM-entry failure, >>> only the exit reason and exit qualification fields are set. All other >>> VM-exit information fields, including "VM-exit interruption >>> information," are unmodified. >> >> >> This log does not reflects what it is doing. The case that >> exit-reason.bit31 = 1 is skipped by the check at the very beginning of >> vmx_complete_atomic_exit(). >> >> Maybe what you want to say is just "exit_intr_info is not valid if vmx >> exit is caused by EXIT_REASON_MCE_DURING_VMENTRY"? >> > > exit_intr_info is not valid for all the vmentry fail. So mce should be > captured by exit reason instead of interpreting exit_intr_info. > > Regards, > Wanpeng Li