Skip to content

RDKB-64189: Enable ZRAM to optimize and reduce RDKB memory usage#291

Open
manedurphy wants to merge 5 commits intodevelopfrom
feature/RDKB-64189
Open

RDKB-64189: Enable ZRAM to optimize and reduce RDKB memory usage#291
manedurphy wants to merge 5 commits intodevelopfrom
feature/RDKB-64189

Conversation

@manedurphy
Copy link
Copy Markdown

@manedurphy manedurphy commented Apr 17, 2026

The following changes are part of a larger effort to enable a ZRAM SWAP partition for Broadband devices:

  • The state of the RFC is managed by newly-defined SysCfg key-value pairs
  • Telemetry markers are reported via a new zram_stats.sh script in test-and-diagnostic which runs on a cron

Copilot AI review requested due to automatic review settings April 17, 2026 18:41
@manedurphy manedurphy requested review from a team as code owners April 17, 2026 18:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enables periodic ZRAM/memory-swap telemetry reporting and introduces default SysCfg keys to control the memory swap feature on ARM targets.

Changes:

  • Add a crond entry for /usr/ccsp/tad/memswap_stats.sh gated by MemorySwapEnable and MemorySwapStatsIntervalMinutes.
  • Define ARM factory defaults for MemorySwap* SysCfg keys (enable flag, disk size, telemetry interval, and tunables).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
source/scripts/init/service.d/service_crond.sh Adds a conditional cron schedule for memory swap telemetry based on SysCfg flags/interval.
source/scripts/init/defaults/system_defaults_arm Adds default SysCfg values controlling memory swap enablement, sizing, reporting interval, and tunables.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/scripts/init/service.d/service_crond.sh
Comment thread source/scripts/init/defaults/system_defaults_arm
Comment thread source/scripts/init/service.d/service_crond.sh Outdated
Copilot AI review requested due to automatic review settings April 20, 2026 20:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/scripts/init/service.d/service_crond.sh
Comment thread source/scripts/init/service.d/service_crond.sh
Comment thread source/scripts/init/service.d/service_crond.sh
Comment thread source/scripts/init/service.d/service_crond.sh
Copy link
Copy Markdown
Contributor

@apattu200 apattu200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok to me

Copilot AI review requested due to automatic review settings April 23, 2026 17:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/scripts/init/service.d/service_crond.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants