cpu-o3: refactor decoupled FTB #231
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change-Id: I7a6d5e69b40a8e484f05e69d241f581af47d3357
ideal_tick() is copied from commit: e7294f1 TODO: Fix bug in ideal_tick(): Bubbles created by generateFinalPredAndCreateBubbles() are lost in the next tick, resulting in almost NO override bubbles. To resolve this, move tryEnqFetchTarget() and tryEnqFetchStream() outside the while loop (before decrementing numOverrideBubbles). Additionally, ensure that TWO FTB entries generated by generateAndSetNewFetchStream() are saved in the current tick, allowing both entries to enqueue to FTQ/FSQ in the next tick. Change-Id: Ibbacd199e2ac9d47fb10c8011ef844a499e918b6
Change-Id: Ide0a7eb0b0d3c31226f57e0a0d64e1d813de506c
Change-Id: I7a6d5e69b40a8e484f05e69d241f581af47d3357
ideal_tick() is copied from commit: e7294f1 TODO: Fix bug in ideal_tick(): Bubbles created by generateFinalPredAndCreateBubbles() are lost in the next tick, resulting in almost NO override bubbles. To resolve this, move tryEnqFetchTarget() and tryEnqFetchStream() outside the while loop (before decrementing numOverrideBubbles). Additionally, ensure that TWO FTB entries generated by generateAndSetNewFetchStream() are saved in the current tick, allowing both entries to enqueue to FTQ/FSQ in the next tick. Change-Id: Ibbacd199e2ac9d47fb10c8011ef844a499e918b6
Change-Id: Ide0a7eb0b0d3c31226f57e0a0d64e1d813de506c
jensen-yan
force-pushed
the
refactor-ideal-frontend
branch
from
3a97593 to
63ba779
Compare
Change-Id: I40d159242ca782b91add8e1483be6a5306f43b0d
Merge branch 'refactor-ideal-frontend' of https://github.com/OpenXiangShan/GEM5 into refactor-ideal-frontend Change-Id: I0055821eb8eb43cf5c80d7326db13be78f83575a
jensen-yan
approved these changes
Dec 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
ideal_tick() is copied from commit: e7294f1
TODO: Fix bug in ideal_tick(): Bubbles created by generateFinalPredAndCreateBubbles() are lost in the next tick, resulting in almost no override bubbles. To resolve this, move tryEnqFetchTarget() and tryEnqFetchStream() outside the while loop (before decrementing numOverrideBubbles). Additionally, ensure that two FTB entries generated by generateAndSetNewFetchStream() are saved in the current tick, allowing both entries to be enqueued to FTQ/FSQ in the next tick.
makeNewPrediction() was split into generateAndSetNewFetchStream() and enqueueFetchStream(). When enableTwoTaken, the stream entry is set in the previous ideal_tick(), so in the current ideal_tick(), we only need to enqueue the fetch stream. When disableTwoTaken, the function flow remains the same as before, where a new prediction is made and the fetch stream is enqueued in the same tick.