]> git.proxmox.com Git - mirror_qemu.git/commit
ppc: Validate compatibility modes when setting
authorDavid Gibson <david@gibson.dropbear.id.au>
Fri, 28 Oct 2016 11:51:46 +0000 (22:51 +1100)
committerDavid Gibson <david@gibson.dropbear.id.au>
Mon, 30 Jan 2017 23:10:13 +0000 (10:10 +1100)
commit9d2179d6f960aef1b8aab4d014fd8385f0a187e5
tree1d518868a6d095c7a3e408bd0f985ace45da5b4c
parent12dbeb16d0984fe03bd4bc5cd952187627a22ce9
ppc: Validate compatibility modes when setting

Current ppc_set_compat() will attempt to set any compatiblity mode
specified, regardless of whether it's available on the CPU.  The caller is
expected to make sure it is setting a possible mode, which is awkwward
because most of the information to make that decision is at the CPU level.

This begins to clean this up by introducing a ppc_check_compat() function
which will determine if a given compatiblity mode is supported on a CPU
(and also whether it lies within specified minimum and maximum compat
levels, which will be useful later).  It also contains an assertion that
the CPU has a "virtual hypervisor"[1], that is, that the guest isn't
permitted to execute hypervisor privilege code.  Without that, the guest
would own the PCR and so could override any mode set here.  Only machine
types which use a virtual hypervisor (i.e. 'pseries') should use
ppc_check_compat().

ppc_set_compat() is modified to validate the compatibility mode it is given
and fail if it's not available on this CPU.

[1] Or user-only mode, which also obviously doesn't allow access to the
hypervisor privileged PCR.  We don't use that now, but could in future.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Alexey Kardashevskiy <aik@ozlabs.ru>
target/ppc/compat.c
target/ppc/cpu.h