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

Jumbo Frame Support - Not Working #204

Open
abu7770 opened this issue Apr 24, 2024 · 2 comments
Open

Jumbo Frame Support - Not Working #204

abu7770 opened this issue Apr 24, 2024 · 2 comments

Comments

@abu7770
Copy link

abu7770 commented Apr 24, 2024

Hi First of all thank you for the flawless ethernet hdl, they work great. Currently I am using 1440 Bytes size for transmitting UDP packets but I need to send more than that say 8640 Bytes. i verified the code and changed the line
parameter TX_FRAME_FIFO = 0,
parameter TX_DROP_OVERSIZE_FRAME = TX_FRAME_FIFO,
in the file "eth_mac_10g_fifo.v"
which was default set to 1 to drop oversize frame if i am correct so i changed it to 0 to support higher byte count. but the fact is when i give higher byte count for single packet after this little change, the Tx tready signal is not going after it goes low in first packet itself(i saw that it initially goes low for ARP response and becomes high ). Do i need to change anything else to support higher byte count.
Thank you

@fpgapsyc
Copy link

Change the FIFO size in the parameter and increase the UDP checksum calculation fifo size. If the UDP checksum fifo is full and the tlast doesn't come. The checksum calculation logic waits for the complete frame to come wgich never comes as the tready goes low after the fifo is full.

So i suggest you to change the fifo sizes to 9000 in both UDP checksum fifo as well as 10G fifo side for you to be able to do the transfer of more than 1440.

Also make sure, you use the eth hdr ready signal to maintain interframe gap for sending the next new frame.

@alexforencich
Copy link
Owner

Yep, TX_FRAME_FIFO must be set, changing the FIFO sizes is the correct solution.

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

3 participants