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

Disable NETIF_F_HW_CSUM in KNET #17080

Merged
merged 1 commit into from
Nov 2, 2023
Merged

Conversation

byu343
Copy link
Contributor

@byu343 byu343 commented Nov 2, 2023

Why I did it

This is CSP CS00012280996.
The issue to fix is that the checksum was incorrect for all TCP packets leaving the system so that the BGP connection cannot be established. We found the issue on BCM56993, and it is possible to affect all platforms using linux_ngknet.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Remove NETIF_F_HW_CSUM

How to verify it

We verified that the BGP connection is established according to 'show ip bgp summary'.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@byu343 byu343 requested a review from lguohan as a code owner November 2, 2023 21:09
@lguohan lguohan merged commit ed07dba into sonic-net:master Nov 2, 2023
@StormLiangMS
Copy link
Contributor

ADO:25760023

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Nov 8, 2023
This is CSP CS00012280996.
The issue to fix is that the checksum was incorrect for all TCP packets leaving the system so that the BGP connection cannot be established. We found the issue on BCM56993, and it is possible to affect all platforms using linux_ngknet.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #17118

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants