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

set genev_sys_6081 tx checksum off #3045

Merged
merged 1 commit into from Jul 18, 2023
Merged

set genev_sys_6081 tx checksum off #3045

merged 1 commit into from Jul 18, 2023

Conversation

hongzhen-ma
Copy link
Collaborator

@hongzhen-ma hongzhen-ma commented Jul 18, 2023

What type of this PR

  • Bug fixes
企业微信截图_df13dd78-cbd8-4f1c-b0a6-49bbf0a1b4f9

Which issue(s) this PR fixes:

Fixes #(issue-number)

WHAT

🤖 Generated by Copilot at 7def5cc

Disable tx checksum for geneve interface on Linux to avoid packet loss. Modify initForOS function in cmd/daemon/init.go to use ethtool command.

🤖 Generated by Copilot at 7def5cc

initForOS changed
checksum off for geneve
buggy in winter

HOW

🤖 Generated by Copilot at 7def5cc

  • Disable tx checksum for geneve interface on Linux (link)

@oilbeater
Copy link
Collaborator

需要对比和开启 offload 的性能差异

@hongzhen-ma
Copy link
Collaborator Author

企业微信截图_2d79f256-5aa4-425b-8a5a-1577d3074422 企业微信截图_e0f537cf-fa9b-4081-ae21-1ed250ae4207

根据结果看,测试结果差异不大

@hongzhen-ma hongzhen-ma merged commit 3be1e84 into master Jul 18, 2023
58 checks passed
@hongzhen-ma hongzhen-ma deleted the checksum branch July 18, 2023 10:39
hongzhen-ma added a commit that referenced this pull request Jul 18, 2023
hongzhen-ma added a commit that referenced this pull request Jul 18, 2023
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.

None yet

2 participants