From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Haozhong Zhang <haozhong.zhang@intel.com>
Cc: xen-devel@lists.xenproject.org
Subject: Re: [XTF PATCH 14/16] vvmx: test vmxon in VMX root w/ CPL = 3 and w/o current VMCS
Date: Mon, 19 Dec 2016 16:07:57 +0000 [thread overview]
Message-ID: <9d83c247-fb38-d3d0-47b9-a69462f5992d@citrix.com> (raw)
In-Reply-To: <20161219034626.d2c7scc7vkg2wrmm@hz-desktop>
On 19/12/16 03:46, Haozhong Zhang wrote:
>> However, I am not sure of its purpose. Why cant you reuse the previous
>> host state area?
>>
>
> Intel SDM says SW should not access or modify the VXMON rgion of a
> logical processor between vmxon and vmxoff. Though I have tested on
> real hardware whether reusing VMXON region would cause any trouble, I
> still want to exclude that possibility/noise from this test.
That is a good reason. Could you please add a comment to this effect?
~Andrew
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-12-19 16:08 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-16 13:43 [XTF PATCH 00/16] Add test cases for nested vmxon Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 01/16] vvmx: test whether VMX feature is present in CPUID Haozhong Zhang
2016-12-16 14:40 ` Andrew Cooper
2016-12-19 1:51 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 02/16] vvmx: test whether MSR_IA32_FEATURE_CONTROL is set correctly Haozhong Zhang
2016-12-16 16:17 ` Andrew Cooper
2016-12-19 2:20 ` Haozhong Zhang
2016-12-19 15:29 ` Andrew Cooper
2016-12-16 13:43 ` [XTF PATCH 03/16] vvmx: test whether MSR_IA32_VMX_BASIC " Haozhong Zhang
2016-12-16 17:19 ` Andrew Cooper
2016-12-19 2:44 ` Haozhong Zhang
2016-12-19 15:41 ` Andrew Cooper
2016-12-20 2:45 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 04/16] vvmx: add C wrappers of vmxon/vmread/vmptrld Haozhong Zhang
2016-12-16 19:03 ` [licensing] was: " Andrew Cooper
2016-12-19 3:20 ` Haozhong Zhang
2016-12-19 14:41 ` Lars Kurth
2016-12-19 15:51 ` Ian Jackson
2016-12-19 15:58 ` Andrew Cooper
2016-12-19 15:20 ` Roger Pau Monné
2016-12-19 15:24 ` Andrew Cooper
2016-12-16 19:47 ` Andrew Cooper
2016-12-19 3:00 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 05/16] vvmx: add a general error handler for VMX instructions Haozhong Zhang
2016-12-16 20:16 ` Andrew Cooper
2016-12-19 3:24 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 06/16] vvmx: test vmxon with CR4.VMXE cleared Haozhong Zhang
2016-12-16 20:25 ` Andrew Cooper
2016-12-19 3:26 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 07/16] vvmx: test vmxon in CPL=3 and out of VMX operation Haozhong Zhang
2016-12-16 20:33 ` Andrew Cooper
2016-12-19 3:35 ` Haozhong Zhang
2016-12-19 16:02 ` Andrew Cooper
2016-12-19 16:02 ` Andrew Cooper
2016-12-16 13:43 ` [XTF PATCH 08/16] vvmx: test vmxon with invalidly wide VMXON region address Haozhong Zhang
2016-12-16 20:40 ` Andrew Cooper
2016-12-19 3:36 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 09/16] vvmx: test vmxon with unaligned " Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 10/16] vvmx: test vmxon with mismatched VMCS revision ID Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 11/16] vvmx: test vmxon with bit 31 of VMCS revision ID set Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 12/16] vvmx: test the correct vmxon Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 13/16] vvmx: test vmxon in VMX root w/ CPL = 0 and w/o current VMCS Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 14/16] vvmx: test vmxon in VMX root w/ CPL = 3 " Haozhong Zhang
2016-12-16 20:59 ` Andrew Cooper
2016-12-19 3:46 ` Haozhong Zhang
2016-12-19 16:07 ` Andrew Cooper [this message]
2016-12-20 2:50 ` Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 15/16] vvmx: test vmxon in VMX root w/ CPL = 0 and w/ " Haozhong Zhang
2016-12-16 13:43 ` [XTF PATCH 16/16] vvmx: test vmxon in VMX root w/ CPL = 3 " Haozhong Zhang
2016-12-16 14:12 ` [XTF PATCH 00/16] Add test cases for nested vmxon Andrew Cooper
2016-12-19 1:44 ` Haozhong Zhang
2016-12-19 16:18 ` Andrew Cooper
2016-12-16 21:26 ` Andrew Cooper
2016-12-19 5:31 ` Haozhong Zhang
2016-12-19 16:34 ` Andrew Cooper
2016-12-20 2:32 ` Haozhong Zhang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9d83c247-fb38-d3d0-47b9-a69462f5992d@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=haozhong.zhang@intel.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).