KVM: s390: some ext irqs have to clear the ext cpu addr

The cpu address of a source cpu (responsible for an external irq) is only to
be stored if bit 6 of the ext irq code is set.

If bit 6 is not set, it is to be zeroed out.

The special external irq code used for virtio and pfault uses the cpu addr as a
parameter field. As bit 6 is set, this implementation is correct.

Reviewed-by: Thomas Huth <thuth@linux.vnet.ibm.com>
Signed-off-by: David Hildenbrand <dahi@linux.vnet.ibm.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>

1 files changed, 3 insertions, 0 deletions

diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c
index b3d4409fcf3f..6c0d14b5fec5 100644
--- a/arch/s390/kvm/interrupt.c
+++ b/arch/s390/kvm/interrupt.c
@@ -281,6 +281,7 @@ static int __must_check __deliver_cpu_timer(struct kvm_vcpu *vcpu)
 
         rc  = put_guest_lc(vcpu, EXT_IRQ_CPU_TIMER,
                            (u16 *)__LC_EXT_INT_CODE);
+        rc |= put_guest_lc(vcpu, 0, (u16 *)__LC_EXT_CPU_ADDR);
         rc |= write_guest_lc(vcpu, __LC_EXT_OLD_PSW,
                              &vcpu->arch.sie_block->gpsw, sizeof(psw_t));
         rc |= read_guest_lc(vcpu, __LC_EXT_NEW_PSW,
@@ -299,6 +300,7 @@ static int __must_check __deliver_ckc(struct kvm_vcpu *vcpu)
 
         rc  = put_guest_lc(vcpu, EXT_IRQ_CLK_COMP,
                            (u16 __user *)__LC_EXT_INT_CODE);
+        rc |= put_guest_lc(vcpu, 0, (u16 *)__LC_EXT_CPU_ADDR);
         rc |= write_guest_lc(vcpu, __LC_EXT_OLD_PSW,
                              &vcpu->arch.sie_block->gpsw, sizeof(psw_t));
         rc |= read_guest_lc(vcpu, __LC_EXT_NEW_PSW,
@@ -585,6 +587,7 @@ static int __must_check __deliver_service(struct kvm_vcpu *vcpu,
                                          inti->ext.ext_params, 0);
 
         rc  = put_guest_lc(vcpu, EXT_IRQ_SERVICE_SIG, (u16 *)__LC_EXT_INT_CODE);
+        rc |= put_guest_lc(vcpu, 0, (u16 *)__LC_EXT_CPU_ADDR);
         rc |= write_guest_lc(vcpu, __LC_EXT_OLD_PSW,
                              &vcpu->arch.sie_block->gpsw, sizeof(psw_t));
         rc |= read_guest_lc(vcpu, __LC_EXT_NEW_PSW,