On Mon, Oct 07, 2019 at 07:29:45pm +0100, Cristian Marussi wrote: > Add some arm64/signal specific boilerplate and utility code to help > further testcases' development. > > Introduce also one simple testcase mangle_pstate_invalid_compat_toggle > and some related helpers: it is a simple mangle testcase which messes > with the ucontext_t from within the signal handler, trying to toggle > PSTATE state bits to switch the system between 32bit/64bit execution > state. Expects SIGSEGV on test PASS. > > Reviewed-by: Dave Martin <Dave.Martin@xxxxxxx> > Signed-off-by: Cristian Marussi <cristian.marussi@xxxxxxx> > --- > v6 --> v7 > - removed ambiguos fprintf in test_init > - fixed spacing [...] > diff --git a/tools/testing/selftests/arm64/signal/test_signals.h b/tools/testing/selftests/arm64/signal/test_signals.h [...] > +/* Local Helpers */ > +#define ID_AA64MMFR1_EL1_PAN_SUPPORTED(val) \ > + (!!((val) & (0xfUL << ID_AA64MMFR1_PAN_SHIFT))) > +#define ID_AA64MMFR2_EL1_UAO_SUPPORTED(val) \ > + (!!((val) & (0xfUL << ID_AA64MMFR2_UAO_SHIFT))) > + > +#define SSBS_SYSREG S3_3_C4_C2_6 /* EL0 supported */ Does anything in the series still use SSBS_SYSREG? If not, please drop it. No need to remove my Reviewed-by just for thato change. [...] Cheers ---Dave