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

Broadcom SAI 9.x/10.1 does not allow switch_id when creating fabric switch #18579

Closed
ysmanman opened this issue Apr 5, 2024 · 4 comments
Closed
Labels
BRCM Master Branch Quality Triaged this issue has been triaged

Comments

@ysmanman
Copy link
Contributor

ysmanman commented Apr 5, 2024

Description

We observed the following syncd crash on sup with SAI 10.1:

Apr  3 00:13:33.568454 nfc407 ERR syncd5#syncd: [1f:00.0] SAI_API_SWITCH:brcm_sai_dnx_create_switch:8075 SAI_SWITCH_ATTR_SWITCH_ID is not valid for SAI_SWITCH_ATTR_TYPE
Apr  3 00:13:33.568689 nfc407 ERR syncd5#syncd: :- sendApiResponse: api SAI_COMMON_API_CREATE failed in syncd mode: SAI_STATUS_INVALID_PARAMETER
Apr  3 00:13:33.569057 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_INIT_SWITCH: true
Apr  3 00:13:33.569256 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_FDB_EVENT_NOTIFY: 0x5615c1cf63d0
Apr  3 00:13:33.569450 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_PORT_STATE_CHANGE_NOTIFY: 0x5615c1cf63e0
Apr  3 00:13:33.569645 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_SWITCH_SHUTDOWN_REQUEST_NOTIFY: 0x5615c1cf6400
Apr  3 00:13:33.569847 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_PORT_HOST_TX_READY_NOTIFY: 0x5615c1cf6460
Apr  3 00:13:33.570062 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_SWITCH_HARDWARE_INFO: 7:49,102,58,48,48,46,48
Apr  3 00:13:33.570281 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_TYPE: SAI_SWITCH_TYPE_FABRIC
Apr  3 00:13:33.570492 nfc407 ERR syncd5#syncd: :- processQuadEvent: attr: SAI_SWITCH_ATTR_SWITCH_ID: 4294967295
Apr  3 00:13:33.575873 nfc407 ERR swss5#orchagent: :- create: create status: SAI_STATUS_INVALID_PARAMETER
Apr  3 00:13:33.576107 nfc407 ERR swss5#orchagent: :- main: Failed to create a switch, rv:-5

According to SAI 10.1 source code, switch_id is not allowed when creating fabric switch.
Opened CSP CS00012344284 to tack the issue.

Steps to reproduce the issue:

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@ysmanman
Copy link
Contributor Author

ysmanman commented Apr 5, 2024

Add @arlakshm @kenneth-arista @arista-nwolfe for viz.

@ysmanman ysmanman changed the title Broadcom SAI 10.1 does not allow switch_id when creating fabric switch Broadcom SAI 9.x/10.1 does not allow switch_id when creating fabric switch Apr 8, 2024
@adyeung adyeung added the BRCM label Apr 9, 2024
@adyeung
Copy link
Collaborator

adyeung commented Apr 9, 2024

Debug and triage will be thru BRCM CSP

@adyeung adyeung added the Triaged this issue has been triaged label Apr 10, 2024
@arlakshm
Copy link
Contributor

arlakshm commented May 8, 2024

Fix available in DNX SAI version 10.1.15. @arista-nwolfe to validate and close this issue

@ysmanman
Copy link
Contributor Author

ysmanman commented May 8, 2024

This is the SONiC PR that extracts the fabric switch_id from config_db and then passes to SAI: sonic-net/sonic-swss#3102

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BRCM Master Branch Quality Triaged this issue has been triaged
Projects
Archived in project
Development

No branches or pull requests

4 participants