On Fri, Feb 20, 2015 at 04:02:10PM -0600, Joel Schopp wrote: > From: David Kaplan <david.kap...@amd.com> > > KVM has nice wrappers to access the register values, clean up a few places > that should use them but currently do not. > > Signed-off-by: David Kaplan <david.kap...@amd.com> > [forward port and testing] > Signed-off-by: Joel Schopp <joel.sch...@amd.com>
Looks good. Acked-by: Borislav Petkov <b...@suse.de> > --- > arch/x86/kvm/svm.c | 19 +++++++++---------- > 1 file changed, 9 insertions(+), 10 deletions(-) > > diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c > index d319e0c..a7d88e4 100644 > --- a/arch/x86/kvm/svm.c > +++ b/arch/x86/kvm/svm.c ... > @@ -3142,8 +3142,8 @@ static int rdmsr_interception(struct vcpu_svm *svm) > } else { > trace_kvm_msr_read(ecx, data); > > - svm->vcpu.arch.regs[VCPU_REGS_RAX] = data & 0xffffffff; > - svm->vcpu.arch.regs[VCPU_REGS_RDX] = data >> 32; > + kvm_register_write(&svm->vcpu, VCPU_REGS_RAX, data & > 0xffffffff); > + kvm_register_write(&svm->vcpu, VCPU_REGS_RDX, data >> 32); Right, kvm is missing kvm_write_edx_eax() in addition to the read variant. Someone might want to do a patch and such, ^^hint hint^^... :-) -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/