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

dwc_eqos - support 9014-byte jumbo frames #36

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Conversation

idigdoug
Copy link
Collaborator

Enable support for 9014-byte jumbo frames.

  • Update INF to allow JumboPacket up to 9014.
  • Tell NetAdapterCx that the receive buffer size needs to be RoundUp8(JumboPacket + 8), where JumboPacket is set in the INF (default = min = 1514, max = 9014).
  • Tell hardware that the receive buffer size is RoundUp8(JumboPacket + 8).

Note that using a full 9022-byte buffer for 1522-byte packets is sub-optimal. NetAdapterCx does not currently support multi-fragment packets if using system-managed buffering. With driver-managed buffering we could fix this, but that probably isn't worthwhile at this point.

Enable support for 9014-byte jumbo frames.

- Update INF to allow JumboPacket up to 9014.
- Tell NetAdapterCx that the receive buffer size needs to be
  RoundUp8(JumboPacket + 8), where JumboPacket is set in the INF
  (default = min = 1514, max = 9014).
- Tell hardware that the receive buffer size is RoundUp8(JumboPacket + 8).

Note that using a full 9022-byte buffer for 1522-byte packets is
sub-optimal. NetAdapterCx does not currently support multi-fragment
packets if using system-managed buffering. With driver-managed buffering
we could fix this, but that probably isn't worthwhile at this point.
@idigdoug idigdoug merged commit 540c49e into master Jan 11, 2024
2 checks passed
@idigdoug idigdoug deleted the dwc_eqos_9014 branch January 11, 2024 19:28
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.

2 participants