* [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes
@ 2020-07-14 0:23 Sean Christopherson
2020-07-14 0:23 ` [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() Sean Christopherson
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Sean Christopherson @ 2020-07-14 0:23 UTC (permalink / raw)
To: Paolo Bonzini
Cc: kvm, Krish Sadhukhan, Karl Heubaum, Sean Christopherson,
Oliver Upton, Jim Mattson
PCIDE fixes for two completely unrelated tests that managed to combine
powers and create a super confusing error where the MTF test loads CR3
with 0 and sends things into the weeds.
Sean Christopherson (2):
nVMX: Restore active host RIP/CR4 after test_host_addr_size()
nVMX: Use the standard non-canonical value in test_mtf3
x86/vmx_tests.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
--
2.26.0
^ permalink raw reply [flat|nested] 12+ messages in thread* [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-14 0:23 [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Sean Christopherson @ 2020-07-14 0:23 ` Sean Christopherson 2020-07-14 4:43 ` Oliver Upton 2020-07-15 18:34 ` Krish Sadhukhan 2020-07-14 0:23 ` [kvm-unit-tests PATCH 2/2] nVMX: Use the standard non-canonical value in test_mtf3 Sean Christopherson 2020-07-15 21:31 ` [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Krish Sadhukhan 2 siblings, 2 replies; 12+ messages in thread From: Sean Christopherson @ 2020-07-14 0:23 UTC (permalink / raw) To: Paolo Bonzini Cc: kvm, Krish Sadhukhan, Karl Heubaum, Sean Christopherson, Oliver Upton, Jim Mattson Perform one last VMX transition to actually load the host's RIP and CR4 at the end of test_host_addr_size(). Simply writing the VMCS doesn't restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, which causes spectacularly confusing explosions when other misguided tests assume setting bit 63 in CR3 will cause a non-canonical #GP. Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> Cc: Karl Heubaum <karl.heubaum@oracle.com> Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> --- x86/vmx_tests.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c index 29f3d0e..cb42a2d 100644 --- a/x86/vmx_tests.c +++ b/x86/vmx_tests.c @@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); vmcs_write(HOST_RIP, rip_saved); vmcs_write(HOST_CR4, cr4_saved); + + /* Restore host's active RIP and CR4 values. */ + report_prefix_pushf("restore host state"); + test_vmx_vmlaunch(0); + report_prefix_pop(); } } -- 2.26.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-14 0:23 ` [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() Sean Christopherson @ 2020-07-14 4:43 ` Oliver Upton 2020-07-15 18:34 ` Krish Sadhukhan 1 sibling, 0 replies; 12+ messages in thread From: Oliver Upton @ 2020-07-14 4:43 UTC (permalink / raw) To: Sean Christopherson Cc: Paolo Bonzini, kvm list, Krish Sadhukhan, Karl Heubaum, Jim Mattson On Mon, Jul 13, 2020 at 5:23 PM Sean Christopherson <sean.j.christopherson@intel.com> wrote: > > Perform one last VMX transition to actually load the host's RIP and CR4 > at the end of test_host_addr_size(). Simply writing the VMCS doesn't > restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, > which causes spectacularly confusing explosions when other misguided > tests assume setting bit 63 in CR3 will cause a non-canonical #GP. > > Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") > Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> > Cc: Karl Heubaum <karl.heubaum@oracle.com> > Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> Reviewed-by: Oliver Upton <oupton@google.com> > --- > x86/vmx_tests.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > index 29f3d0e..cb42a2d 100644 > --- a/x86/vmx_tests.c > +++ b/x86/vmx_tests.c > @@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) > vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); > vmcs_write(HOST_RIP, rip_saved); > vmcs_write(HOST_CR4, cr4_saved); > + > + /* Restore host's active RIP and CR4 values. */ > + report_prefix_pushf("restore host state"); > + test_vmx_vmlaunch(0); > + report_prefix_pop(); > } > } > > -- > 2.26.0 > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-14 0:23 ` [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() Sean Christopherson 2020-07-14 4:43 ` Oliver Upton @ 2020-07-15 18:34 ` Krish Sadhukhan 2020-07-15 18:48 ` Sean Christopherson 1 sibling, 1 reply; 12+ messages in thread From: Krish Sadhukhan @ 2020-07-15 18:34 UTC (permalink / raw) To: Sean Christopherson, Paolo Bonzini Cc: kvm, Karl Heubaum, Oliver Upton, Jim Mattson On 7/13/20 5:23 PM, Sean Christopherson wrote: > Perform one last VMX transition to actually load the host's RIP and CR4 > at the end of test_host_addr_size(). Simply writing the VMCS doesn't > restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, > which causes spectacularly confusing explosions when other misguided > tests assume setting bit 63 in CR3 will cause a non-canonical #GP. > > Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") > Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> > Cc: Karl Heubaum <karl.heubaum@oracle.com> > Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> > --- > x86/vmx_tests.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > index 29f3d0e..cb42a2d 100644 > --- a/x86/vmx_tests.c > +++ b/x86/vmx_tests.c > @@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) > vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); > vmcs_write(HOST_RIP, rip_saved); > vmcs_write(HOST_CR4, cr4_saved); > + > + /* Restore host's active RIP and CR4 values. */ > + report_prefix_pushf("restore host state"); > + test_vmx_vmlaunch(0); > + report_prefix_pop(); > } > } > Just for my understanding. When you say, "other misguided tests", which tests are you referring to ? In the current sequence of tests in vmx_host_state_area_test(), test_load_host_perf_global_ctrl() is the one that follows and it runs fine. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-15 18:34 ` Krish Sadhukhan @ 2020-07-15 18:48 ` Sean Christopherson 2020-07-15 21:34 ` Krish Sadhukhan 0 siblings, 1 reply; 12+ messages in thread From: Sean Christopherson @ 2020-07-15 18:48 UTC (permalink / raw) To: Krish Sadhukhan Cc: Paolo Bonzini, kvm, Karl Heubaum, Oliver Upton, Jim Mattson On Wed, Jul 15, 2020 at 11:34:46AM -0700, Krish Sadhukhan wrote: > > On 7/13/20 5:23 PM, Sean Christopherson wrote: > >Perform one last VMX transition to actually load the host's RIP and CR4 > >at the end of test_host_addr_size(). Simply writing the VMCS doesn't > >restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, > >which causes spectacularly confusing explosions when other misguided > >tests assume setting bit 63 in CR3 will cause a non-canonical #GP. > > > >Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") > >Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> > >Cc: Karl Heubaum <karl.heubaum@oracle.com> > >Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> > >--- > > x86/vmx_tests.c | 5 +++++ > > 1 file changed, 5 insertions(+) > > > >diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > >index 29f3d0e..cb42a2d 100644 > >--- a/x86/vmx_tests.c > >+++ b/x86/vmx_tests.c > >@@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) > > vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); > > vmcs_write(HOST_RIP, rip_saved); > > vmcs_write(HOST_CR4, cr4_saved); > >+ > >+ /* Restore host's active RIP and CR4 values. */ > >+ report_prefix_pushf("restore host state"); > >+ test_vmx_vmlaunch(0); > >+ report_prefix_pop(); > > } > > } > Just for my understanding. When you say, "other misguided tests", which > tests are you referring to ? In the current sequence of tests in > vmx_host_state_area_test(), test_load_host_perf_global_ctrl() is the one > that follows and it runs fine. See test_mtf_guest() in patch 2/2. https://patchwork.kernel.org/patch/11661189/ ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-15 18:48 ` Sean Christopherson @ 2020-07-15 21:34 ` Krish Sadhukhan 2020-07-15 22:22 ` Sean Christopherson 0 siblings, 1 reply; 12+ messages in thread From: Krish Sadhukhan @ 2020-07-15 21:34 UTC (permalink / raw) To: Sean Christopherson Cc: Paolo Bonzini, kvm, Karl Heubaum, Oliver Upton, Jim Mattson On 7/15/20 11:48 AM, Sean Christopherson wrote: > On Wed, Jul 15, 2020 at 11:34:46AM -0700, Krish Sadhukhan wrote: >> On 7/13/20 5:23 PM, Sean Christopherson wrote: >>> Perform one last VMX transition to actually load the host's RIP and CR4 >>> at the end of test_host_addr_size(). Simply writing the VMCS doesn't >>> restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, >>> which causes spectacularly confusing explosions when other misguided >>> tests assume setting bit 63 in CR3 will cause a non-canonical #GP. >>> >>> Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") >>> Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> >>> Cc: Karl Heubaum <karl.heubaum@oracle.com> >>> Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> >>> --- >>> x86/vmx_tests.c | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c >>> index 29f3d0e..cb42a2d 100644 >>> --- a/x86/vmx_tests.c >>> +++ b/x86/vmx_tests.c >>> @@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) >>> vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); >>> vmcs_write(HOST_RIP, rip_saved); >>> vmcs_write(HOST_CR4, cr4_saved); >>> + >>> + /* Restore host's active RIP and CR4 values. */ >>> + report_prefix_pushf("restore host state"); >>> + test_vmx_vmlaunch(0); >>> + report_prefix_pop(); >>> } >>> } >> Just for my understanding. When you say, "other misguided tests", which >> tests are you referring to ? In the current sequence of tests in >> vmx_host_state_area_test(), test_load_host_perf_global_ctrl() is the one >> that follows and it runs fine. > See test_mtf_guest() in patch 2/2. https://patchwork.kernel.org/patch/11661189/ I ran the two tests as follows but couldn't reproduce it: ./x86/run x86/vmx.flat -smp 1 -cpu host,+vmx -append "vmx_host_state_area_test vmx_mtf_test" How did you run the them ? ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-15 21:34 ` Krish Sadhukhan @ 2020-07-15 22:22 ` Sean Christopherson 2020-07-16 0:41 ` Krish Sadhukhan 0 siblings, 1 reply; 12+ messages in thread From: Sean Christopherson @ 2020-07-15 22:22 UTC (permalink / raw) To: Krish Sadhukhan Cc: Paolo Bonzini, kvm, Karl Heubaum, Oliver Upton, Jim Mattson On Wed, Jul 15, 2020 at 02:34:23PM -0700, Krish Sadhukhan wrote: > > On 7/15/20 11:48 AM, Sean Christopherson wrote: > >On Wed, Jul 15, 2020 at 11:34:46AM -0700, Krish Sadhukhan wrote: > >>On 7/13/20 5:23 PM, Sean Christopherson wrote: > >>>Perform one last VMX transition to actually load the host's RIP and CR4 > >>>at the end of test_host_addr_size(). Simply writing the VMCS doesn't > >>>restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, > >>>which causes spectacularly confusing explosions when other misguided > >>>tests assume setting bit 63 in CR3 will cause a non-canonical #GP. > >>> > >>>Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") > >>>Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> > >>>Cc: Karl Heubaum <karl.heubaum@oracle.com> > >>>Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> > >>>--- > >>> x86/vmx_tests.c | 5 +++++ > >>> 1 file changed, 5 insertions(+) > >>> > >>>diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > >>>index 29f3d0e..cb42a2d 100644 > >>>--- a/x86/vmx_tests.c > >>>+++ b/x86/vmx_tests.c > >>>@@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) > >>> vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); > >>> vmcs_write(HOST_RIP, rip_saved); > >>> vmcs_write(HOST_CR4, cr4_saved); > >>>+ > >>>+ /* Restore host's active RIP and CR4 values. */ > >>>+ report_prefix_pushf("restore host state"); > >>>+ test_vmx_vmlaunch(0); > >>>+ report_prefix_pop(); > >>> } > >>> } > >>Just for my understanding. When you say, "other misguided tests", which > >>tests are you referring to ? In the current sequence of tests in > >>vmx_host_state_area_test(), test_load_host_perf_global_ctrl() is the one > >>that follows and it runs fine. > >See test_mtf_guest() in patch 2/2. https://patchwork.kernel.org/patch/11661189/ > > I ran the two tests as follows but couldn't reproduce it: > > ./x86/run x86/vmx.flat -smp 1 -cpu host,+vmx -append > "vmx_host_state_area_test vmx_mtf_test" > > > How did you run the them ? I ran the VMX testcase from x86/unittest.cfg (below) on HSW. I eventually narrowed it down to just test_host_addr_size() and the MTF test. Note, the failure signature will change depending on whether vmx_cr_load_test() is run between those two. If it's not run, the failure is a straightforward triple fault. If it is run, for me the failure morphed into a an emulation error because the unit test was able to generate a valid translation out of CR3=0 and hit a non-existent memslot, which was all kinds of confusing. ./x86/run x86/vmx.flat -smp 1 -cpu host,+vmx -append "-exit_monitor_from_l2_test -ept_access* -vmx_smp* -vmx_vmcs_shadow_test -atomic_switch_overflow_msrs_test -vmx_init_signal_test -vmx_apic_passthrough_tpr_threshold_test" ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() 2020-07-15 22:22 ` Sean Christopherson @ 2020-07-16 0:41 ` Krish Sadhukhan 0 siblings, 0 replies; 12+ messages in thread From: Krish Sadhukhan @ 2020-07-16 0:41 UTC (permalink / raw) To: Sean Christopherson Cc: Paolo Bonzini, kvm, Karl Heubaum, Oliver Upton, Jim Mattson On 7/15/20 3:22 PM, Sean Christopherson wrote: > On Wed, Jul 15, 2020 at 02:34:23PM -0700, Krish Sadhukhan wrote: >> On 7/15/20 11:48 AM, Sean Christopherson wrote: >>> On Wed, Jul 15, 2020 at 11:34:46AM -0700, Krish Sadhukhan wrote: >>>> On 7/13/20 5:23 PM, Sean Christopherson wrote: >>>>> Perform one last VMX transition to actually load the host's RIP and CR4 >>>>> at the end of test_host_addr_size(). Simply writing the VMCS doesn't >>>>> restore the values in hardware, e.g. as is, CR4.PCIDE can be left set, >>>>> which causes spectacularly confusing explosions when other misguided >>>>> tests assume setting bit 63 in CR3 will cause a non-canonical #GP. >>>>> >>>>> Fixes: 0786c0316ac05 ("kvm-unit-test: nVMX: Check Host Address Space Size on vmentry of nested guests") >>>>> Cc: Krish Sadhukhan <krish.sadhukhan@oracle.com> >>>>> Cc: Karl Heubaum <karl.heubaum@oracle.com> >>>>> Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> >>>>> --- >>>>> x86/vmx_tests.c | 5 +++++ >>>>> 1 file changed, 5 insertions(+) >>>>> >>>>> diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c >>>>> index 29f3d0e..cb42a2d 100644 >>>>> --- a/x86/vmx_tests.c >>>>> +++ b/x86/vmx_tests.c >>>>> @@ -7673,6 +7673,11 @@ static void test_host_addr_size(void) >>>>> vmcs_write(ENT_CONTROLS, entry_ctrl_saved | ENT_GUEST_64); >>>>> vmcs_write(HOST_RIP, rip_saved); >>>>> vmcs_write(HOST_CR4, cr4_saved); >>>>> + >>>>> + /* Restore host's active RIP and CR4 values. */ >>>>> + report_prefix_pushf("restore host state"); >>>>> + test_vmx_vmlaunch(0); >>>>> + report_prefix_pop(); >>>>> } >>>>> } >>>> Just for my understanding. When you say, "other misguided tests", which >>>> tests are you referring to ? In the current sequence of tests in >>>> vmx_host_state_area_test(), test_load_host_perf_global_ctrl() is the one >>>> that follows and it runs fine. >>> See test_mtf_guest() in patch 2/2. https://patchwork.kernel.org/patch/11661189/ >> I ran the two tests as follows but couldn't reproduce it: >> >> ./x86/run x86/vmx.flat -smp 1 -cpu host,+vmx -append >> "vmx_host_state_area_test vmx_mtf_test" >> >> >> How did you run the them ? > I ran the VMX testcase from x86/unittest.cfg (below) on HSW. I eventually > narrowed it down to just test_host_addr_size() and the MTF test. Note, the > failure signature will change depending on whether vmx_cr_load_test() is > run between those two. If it's not run, the failure is a straightforward > triple fault. If it is run, for me the failure morphed into a an emulation > error because the unit test was able to generate a valid translation out of > CR3=0 and hit a non-existent memslot, which was all kinds of confusing. > > ./x86/run x86/vmx.flat -smp 1 -cpu host,+vmx -append "-exit_monitor_from_l2_test -ept_access* -vmx_smp* -vmx_vmcs_shadow_test -atomic_switch_overflow_msrs_test -vmx_init_signal_test -vmx_apic_passthrough_tpr_threshold_test" Thanks. I see it now, after I comment out test_load_host_perf_global_ctrl(). If any test calls enter_guest() right after test_host_addr_size(), this problem will manifest. I didn't think about this sequence of tests when adding test_host_addr_size() or any host-state-area tests for that matter. ^ permalink raw reply [flat|nested] 12+ messages in thread
* [kvm-unit-tests PATCH 2/2] nVMX: Use the standard non-canonical value in test_mtf3 2020-07-14 0:23 [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Sean Christopherson 2020-07-14 0:23 ` [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() Sean Christopherson @ 2020-07-14 0:23 ` Sean Christopherson 2020-07-14 4:41 ` Oliver Upton 2020-07-15 21:31 ` [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Krish Sadhukhan 2 siblings, 1 reply; 12+ messages in thread From: Sean Christopherson @ 2020-07-14 0:23 UTC (permalink / raw) To: Paolo Bonzini Cc: kvm, Krish Sadhukhan, Karl Heubaum, Sean Christopherson, Oliver Upton, Jim Mattson Use the standard non-canonical value of repeating 'a' instead of a custom (1 << 63) value in test_mtf3. When PCID is enabled, bit 63 is a flag that controls TLB swithching on MOV CR3 and is not included in the canonical check of CR3, i.e. if CR4.PCIDE=1 then the test will load 0 into CR3 and all manner of confusion things happen. Fixes: 46cc038c6afb8 ("x86: VMX: Add tests for monitor trap flag") Cc: Oliver Upton <oupton@google.com> Cc: Jim Mattson <jmattson@google.com> Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> --- x86/vmx_tests.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c index cb42a2d..32e3d4f 100644 --- a/x86/vmx_tests.c +++ b/x86/vmx_tests.c @@ -5107,7 +5107,7 @@ static void test_mtf_guest(void) * MOV RAX is done before the VMCALL such that MTF is only enabled * for the instruction under test. */ - "mov $0x8000000000000000, %rax;\n\t" + "mov $0xaaaaaaaaaaaaaaaa, %rax;\n\t" "vmcall;\n\t" "mov %rax, %cr3;\n\t" "test_mtf3:\n\t" -- 2.26.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 2/2] nVMX: Use the standard non-canonical value in test_mtf3 2020-07-14 0:23 ` [kvm-unit-tests PATCH 2/2] nVMX: Use the standard non-canonical value in test_mtf3 Sean Christopherson @ 2020-07-14 4:41 ` Oliver Upton 0 siblings, 0 replies; 12+ messages in thread From: Oliver Upton @ 2020-07-14 4:41 UTC (permalink / raw) To: Sean Christopherson Cc: Paolo Bonzini, kvm list, Krish Sadhukhan, Karl Heubaum, Jim Mattson On Mon, Jul 13, 2020 at 5:24 PM Sean Christopherson <sean.j.christopherson@intel.com> wrote: > > Use the standard non-canonical value of repeating 'a' instead of a > custom (1 << 63) value in test_mtf3. When PCID is enabled, bit 63 is > a flag that controls TLB swithching on MOV CR3 and is not included in > the canonical check of CR3, i.e. if CR4.PCIDE=1 then the test will load > 0 into CR3 and all manner of confusion things happen. > > Fixes: 46cc038c6afb8 ("x86: VMX: Add tests for monitor trap flag") > Cc: Oliver Upton <oupton@google.com> > Cc: Jim Mattson <jmattson@google.com> > Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> Reviewed-by: Oliver Upton <oupton@google.com> > --- > x86/vmx_tests.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > index cb42a2d..32e3d4f 100644 > --- a/x86/vmx_tests.c > +++ b/x86/vmx_tests.c > @@ -5107,7 +5107,7 @@ static void test_mtf_guest(void) > * MOV RAX is done before the VMCALL such that MTF is only enabled > * for the instruction under test. > */ > - "mov $0x8000000000000000, %rax;\n\t" > + "mov $0xaaaaaaaaaaaaaaaa, %rax;\n\t" > "vmcall;\n\t" > "mov %rax, %cr3;\n\t" > "test_mtf3:\n\t" > -- > 2.26.0 > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes 2020-07-14 0:23 [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Sean Christopherson 2020-07-14 0:23 ` [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() Sean Christopherson 2020-07-14 0:23 ` [kvm-unit-tests PATCH 2/2] nVMX: Use the standard non-canonical value in test_mtf3 Sean Christopherson @ 2020-07-15 21:31 ` Krish Sadhukhan 2020-07-28 21:26 ` Paolo Bonzini 2 siblings, 1 reply; 12+ messages in thread From: Krish Sadhukhan @ 2020-07-15 21:31 UTC (permalink / raw) To: Sean Christopherson, Paolo Bonzini Cc: kvm, Karl Heubaum, Oliver Upton, Jim Mattson On 7/13/20 5:23 PM, Sean Christopherson wrote: > PCIDE fixes for two completely unrelated tests that managed to combine > powers and create a super confusing error where the MTF test loads CR3 > with 0 and sends things into the weeds. > > Sean Christopherson (2): > nVMX: Restore active host RIP/CR4 after test_host_addr_size() > nVMX: Use the standard non-canonical value in test_mtf3 > > x86/vmx_tests.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > Reviewed-by: Krish Sadhukhan <krish.sadhukhan@oracle.com> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes 2020-07-15 21:31 ` [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Krish Sadhukhan @ 2020-07-28 21:26 ` Paolo Bonzini 0 siblings, 0 replies; 12+ messages in thread From: Paolo Bonzini @ 2020-07-28 21:26 UTC (permalink / raw) To: Krish Sadhukhan, Sean Christopherson Cc: kvm, Karl Heubaum, Oliver Upton, Jim Mattson On 15/07/20 23:31, Krish Sadhukhan wrote: > > On 7/13/20 5:23 PM, Sean Christopherson wrote: >> PCIDE fixes for two completely unrelated tests that managed to combine >> powers and create a super confusing error where the MTF test loads CR3 >> with 0 and sends things into the weeds. >> >> Sean Christopherson (2): >> nVMX: Restore active host RIP/CR4 after test_host_addr_size() >> nVMX: Use the standard non-canonical value in test_mtf3 >> >> x86/vmx_tests.c | 7 ++++++- >> 1 file changed, 6 insertions(+), 1 deletion(-) >> > Reviewed-by: Krish Sadhukhan <krish.sadhukhan@oracle.com> > Queued, thanks. Paolo ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2020-07-28 21:33 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-07-14 0:23 [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Sean Christopherson 2020-07-14 0:23 ` [kvm-unit-tests PATCH 1/2] nVMX: Restore active host RIP/CR4 after test_host_addr_size() Sean Christopherson 2020-07-14 4:43 ` Oliver Upton 2020-07-15 18:34 ` Krish Sadhukhan 2020-07-15 18:48 ` Sean Christopherson 2020-07-15 21:34 ` Krish Sadhukhan 2020-07-15 22:22 ` Sean Christopherson 2020-07-16 0:41 ` Krish Sadhukhan 2020-07-14 0:23 ` [kvm-unit-tests PATCH 2/2] nVMX: Use the standard non-canonical value in test_mtf3 Sean Christopherson 2020-07-14 4:41 ` Oliver Upton 2020-07-15 21:31 ` [kvm-unit-tests PATCH 0/2] nVMX: Two PCIDE related fixes Krish Sadhukhan 2020-07-28 21:26 ` Paolo Bonzini
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).