Skip to content

[action] [PR:1723] Fix deadlock between syncd and orchagent syncd during initialization failure#104

Merged
mssonicbld merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/1723
Jan 24, 2026
Merged

[action] [PR:1723] Fix deadlock between syncd and orchagent syncd during initialization failure#104
mssonicbld merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/1723

Conversation

@mssonicbld
Copy link
Collaborator

When syncd requests a shutdown, orchagent may be blocked waiting for a response to an init view (or other NOTIFY) command. Since syncd stops processing commands while waiting for the shutdown response, orchagent never receives its response and cannot acknowledge the shutdown request - resulting in a deadlock.

This fix adds the selectable channel to the select loop during shutdown-wait mode and handles incoming commands appropriately:

  • NOTIFY commands receive a SAI_STATUS_FAILURE response to unblock the waiting orchagent
  • Other commands are logged and ignored

This prevents orchagent from hanging until timout when syncd is failing.

This fix - sonic-net/sonic-buildimage#24799

…failure

When syncd requests a shutdown, orchagent may be blocked waiting for a response to an init view (or other NOTIFY) command. Since syncd stops processing commands while waiting for the shutdown response, orchagent never receives its response and cannot acknowledge the shutdown request - resulting in a deadlock.

This fix adds the selectable channel to the select loop during shutdown-wait mode and handles incoming commands appropriately:
- NOTIFY commands receive a SAI_STATUS_FAILURE response to unblock the waiting orchagent
- Other commands are logged and ignored

This prevents orchagent from hanging until timout when syncd is failing.

This fix - sonic-net/sonic-buildimage#24799
@mssonicbld
Copy link
Collaborator Author

Original PR: sonic-net/sonic-sairedis#1723

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 39c3fdc into Azure:202412 Jan 24, 2026
3 of 6 checks passed
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.

1 participant