Re: [PATCH 2/7] KVM: SVM: Use host_vmcb_pa for vmload and vmsave

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/14/2011 04:10 PM, Joerg Roedel wrote:
On Thu, Jul 14, 2011 at 02:29:36PM +0300, Avi Kivity wrote:
>  On 07/13/2011 06:32 PM, Joerg Roedel wrote:
>>  This saves copying over the vmload/vmsave switched part from
>>  the host to the guest vmcb later.
>>
>>  Signed-off-by: Joerg Roedel<joerg.roedel@xxxxxxx>
>>  ---
>>    arch/x86/kvm/svm.c |    7 ++++++-
>>    1 files changed, 6 insertions(+), 1 deletions(-)
>>
>>  diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c
>>  index 3d5990f..dc703ac 100644
>>  --- a/arch/x86/kvm/svm.c
>>  +++ b/arch/x86/kvm/svm.c
>>  @@ -3704,9 +3704,13 @@ static void svm_vcpu_run(struct kvm_vcpu *vcpu)
>>
>>    		/* Enter guest mode */
>>    		"push %%"R"ax \n\t"
>>  -		"mov %c[vmcb](%[svm]), %%"R"ax \n\t"
>>  +		"mov %c[host_vmcb](%[svm]), %%"R"ax \n\t"
>>    		__ex(SVM_VMLOAD) "\n\t"
>>  +		"mov (%%"R"sp), %%"R"ax\n\t"
>>  +		"mov %c[vmcb](%[svm]), %%"R"ax \n\t"
>>    		__ex(SVM_VMRUN) "\n\t"
>>  +		"mov (%%"R"sp), %%"R"ax\n\t"
>>  +		"mov %c[host_vmcb](%[svm]), %%"R"ax \n\t"
>>    		__ex(SVM_VMSAVE) "\n\t"
>>    		"pop %%"R"ax \n\t"
>>
>
>  Okay, so the plan is to split L2 state between ->vmcb and ->host_vmcb?

Yes, otherwise we need to copy the vmload/vmsave switched state back and
forth between both VMCBs which is a waste of cycles.

Just to be sure I understand this: the root cause is because VMRUN doesn't actually switch this state. So we have to copy the state. Okay.

What about an L2 guest executing VMLOAD or VMSAVE which isn't intercepted? Don't we have to redirect it's reads and writes to host_vmcb?

>  In that case my suggestion for patch 1 doesn't apply.  But the name
>  still is confusing.  If we don't find a better one, I want a fat comment
>  explaining how state is split.

Hmm, how about naming them l1_vmcb and l2_vmcb? The comment explaining
why vmload/vmsave always happens on l1_vmcb is needed anyway then.

In a later patch you introduce n_vmcb. I think it makes sense to name that vmcb02?

>  (would be good to have documentation for the overall strategy of nsvm,
>  like we have for nvmx and nmmu).

There is not much to document about future plans for nested-svm. At the
moment I try to add emulation code for new SVM features when there is
some time left. Live migration support is also on the list.


Even the exising code would be good to document. So when a reader sees some bit, they can compare it to the document and see why it's that way.

The long-term plan is certainly to merge code with nested-vmx where
possible and move logic into generic KVM code. The first item that comes
to mind here is to create a single place where a vmexit is emulated and
let all other place which do that today just signal that it is required.

I'm not very concerned about reuse with nvmx except for architectural code like interrupts. Of course, if it turns out simple I'm all for it, but if it's hard or uglifies the code, let it be.

--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux