On Fri, Dec 06, 2019 at 12:12:10PM +0000, Marc Zyngier wrote:
> On 2019-12-06 11:57, Catalin Marinas wrote:
> > On Thu, Nov 28, 2019 at 08:58:05PM +0100, Sebastian Andrzej Siewior
> > wrote:
> > > @@ -408,6 +410,8 @@ static void __init hyp_mode_check(void)
> > > "CPU: CPUs started in inconsistent modes");
> > > else
> > > pr_info("CPU: All CPU(s) started at EL1\n");
> > > + if (IS_ENABLED(CONFIG_KVM_ARM_HOST))
> > > + kvm_compute_layout();
> > > }
> >
> > It looks like we call this unconditionally here even if the kernel was
> > booted at EL1.
>
> It has always been the case. My motivation was to be able to debug
> this in a guest, because doing this on the host is... painful! ;-)
>
> Feel free to condition it on !EL1 though.
I'll leave the patch as is. Thanks for confirming.
--
Catalin
_______________________________________________
kvmarm mailing list
kvmarm@xxxxxxxxxxxxxxxxxxxxx
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
