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

[master] Upgrade xgs SAI version to 10.1.21.0 #19334

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

Gfrom2016
Copy link
Contributor

@Gfrom2016 Gfrom2016 commented Jun 18, 2024

Why I did it

Upgrade the xgs SAI version to 10.1.21.0 to include the following changes:

  • 10.1.8.0: Support _brcm_sai_port_unlos_set when external port_phy_addr is 0xff
  • 10.1.9.0: [CS00012330893]With PFC test, egress drop observed for lossless queues
  • 10.1.11.0: Include the bcmsim in the odp package
  • 10.1.13.0: handle bind of egress and ingress groups to same lag
  • 10.1.14.0: Backport SDK-381920 to rel_ocp_sai_10_1 High CPU due to SDK calling soc_dnxc_port_resource_get
  • 10.1.16.0: [CSP CS00012336644] callback called twice with same event type
  • 10.1.17.0: [CSP 00012320736][SAI_BRANCH rel_ocp_sai_10_1]Back port MMU related Fixes to 10.1 specific to LTSW/TH4 devices-HSDK
  • 10.1.18.0: [CSP CS00012351984] Backport SONIC-81526 to 10.1 Fixes for rate class bound setting and UCQ stats
  • 10.1.19.0: Reset EGR_L3_NEXT_HOP_2 for TH3
  • 10.1.20.0: Sonic crash due to unsupported return from SAI_SWITCH_ATTR_ECMP_DEFAULT_HASH_OFFSET
Work item tracking
  • Microsoft ADO (number only): 28356083

How I did it

Upgrade the SAI version in the sai.mk file.

How to verify it

Run the SONiC basic test:
https://elastictest.org/scheduler/testplan/66742d7ef6db3102cef61056

Signed-off-by: zitingguo-ms [email protected]

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)

@Gfrom2016 Gfrom2016 changed the title [build only]change xgs SAI version to 10.1.20.0 [build only]Upgrade xgs SAI version to 10.1.20.0 Jun 20, 2024
@Gfrom2016
Copy link
Contributor Author

/azp run ms_conflict

Copy link

Commenter does not have sufficient privileges for PR 19334 in repo sonic-net/sonic-buildimage

@Gfrom2016 Gfrom2016 changed the title [build only]Upgrade xgs SAI version to 10.1.20.0 [master] Upgrade xgs SAI version to 10.1.20.0 Jun 20, 2024
@Gfrom2016 Gfrom2016 marked this pull request as ready for review June 21, 2024 01:23
@Gfrom2016 Gfrom2016 requested a review from lguohan as a code owner June 21, 2024 01:23
@StormLiangMS
Copy link
Contributor

/azp run ms_conflict

Copy link

No pipelines are associated with this pull request.

@StormLiangMS
Copy link
Contributor

/azpw ms_conflict

1 similar comment
@Gfrom2016
Copy link
Contributor Author

/azpw ms_conflict

@Gfrom2016 Gfrom2016 changed the title [master] Upgrade xgs SAI version to 10.1.20.0 [master] Upgrade xgs SAI version to 10.1.21.0 Jun 26, 2024
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Gfrom2016
Copy link
Contributor Author

This has been cherry-picked to 202311 #19322, remove Request for 202311 Branch label

@Gfrom2016
Copy link
Contributor Author

hi @StormLiangMS, could you help merge this?

@StormLiangMS StormLiangMS merged commit 0bcf2de into sonic-net:master Jul 3, 2024
11 checks passed
r12f pushed a commit to r12f/sonic-buildimage that referenced this pull request Jul 17, 2024
Why I did it
Upgrade the xgs SAI version to 10.1.21.0 to include the following changes:

10.1.8.0: Support _brcm_sai_port_unlos_set when external port_phy_addr is 0xff
10.1.9.0: [CS00012330893]With PFC test, egress drop observed for lossless queues
10.1.11.0: Include the bcmsim in the odp package
10.1.13.0: handle bind of egress and ingress groups to same lag
10.1.14.0: Backport SDK-381920 to rel_ocp_sai_10_1 High CPU due to SDK calling soc_dnxc_port_resource_get
10.1.16.0: [CSP CS00012336644] callback called twice with same event type
10.1.17.0: [CSP 00012320736][SAI_BRANCH rel_ocp_sai_10_1]Back port MMU related Fixes to 10.1 specific to LTSW/TH4 devices-HSDK
10.1.18.0: [CSP CS00012351984] Backport SONIC-81526 to 10.1 Fixes for rate class bound setting and UCQ stats
10.1.19.0: Reset EGR_L3_NEXT_HOP_2 for TH3
10.1.20.0: Sonic crash due to unsupported return from SAI_SWITCH_ATTR_ECMP_DEFAULT_HASH_OFFSET
Work item tracking
Microsoft ADO (number only): 28356083
How I did it
Upgrade the SAI version in the sai.mk file.

How to verify it
Run the SONiC basic test:
https://elastictest.org/scheduler/testplan/66742d7ef6db3102cef61056
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
Why I did it
Upgrade the xgs SAI version to 10.1.21.0 to include the following changes:

10.1.8.0: Support _brcm_sai_port_unlos_set when external port_phy_addr is 0xff
10.1.9.0: [CS00012330893]With PFC test, egress drop observed for lossless queues
10.1.11.0: Include the bcmsim in the odp package
10.1.13.0: handle bind of egress and ingress groups to same lag
10.1.14.0: Backport SDK-381920 to rel_ocp_sai_10_1 High CPU due to SDK calling soc_dnxc_port_resource_get
10.1.16.0: [CSP CS00012336644] callback called twice with same event type
10.1.17.0: [CSP 00012320736][SAI_BRANCH rel_ocp_sai_10_1]Back port MMU related Fixes to 10.1 specific to LTSW/TH4 devices-HSDK
10.1.18.0: [CSP CS00012351984] Backport SONIC-81526 to 10.1 Fixes for rate class bound setting and UCQ stats
10.1.19.0: Reset EGR_L3_NEXT_HOP_2 for TH3
10.1.20.0: Sonic crash due to unsupported return from SAI_SWITCH_ATTR_ECMP_DEFAULT_HASH_OFFSET
Work item tracking
Microsoft ADO (number only): 28356083
How I did it
Upgrade the SAI version in the sai.mk file.

How to verify it
Run the SONiC basic test:
https://elastictest.org/scheduler/testplan/66742d7ef6db3102cef61056
@Gfrom2016 Gfrom2016 deleted the test_10.1.20.0 branch July 30, 2024 12:17
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Aug 1, 2024
Why I did it
Upgrade the xgs SAI version to 10.1.21.0 to include the following changes:

10.1.8.0: Support _brcm_sai_port_unlos_set when external port_phy_addr is 0xff
10.1.9.0: [CS00012330893]With PFC test, egress drop observed for lossless queues
10.1.11.0: Include the bcmsim in the odp package
10.1.13.0: handle bind of egress and ingress groups to same lag
10.1.14.0: Backport SDK-381920 to rel_ocp_sai_10_1 High CPU due to SDK calling soc_dnxc_port_resource_get
10.1.16.0: [CSP CS00012336644] callback called twice with same event type
10.1.17.0: [CSP 00012320736][SAI_BRANCH rel_ocp_sai_10_1]Back port MMU related Fixes to 10.1 specific to LTSW/TH4 devices-HSDK
10.1.18.0: [CSP CS00012351984] Backport SONIC-81526 to 10.1 Fixes for rate class bound setting and UCQ stats
10.1.19.0: Reset EGR_L3_NEXT_HOP_2 for TH3
10.1.20.0: Sonic crash due to unsupported return from SAI_SWITCH_ATTR_ECMP_DEFAULT_HASH_OFFSET
Work item tracking
Microsoft ADO (number only): 28356083
How I did it
Upgrade the SAI version in the sai.mk file.

How to verify it
Run the SONiC basic test:
https://elastictest.org/scheduler/testplan/66742d7ef6db3102cef61056
@bingwang-ms
Copy link
Contributor

@Gfrom2016 Please file another PR for 202405 branch to address cherry-pick conflict.

@Gfrom2016
Copy link
Contributor Author

@bingwang-ms 202405 branch already been upgraded in this PR #19635.

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.

4 participants