RDKB-63687 Code development to reduce SM app memory usage#921
RDKB-63687 Code development to reduce SM app memory usage#921skarth597 wants to merge 10 commits intordkcentral:developfrom
Conversation
Reason for change: for onboarding Test Procedure: Debug Priority: P0 Risks: Low Signed-off-by: Samyuktha Karthikeyan <samyuktha_karthikeyan@comcast.com>
There was a problem hiding this comment.
Pull request overview
This PR reduces SM app memory usage by changing client_assoc_stats from a fixed-size per-MAX_NUM_RADIOS global array to a dynamically allocated buffer sized to the runtime number of radios.
Changes:
- Replaced
client_assoc_stats[MAX_NUM_RADIOS]with a heap-allocatedclient_assoc_statspointer. - Allocated
client_assoc_statsduringsm_init()usinggetNumberRadios()and freed it duringsm_deinit().
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Reason for change: To reduce the SM app memory usage Test Procedure: Debug Priority: P1 Risks: Low Signed-off-by: Samyuktha Karthikeyan <samyuktha_karthikeyan@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Reason for change: To reduce the SM app memory usage Test Procedure: Debug Priority: P1 Risks: Low Signed-off-by: Samyuktha Karthikeyan <samyuktha_karthikeyan@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Reason for change: To reduce the SM app memory usage
Test Procedure: Debug
Priority: P1
Risks: Low
Signed-off-by: Samyuktha Karthikeyan samyuktha_karthikeyan@comcast.com