On Mon, Feb 10, 2020 at 06:07:41PM +0000, Will Deacon wrote:

On Mon, Feb 10, 2020 at 06:03:28PM +0000, Mark Rutland wrote:

quoted

On Mon, Feb 10, 2020 at 05:52:15PM +0000, Will Deacon wrote:

quoted

On Mon, Feb 10, 2020 at 05:18:58PM +0000, James Morse wrote:

quoted

On 28/01/2020 18:49, Sami Tolvanen wrote:

quoted

Filter out CC_FLAGS_SCS and -ffixed-x18 for code that runs at a
different exception level.

Hmmm, there are two things being disabled here.

Stashing the lr in memory pointed to by VA won't work transparently at EL2 ... but
shouldn't KVM's C code still treat x18 as a fixed register?

My review of v6 suggested dropping the -ffixed-x18 as well, since it's only
introduced by SCS (in patch 5) and so isn't required by anything else. Why
do you think it's needed?

When EL1 code calls up to hyp, it expects x18 to be preserved across the
call, so hyp needs to either preserve it explicitly across a transitions
from/to EL1 or always preserve it.

I thought we explicitly saved/restored it across the call after
af12376814a5 ("arm64: kvm: stop treating register x18 as caller save"). Is
that not sufficient?

That covers the hyp->guest->hyp round trip, but not the host->hyp->host
portion surrounding that.

Anywhere we use __call_hyp() expects x18 to be preserved across the
call, and that's not only used to enter the guest. If we don't want to
do that naturally at EL2, we'd probably have to add that to
do_el2_call in arch/arm64/kvm/hyp/hyp-entry.S.

Thanks,
Mark.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel