From 157d6f9c2fb203dd95fe53c3481e851ea55272ef Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Sun, 10 Oct 2010 20:38:05 -0700 Subject: [PATCH] SPARC: Make SPARC's ISA's clear function initialize everything it should. Also make it not set some pointers to NULL potentially introducing a memory leak. That should be done in the constructor. --- src/arch/alpha/isa.hh | 2 ++ src/arch/sparc/isa.cc | 18 +++++++++++++++--- src/arch/sparc/isa.hh | 6 ++++++ 3 files changed, 23 insertions(+), 3 deletions(-) diff --git a/src/arch/alpha/isa.hh b/src/arch/alpha/isa.hh index 01666c783..cd2ece22d 100644 --- a/src/arch/alpha/isa.hh +++ b/src/arch/alpha/isa.hh @@ -31,6 +31,8 @@ #ifndef __ARCH_ALPHA_ISA_HH__ #define __ARCH_ALPHA_ISA_HH__ +#include + #include #include diff --git a/src/arch/sparc/isa.cc b/src/arch/sparc/isa.cc index 3226b4e42..9d4490fa3 100644 --- a/src/arch/sparc/isa.cc +++ b/src/arch/sparc/isa.cc @@ -99,6 +99,7 @@ ISA::clear() memset(tnpc, 0, sizeof(tnpc)); memset(tstate, 0, sizeof(tstate)); memset(tt, 0, sizeof(tt)); + tba = 0; pstate = 0; tl = 0; pil = 0; @@ -123,10 +124,21 @@ ISA::clear() lsuCtrlReg = 0; memset(scratchPad, 0, sizeof(scratchPad)); + + cpu_mondo_head = 0; + cpu_mondo_tail = 0; + dev_mondo_head = 0; + dev_mondo_tail = 0; + res_error_head = 0; + res_error_tail = 0; + nres_error_head = 0; + nres_error_tail = 0; + #if FULL_SYSTEM - tickCompare = NULL; - sTickCompare = NULL; - hSTickCompare = NULL; + // If one of these events is active, it's not obvious to me how to get + // rid of it cleanly. For now we'll just assert that they're not. + if (tickCompare != NULL && sTickCompare != NULL && hSTickCompare != NULL) + panic("Tick comparison event active when clearing the ISA object.\n"); #endif } diff --git a/src/arch/sparc/isa.hh b/src/arch/sparc/isa.hh index 9b4fd50d0..5fe57773c 100644 --- a/src/arch/sparc/isa.hh +++ b/src/arch/sparc/isa.hh @@ -205,6 +205,12 @@ namespace SparcISA ISA() { +#if FULL_SYSTEM + tickCompare = NULL; + sTickCompare = NULL; + hSTickCompare = NULL; +#endif + clear(); } }; -- 2.30.2