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

LOTR - Enable Stall #170

Open
adlv888 opened this issue Mar 3, 2022 · 3 comments
Open

LOTR - Enable Stall #170

adlv888 opened this issue Mar 3, 2022 · 3 comments
Assignees
@amichai-bd
Labels
bug Something isn't working Design task RTL design tasks

Comments

@adlv888
Copy link
Collaborator

adlv888 commented Mar 3, 2022

No description provided.

@adlv888 adlv888 self-assigned this Mar 3, 2022
@adlv888
Copy link
Collaborator Author

adlv888 commented May 1, 2022

@amichaibd can you please write a brief explain about this issue?

@amichai-bd
Copy link
Owner

currently, nothing is stopping the core to send as many requests to the ring.
This is a problem - what if the C2F is full and the core wants to send another write request?
This si why we need a "stall" to make sure the core will not send request when the C2F buffer is already full

@amichai-bd amichai-bd added the bug Something isn't working label Jun 7, 2022
@amichai-bd amichai-bd assigned amichai-bd and unassigned adlv888 Jan 21, 2023
@amichai-bd
Copy link
Owner

Still a issue.
As long the RING is not very traffic-loaded, things are working for now.
But this should be fixed!

@amichai-bd amichai-bd added the Design task RTL design tasks label Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amichai-bd amichai-bd

Labels
bug Something isn't working Design task RTL design tasks
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@amichai-bd @adlv888