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

RISC-V default TLB size documentation #50

Closed
HanyzPAPU opened this issue Oct 5, 2023 · 2 comments
Closed

RISC-V default TLB size documentation #50

HanyzPAPU opened this issue Oct 5, 2023 · 2 comments

Comments

@HanyzPAPU
Copy link
Collaborator

The documentation regarding RICV-V TLB default size mentions a Kilo-TLB and Mega-TLB size, which is a remnant of previous implementation.

@HanyzPAPU HanyzPAPU changed the title RISC-V default tlb size RISC-V default TLB size documentation Oct 5, 2023
@vhotspur
Copy link
Member

vhotspur commented Oct 5, 2023

Would the following work or have I overlooked something?

diff --git a/doc/reference/riscv_constants.rst b/doc/reference/riscv_constants.rst
index 70a480e..c01a6c2 100644
--- a/doc/reference/riscv_constants.rst
+++ b/doc/reference/riscv_constants.rst
@@ -43,5 +43,4 @@ Waiting cycles
 Default TLB size
 ----------------
 
-Kilo-TLB - 256 entries
-Mega-TLB - 32 entries
+Default TLB size is 48 entries.

@HanyzPAPU
Copy link
Collaborator Author

Yes, that should fix the problem. Thank you

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

2 participants