diff options
| author | Jens Freimann | 2014-04-17 10:10:30 +0200 |
|---|---|---|
| committer | Christian Borntraeger | 2014-08-25 14:35:56 +0200 |
| commit | 7939503147f89f0799ddc89afec2aeae57dd7e2c (patch) | |
| tree | 5e314444b3142d0b96cbeb748c815a9c432ca20a /arch/s390/kvm/kvm-s390.c | |
| parent | KVM: s390: implement KVM_REQ_TLB_FLUSH and make use of it (diff) | |
| download | kernel-qcow2-linux-7939503147f89f0799ddc89afec2aeae57dd7e2c.tar.gz kernel-qcow2-linux-7939503147f89f0799ddc89afec2aeae57dd7e2c.tar.xz kernel-qcow2-linux-7939503147f89f0799ddc89afec2aeae57dd7e2c.zip | |
KVM: s390: return -EFAULT if lowcore is not mapped during irq delivery
Currently we just kill the userspace process and exit the thread
immediatly without making sure that we don't hold any locks etc.
Improve this by making KVM_RUN return -EFAULT if the lowcore is not
mapped during interrupt delivery. To achieve this we need to pass
the return code of guest memory access routines used in interrupt
delivery all the way back to the KVM_RUN ioctl.
Signed-off-by: Jens Freimann <jfrei@linux.vnet.ibm.com>
Reviewed-by: David Hildenbrand <dahi@linux.vnet.ibm.com>
Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Diffstat (limited to 'arch/s390/kvm/kvm-s390.c')
| -rw-r--r-- | arch/s390/kvm/kvm-s390.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index 56193becdca5..c2caa175320c 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c @@ -1198,8 +1198,11 @@ static int vcpu_pre_run(struct kvm_vcpu *vcpu) if (test_cpu_flag(CIF_MCCK_PENDING)) s390_handle_mcck(); - if (!kvm_is_ucontrol(vcpu->kvm)) - kvm_s390_deliver_pending_interrupts(vcpu); + if (!kvm_is_ucontrol(vcpu->kvm)) { + rc = kvm_s390_deliver_pending_interrupts(vcpu); + if (rc) + return rc; + } rc = kvm_s390_handle_requests(vcpu); if (rc) |