-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Syncd container exit on broadcom platform #21680
Comments
Met same issue too.
syslog:
|
@adyeung Can you please take a look? Looks like a SAI implementation issue. |
Current latest SONiC master is on SAI header 1.15 SAI header 1.15.x does not have SAI_API_PREFIX_COMPRESSION 53 definition (despite it was merged 3 months ago), only the sai master has it https://github.com/opencomputeproject/SAI/blob/v1.15.0/inc/sai.h I am not sure how syncd is reaching type 53 |
this seems like brcm log_set issue, it's not supporting that index and probably getting out of array causing sigsegv |
@kcudnik @prabhataravind the code PR below moved SAI pairing to commit id 0540855, that contains SAI_API_PREFIX_COMPRESSION 53 in sai.h, see below https://github.com/opencomputeproject/SAI/blob/054085547c2042b95ab0fa1a75a44da3288b26bf/inc/sai.h However, official SAI 1.15.4 sai.h is not the same as 0540855 https://github.com/opencomputeproject/SAI/blob/v1.15.4/inc/sai.h Is the intend to pair SONiC master with SAI 1.15.x or SAI Master? If it's the former, SAI commit id 0540855 pairing seems to be off. sai_log_set() out-of-bound handling needs to be fixed separately, but please check what is the intended SAI header pairing |
Broadcom platform, the sai seems not support 'SAI_API_PREFIX_COMPRESSION', API 53, the idx is out of range:
Failed log:
version info
The text was updated successfully, but these errors were encountered: