On Thu, Nov 30, 2017 at 04:39:41PM +0000, Will Deacon wrote: > .macro kernel_ventry, el, label, regsize = 64 > .align 7 > +#ifdef CONFIG_UNMAP_KERNEL_AT_EL0 > + .if \el == 0 > + .if \regsize == 64 > + mrs x30, tpidrro_el0 > + msr tpidrro_el0, xzr > + .else > + mov x30, xzr
I guess that's just to prevent acccidental leaks if we dump registers somewhere, since we used x30 as a scratch register? > + .macro tramp_alias, dst, sym > + mov_q \dst, TRAMP_VALIAS > + add \dst, \dst, #(\sym - .entry.tramp.text) > + .endm I didn't realise you could refer to sections like this; neat! Otherwise, looks fine to me. Thanks, Mark.

