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

Use of 15 vs. 16 entries in the binned memories #171

Open
aryd opened this issue Jun 24, 2021 · 0 comments
Open

Use of 15 vs. 16 entries in the binned memories #171

aryd opened this issue Jun 24, 2021 · 0 comments
Labels

Comments

@aryd
Copy link
Contributor

aryd commented Jun 24, 2021

We have changed in the emulation (for the fw_synch_210613 tag) the number of stubs that are written to the binned memories to be maximum of 15. This addressed a problem in the ME when we had 16 stubs. I (aryd) was confused about what we could do in the HLS code - 16 should indeed be possible, we have 5 bits for the counter in the standard config and 4 bits + 1 but for empty/non-empty in the combined modules configuration. This should be sufficient to make this work. But for now we changed the VMR to produce at most 15 stubs. This should be fixed at some point, but not the highest priority at this point so we created this issue not to forget about this.

@pwittich pwittich added the stale label Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants