Skip to content

Remove SHARED_LIMIT_DYNAMIC for MC queue on TH5#23327

Merged
yxieca merged 3 commits intosonic-net:masterfrom
bingwang-ms:rm_shared_limit
Jul 17, 2025
Merged

Remove SHARED_LIMIT_DYNAMIC for MC queue on TH5#23327
yxieca merged 3 commits intosonic-net:masterfrom
bingwang-ms:rm_shared_limit

Conversation

@bingwang-ms
Copy link
Copy Markdown
Contributor

@bingwang-ms bingwang-ms commented Jul 15, 2025

Why I did it

This PR is to fix issue https://github.com/aristanetworks/sonic-qual.msft/issues/684

Work item tracking
  • Microsoft ADO 33797172:

How I did it

Update bcm.config and remove SHARED_LIMIT_DYNAMIC: ALPHA_2.

How to verify it

The change is verified by running everflow test.

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Tested branch (Please provide the tested image version)

  • 202503

Description for the changelog

Remove SHARED_LIMIT_DYNAMIC for MC queue on TH5

Link to config_db schema for YANG module changes

No config_db change.

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

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Copy Markdown
Contributor Author

@aaronpayment , @davidm-arista Can you help review?

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Copy Markdown
Contributor Author

@r12f Can you help review please?

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 0f77fa3 into sonic-net:master Jul 17, 2025
20 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to msft-202503: Azure/sonic-buildimage-msft#1364

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #23374

r12f added a commit to Azure/sonic-buildimage-msft that referenced this pull request Jul 18, 2025
)

Why I did it
This PR is to fix issue aristanetworks/sonic-qual.msft#684
Cherry-pick sonic-net/sonic-buildimage#23327

Work item tracking
Microsoft ADO 33797172:

How I did it
Update bcm.config and remove SHARED_LIMIT_DYNAMIC: ALPHA_2.

How to verify it
The change is verified by running everflow test.

<!--
Please make sure you've read and understood our contributing guidelines:
     https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

** Make sure all your commits include a signature generated with `git
commit -s` **

If this is a bug fix, make sure your description includes "fixes #xxxx",
or
     "closes #xxxx" or "resolves #xxxx"

     Please provide the following information:
-->

#### Why I did it

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it

#### How to verify it

<!--
If PR needs to be backported, then the PR must be tested against the
base branch and the earliest backport release branch and provide tested
image version on these two branches. For example, if the PR is requested
for master, 202211 and 202012, then the requester needs to provide test
results on master and 202012.
-->

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

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [ ] 202205
- [ ] 202211

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [ ] <!-- image version 1 -->
- [ ] <!-- image version 2 -->

#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->

<!--
Ensure to add label/tag for the feature raised. example - PR#2174 under
sonic-utilities repo. where, Generic Config and Update feature has been
labelled as GCU.
-->

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on
https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->

#### A picture of a cute animal (not mandatory but encouraged)
bingwang-ms added a commit to Azure/sonic-buildimage-msft that referenced this pull request Aug 14, 2025
<!--
Please make sure you've read and understood our contributing guidelines:
     https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

** Make sure all your commits include a signature generated with `git
commit -s` **

If this is a bug fix, make sure your description includes "fixes #xxxx",
or
     "closes #xxxx" or "resolves #xxxx"

     Please provide the following information:
-->

#### Why I did it
This PR is to fix issue
aristanetworks/sonic-qual.msft#684
Cherry-pick sonic-net/sonic-buildimage#23327

##### Work item tracking
- Microsoft ADO **33797172**:

#### How I did it
Update bcm.config and remove `SHARED_LIMIT_DYNAMIC: ALPHA_2`.

#### How to verify it
The change is verified by running everflow test.

<!--
If PR needs to be backported, then the PR must be tested against the
base branch and the earliest backport release branch and provide tested
image version on these two branches. For example, if the PR is requested
for master, 202211 and 202012, then the requester needs to provide test
results on master and 202012.
-->

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

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [ ] 202205
- [ ] 202211
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [ ] 202505

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [x] 202503


#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->

<!--
Ensure to add label/tag for the feature raised. example - PR#2174 under
sonic-utilities repo. where, Generic Config and Update feature has been
labelled as GCU.
-->
Remove SHARED_LIMIT_DYNAMIC for MC queue on TH5

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on
https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->
No config_db change.

#### A picture of a cute animal (not mandatory but encouraged)
tshalvi pushed a commit to tshalvi/sonic-buildimage that referenced this pull request Aug 25, 2025
Why I did it
This PR is to fix issue aristanetworks/sonic-qual.msft#684

Work item tracking
Microsoft ADO 33797172:

How I did it
Update bcm.config and remove SHARED_LIMIT_DYNAMIC: ALPHA_2.

How to verify it
The change is verified by running everflow test.
dakotac-arista added a commit to dakotac-arista/sonic-buildimage that referenced this pull request Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment