Skip to content

Conversation

@cel-gl
Copy link
Contributor

@cel-gl cel-gl commented Dec 9, 2025

Pre-submission checklist
[heshiw@AZUHPS26 fboss]$ git commit -m "Add icetea800bc Si setting"
clang-format.............................................................Passed
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check json...............................................................Passed
check for merge conflicts................................................Passed

Summary

Add icetea800bc Si setting,Update the parameters provided by the SI team into FBOSS to improve the port's BER performance.

Test Plan

Update the CSV file containing the SI parameters, then regenerate icetea800bc_platform_mapping.json.
Integrate the JSON file into the C++ code and compile to generate wedge_agent-sai_impl.
Run the agent and verify in bcmshell that the SI parameters are correctly applied.
front port
OM-test (1).txt when lane work at 212G, it's good, when work at 106G not good enough, dvt will new be parameters
backplane
The backplane fix setting BER test setting is precoding enabled at LT disabled and ER mode enabled. (the precoding is disable if only disable LT)
The backplane fix setting BER test log:
image

@meta-cla meta-cla bot added the CLA Signed label Dec 9, 2025
@meta-codesync
Copy link

meta-codesync bot commented Dec 9, 2025

@shiva-menta has imported this pull request. If you are a Meta employee, you can view this in D88746003.

@shiva-menta
Copy link
Contributor

@cel-gl I tried running with these new settings internally and we're seeing issues with link up reliability. Seems like around 50% of the time, EmptyLinkTest.CheckInit fails with these new SI settings. I ran the test 16 times and saw link up issues 6/16 times.

When testing with the old SI settings, I did not see link up issues 20/20 times. Did you ever see these issues on your side?

@cel-gl
Copy link
Contributor Author

cel-gl commented Dec 10, 2025

@cel-gl I tried running with these new settings internally and we're seeing issues with link up reliability. Seems like around 50% of the time, EmptyLinkTest.CheckInit fails with these new SI settings. I ran the test 16 times and saw link up issues 6/16 times.

When testing with the old SI settings, I did not see link up issues 20/20 times. Did you ever see these issues on your side?

yes i have tested, , some port not good than old si seeting , SI team confirmed this , they suggest set OM TXFIR to 001
on optical module , I'm trying to set it up in fboss and test again

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