s390/kvm: cleanup/fix handle_tpi()
- add missing specification exception check - remove one level of indentation - use defines instead of magic numbers Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com> Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
This commit is contained in:
parent
f9dc72e82d
commit
7c959e82ac
|
@ -14,6 +14,7 @@
|
||||||
#include <linux/kvm.h>
|
#include <linux/kvm.h>
|
||||||
#include <linux/gfp.h>
|
#include <linux/gfp.h>
|
||||||
#include <linux/errno.h>
|
#include <linux/errno.h>
|
||||||
|
#include <asm/asm-offsets.h>
|
||||||
#include <asm/current.h>
|
#include <asm/current.h>
|
||||||
#include <asm/debug.h>
|
#include <asm/debug.h>
|
||||||
#include <asm/ebcdic.h>
|
#include <asm/ebcdic.h>
|
||||||
|
@ -129,14 +130,20 @@ static int handle_skey(struct kvm_vcpu *vcpu)
|
||||||
|
|
||||||
static int handle_tpi(struct kvm_vcpu *vcpu)
|
static int handle_tpi(struct kvm_vcpu *vcpu)
|
||||||
{
|
{
|
||||||
u64 addr;
|
|
||||||
struct kvm_s390_interrupt_info *inti;
|
struct kvm_s390_interrupt_info *inti;
|
||||||
|
u64 addr;
|
||||||
int cc;
|
int cc;
|
||||||
|
|
||||||
addr = kvm_s390_get_base_disp_s(vcpu);
|
addr = kvm_s390_get_base_disp_s(vcpu);
|
||||||
|
if (addr & 3) {
|
||||||
|
kvm_s390_inject_program_int(vcpu, PGM_SPECIFICATION);
|
||||||
|
goto out;
|
||||||
|
}
|
||||||
|
cc = 0;
|
||||||
inti = kvm_s390_get_io_int(vcpu->kvm, vcpu->run->s.regs.crs[6], 0);
|
inti = kvm_s390_get_io_int(vcpu->kvm, vcpu->run->s.regs.crs[6], 0);
|
||||||
if (inti) {
|
if (!inti)
|
||||||
|
goto no_interrupt;
|
||||||
|
cc = 1;
|
||||||
if (addr) {
|
if (addr) {
|
||||||
/*
|
/*
|
||||||
* Store the two-word I/O interruption code into the
|
* Store the two-word I/O interruption code into the
|
||||||
|
@ -150,18 +157,17 @@ static int handle_tpi(struct kvm_vcpu *vcpu)
|
||||||
* Store the three-word I/O interruption code into
|
* Store the three-word I/O interruption code into
|
||||||
* the appropriate lowcore area.
|
* the appropriate lowcore area.
|
||||||
*/
|
*/
|
||||||
put_guest(vcpu, inti->io.subchannel_id, (u16 *) 184);
|
put_guest(vcpu, inti->io.subchannel_id, (u16 *) __LC_SUBCHANNEL_ID);
|
||||||
put_guest(vcpu, inti->io.subchannel_nr, (u16 *) 186);
|
put_guest(vcpu, inti->io.subchannel_nr, (u16 *) __LC_SUBCHANNEL_NR);
|
||||||
put_guest(vcpu, inti->io.io_int_parm, (u32 *) 188);
|
put_guest(vcpu, inti->io.io_int_parm, (u32 *) __LC_IO_INT_PARM);
|
||||||
put_guest(vcpu, inti->io.io_int_word, (u32 *) 192);
|
put_guest(vcpu, inti->io.io_int_word, (u32 *) __LC_IO_INT_WORD);
|
||||||
}
|
}
|
||||||
cc = 1;
|
|
||||||
} else
|
|
||||||
cc = 0;
|
|
||||||
kfree(inti);
|
kfree(inti);
|
||||||
|
no_interrupt:
|
||||||
/* Set condition code and we're done. */
|
/* Set condition code and we're done. */
|
||||||
vcpu->arch.sie_block->gpsw.mask &= ~(3ul << 44);
|
vcpu->arch.sie_block->gpsw.mask &= ~(3ul << 44);
|
||||||
vcpu->arch.sie_block->gpsw.mask |= (cc & 3ul) << 44;
|
vcpu->arch.sie_block->gpsw.mask |= (cc & 3ul) << 44;
|
||||||
|
out:
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue