forked from sonic-net/sonic-swss
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Improve the way of handling BUFFER_PG during PFC storm (sonic-net#1480)
* Fix the issue: failed to updating BUFFER_PG during PFC storm One action for the PFC storm is to set a zero buffer profile on the PG under PFC storm. The zero buffer won't be removed until the PFC storm has gone. If the user wants to modify the buffer profile for the PG, the bufferorch will return "task_need_retry". General speaking it doesn't matter unless that: - the system can't be warm rebooted until the PFC storm has gone. - the "task_need_retry" will block the update of the entire BUFFER_PG table from being programmed to ASIC. In this sense, we need a better solution. The new solution will: - record the new profile user wants to apply during PFC storm as the "pending profile" for that PG and return "task_success" if the PG is under PFC storm. - apply the pending profile once the PG is unlocked. - the latest pending profile will take effect in case user tries updating the profile for more than 1 times. Signed-off-by: Stephen Sun <[email protected]> * Address review comments: add a pair of brackets for the if-block Signed-off-by: Stephen Sun <[email protected]> * Fix ut error Signed-off-by: Stephen Sun <[email protected]> * Fix typo Signed-off-by: Stephen Sun <[email protected]>
- Loading branch information
Showing
5 changed files
with
51 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters