KVM: s390: sthyi: remove invalid guest write access
authorHeiko Carstens <heiko.carstens@de.ibm.com>
Thu, 3 Aug 2017 11:05:40 +0000 (13:05 +0200)
committerChristian Borntraeger <borntraeger@de.ibm.com>
Tue, 29 Aug 2017 13:15:56 +0000 (15:15 +0200)
commit631aebfee8261d66597ce0f88ba046c77a0e415e
treeeede4dee7ce0db5beeba074cceff099a38eee01a
parent8fa1696ea78162ca3112a26879d9379483443c85
KVM: s390: sthyi: remove invalid guest write access

handle_sthyi() always writes to guest memory if the sthyi function
code is zero in order to fault in the page that later is written to.

However a function code of zero does not necessarily mean that a write
to guest memory happens: if the KVM host is running as a second level
guest under z/VM 6.2 the sthyi instruction is indicated to be
available to the KVM host, however if the instruction is executed it
will always return with a return code that indicates "unsupported
function code".

In such a case handle_sthyi() must not write to guest memory. This
means that the prior write access to fault in the guest page may
result in invalid guest exceptions, and/or invalid data modification.

In order to be architecture compliant simply remove the write_guest()
call.

Given that the guest assumed a write access anyway, this fix does not
qualify for -stable. This just makes sure the sthyi handler is
architecture compliant.

Fixes: 95ca2cb57985 ("KVM: s390: Add sthyi emulation")
Reviewed-by: Janosch Frank <frankja@linux.vnet.ibm.com>
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
arch/s390/kvm/sthyi.c