On Wed, Dec 02, 2020 at 08:04:02PM +0100, Alexander Graf wrote:
> In Hypervisor.framework, we just pass PSCI calls straight on to the QEMU
> emulation
> of it. That means, if TCG is compatible with PSCI 0.2, so are we. Let's
> transpose
> that fact in code too.
>
> Signed-off-by: Alexander Graf <ag...@csgraf.de>
> ---
> target/arm/cpu.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/target/arm/cpu.c b/target/arm/cpu.c
> index 07492e9f9a..db6f7c34ed 100644
> --- a/target/arm/cpu.c
> +++ b/target/arm/cpu.c
> @@ -1062,6 +1062,10 @@ static void arm_cpu_initfn(Object *obj)
> if (tcg_enabled()) {
> cpu->psci_version = 2; /* TCG implements PSCI 0.2 */
> }
> +
> + if (hvf_enabled()) {
> + cpu->psci_version = 2; /* HVF uses TCG's PSCI */
> + }
If HVF is piggybacking on TCG and they're both the same versions would
it be better if:
> - if (tcg_enabled()) {
> - cpu->psci_version = 2; /* TCG implements PSCI 0.2 */
> + if (tcg_enabled() || hvf_enabled()) {
> + cpu->psci_version = 2; /* TCG implements PSCI 0.2 */
> }
That'd avoid a case where they get out of sync.
Thanks,
Roman
> }
>
> static Property arm_cpu_gt_cntfrq_property =
> --
> 2.24.3 (Apple Git-128)
>
