Skip to content

Bump up /dev/shm size for Broadcom platforms to 512MB by default#21772

Merged
yxieca merged 1 commit intosonic-net:masterfrom
saiarcot895:fix-syncd-dev-shm
Apr 1, 2025
Merged

Bump up /dev/shm size for Broadcom platforms to 512MB by default#21772
yxieca merged 1 commit intosonic-net:masterfrom
saiarcot895:fix-syncd-dev-shm

Conversation

@saiarcot895
Copy link
Contributor

@saiarcot895 saiarcot895 commented Feb 18, 2025

Why I did it

In the case of ASIC detection failures on Broadcom (or if the ASIC couldn't be detected in time), the /dev/shm partition in the syncd container will be only 64MB, which might cause issues if syncd/Broadcom SAI library needs more space than that.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Since using a larger /dev/shm on its own doesn't cause any issues, bump up the default to 512MB. This should be enough for most platforms.

How to verify it

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)

@prgeor
Copy link
Contributor

prgeor commented Feb 18, 2025

@saiarcot895 we have few platforms with very less memory. have we studied the impact on them?

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@saiarcot895
Copy link
Contributor Author

@saiarcot895 we have few platforms with very less memory. have we studied the impact on them?

Increasing the /dev/shm on its own doesn't automatically take memory away from the system. Memory will be taken only if something is using it.

@saiarcot895
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@saiarcot895 saiarcot895 requested review from Gfrom2016 and yxieca April 1, 2025 00:48
Copy link
Contributor

@Gfrom2016 Gfrom2016 left a comment

Choose a reason for hiding this comment

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

LGTM

@yxieca yxieca merged commit b7edc45 into sonic-net:master Apr 1, 2025
23 checks passed
@saiarcot895 saiarcot895 deleted the fix-syncd-dev-shm branch April 3, 2025 06:06
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #22546

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #22547

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.

8 participants