Skip to content

[action] [PR:23343] TH5-512 ingress and egress lossy queue buffer size change#1380

Merged
r12f merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/23343
Jul 21, 2025
Merged

[action] [PR:23343] TH5-512 ingress and egress lossy queue buffer size change#1380
r12f merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/23343

Conversation

@mssonicbld
Copy link
Collaborator

Why I did it

BRCM SAI 13.2.1 increases the pkt trim buffer pool to 1.7MB, leading to the reduction of available lossy queue buffer.

Without this change, there will be a syncd crash on default buffer profile pairing with SAI 13.2.1.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Reduce default ingress and egress lossy buffer size from 165307264 with 161999982

How to verify it

Verified on Arista-7060X6-64PE-B-C512S2

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

  • 202412

Tested branch (Please provide the tested image version)

  • 202412.b27

Description for the changelog

BRCM SAI 13.2.1 increases the pkt trim buffer pool to 1.7MB, available lossy queue buffer size to be reduced accordingly.

Link to config_db schema for YANG module changes

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

#### Why I did it

BRCM SAI 13.2.1 increases the pkt trim buffer pool to 1.7MB, leading to the reduction of available lossy queue buffer.

Without this change, there will be a syncd crash on default buffer profile pairing with SAI 13.2.1.

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

#### How I did it

Reduce default ingress and egress lossy buffer size from 165307264 with 161999982

#### How to verify it

Verified on Arista-7060X6-64PE-B-C512S2

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

- [x] 202412

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

- [x] 202412.b27

#### Description for the changelog

BRCM SAI 13.2.1 increases the pkt trim buffer pool to 1.7MB, available lossy queue buffer size to be reduced accordingly.

#### Link to config_db schema for YANG module changes

#### A picture of a cute animal (not mandatory but encouraged)
@mssonicbld
Copy link
Collaborator Author

Original PR: sonic-net/sonic-buildimage#23343

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link

@r12f r12f left a comment

Choose a reason for hiding this comment

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

CI failed due to known test issues. bypassing and approved.

@r12f r12f merged commit e8d9805 into Azure:202412 Jul 21, 2025
12 of 16 checks passed
mssonicbld added a commit that referenced this pull request Jul 22, 2025
…03 (#1387)

```<br>* a44672c - (HEAD -> 202503) Merge branch '202412' of https://github.com/Azure/sonic-buildimage-msft into 202503 (2025-07-22) [Sonic Automation]
* e8d9805 - (origin/202412, 202412) [action] [PR:23343] TH5-512 ingress and egress lossy queue buffer size change (#1380) (2025-07-21) [Riff]
|\ 
| failure_prs.log skip_prs.log 3c745e7 - TH5-512 ingress and egress lossy queue buffer size change (2025-07-19) [Sonic Build Admin]
* 155c4f8 - [submodule][202412] Update submodule sonic-sairedis to the latest HEAD automatically (#1374) (2025-07-21) [Riff]
|\ 
| failure_prs.log skip_prs.log 4309cb7 - [submodule] Update submodule sonic-sairedis to the latest HEAD automatically (2025-07-21) [Sonic Automation]
* 3648500 - [submodule][202412] Update submodule sonic-linux-kernel to the latest HEAD automatically (#1375) (2025-07-21) [Riff]
* 183c067 - [submodule] Update submodule sonic-linux-kernel to the latest HEAD automatically (2025-07-21) [Sonic Automation]<br>```
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.

2 participants