* [Bug 53631] New: nVMX: Support exit/entry MSR load/store
@ 2013-02-11 14:21 bugzilla-daemon
2013-02-11 14:21 ` [Bug 53631] " bugzilla-daemon
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: bugzilla-daemon @ 2013-02-11 14:21 UTC (permalink / raw)
To: kvm
https://bugzilla.kernel.org/show_bug.cgi?id=53631
Summary: nVMX: Support exit/entry MSR load/store
Product: Virtualization
Version: unspecified
Platform: All
OS/Version: Linux
Tree: Mainline
Status: NEW
Severity: normal
Priority: P1
Component: kvm
AssignedTo: virtualization_kvm@kernel-bugs.osdl.org
ReportedBy: nyh@math.technion.ac.il
Regression: No
Currently, nested VMX does not allow L1 to use the MSR load/store feature on
entry/exit. The code fails entry (from L1 to L2) if these VMCS features are
used:
if (vmcs12->vm_entry_msr_load_count > 0 ||
vmcs12->vm_exit_msr_load_count > 0 ||
vmcs12->vm_exit_msr_store_count > 0) {
pr_warn_ratelimited("%s: VMCS MSR_{LOAD,STORE} unsupported\n",
__func__);
nested_vmx_failValid(vcpu, VMXERR_ENTRY_INVALID_CONTROL_FIELD);
return 1;
}
This was not a big problem if L1 is also KVM, because KVM didn't use this
feature - it only uses it in case of EPT for switching the EFER (see
explanation in http://kerneltrap.org/mailarchive/linux-kvm/2010/5/2/6261577),
and in that case there is a simpler alternative: supporting
VM_ENTRY/EXIT_LOAD_IA32_EFER is enough. So this is what we did in the nested
EPT patches proposed in bug 53611.
However, it is likely that for different L1s (or even KVM in the future), we'll
need to support the generic MSR load/store feature.
--
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug 53631] nVMX: Support exit/entry MSR load/store
2013-02-11 14:21 [Bug 53631] New: nVMX: Support exit/entry MSR load/store bugzilla-daemon
@ 2013-02-11 14:21 ` bugzilla-daemon
2013-02-26 15:21 ` bugzilla-daemon
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon @ 2013-02-11 14:21 UTC (permalink / raw)
To: kvm
https://bugzilla.kernel.org/show_bug.cgi?id=53631
Nadav Har'El <nyh@math.technion.ac.il> changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |53601
--
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug 53631] nVMX: Support exit/entry MSR load/store
2013-02-11 14:21 [Bug 53631] New: nVMX: Support exit/entry MSR load/store bugzilla-daemon
2013-02-11 14:21 ` [Bug 53631] " bugzilla-daemon
@ 2013-02-26 15:21 ` bugzilla-daemon
2015-03-17 3:53 ` bugzilla-daemon
2015-04-08 8:57 ` bugzilla-daemon
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon @ 2013-02-26 15:21 UTC (permalink / raw)
To: kvm
https://bugzilla.kernel.org/show_bug.cgi?id=53631
--- Comment #1 from Nadav Har'El <nyh@math.technion.ac.il> 2013-02-26 15:21:19 ---
To support this feature correctly, I think we can't give the msr array address
given by L1 (vmcs12) directly to the processor (vmcs02), but rather we should
loop on the entries in the array given by L1, using KVM's writemsr/readmsr.
--
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug 53631] nVMX: Support exit/entry MSR load/store
2013-02-11 14:21 [Bug 53631] New: nVMX: Support exit/entry MSR load/store bugzilla-daemon
2013-02-11 14:21 ` [Bug 53631] " bugzilla-daemon
2013-02-26 15:21 ` bugzilla-daemon
@ 2015-03-17 3:53 ` bugzilla-daemon
2015-04-08 8:57 ` bugzilla-daemon
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon @ 2015-03-17 3:53 UTC (permalink / raw)
To: kvm
https://bugzilla.kernel.org/show_bug.cgi?id=53631
Bandan Das <bsd@makefile.in> changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |94971
--
You are receiving this mail because:
You are watching the assignee of the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug 53631] nVMX: Support exit/entry MSR load/store
2013-02-11 14:21 [Bug 53631] New: nVMX: Support exit/entry MSR load/store bugzilla-daemon
` (2 preceding siblings ...)
2015-03-17 3:53 ` bugzilla-daemon
@ 2015-04-08 8:57 ` bugzilla-daemon
3 siblings, 0 replies; 5+ messages in thread
From: bugzilla-daemon @ 2015-04-08 8:57 UTC (permalink / raw)
To: kvm
https://bugzilla.kernel.org/show_bug.cgi?id=53631
Paolo Bonzini <bonzini@gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
CC| |bonzini@gnu.org
Kernel Version| |3.19
Resolution|--- |CODE_FIX
--- Comment #2 from Paolo Bonzini <bonzini@gnu.org> ---
Fixed by commit ff651cb613b4 (KVM: nVMX: Add nested msr load/restore algorithm,
2014-12-11).
--
You are receiving this mail because:
You are watching the assignee of the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-04-08 8:57 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-11 14:21 [Bug 53631] New: nVMX: Support exit/entry MSR load/store bugzilla-daemon
2013-02-11 14:21 ` [Bug 53631] " bugzilla-daemon
2013-02-26 15:21 ` bugzilla-daemon
2015-03-17 3:53 ` bugzilla-daemon
2015-04-08 8:57 ` bugzilla-daemon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.