Skip to content

[Dynamic Buffer][Mellanox] Enhance port shared headroom pool check#3272

Merged
prsunny merged 3 commits intosonic-net:masterfrom
stephenxs:enhance_port_shp_check
Oct 7, 2024
Merged

[Dynamic Buffer][Mellanox] Enhance port shared headroom pool check#3272
prsunny merged 3 commits intosonic-net:masterfrom
stephenxs:enhance_port_shp_check

Conversation

@stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Aug 30, 2024

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash

How I verified it

Manual test
Regression test sonic-net/sonic-mgmt#14479

Details if related

Copy link
Contributor

@kperumalbfn kperumalbfn left a comment

Choose a reason for hiding this comment

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

LGTM

@stephenxs
Copy link
Collaborator Author

The failures do not look relevant to the PR. Retrying.

@stephenxs
Copy link
Collaborator Author

/azpw run azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/AzurePipelines run azure.sonic-swss

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Stephen Sun <[email protected]>
…rofile has not applied to the database

Signed-off-by: Stephen Sun <[email protected]>
@stephenxs stephenxs force-pushed the enhance_port_shp_check branch from ec0703a to 84d28ef Compare October 6, 2024 08:02
@prsunny prsunny merged commit c553d8d into sonic-net:master Oct 7, 2024
@stephenxs stephenxs deleted the enhance_port_shp_check branch October 7, 2024 23:32
mssonicbld pushed a commit to mssonicbld/sonic-swss that referenced this pull request Nov 15, 2024
…onic-net#3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #3374

mssonicbld pushed a commit that referenced this pull request Nov 15, 2024
…3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash
divyachandralekha pushed a commit to divyachandralekha/sonic-swss that referenced this pull request Dec 12, 2024
…onic-net#3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash
divyachandralekha pushed a commit to divyachandralekha/sonic-swss that referenced this pull request Dec 12, 2024
…onic-net#3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash
shiraez pushed a commit to Marvell-switching/sonic-swss that referenced this pull request Feb 17, 2025
…onic-net#3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
…onic-net#3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash
baorliu pushed a commit to baorliu/sonic-swss that referenced this pull request Feb 23, 2026
…onic-net#3272)

What I did

Check maximum port headroom pool in dynamic buffer calculation
Depends on sonic-net/sonic-buildimage#20085

Why I did it

To avoid orchagent from crash

Signed-off-by: Baorong Liu <[email protected]>
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.

5 participants