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

Update Dynamic NoC sw counter before issue NoC txns #18794

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

yugaoTT
Copy link
Contributor

@yugaoTT yugaoTT commented Mar 7, 2025

We have a race condition between l1 write of sw counter and issue noc txn.
if noc0 issues a write after noc1, noc1 does the barrier and the barrier succeeds it doesn’t mean that noc1's write landed.
image

Solution, update counter before issue noc txn (swap the green box)

NOC transaction:
Write the SW counter
Issue NOC transaction
Barrier:
while !done
Read HW counter
Read SW counter

Checklist

@yugaoTT yugaoTT self-assigned this Mar 7, 2025
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.

1 participant