[PATCH] KVM: PPC: Fix SPRG get/set for Book3S and BookE

[Peter Tyser]

Previously SPRGs 4-7 were improperly read and written in
kvm_arch_vcpu_ioctl_get_regs() and kvm_arch_vcpu_ioctl_set_regs();

Signed-off-by: Peter Tyser <pty...@xes-inc.com>
---
I noticed this while grepping for somthing unrelated and assume its
a typo.  Feel free to add to the patch description; I don't use KVM
so don't know what the high-level consequences of this change are.

 arch/powerpc/kvm/book3s.c |   14 ++++++++------
 arch/powerpc/kvm/booke.c  |   14 ++++++++------
 2 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c
index e316847..5eb86c9 100644
--- a/arch/powerpc/kvm/book3s.c
+++ b/arch/powerpc/kvm/book3s.c
@@ -1141,9 +1141,10 @@ int kvm_arch_vcpu_ioctl_get_regs(struct kvm_vcpu *vcpu, 
struct kvm_regs *regs)
        regs->sprg1 = vcpu->arch.shared->sprg1;
        regs->sprg2 = vcpu->arch.shared->sprg2;
        regs->sprg3 = vcpu->arch.shared->sprg3;
-       regs->sprg5 = vcpu->arch.sprg4;
-       regs->sprg6 = vcpu->arch.sprg5;
-       regs->sprg7 = vcpu->arch.sprg6;
+       regs->sprg4 = vcpu->arch.sprg4;
+       regs->sprg5 = vcpu->arch.sprg5;
+       regs->sprg6 = vcpu->arch.sprg6;
+       regs->sprg7 = vcpu->arch.sprg7;
 
        for (i = 0; i < ARRAY_SIZE(regs->gpr); i++)
                regs->gpr[i] = kvmppc_get_gpr(vcpu, i);
@@ -1167,9 +1168,10 @@ int kvm_arch_vcpu_ioctl_set_regs(struct kvm_vcpu *vcpu, 
struct kvm_regs *regs)
        vcpu->arch.shared->sprg1 = regs->sprg1;
        vcpu->arch.shared->sprg2 = regs->sprg2;
        vcpu->arch.shared->sprg3 = regs->sprg3;
-       vcpu->arch.sprg5 = regs->sprg4;
-       vcpu->arch.sprg6 = regs->sprg5;
-       vcpu->arch.sprg7 = regs->sprg6;
+       vcpu->arch.sprg4 = regs->sprg4;
+       vcpu->arch.sprg5 = regs->sprg5;
+       vcpu->arch.sprg6 = regs->sprg6;
+       vcpu->arch.sprg7 = regs->sprg7;
 
        for (i = 0; i < ARRAY_SIZE(regs->gpr); i++)
                kvmppc_set_gpr(vcpu, i, regs->gpr[i]);
diff --git a/arch/powerpc/kvm/booke.c b/arch/powerpc/kvm/booke.c
index 77575d0..ef76acb 100644
--- a/arch/powerpc/kvm/booke.c
+++ b/arch/powerpc/kvm/booke.c
@@ -546,9 +546,10 @@ int kvm_arch_vcpu_ioctl_get_regs(struct kvm_vcpu *vcpu, 
struct kvm_regs *regs)
        regs->sprg1 = vcpu->arch.shared->sprg1;
        regs->sprg2 = vcpu->arch.shared->sprg2;
        regs->sprg3 = vcpu->arch.shared->sprg3;
-       regs->sprg5 = vcpu->arch.sprg4;
-       regs->sprg6 = vcpu->arch.sprg5;
-       regs->sprg7 = vcpu->arch.sprg6;
+       regs->sprg4 = vcpu->arch.sprg4;
+       regs->sprg5 = vcpu->arch.sprg5;
+       regs->sprg6 = vcpu->arch.sprg6;
+       regs->sprg7 = vcpu->arch.sprg7;
 
        for (i = 0; i < ARRAY_SIZE(regs->gpr); i++)
                regs->gpr[i] = kvmppc_get_gpr(vcpu, i);
@@ -572,9 +573,10 @@ int kvm_arch_vcpu_ioctl_set_regs(struct kvm_vcpu *vcpu, 
struct kvm_regs *regs)
        vcpu->arch.shared->sprg1 = regs->sprg1;
        vcpu->arch.shared->sprg2 = regs->sprg2;
        vcpu->arch.shared->sprg3 = regs->sprg3;
-       vcpu->arch.sprg5 = regs->sprg4;
-       vcpu->arch.sprg6 = regs->sprg5;
-       vcpu->arch.sprg7 = regs->sprg6;
+       vcpu->arch.sprg4 = regs->sprg4;
+       vcpu->arch.sprg5 = regs->sprg5;
+       vcpu->arch.sprg6 = regs->sprg6;
+       vcpu->arch.sprg7 = regs->sprg7;
 
        for (i = 0; i < ARRAY_SIZE(regs->gpr); i++)
                kvmppc_set_gpr(vcpu, i, regs->gpr[i]);
-- 
1.7.0.4

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev