Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

perf(StoreUnit): writeback at s2 #4068

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

perf(StoreUnit): writeback at s2 #4068

wants to merge 6 commits into from

Conversation

cz4e
Copy link
Contributor

@cz4e cz4e commented Dec 19, 2024 •
edited
Loading

366841734594531_ pic

@happy-lx shows gem5 store pipeline writeback at s2 and at s4 speedup.

  • LoadQueueRAW entries: 64 to 32 @jin120811

  • LoadQueueRAR entreies: 72 to 64 @jin120811

  • LoadQueueUncache entries: 20 to 4 @jin120811

  • LoadQueueRAW select group size: 8 to 16, wait for timing evaluation.

  • change the entries of LoadQueueRAW will reduce 1 cycle for store unit

  • change the size of select group will reduce 1 cycle for store unit

@cz4e cz4e added the do not merge Do not merge this pull request label Dec 19, 2024
linjuanZ
linjuanZ requested changes Dec 19, 2024
View reviewed changes
Copy link
Member

@linjuanZ linjuanZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Detailed description, please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@linjuanZ linjuanZ linjuanZ requested changes

@good-circle good-circle Awaiting requested review from good-circle good-circle is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
do not merge Do not merge this pull request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cz4e @linjuanZ