On 07/20/2016 06:10 PM, Julien Grall wrote:
> The function p2m_restore_state could be called with an idle vCPU in
> arguments (when called by construct_dom0). However, we will never return
> to EL0/EL1 in this case, so it is not necessary to restore the p2m
> registers.
>
I absolutely agree.
Cheers,
~Sergej
> Signed-off-by: Julien Grall <julien.gr...@arm.com>
> ---
> xen/arch/arm/p2m.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c
> index c52081a..d1b6009 100644
> --- a/xen/arch/arm/p2m.c
> +++ b/xen/arch/arm/p2m.c
> @@ -127,6 +127,9 @@ void p2m_restore_state(struct vcpu *n)
> {
> register_t hcr;
>
> + if ( is_idle_vcpu(n) )
> + return;
> +
> hcr = READ_SYSREG(HCR_EL2);
> WRITE_SYSREG(hcr & ~HCR_VM, HCR_EL2);
> isb();
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
