From 536af1381a1eb96916e391ea488f7a74ad95fe4b Mon Sep 17 00:00:00 2001 From: fros4943 Date: Thu, 12 Mar 2009 17:47:57 +0000 Subject: [PATCH] temporary fix for catching new emulationexception --- .../src/se/sics/cooja/mspmote/MspMote.java | 10 ++++- .../cooja/mspmote/plugins/MspCodeWatcher.java | 37 ++++++++++++------- 2 files changed, 31 insertions(+), 16 deletions(-) diff --git a/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/MspMote.java b/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/MspMote.java index ad6ecb940..e1360210a 100644 --- a/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/MspMote.java +++ b/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/MspMote.java @@ -26,7 +26,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * - * $Id: MspMote.java,v 1.22 2009/03/09 17:12:27 fros4943 Exp $ + * $Id: MspMote.java,v 1.23 2009/03/12 17:47:58 fros4943 Exp $ */ package se.sics.cooja.mspmote; @@ -51,6 +51,7 @@ import se.sics.cooja.mspmote.interfaces.TR1001Radio; import se.sics.mspsim.cli.CommandHandler; import se.sics.mspsim.cli.LineListener; import se.sics.mspsim.cli.LineOutputStream; +import se.sics.mspsim.core.EmulationException; import se.sics.mspsim.core.MSP430; import se.sics.mspsim.platform.GenericNode; import se.sics.mspsim.util.ConfigManager; @@ -327,7 +328,12 @@ public abstract class MspMote implements Mote { } myMoteInterfaceHandler.doActiveActionsBeforeTick(); - cpu.step(cycleCounter); + try { + cpu.step(cycleCounter); + } catch (EmulationException e) { + throw (RuntimeException) + new RuntimeException("Emulated exception: " + e.getMessage()).initCause(e); + } /* Check if radio has pending incoming bytes */ if (myRadio != null && myRadio.hasPendingBytes()) { diff --git a/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/plugins/MspCodeWatcher.java b/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/plugins/MspCodeWatcher.java index 3edcd9fcd..22e69309c 100644 --- a/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/plugins/MspCodeWatcher.java +++ b/tools/cooja/apps/mspsim/src/se/sics/cooja/mspmote/plugins/MspCodeWatcher.java @@ -26,7 +26,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * - * $Id: MspCodeWatcher.java,v 1.15 2009/03/12 15:12:23 fros4943 Exp $ + * $Id: MspCodeWatcher.java,v 1.16 2009/03/12 17:47:57 fros4943 Exp $ */ package se.sics.cooja.mspmote.plugins; @@ -52,6 +52,7 @@ import se.sics.cooja.*; import se.sics.cooja.mspmote.MspMote; import se.sics.cooja.mspmote.MspMoteType; import se.sics.mspsim.core.CPUMonitor; +import se.sics.mspsim.core.EmulationException; import se.sics.mspsim.core.MSP430; import se.sics.mspsim.core.MSP430Core; import se.sics.mspsim.ui.DebugUI; @@ -190,7 +191,11 @@ public class MspCodeWatcher extends VisPlugin { stepButton.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { // TODO Perform single step here - mspMote.getCPU().step(mspMote.getCPU().cycles+1); + try { + mspMote.getCPU().step(mspMote.getCPU().cycles+1); + } catch (EmulationException e1) { + logger.fatal("Error: ", e1); + } updateInfo(); } }); @@ -223,20 +228,24 @@ public class MspCodeWatcher extends VisPlugin { return; } - while (count++ < max) { - cpu.step(mspMote.getCPU().cycles+1); - pc = cpu.readRegister(MSP430Core.PC); - instruction = cpu.memory[pc] + (cpu.memory[pc + 1] << 8); - if ((instruction & 0xff80) == MSP430.CALL) { - depth++; - } else if (instruction == MSP430.RETURN) { - depth--; - if (depth < 0) { - updateInfo(); - return; - } + try { + while (count++ < max) { + cpu.step(mspMote.getCPU().cycles+1); + pc = cpu.readRegister(MSP430Core.PC); + instruction = cpu.memory[pc] + (cpu.memory[pc + 1] << 8); + if ((instruction & 0xff80) == MSP430.CALL) { + depth++; + } else if (instruction == MSP430.RETURN) { + depth--; + if (depth < 0) { + updateInfo(); + return; + } + } } + } catch (EmulationException e1) { + logger.fatal("Error: ", e1); } logger.fatal("Function '" + functionName + "' did not return within " + max + " instructions");