On Fri, 10 May 2019 at 02:25, Richard Henderson <richard.hender...@linaro.org> wrote: > > Cc: qemu-...@nongnu.org > Cc: Peter Maydell <peter.mayd...@linaro.org> > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > v3: Log errors with -d unimp, for lack of a better flag. > --- > target/arm/cpu.h | 5 +++++ > target/arm/cpu64.c | 1 + > target/arm/helper.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 50 insertions(+)
> +/* We do not support re-seeding, so the two registers operate the same. */ > +static const ARMCPRegInfo rndr_reginfo[] = { > + { .name = "RNDR", .state = ARM_CP_STATE_AA64, > + .type = ARM_CP_NO_RAW | ARM_CP_SUPPRESS_TB_END, > + .opc0 = 3, .opc1 = 3, .crn = 2, .crm = 4, .opc2 = 0, > + .access = PL0_R, .readfn = rndr_readfn }, > + { .name = "RNDRRS", .state = ARM_CP_STATE_AA64, > + .type = ARM_CP_NO_RAW | ARM_CP_SUPPRESS_TB_END, > + .opc0 = 3, .opc1 = 3, .crn = 2, .crm = 4, .opc2 = 1, > + .access = PL0_R, .readfn = rndr_readfn }, Don't these need to be marked ARM_CP_IO for the benefit of -icount ? Otherwise Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM