Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workaround getSwitchType failure due to NPU SAI not yet supporting it #647

Merged
merged 1 commit into from
Jul 27, 2020
Merged

Conversation

gechiang
Copy link
Contributor

Changes from #632 is causing BRCM DUT not able to bootup.
The newly introduced getSwithType at SONiC boot time to support BRCM GearBox SAI is not yet supported by ASIC SAI result to error that causes Orchagent termination at boot up.
This workaround is added to default the switch type to NPU in case receiving a failure from the SAI call so that BRCM DUT can boot up successfully.

I have validated this change using the Force10-S6000 (BRCM ASIC).

Kamil has also reviewed and OKed this change.

@gechiang gechiang merged commit c810e5f into sonic-net:master Jul 27, 2020
gechiang added a commit to sonic-net/sonic-buildimage that referenced this pull request Jul 28, 2020
Advanced Submodule head of Sonic-Sairedis and picked up the following changes:

Workaround getSwitchType failure due to NPU SAI not yet supporting it (#647)
sonic-net/sonic-sairedis#647

Adding support for new SAI API sai_query_attribute_capability() (#645)
sonic-net/sonic-sairedis#645

[syncd] ARM32 bit fixes, for 64bit printf format specifier. (#643)  
Signed-off-by: Rajkumar Pennadam Ramamoorthy <[email protected]>
sonic-net/sonic-sairedis#643

[syncd] Add support for synchronous bulk api (#640)
sonic-net/sonic-sairedis#640
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants