summaryrefslogtreecommitdiffstats
path: root/target-ppc
diff options
context:
space:
mode:
authorAlexander Graf2011-10-24 20:43:22 +0200
committerAlexander Graf2011-10-30 20:03:27 +0100
commit70bca53ffb811ea59dc090b3ca7825cf0bf346a7 (patch)
treed8d59f592f44fe43841913927083226fc1ff26f7 /target-ppc
parentppc: Fix up usermode only builds (diff)
downloadqemu-70bca53ffb811ea59dc090b3ca7825cf0bf346a7.tar.gz
qemu-70bca53ffb811ea59dc090b3ca7825cf0bf346a7.tar.xz
qemu-70bca53ffb811ea59dc090b3ca7825cf0bf346a7.zip
KVM: PPC: Override host vmx/vsx/dfp only when information known
The -cpu host feature tries to find out the host capabilities based on device tree information. However, we don't always have that available because it's an optional property in dt. So instead of force unsetting values depending on an unreliable source of information, let's just try to be clever about it and not override capabilities when we don't know the device tree pieces. This fixes altivec with -cpu host on YDL PowerStations. Reported-by: Nishanth Aravamudan <nacc@us.ibm.com> Acked-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Alexander Graf <agraf@suse.de>
Diffstat (limited to 'target-ppc')
-rw-r--r--target-ppc/kvm.c12
1 files changed, 9 insertions, 3 deletions
diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c
index a090d79ad4..f3d0861a11 100644
--- a/target-ppc/kvm.c
+++ b/target-ppc/kvm.c
@@ -912,9 +912,15 @@ const ppc_def_t *kvmppc_host_cpu_def(void)
/* Now fix up the spec with information we can query from the host */
- alter_insns(&spec->insns_flags, PPC_ALTIVEC, vmx > 0);
- alter_insns(&spec->insns_flags2, PPC2_VSX, vmx > 1);
- alter_insns(&spec->insns_flags2, PPC2_DFP, dfp);
+ if (vmx != -1) {
+ /* Only override when we know what the host supports */
+ alter_insns(&spec->insns_flags, PPC_ALTIVEC, vmx > 0);
+ alter_insns(&spec->insns_flags2, PPC2_VSX, vmx > 1);
+ }
+ if (dfp != -1) {
+ /* Only override when we know what the host supports */
+ alter_insns(&spec->insns_flags2, PPC2_DFP, dfp);
+ }
return spec;
}