Skip to content

[7060X6] Added custom SI settings for the host side serdes and LPO mo…#23841

Merged
yxieca merged 1 commit intosonic-net:masterfrom
prgeor:7060x6_lpo
Sep 26, 2025
Merged

[7060X6] Added custom SI settings for the host side serdes and LPO mo…#23841
yxieca merged 1 commit intosonic-net:masterfrom
prgeor:7060x6_lpo

Conversation

@prgeor
Copy link
Copy Markdown
Contributor

@prgeor prgeor commented Aug 28, 2025

…dules

Why I did it

Added SI settings for the 1x800G LPO module on Arista 7060X6

Signed-off-by: Prince George [email protected]

Work item tracking
  • Microsoft ADO (number only):

How I did it

  1. Added the media_settings.json
  2. Added the optics_si_settings.json

How to verify it

Tested the following

  1. Multiple switch reboots
  2. 48+ hours soak test and ensured there ae no link flap, FEC histogram Bin <= 4, MAX pre-FEC BER < 4.01E-8 on all 64 ports , real time pre-FEC BER < 1E-8
  3. Able to upgrade the CDB firmware

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)

Description for the changelog

Link to config_db schema for YANG module changes

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).

@kewei-arista
Copy link
Copy Markdown
Contributor

Thanks @prgeor. Keys used in the media_settings.json lgtm. Did you have a chance to run dsc command to confirm these tuning values are correctly applied when the corresponding modules are inserted?

@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).

@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).

@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).

@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).

@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).

@prgeor
Copy link
Copy Markdown
Contributor Author

prgeor commented Sep 23, 2025

Thanks @prgeor. Keys used in the media_settings.json lgtm. Did you have a chance to run dsc command to confirm these tuning values are correctly applied when the corresponding modules are inserted?

@kewei-arista I have now tested the changes for both LPO vendors. if looks good can you approve ?

@kewei-arista
Copy link
Copy Markdown
Contributor

@prgeor lgtm, but I don't think I have the access to approve this PR.

@prgeor
Copy link
Copy Markdown
Contributor Author

prgeor commented Sep 25, 2025

"/azpw_ms_conflict"

@prgeor
Copy link
Copy Markdown
Contributor Author

prgeor commented Sep 25, 2025

"/azpw ms_conflict"

bingwang-ms added a commit to Azure/sonic-buildimage-msft that referenced this pull request Sep 26, 2025
…d LPO modules (#1667)

202503 cherry-pick for
sonic-net/sonic-buildimage#23841

<!--
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
Added SI settings for the 1x800G LPO module on Arista 7060X6

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

#### How I did it
Added the media_settings.json
Added the optics_si_settings.json

#### 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)
@yxieca yxieca merged commit 0327fd7 into sonic-net:master Sep 26, 2025
19 of 20 checks passed
@liushilongbuaa
Copy link
Copy Markdown
Contributor

@prgeor , there is code conflict between ms repo and github repo.
I merged to ms repo ignoring difference in this PR. Please check internal branch carefully!

@liushilongbuaa
Copy link
Copy Markdown
Contributor

Please add file in github repo first and change in ms repo to avoid code conflict.

@kewei-arista
Copy link
Copy Markdown
Contributor

@prgeor Do we plan to backport this to 202505?

FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Dec 4, 2025
…dules (sonic-net#23841)

Why I did it
Added SI settings for the 1x800G LPO module on Arista 7060X6

How I did it
Added the media_settings.json
Added the optics_si_settings.json
How to verify it
Tested the following

Multiple switch reboots
48+ hours soak test and ensured there ae no link flap, FEC histogram Bin <= 4, MAX pre-FEC BER < 4.01E-8 on all 64 ports , real time pre-FEC BER < 1E-8
Able to upgrade the CDB firmware

Signed-off-by: Prince George <[email protected]>
Signed-off-by: Feng Pan <[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.

7 participants