diff options
author | David Hildenbrand <dahi@linux.vnet.ibm.com> | 2016-01-15 00:12:47 +0300 |
---|---|---|
committer | Christian Borntraeger <borntraeger@de.ibm.com> | 2016-01-26 17:40:21 +0300 |
commit | 9abc2a08a7d665b02bdde974fd6c44aae86e923e (patch) | |
tree | 16b5d27bc327754fc87c261551c55dffd44497b3 /arch/arm/kernel/cpuidle.c | |
parent | 14b0b4ac37cbb3e57f564da30ece5626d1e3767a (diff) | |
download | linux-9abc2a08a7d665b02bdde974fd6c44aae86e923e.tar.xz |
KVM: s390: fix memory overwrites when vx is disabled
The kernel now always uses vector registers when available, however KVM
has special logic if support is really enabled for a guest. If support
is disabled, guest_fpregs.fregs will only contain memory for the fpu.
The kernel, however, will store vector registers into that area,
resulting in crazy memory overwrites.
Simply extending that area is not enough, because the format of the
registers also changes. We would have to do additional conversions, making
the code even more complex. Therefore let's directly use one place for
the vector/fpu registers + fpc (in kvm_run). We just have to convert the
data properly when accessing it. This makes current code much easier.
Please note that vector/fpu registers are now always stored to
vcpu->run->s.regs.vrs. Although this data is visible to QEMU and
used for migration, we only guarantee valid values to user space when
KVM_SYNC_VRS is set. As that is only the case when we have vector
register support, we are on the safe side.
Fixes: b5510d9b68c3 ("s390/fpu: always enable the vector facility if it is available")
Cc: stable@vger.kernel.org # v4.4 d9a3a09af54d s390/kvm: remove dependency on struct save_area definition
Signed-off-by: David Hildenbrand <dahi@linux.vnet.ibm.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
[adopt to d9a3a09af54d]
Diffstat (limited to 'arch/arm/kernel/cpuidle.c')
0 files changed, 0 insertions, 0 deletions