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

BOOMv3 TODOs #152

Closed
14 tasks done
jerryz123 opened this issue Jan 12, 2019 · 5 comments
Closed
14 tasks done

BOOMv3 TODOs #152

jerryz123 opened this issue Jan 12, 2019 · 5 comments

Comments

@jerryz123
Copy link
Contributor

jerryz123 commented Jan 12, 2019

I think once we get the following features completed, we can mark a new version of BOOM.

@abejgonzalez
Copy link
Contributor

I would also recommend having HPM's working, TAGE verified and working, and bumped rocket-chip added to the list.

@jerryz123
Copy link
Contributor Author

Do we have documentation of the RAS bug anywhere? What is the broken behavior?

How difficult would it be to re-enable HPMs. I see a bunch of commented out code, but I don't know how much work it would be to re-enable these.

Also lets just use the same rocket-chip as whatever FireSim is using. I don't want to bump past the version they have.

@jerryz123 jerryz123 pinned this issue Jan 12, 2019
@abejgonzalez
Copy link
Contributor

I don't think HPM's would be that hard. Theoretically, they should already be enabled. It was just when I was trying to read them that I was getting garbage out. As for the RAS, it is just that is it commented out with older signal names. So the port would be to uncomment then fix signal names and verify. As for FireChip, that sounds good with me.

@abejgonzalez abejgonzalez changed the title BOOM next version TODOs BOOMv3 TODOs Jan 12, 2019
@heiner-bauer
Copy link

After reading chapter 12 of the documentation I have several questions regarding HPMs / HPEs:

  • which events does BOOM track ?
    I have no clue of chisel but the code in exu/core.scala
    new freechips.rocketchip.rocket.EventSet((mask, hits) => (mask & hits).orR, Seq(
    does not look like it matches the number and ordering of the SiFive U54 events
  • how many independent counters does BOOM support ?
  • do you happen to have some kind of library to use these counters (configuring and reading these CSRs is kind of fiddly) ?

@abejgonzalez
Copy link
Contributor

@heiner-bauer Moved the conversation to a new issue thread to keep this thread on "ToDos"

@jerryz123 jerryz123 unpinned this issue Nov 17, 2019
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

No branches or pull requests

3 participants