KVM: PPC: Book3S HV: Stop returning internal values to userspace
authorFabiano Rosas <farosas@linux.ibm.com>
Tue, 25 Jan 2022 21:56:51 +0000 (18:56 -0300)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 31 Jan 2022 02:42:59 +0000 (13:42 +1100)
Our kvm_arch_vcpu_ioctl_run currently returns the RESUME_HOST values
to userspace, against the API of the KVM_RUN ioctl which returns 0 on
success.

Signed-off-by: Fabiano Rosas <farosas@linux.ibm.com>
Reviewed-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20220125215655.1026224-2-farosas@linux.ibm.com
arch/powerpc/kvm/powerpc.c

index 2ad0ccd202d5dda9be350c8b22b640e54f7093c5..50414fb2a5eaa957c86821d5a3c9d6a662fadebb 100644 (file)
@@ -1841,6 +1841,14 @@ int kvm_arch_vcpu_ioctl_run(struct kvm_vcpu *vcpu)
 #ifdef CONFIG_ALTIVEC
 out:
 #endif
+
+       /*
+        * We're already returning to userspace, don't pass the
+        * RESUME_HOST flags along.
+        */
+       if (r > 0)
+               r = 0;
+
        vcpu_put(vcpu);
        return r;
 }