From 8030f85769ed93908f16696615ed22242f861eba Mon Sep 17 00:00:00 2001 From: Ben Elliston Date: Mon, 19 Feb 2001 21:57:03 +0000 Subject: [PATCH] 2001-02-19 Ben Elliston * sim-main.h (sim_monitor): Return an int. * interp.c (sim_monitor): Add return values. (signal_exception): Handle error conditions from sim_monitor. --- sim/mips/ChangeLog | 6 ++++++ sim/mips/interp.c | 14 ++++++++------ sim/mips/sim-main.h | 2 +- 3 files changed, 15 insertions(+), 7 deletions(-) diff --git a/sim/mips/ChangeLog b/sim/mips/ChangeLog index 8a3e458f20a..f3cb60918dc 100644 --- a/sim/mips/ChangeLog +++ b/sim/mips/ChangeLog @@ -1,3 +1,9 @@ +2001-02-19 Ben Elliston + + * sim-main.h (sim_monitor): Return an int. + * interp.c (sim_monitor): Add return values. + (signal_exception): Handle error conditions from sim_monitor. + 2001-02-08 Ben Elliston * sim-main.c (load_memory): Pass cia to sim_core_read* functions. diff --git a/sim/mips/interp.c b/sim/mips/interp.c index 382ad7e162a..a0327636506 100644 --- a/sim/mips/interp.c +++ b/sim/mips/interp.c @@ -1128,7 +1128,7 @@ Recognized firmware names are: `idt', `pmon', `lsipmon', and `none'.\n", /* Simple monitor interface (currently setup for the IDT and PMON monitors) */ -void +int sim_monitor (SIM_DESC sd, sim_cpu *cpu, address_word cia, @@ -1358,11 +1358,10 @@ sim_monitor (SIM_DESC sd, } default: - sim_io_error (sd, "TODO: sim_monitor(%d) : PC = 0x%s\n", - reason, pr_addr(cia)); - break; + /* Unknown reason. */ + return 0; } - return; + return 1; } /* Store a word into memory. */ @@ -1713,7 +1712,10 @@ signal_exception (SIM_DESC sd, perform this magic. */ if ((instruction & RSVD_INSTRUCTION_MASK) == RSVD_INSTRUCTION) { - sim_monitor (SD, CPU, cia, ((instruction >> RSVD_INSTRUCTION_ARG_SHIFT) & RSVD_INSTRUCTION_ARG_MASK) ); + int reason = (instruction >> RSVD_INSTRUCTION_ARG_SHIFT) & RSVD_INSTRUCTION_ARG_MASK; + if (!sim_monitor (SD, CPU, cia, reason)) + sim_io_error (sd, "sim_monitor: unhandled reason = %d, pc = 0x%s\n", reason, pr_addr (cia)); + /* NOTE: This assumes that a branch-and-link style instruction was used to enter the vector (which is the case with the current IDT monitor). */ diff --git a/sim/mips/sim-main.h b/sim/mips/sim-main.h index 7db51a14135..9e87fbf109f 100644 --- a/sim/mips/sim-main.h +++ b/sim/mips/sim-main.h @@ -690,7 +690,7 @@ void decode_coproc PARAMS ((SIM_DESC sd, sim_cpu *cpu, address_word cia, unsigne #define DecodeCoproc(instruction) \ decode_coproc (SD, CPU, cia, (instruction)) -void sim_monitor (SIM_DESC sd, sim_cpu *cpu, address_word cia, unsigned int arg); +int sim_monitor (SIM_DESC sd, sim_cpu *cpu, address_word cia, unsigned int arg); -- 2.30.2