mbox series

[0/4,kvm-unit-test,nVMX] : Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry control on vmentry of nested guests

Message ID 20190509212055.29933-1-krish.sadhukhan@oracle.com (mailing list archive)
Headers show
Series : Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry control on vmentry of nested guests | expand

Message

Krish Sadhukhan May 9, 2019, 9:20 p.m. UTC
This set contains the unit test, and related changes, for the "load
IA32_PERF_GLOBAL_CONTROL" VM-entry control that was enabled in my previous
patchset titled:

	[KVM nVMX]: Check "load IA32_PERF_GLOBAL_CTRL" on vmentry of nested guests


[PATCH 1/4][kvm-unit-test nVMX]: Rename guest_pat_main to guest_state_test_main
[PATCH 2/4][kvm-unit-test nVMX]: Rename report_guest_pat_test to
[PATCH 3/4][kvm-unit-test nVMX]: Add #define for "load IA32_PERF_GLOBAL_CONTROL" bit
[PATCH 4/4][kvm-unit-test nVMX]: Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry

 x86/vmx.h       |   1 +
 x86/vmx_tests.c | 108 +++++++++++++++++++++++++++++++++++++++++++-----------
 2 files changed, 87 insertions(+), 22 deletions(-)

Krish Sadhukhan (4):
      nVMX: Rename guest_pat_main to guest_state_test_main
      nVMX: Rename report_guest_pat_test to report_guest_state_test
      nVMX: Add #define for "load IA32_PERF_GLOBAL_CONTROL" bit
      nVMX: Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry control on vmentry of nested guests

Comments

Paolo Bonzini June 6, 2019, 12:39 p.m. UTC | #1
On 09/05/19 23:20, Krish Sadhukhan wrote:
> This set contains the unit test, and related changes, for the "load
> IA32_PERF_GLOBAL_CONTROL" VM-entry control that was enabled in my previous
> patchset titled:
> 
> 	[KVM nVMX]: Check "load IA32_PERF_GLOBAL_CTRL" on vmentry of nested guests
> 
> 
> [PATCH 1/4][kvm-unit-test nVMX]: Rename guest_pat_main to guest_state_test_main
> [PATCH 2/4][kvm-unit-test nVMX]: Rename report_guest_pat_test to
> [PATCH 3/4][kvm-unit-test nVMX]: Add #define for "load IA32_PERF_GLOBAL_CONTROL" bit
> [PATCH 4/4][kvm-unit-test nVMX]: Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry
> 
>  x86/vmx.h       |   1 +
>  x86/vmx_tests.c | 108 +++++++++++++++++++++++++++++++++++++++++++-----------
>  2 files changed, 87 insertions(+), 22 deletions(-)
> 
> Krish Sadhukhan (4):
>       nVMX: Rename guest_pat_main to guest_state_test_main
>       nVMX: Rename report_guest_pat_test to report_guest_state_test
>       nVMX: Add #define for "load IA32_PERF_GLOBAL_CONTROL" bit
>       nVMX: Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry control on vmentry of nested guests
> 

Queued 1-3, but patch 4 does not apply.  It seems like you have another
patch that this sits on top of?

Paolo
Krish Sadhukhan June 8, 2019, 1:06 a.m. UTC | #2
On 06/06/2019 05:39 AM, Paolo Bonzini wrote:
> On 09/05/19 23:20, Krish Sadhukhan wrote:
>> This set contains the unit test, and related changes, for the "load
>> IA32_PERF_GLOBAL_CONTROL" VM-entry control that was enabled in my previous
>> patchset titled:
>>
>> 	[KVM nVMX]: Check "load IA32_PERF_GLOBAL_CTRL" on vmentry of nested guests
>>
>>
>> [PATCH 1/4][kvm-unit-test nVMX]: Rename guest_pat_main to guest_state_test_main
>> [PATCH 2/4][kvm-unit-test nVMX]: Rename report_guest_pat_test to
>> [PATCH 3/4][kvm-unit-test nVMX]: Add #define for "load IA32_PERF_GLOBAL_CONTROL" bit
>> [PATCH 4/4][kvm-unit-test nVMX]: Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry
>>
>>   x86/vmx.h       |   1 +
>>   x86/vmx_tests.c | 108 +++++++++++++++++++++++++++++++++++++++++++-----------
>>   2 files changed, 87 insertions(+), 22 deletions(-)
>>
>> Krish Sadhukhan (4):
>>        nVMX: Rename guest_pat_main to guest_state_test_main
>>        nVMX: Rename report_guest_pat_test to report_guest_state_test
>>        nVMX: Add #define for "load IA32_PERF_GLOBAL_CONTROL" bit
>>        nVMX: Test "load IA32_PERF_GLOBAL_CONTROL" VM-entry control on vmentry of nested guests
>>
> Queued 1-3, but patch 4 does not apply.  It seems like you have another
> patch that this sits on top of?

Yes, the following patchset:

             [KVM nVMX]: Check "load IA32_PERF_GLOBAL_CTRL" on vmentry 
of nested guests

I need to send out v2 of that set and patch# 4 of this set builds on top 
of that.

You don't need to pull 1-3 out because they are independent of the 
pending patchset. I will attach patch# 4 of this set to the v2 of the 
pending set.

>
> Paolo