sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

o3, inorder: fix FS bug due to initializing ThreadState to Halted.

Browse source 
For some reason o3 FS init() only called initCPU if the thread state
was Suspended, which was no longer the case.  There's no apparent
reason to check, so I whacked the test completely rather than
changing the check to Halted.
The inorder init() was also updated to be symmetric, though the
previous code was just a fancy no-op.
This commit is contained in:
Steve Reinhardt 2009-04-17 16:54:58 -07:00
parent b146131d18
commit 14808ecac9
2 changed files with 8 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/cpu/inorder/cpu.cc
View file

@ -29,6 +29,8 @@
*
*/
#include "config/full_system.hh"
#include "arch/utility.hh"
#include "cpu/exetrace.hh"
#include "cpu/activity.hh"
@ -420,16 +422,12 @@ InOrderCPU::init()
for (int i = 0; i < number_of_threads; ++i)
thread[i]->inSyscall = true;
#if FULL_SYSTEM
for (int tid=0; tid < number_of_threads; tid++) {
ThreadContext *src_tc = thread[tid]->getTC();
// Threads start in the Suspended State
if (src_tc->status() != ThreadContext::Suspended) {
continue;
}
ThreadContext *src_tc = threadContexts[tid];
TheISA::initCPU(src_tc, src_tc->contextId());
}
#endif
// Clear inSyscall.
for (int i = 0; i < number_of_threads; ++i)

13
src/cpu/o3/cpu.cc
View file

@ -568,21 +568,12 @@ FullO3CPU<Impl>::init()
for (int i = 0; i < number_of_threads; ++i)
thread[i]->inSyscall = true;
#if FULL_SYSTEM
for (int tid=0; tid < number_of_threads; tid++) {
#if FULL_SYSTEM
ThreadContext *src_tc = threadContexts[tid];
#else
ThreadContext *src_tc = thread[tid]->getTC();
#endif
// Threads start in the Suspended State
if (src_tc->status() != ThreadContext::Suspended) {
continue;
}
#if FULL_SYSTEM
TheISA::initCPU(src_tc, src_tc->contextId());
#endif
}
#endif
// Clear inSyscall.
for (int i = 0; i < number_of_threads; ++i)