cpu-o3: Clear thread state in time buffers on thread exit #1052
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #1049. Clear only thread-specific state from the O3 CPU time buffers, rather than clearing all state for all threads.
Specifically, this patch adds a
clearStates()
method for all O3 time buffer structs in src/cpu/o3/comm.hh. Upon thread exit,CPU::removeThread()
now invokes this method for all structs in each time buffer, rather than flushing out the time buffers (which nukes the states for all threads, not just the exiting one).Change-Id: I48cd50e39b3cd5e0068ddfc19a03d9bbd3f31bd5