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.

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

Ultrawide Core Fixes #476

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

Ultrawide Core Fixes #476

wants to merge 10 commits into from

Conversation

bkorpan
Copy link
Contributor

@bkorpan bkorpan commented Jul 18, 2020

Add configs with 256-bit fetch and >2 memory pipes.

@jerryz123
Copy link
Contributor

Does this actually work? Its pretty cool if it does.

I suspect there may be some bug in the global history maintenance logic that reduces bpred performance, I assume its 2x 128b banks?
Also, I'm a bit concerned about moving the store pipe to the last pipe in the cache, that was extremely finicky to get right, but I can't think of a case of the top of my head where it might break something.

@bkorpan
Copy link
Contributor Author

bkorpan commented Jul 19, 2020

Does this actually work? Its pretty cool if it does.

I suspect there may be some bug in the global history maintenance logic that reduces bpred performance, I assume its 2x 128b banks?
Also, I'm a bit concerned about moving the store pipe to the last pipe in the cache, that was extremely finicky to get right, but I can't think of a case of the top of my head where it might break something.

I was able to run Dhrystone before doing the store pipe thing - got a score of 11500 Dhrystones/MHz. The store pipe thing doesn't work yet.

jerryz123 pushed a commit that referenced this pull request Mar 11, 2021
bump to firesim dev that will become master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants