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

Increase address width in transactor interface #117

Closed
tswilliams opened this issue Jul 22, 2019 · 1 comment
Closed

Increase address width in transactor interface #117

tswilliams opened this issue Jul 22, 2019 · 1 comment
Assignees
Milestone

Comments

@tswilliams
Copy link
Collaborator

Currently, the addr_width constant in ipbus_trans_decl is equal to 12, meaning that request/reply packets cannot be larger than 2048 words (i.e. 8192 bytes).

Depending on the time taken to send & receive packets, which can vary greatly between different setups (based on e.g. CPU & transport medium), in order to achieve optimal performance larger packets (e.g. 4096 or 8192 words) are sometimes required.

Since the length of the packet header & body (in 32 bit words) are each encoded in 16-bit fields (in the first word of the transactor buffers), 16 is a natural new value for addr_width; addr_width=16 corresponds to maximum packet size of 65536 32-bit words (approx 262kBytes) which more than sufficient for all setups encountered so far (with margin of a few factors of 2).

@tswilliams
Copy link
Collaborator Author

All done. Closing.

@tswilliams tswilliams self-assigned this Aug 20, 2019
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

1 participant