Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[bufferpool] Support for buffer pool type both (#1735)
This PR is fix for the issue sonic-net/sonic-buildimage#7497 Signed-off-by: vedganes <vedavinayagam.ganesan@nokia.com> What I did Changes done to allow orchagent accepting "both" type for buffer pool configuration. Why I did it Currently sonic supports only 'ingress' and 'egress' types of buffer pools. Since some devices like DNX based switches only support buffer pool type 'both', we are unable to create buffer pools for these devices. This patch fixes this problem by enhancing buffer pool configuration to accept "both" type. How I verified it In a switch that uses DNX arch based chips (like VOQ chassis using line cards with Jerico family of switches), the following configuration is applied "BUFFER_POOL": { "lossless_pool": { "size": "12766208", "type": "both", "mode": "dynamic" } } when the switch became operational, it was observed, there are no error logs indicating invalid parameter or failure of buffer pool creation.
- Loading branch information