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

drivers: wifi: Disable TCP/IP checksum offload with IP fragmentation #11876

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

krish2718
Copy link
Contributor

When IP fragementation is enabled, the checksum needs to computed before fragmentation and verified only after re-assembly, but as fragmentation/re-assembly are done in Zephy, offloaded checksum won't work.

Till we figure out a permanent solution for this, disable checksum offload when IP fragmentation is enabled.

Fixes SHEL-1800.

When IP fragementation is enabled, the checksum needs to
computed before fragmentation and verified only after re-assembly, but
as fragmentation/re-assembly are done in Zephy, offloaded checksum won't
work.

Till we figure out a permanent solution for this, disable checksum
offload when IP fragmentation is enabled.

Fixes SHEL-1800.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 26, 2023
@NordicBuilder
Copy link
Contributor

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-chip X
test-sdk-wifi X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@nordicjm nordicjm merged commit e661512 into nrfconnect:main Jul 26, 2023
13 checks passed
@krish2718 krish2718 deleted the disable_csum_offload branch December 14, 2023 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants