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

Describe CVA6 for write-through and write-back cache versions #603

Closed
MikeOpenHWGroup opened this issue Feb 1, 2021 · 7 comments
Closed
Assignees
Labels
Component:Doc For issues in the Documentation (e.g. for README.md files) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Enhancement For feature requests and enhancements

Comments

@MikeOpenHWGroup
Copy link
Member

In #602, there was a conversation like this:

Is the list of files in the manifest complete?

Looks okay for the write-through cache version. Of course, the write-back cache needs different files.

AFAICT, the ability to compile CVA6 for either write-through and write-back cache versions is not described in the User Manual. The User Manual should be updated to explain:

  1. How to compile the CVA6 for write-through and write-back cache versions.
  2. Theory of operation for the write-through cache controller.
  3. Theory of operation for the write-back cache controller.
@MikeOpenHWGroup MikeOpenHWGroup added the Component:Doc For issues in the Documentation (e.g. for README.md files) label Feb 1, 2021
@j-silv
Copy link

j-silv commented Feb 26, 2022

  1. How to compile the CVA6 for write-through and write-back cache versions.

For an FPGA emulation (say for example, using the Genesys 2 board) can we enable the write-back cache by simply commenting out the `define WT_DCACHE line in the genesysii.svh file?

It seems as though the WB_DCACHE define is not mentioned anywhere else in the source code, so I'm assuming we don't need to uncomment that line.

@suppamax
Copy link

suppamax commented Nov 8, 2022

@j-silv yes, by default the write-back cache is selected; you have compile the design with WT_DCACHE defined if you want to use a write-through cache

@suppamax
Copy link

suppamax commented Nov 8, 2022

We've just published our notes about the WB cache.
And here's an interesting paper about the WT cache.

Hope it helps

@MikeOpenHWGroup MikeOpenHWGroup added Type:Enhancement For feature requests and enhancements Status:In Progress Work on this issue has started, but is not complete. labels Feb 17, 2023
@MikeOpenHWGroup
Copy link
Member Author

Hi @JeanRochCoulon, this is an old issue with new information. Let's get this into the User Manual.

@JeanRochCoulon
Copy link
Contributor

I agree. The cache type should be addded in the CVA6 parameter chapter of user manual. I take the point !

@JeanRochCoulon
Copy link
Contributor

Hi, I have just merged the #1127 to replace the define by a localparam into the config pkg. On my side, the issue is solved. Mike, up to you to take the decidsion to close the issue.

@MikeOpenHWGroup
Copy link
Member Author

Thanks everyone! Closing.

@MikeOpenHWGroup MikeOpenHWGroup added Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation and removed Status:In Progress Work on this issue has started, but is not complete. labels Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:Doc For issues in the Documentation (e.g. for README.md files) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Enhancement For feature requests and enhancements
Projects
None yet
Development

No branches or pull requests

5 participants