On Thu, Sep 2, 2021 at 9:19 AM Philippe Mathieu-Daudé <f4...@amsat.org>
wrote:
> Restrict cpu_exec_interrupt() and its callees to sysemu.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> target/rx/cpu.h | 2 ++
> target/rx/cpu.c | 2 +-
> target/rx/helper.c | 4 ++++
> 3 files changed, 7 insertions(+), 1 deletion(-)
>
Reviewed-by: Warner Losh <i...@bsdimp.com>
> diff --git a/target/rx/cpu.h b/target/rx/cpu.h
> index 0b4b998c7be..faa3606f52f 100644
> --- a/target/rx/cpu.h
> +++ b/target/rx/cpu.h
> @@ -124,8 +124,10 @@ typedef RXCPU ArchCPU;
> #define CPU_RESOLVING_TYPE TYPE_RX_CPU
>
> const char *rx_crname(uint8_t cr);
> +#ifndef CONFIG_USER_ONLY
> void rx_cpu_do_interrupt(CPUState *cpu);
> bool rx_cpu_exec_interrupt(CPUState *cpu, int int_req);
> +#endif /* !CONFIG_USER_ONLY */
> void rx_cpu_dump_state(CPUState *cpu, FILE *f, int flags);
> int rx_cpu_gdb_read_register(CPUState *cpu, GByteArray *buf, int reg);
> int rx_cpu_gdb_write_register(CPUState *cpu, uint8_t *buf, int reg);
> diff --git a/target/rx/cpu.c b/target/rx/cpu.c
> index 96cc96e514f..25a4aa2976d 100644
> --- a/target/rx/cpu.c
> +++ b/target/rx/cpu.c
> @@ -186,10 +186,10 @@ static const struct SysemuCPUOps rx_sysemu_ops = {
> static const struct TCGCPUOps rx_tcg_ops = {
> .initialize = rx_translate_init,
> .synchronize_from_tb = rx_cpu_synchronize_from_tb,
> - .cpu_exec_interrupt = rx_cpu_exec_interrupt,
> .tlb_fill = rx_cpu_tlb_fill,
>
> #ifndef CONFIG_USER_ONLY
> + .cpu_exec_interrupt = rx_cpu_exec_interrupt,
> .do_interrupt = rx_cpu_do_interrupt,
> #endif /* !CONFIG_USER_ONLY */
> };
> diff --git a/target/rx/helper.c b/target/rx/helper.c
> index db6b07e3890..f34945e7e2c 100644
> --- a/target/rx/helper.c
> +++ b/target/rx/helper.c
> @@ -40,6 +40,8 @@ void rx_cpu_unpack_psw(CPURXState *env, uint32_t psw,
> int rte)
> env->psw_c = FIELD_EX32(psw, PSW, C);
> }
>
> +#ifndef CONFIG_USER_ONLY
> +
> #define INT_FLAGS (CPU_INTERRUPT_HARD | CPU_INTERRUPT_FIR)
> void rx_cpu_do_interrupt(CPUState *cs)
> {
> @@ -142,6 +144,8 @@ bool rx_cpu_exec_interrupt(CPUState *cs, int
> interrupt_request)
> return false;
> }
>
> +#endif /* !CONFIG_USER_ONLY */
> +
> hwaddr rx_cpu_get_phys_page_debug(CPUState *cs, vaddr addr)
> {
> return addr;
> --
> 2.31.1
>
>
